Changeset 63562 in vbox for trunk/src/VBox/Devices/PC/BIOS/system.c

Timestamp:

Aug 16, 2016 2:04:03 PM (8 years ago)

Author:

vboxsync

Message:

scm: cleaning up todos

File:

• trunk/src/VBox/Devices/PC/BIOS/system.c (modified) (6 diffs)

trunk/src/VBox/Devices/PC/BIOS/system.c

@@ -53 +53 @@
 #define UNSUPPORTED_FUNCTION    0x86    /* Specific to INT 15h. */
 
-#define BIOS_CONFIG_TABLE       0xe6f5  /* TODO: configurable? put elsewhere? */
-
-#define ACPI_DATA_SIZE    0x00010000L   /* TODO: configurable? put elsewhere? */
+#define BIOS_CONFIG_TABLE       0xe6f5  /** @todo configurable? put elsewhere? */
+
+#define ACPI_DATA_SIZE    0x00010000L   /** @todo configurable? put elsewhere? */
 
 #define BX_CPU                  3
@@ -266 +266 @@
     parm [ax] modify nomemory aborts;
 
-// @todo: This method is silly. The RTC should be programmed to fire an interrupt
+/// @todo This method is silly. The RTC should be programmed to fire an interrupt
 // instead of hogging the CPU with inaccurate code.
 void timer_wait(uint32_t usec_wait)
@@ -328 +328 @@
 }
 
-// @todo: move elsewhere?
+/// @todo move elsewhere?
 #define AX      r.gr.u.r16.ax
 #define BX      r.gr.u.r16.bx
@@ -400 +400 @@
         break;
 
-    //@todo: Why does this need special handling? All we need is to set CF
+    /// @todo Why does this need special handling? All we need is to set CF
     //       but not handle this as an unknown function (regardless of CPU type).
     case 0x4f:
@@ -478 +478 @@
 
         // turn off interrupts
-        int_disable();  //@todo: aren't they off already?
+        int_disable();  /// @todo aren't they off already?
 
         set_enable_a20(1); // enable A20 line; we're supposed to fail if that fails
@@ -834 +834 @@
 
     // turn off interrupts
-    int_disable();    //@todo: aren't they disabled already?
+    int_disable();    /// @todo aren't they disabled already?
 
     prev_a20_enable = set_enable_a20(1); // enable A20 line