Changeset 44693 in vbox for trunk

Timestamp:

Feb 14, 2013 5:37:34 PM (12 years ago)

Author:

vboxsync

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

83791

Message:

DevLPC.cpp: some small adjustment, not worth doing much here.

File:

• trunk/src/VBox/Devices/PC/DevLPC.cpp (modified) (9 diffs)

trunk/src/VBox/Devices/PC/DevLPC.cpp

@@ -2 +2 @@
 /** @file
  * DevLPC - LPC device emulation
+ *
+ * @todo This needs to be _replaced_ by a proper chipset device one day. There
+ *       are less than 10 C/C++ statements in this file doing active emulation.
  */
 
 /*
- * Copyright (C) 2006-2011 Oracle Corporation
+ * Copyright (C) 2006-2013 Oracle Corporation
  *
  * This file is part of VirtualBox Open Source Edition (OSE), as
@@ -56 +59 @@
 #include "VBoxDD2.h"
 
-#define RCBA_BASE                0xFED1C000
+#define RCBA_BASE                UINT32_C(0xFED1C000)
 
 typedef struct
@@ -130 +133 @@
 {
     LPCState *s = PDMINS_2_DATA(pDevIns, LPCState*);
-    switch (cb)
-    {
-        case 1:
-        case 2:
-            break;
-
-        case 4:
-        {
-            *(uint32_t*)pv = rcba_ram_readl(s, GCPhysAddr);
-            break;
-        }
-
-        default:
-            AssertReleaseMsgFailed(("cb=%d\n", cb)); /* for now we assume simple accesses. */
-            return VERR_INTERNAL_ERROR;
-    }
+    Assert(cb == 4); Assert(!(GCPhysAddr & 3));
+    *(uint32_t*)pv = rcba_ram_readl(s, GCPhysAddr);
     return VINF_SUCCESS;
 }
@@ -171 +160 @@
             break;
         case 4:
-        {
-            /** @todo: locking? */
             rcba_ram_writel(s, GCPhysAddr, *(uint32_t *)pv);
             break;
-        }
 
         default:
@@ -185 +171 @@
 
 #ifdef IN_RING3
-/**
- * Reset notification.
- *
- * @returns VBox status.
- * @param   pDevIns     The device instance data.
- */
-static DECLCALLBACK(void) lpcReset(PPDMDEVINS pDevIns)
-{
-    LPCState *pThis = PDMINS_2_DATA(pDevIns, LPCState *);
-    LogFlow(("lpcReset: \n"));
-}
 
 /**
@@ -255 +230 @@
     PCIDevSetStatus           (&pThis->dev, 0x0200); /* PCI_status_devsel_medium */
 
-    /** @todo: rewrite using PCI accessors */
+    /** @todo rewrite using PCI accessors; Update, rewrite this device from
+     *        scratch! Possibly against ICH9 or something else matching our
+     *        chipset of choice. (Note that the exteremely partial emulation here
+     *        is supposed to be of ICH7 if what's on the top of the file is
+     *        anything to go by.) */
     /* See p. 427 of ICH9 specification for register description */
 
@@ -337 +316 @@
      */
     rc = PDMDevHlpMMIORegister(pDevIns, RCBA_BASE, 0x4000, pThis,
-                               IOMMMIO_FLAGS_READ_PASSTHRU | IOMMMIO_FLAGS_WRITE_PASSTHRU,
+                               IOMMMIO_FLAGS_READ_DWORD | IOMMMIO_FLAGS_WRITE_PASSTHRU,
                                lpcMMIOWrite, lpcMMIORead, "LPC Memory");
     if (RT_FAILURE(rc))
@@ -343 +322 @@
 
     /* No state in the LPC right now */
-
-    /*
-     * Initialize the device state.
-     */
-    lpcReset(pDevIns);
 
     /**
@@ -392 +366 @@
     NULL,
     /* pfnReset */
-    lpcReset,
+    NULL,
     /* pfnSuspend */
     NULL,