Changeset 33142 in vbox for trunk/src/VBox/Devices/Bus

Timestamp:

Oct 14, 2010 5:25:09 PM (14 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

66674

Message:

PCI: new bus implementation can be used with IDE controller too

File:

• trunk/src/VBox/Devices/Bus/DevPciIch9.cpp (modified) (5 diffs)

trunk/src/VBox/Devices/Bus/DevPciIch9.cpp

@@ -415 +415 @@
     return VINF_SUCCESS;
 }
-
 
 static int ich9pciDataRead(PPCIGLOBALS pGlobals, uint32_t addr, int len, uint32_t *pu32)
@@ -1353 +1352 @@
 static uint32_t ich9pciConfigRead(PPCIGLOBALS pGlobals, uint8_t uBus, uint8_t uDevFn, uint32_t addr, uint32_t len)
 {
-    uint32_t u32Val;
+    uint32_t   u32Val = 0xffffffff;
     PciAddress aPciAddr;
 
@@ -1862 +1861 @@
 } PciSlotAssignments[] = {
     {
-        "piix3ide", 1, 1 // do we really need it?
-    },
-    {
         "lan",      25, 0 /* LAN controller */
     },
@@ -1873 +1869 @@
         "i82801",   30, 0 /* Host Controller */
     },
+    /**
+     *  Please note, that for devices being functions, like we do here, device 0
+     *  must be multifunction, i.e. have header type 0x80. Our LPC device is.
+     *  Alternative approach is to assign separate slot to each device.
+     */
     {
         "lpc",      31, 0 /* Low Pin Count bus */
     },
     {
+        "piix3ide", 31, 1 /* IDE controller */
+    },
+#if 0
+    {
         "ahci",     31, 2 /* SATA controller */
     },
+#endif
     {
         "smbus",    31, 3 /* System Management Bus */
@@ -1939 +1945 @@
      * Find device position
      */
-    if (iDev < 0)
+    if (iDev < 0 || !strcmp(pszName, "piix3ide"))
     {
         iDev = assignPosition(pBus, pPciDev, pszName);