Changeset 63562 in vbox for trunk/src/VBox/Devices/PC

Timestamp:

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

Author:

vboxsync

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

110217

Message:

scm: cleaning up todos

Location:

trunk/src/VBox/Devices/PC

Files:

• BIOS/ahci.c (modified) (8 diffs)
• BIOS/apm.c (modified) (5 diffs)
• BIOS/ata.c (modified) (2 diffs)
• BIOS/bios.c (modified) (1 diff)
• BIOS/boot.c (modified) (3 diffs)
• BIOS/disk.c (modified) (5 diffs)
• BIOS/eltorito.c (modified) (14 diffs)
• BIOS/floppy.c (modified) (12 diffs)
• BIOS/keyboard.c (modified) (8 diffs)
• BIOS/logo.c (modified) (3 diffs)
• BIOS/pcibios.c (modified) (3 diffs)
• BIOS/scsi.c (modified) (1 diff)
• BIOS/system.c (modified) (6 diffs)
• BIOS/timepci.c (modified) (1 diff)
• DevDMA.cpp (modified) (1 diff)
• DevHPET.cpp (modified) (3 diffs)

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

@@ -130 +130 @@
 #define AHCI_HBA_SIZE 0x100
 
-//@todo: what are the casts good for?
+/// @todo what are the casts good for?
 #define AHCI_REG_CAP ((uint32_t)0x00)
 #define AHCI_REG_GHC ((uint32_t)0x04)
@@ -443 +443 @@
          * address registers.
          */
-        //@todo: merge memsets?
+        /// @todo merge memsets?
         _fmemset(&ahci->aCmdHdr[0], 0, sizeof(ahci->aCmdHdr));
         _fmemset(&ahci->abCmd[0], 0, sizeof(ahci->abCmd));
@@ -484 +484 @@
      * address registers.
      */
-    //@todo: just one memset?
+    /// @todo just one memset?
     _fmemset(&ahci->aCmdHdr[0], 0, sizeof(ahci->aCmdHdr));
     _fmemset(&ahci->abCmd[0], 0, sizeof(ahci->abCmd));
@@ -569 +569 @@
 }
 
-//@todo: move
+/// @todo move
 #define ATA_DATA_NO      0x00
 #define ATA_DATA_IN      0x01
@@ -601 +601 @@
              bios_dsk->drqp.nsect, bios_dsk->drqp.sect_sz);
 
-    bios_dsk->drqp.lba     = length << 8;     //@todo: xfer length limit
+    bios_dsk->drqp.lba     = length << 8;     /// @todo xfer length limit
     bios_dsk->drqp.buffer  = buffer;
     bios_dsk->drqp.nsect   = length / bios_dsk->drqp.sect_sz;
@@ -615 +615 @@
 
     /* Reset transferred counts. */
-    // @todo: clear in calling code?
+    /// @todo clear in calling code?
     bios_dsk->drqp.trsfsectors = 0;
     bios_dsk->drqp.trsfbytes   = 0;
@@ -678 +678 @@
         DBG_AHCI("AHCI: Device detected on port %d\n", u8Port);
 
-        //@todo: Merge common HD/CDROM detection code
+        /// @todo Merge common HD/CDROM detection code
         if (devcount_ahci < BX_MAX_AHCI_DEVICES)
         {
@@ -877 +877 @@
      * blocks from 2048-byte CD sectors.
      */
-    ahci->sink_buf_phys = 0xCC000;  //@todo: find some better place!
+    ahci->sink_buf_phys = 0xCC000;  /// @todo find some better place!
 
     /* Reset the controller. */

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

@@ -79 +79 @@
 #define APM_PORT        0x8900      /* Bochs power control port. */
 
-// @todo: merge with system.c
+/// @todo merge with system.c
 #define AX      r.gr.u.r16.ax
 #define BX      r.gr.u.r16.bx
@@ -160 +160 @@
         break;
     case APM_RM_CONN:
-        // @todo: validate device ID
-        // @todo: validate current connection state
-        // @todo: change connection state
+        /// @todo validate device ID
+        /// @todo validate current connection state
+        /// @todo change connection state
         break;
     case APM_PM_CONN:
-        // @todo: validate device ID
-        // @todo: validate current connection state
-        // @todo: change connection state
+        /// @todo validate device ID
+        /// @todo validate current connection state
+        /// @todo change connection state
         AX = APM_BIOS_SEG;              /* 16-bit PM code segment (RM segment base). */
         BX = (uint16_t)apm_pm16_entry;  /* 16-bit PM entry point offset. */
@@ -175 +175 @@
         break;
     case APM_32_CONN:
-        // @todo: validate device ID
-        // @todo: validate current connection state
-        // @todo: change connection state
+        /// @todo validate device ID
+        /// @todo validate current connection state
+        /// @todo change connection state
         AX = APM_BIOS_SEG;              /* 32-bit PM code segment (RM segment base). */
         BX = (uint16_t)apm_pm32_entry;  /* 32-bit entry point offset. */
@@ -192 +192 @@
         break;
     case APM_SET_PWR:
-        // @todo: validate device ID
-        // @todo: validate current connection state
+        /// @todo validate device ID
+        /// @todo validate current connection state
         switch (CX) {
         case APM_PS_STANDBY:
@@ -210 +210 @@
         break;
     case APM_DRV_VER:
-        AX = 0x0102;    // @todo: Not right - must take driver version into account!
+        AX = 0x0102;    /// @todo Not right - must take driver version into account!
         break;
     case APM_DISCONN:
-        // @todo: actually perform a disconnect...
+        /// @todo actually perform a disconnect...
     case APM_BUSY:      /* Nothing to do as APM Idle doesn't slow CPU clock. */
         break;
     case APM_GET_EVT:
-        // @todo: error should be different if interface not connected + engaged
+        /// @todo error should be different if interface not connected + engaged
         SET_AH(APM_ERR_NO_EVENTS);  /* PM events don't happen. */
         SET_CF();

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

@@ -379 +379 @@
     bios_dsk->channels[1].irq     = 15;
 #endif
-#if 0   //@todo - temporarily removed to avoid conflict with AHCI
+#if 0   /// @todo - temporarily removed to avoid conflict with AHCI
 #if BX_MAX_ATA_INTERFACES > 2
     bios_dsk->channels[2].iface   = ATA_IFACE_ISA;
@@ -937 +937 @@
 
     // Reset count of transferred data
-    // @todo: clear in calling code?
+    /// @todo clear in calling code?
     bios_dsk->drqp.trsfsectors = 0;
     bios_dsk->drqp.trsfbytes   = 0;

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

@@ -155 +155 @@
     parm [al] modify [ax];
 
-//@todo: restore
+/// @todo restore
 //#undef VBOX
 

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

@@ -92 +92 @@
 static const char drivetypes[][10]={"Floppy","Hard Disk","CD-ROM","LAN"};
 
-//@todo: pass inputs as bit flags rather than bytes?
+/// @todo pass inputs as bit flags rather than bytes?
 void print_boot_device(uint8_t cdboot, uint8_t lanboot, uint8_t drive)
 {
@@ -114 +114 @@
 //   displays the reason why boot failed
 //--------------------------------------------------------------------------
-//@todo: pass inputs as bit flags rather than bytes?
+/// @todo pass inputs as bit flags rather than bytes?
 void print_boot_failure(uint8_t cdboot, uint8_t lanboot, uint8_t drive,
                         uint8_t reason, uint8_t lastdrive)
@@ -156 +156 @@
 uint32_t BIOSCALL int19_function(uint8_t bseqnr)
 {
-    //@todo: common code for getting the EBDA segment
+    /// @todo common code for getting the EBDA segment
     uint16_t    ebda_seg=read_word(0x0040,0x000E);
     uint16_t    bootseq;

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

@@ -80 +80 @@
 
 
-//@todo: put in a header
+/// @todo put in a header
 #define AX      r.gr.u.r16.ax
 #define BX      r.gr.u.r16.bx
@@ -203 +203 @@
 
         /* If required, translate LCHS to LBA and execute command. */
-        //@todo: The IS_SCSI_DEVICE check should be redundant...
+        /// @todo The IS_SCSI_DEVICE check should be redundant...
         if (( (bios_dsk->devices[device].pchs.heads != nlh) || (bios_dsk->devices[device].pchs.spt != nlspt)) || VBOX_IS_SCSI_DEVICE(device)) {
             lba = ((((uint32_t)cylinder * (uint32_t)nlh) + (uint32_t)head) * (uint32_t)nlspt) + (uint32_t)sector - 1;
@@ -220 +220 @@
         bios_dsk->drqp.buffer   = MK_FP(ES, BX);
         bios_dsk->drqp.nsect    = count;
-        bios_dsk->drqp.sect_sz  = 512;  //@todo: device specific?
+        bios_dsk->drqp.sect_sz  = 512;  /// @todo device specific?
         bios_dsk->drqp.cylinder = cylinder;
         bios_dsk->drqp.head     = head;
@@ -262 +262 @@
 
         // FIXME should set ES & DI
-        // @todo: Actually, the above comment is nonsense.
+        /// @todo Actually, the above comment is nonsense.
 
         goto int13_success;
@@ -411 +411 @@
         bios_dsk->drqp.buffer  = MK_FP(segment, offset);
         bios_dsk->drqp.nsect   = count;
-        bios_dsk->drqp.sect_sz = 512;   //@todo: device specific?
+        bios_dsk->drqp.sect_sz = 512;   /// @todo device specific?
         bios_dsk->drqp.sector  = 0;     /* Indicate LBA. */
         bios_dsk->drqp.dev_id  = device;

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

@@ -66 +66 @@
 
 
-//@todo: put in a header
+/// @todo put in a header
 #define AX      r.gr.u.r16.ax
 #define BX      r.gr.u.r16.bx
@@ -142 +142 @@
 void BIOSCALL cdemu_init(void)
 {
-    // @TODO: a macro or a function for getting the EBDA segment
+    /// @todo a macro or a function for getting the EBDA segment
     uint16_t    ebda_seg = read_word(0x0040,0x000E);
 
@@ -151 +151 @@
 uint8_t BIOSCALL cdemu_isactive(void)
 {
-    // @TODO: a macro or a function for getting the EBDA segment
+    /// @todo a macro or a function for getting the EBDA segment
     uint16_t    ebda_seg = read_word(0x0040,0x000E);
 
@@ -159 +159 @@
 uint8_t BIOSCALL cdemu_emulated_drive(void)
 {
-    // @TODO: a macro or a function for getting the EBDA segment
+    /// @todo a macro or a function for getting the EBDA segment
     uint16_t    ebda_seg = read_word(0x0040,0x000E);
 
@@ -171 +171 @@
 void BIOSCALL int13_eltorito(disk_regs_t r)
 {
-    // @TODO: a macro or a function for getting the EBDA segment
+    /// @todo a macro or a function for getting the EBDA segment
     uint16_t        ebda_seg=read_word(0x0040,0x000E);
     cdemu_t __far   *cdemu;
@@ -193 +193 @@
     case 0x4b: // ElTorito - Terminate disk emu
         // FIXME ElTorito Hardcoded
-        //@todo: maybe our cdemu struct should match El Torito to allow memcpy()?
+        /// @todo maybe our cdemu struct should match El Torito to allow memcpy()?
         write_byte(DS,SI+0x00,0x13);
         write_byte(DS,SI+0x01,cdemu->media);
@@ -240 +240 @@
 
 /* Utility routine to check if a device is a CD-ROM. */
-//@todo: this function is kinda useless as the ATAPI type check is obsolete.
+/// @todo this function is kinda useless as the ATAPI type check is obsolete.
 static uint16_t device_is_cdrom(uint8_t device)
 {
@@ -269 +269 @@
 uint16_t cdrom_boot(void)
 {
-    // @TODO: a macro or a function for getting the EBDA segment
+    /// @todo a macro or a function for getting the EBDA segment
     uint16_t            ebda_seg=read_word(0x0040,0x000E);
     uint8_t             buffer[2048];
@@ -314 +314 @@
     if (buffer[0] != 0)
         return 4;
-    //@todo: what's wrong with memcmp()?
+    /// @todo what's wrong with memcmp()?
     for (i = 0; i < 5; ++i) {
         if (buffer[1+i] != isotag[i])
@@ -341 +341 @@
         return 7;
 
-    //@todo: Define a struct for the Boot Catalog, the hardcoded offsets are so dumb...
+    /// @todo Define a struct for the Boot Catalog, the hardcoded offsets are so dumb...
 
     /* Check if the Boot Catalog looks valid. */
@@ -464 +464 @@
 void BIOSCALL int13_cdemu(disk_regs_t r)
 {
-    // @TODO: a macro or a function for getting the EBDA segment
+    /// @todo a macro or a function for getting the EBDA segment
     uint16_t            ebda_seg=read_word(0x0040,0x000E);
     uint8_t             device, status;
@@ -628 +628 @@
         /* Only set the DPT pointer for emulated floppies. */
         if (cdemu->media < 4) {
-            DI = (uint16_t)&diskette_param_table;   // @todo: should this depend on emulated medium?
-            ES = 0xF000;                            // @todo: how to make this relocatable?
+            DI = (uint16_t)&diskette_param_table;   /// @todo should this depend on emulated medium?
+            ES = 0xF000;                            /// @todo how to make this relocatable?
         }
         goto int13_success;
@@ -852 +852 @@
         // FIXME should handle 0xb5 valid request failed
 
-#if 0 //@todo: implement!
+#if 0 /// @todo implement!
         // Call removable media eject
         ASM_START
@@ -876 +876 @@
         break;
 
-    //@todo: Part of this should be merged with analogous code in disk.c
+    /// @todo Part of this should be merged with analogous code in disk.c
     case 0x48: // IBM/MS get drive parameters
         dpt = DS :> (dpt_t *)SI;

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

@@ -345 +345 @@
     //    111 all other formats/drives
 
-    // @todo: break out drive type determination
+    /// @todo break out drive type determination
     drive_type = inb_cmos(0x10);
     if (drive == 0)
@@ -431 +431 @@
 
     // check CMOS to see if drive exists
-    // @todo: break out drive type determination
+    /// @todo break out drive type determination
     drive_type = inb_cmos(0x10);
     if (drive == 0)
@@ -440 +440 @@
 }
 
-//@todo: put in a header
+/// @todo put in a header
 #define AX      r.gr.u.r16.ax
 #define BX      r.gr.u.r16.bx
@@ -477 +477 @@
             return;
         }
-        // @todo: break out drive type determination
+        /// @todo break out drive type determination
         drive_type = inb_cmos(0x10);
         if (drive == 0)
@@ -559 +559 @@
             // port 04: DMA-1 base and current address, channel 2
             // port 05: DMA-1 base and current count, channel 2
-            // @todo: merge/factor out pointer normalization
+            /// @todo merge/factor out pointer normalization
             page = (ES >> 12);              // upper 4 bits
             base_es = (ES << 4);            // lower 16bits contributed by ES
@@ -706 +706 @@
             // port 04: DMA-1 base and current address, channel 2
             // port 05: DMA-1 base and current count, channel 2
-            // @todo: merge/factor out pointer normalization
+            /// @todo merge/factor out pointer normalization
             page = (ES >> 12);              // upper 4 bits
             base_es = (ES << 4);            // lower 16bits contributed by ES
@@ -882 +882 @@
 
         // set up DMA controller for transfer
-        // @todo: merge/factor out pointer normalization
+        /// @todo merge/factor out pointer normalization
         page = (ES >> 12);              // upper 4 bits
         base_es = (ES << 4);            // lower 16bits contributed by ES
@@ -1016 +1016 @@
         }
 
-        // @todo: break out drive type determination
+        /// @todo break out drive type determination
         drive_type = inb_cmos(0x10);
         num_floppies = 0;
@@ -1076 +1076 @@
 
         /* set es & di to point to 11 byte diskette param table in ROM */
-        ES = 0xF000;    // @todo: any way to make this relocatable?
+        ES = 0xF000;    /// @todo any way to make this relocatable?
         DI = get_floppy_dpt(drive_type);
         CLEAR_CF(); // success
@@ -1091 +1091 @@
             return;
         }
-        // @todo: break out drive type determination
+        /// @todo break out drive type determination
         drive_type = inb_cmos(0x10);
         if (drive == 0)
@@ -1235 +1235 @@
         }
 
-        // @todo: break out drive type determination
+        /// @todo break out drive type determination
         drive_type = inb_cmos(0x10);
         if (drive == 0)
@@ -1305 +1305 @@
 
         // set es & di to point to 11 byte diskette param table in ROM
-        ES = 0xF000;    // @todo: any way to make this relocatable?
+        ES = 0xF000;    /// @todo any way to make this relocatable?
         DI = get_floppy_dpt(drive_type);
 

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

@@ -354 +354 @@
 
 /* Keyboard hardware interrupt handler. */
-//@todo: should this be declared as taking arguments at all?
+/// @todo should this be declared as taking arguments at all?
 void BIOSCALL int09_function(uint16_t ES, uint16_t DI, uint16_t SI, uint16_t BP, uint16_t SP,
                              uint16_t BX, uint16_t DX, uint16_t CX, uint16_t AX)
@@ -474 +474 @@
                 ;   /* Hold on and wait... */
 
-            //@todo: We will send EOI again (and enable keyboard) on the way out; we shouldn't
+            /// @todo We will send EOI again (and enable keyboard) on the way out; we shouldn't
         }
         break;
@@ -512 +512 @@
             mf2_flags |= 0x04;
             write_byte(0x0040, 0x18, mf2_flags);
-            //@todo: EOI/enable kbd/enable interrupts/call INT 15h/8500h
+            /// @todo EOI/enable kbd/enable interrupts/call INT 15h/8500h
         }
         break;
@@ -519 +519 @@
         mf2_flags &= ~0x04;
         write_byte(0x0040, 0x18, mf2_flags);
-        //@todo: EOI/enable kbd/enable interrupts/call INT 15h/8501h
+        /// @todo EOI/enable kbd/enable interrupts/call INT 15h/8501h
         break;
 
@@ -635 +635 @@
 
 
-//@todo: move somewhere else?
+/// @todo move somewhere else?
 #define AX      r.gr.u.r16.ax
 #define BX      r.gr.u.r16.bx
@@ -658 +658 @@
     led_flags   = read_byte(0x0040, 0x97);
     if ((((shift_flags >> 4) & 0x07) ^ (led_flags & 0x07)) != 0) {
-        int_disable();    //@todo: interrupts should be disabled already??
+        int_disable();    /// @todo interrupts should be disabled already??
         outb(0x60, 0xed);
         while ((inb(0x64) & 0x01) == 0) outb(0x80, 0x21);
@@ -730 +730 @@
         count = 2;
         kbd_code = 0x0;
-        //@todo: Might be better to just mask the KB interrupt
+        /// @todo Might be better to just mask the KB interrupt
         int_disable();
         outb(0x60, 0xf2);
@@ -791 +791 @@
         break;
 
-    //@todo: what's the point of handling this??
+    /// @todo what's the point of handling this??
 #if 0
     case 0x6F:

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

@@ -66 +66 @@
  * @returns    True if keystroke available, False if not.
  */
-//@todo: INT 16h should already be returning the right value in al; could also use setz
+/// @todo INT 16h should already be returning the right value in al; could also use setz
 uint8_t check_for_keystroke(void);
 #pragma aux check_for_keystroke =   \
@@ -92 +92 @@
 
 
-//@todo: This whole business with reprogramming the PIT is rather suspect.
+/// @todo This whole business with reprogramming the PIT is rather suspect.
 // The BIOS already has waiting facilities in INT 15h (fn 83h, 86h) which
 // should be utilized instead.
@@ -107 +107 @@
     modify [ax] nomemory;
 
-//@todo: using this private interface is not great
+/// @todo using this private interface is not great
 extern void rtc_post(void);
 #pragma aux rtc_post "*";

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

@@ -53 +53 @@
 };
 
-// @todo: merge with system.c
+/// @todo merge with system.c
 #define AX      r.gr.u.r16.ax
 #define BX      r.gr.u.r16.bx
@@ -134 +134 @@
 #else
 
-//@todo: merge with AHCI code
+/// @todo merge with AHCI code
 
 /* Warning: Destroys high bits of EAX. */
@@ -323 +323 @@
         AX  = 0x0001;   /* Configuration mechanism #1 supported. */
         BX  = 0x0210;   /* Version 2.1. */
-        //@todo: return true max bus # in CL
+        /// @todo return true max bus # in CL
         CX  = 0;        /* Maximum bus number. */
         EDX = 'P' | ('C' << 8) | ((uint32_t)'I' << 16) | ((uint32_t)' ' << 24);

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

@@ -295 +295 @@
 
 
-//@todo: move
+/// @todo move
 #define ATA_DATA_NO      0x00
 #define ATA_DATA_IN      0x01

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

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

@@ -143 +143 @@
 }
 
-// @todo: the coding style WRT register access is totally inconsistent
+/// @todo the coding style WRT register access is totally inconsistent
 // in the following routines
 

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

@@ -771 +771 @@
     if (IS_MODE_DEC(ch->u8Mode))
     {
-        //@todo: This would need a temporary buffer.
+        /// @todo This would need a temporary buffer.
         Assert(0);
 #if 0

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

@@ -415 +415 @@
      * HACK ALERT! Avoid killing VM with interrupts.
      */
-#if 1 /** @todo: HACK, rethink, may have negative impact on the guest */
+#if 1 /** @todo HACK, rethink, may have negative impact on the guest */
     if (u64Diff == 0)
         u64Diff = 100000; /* 1 millisecond */
@@ -994 +994 @@
 static void hpetR3TimerUpdateIrq(HPET *pThis, struct HPETTIMER *pHpetTimer)
 {
-    /** @todo: is it correct? */
+    /** @todo is it correct? */
     if (   !!(pHpetTimer->u64Config & HPET_TN_ENABLE)
         && !!(pThis->u64HpetConfig & HPET_CFG_ENABLE))
@@ -1011 +1011 @@
         else
             AssertFailed();
-        /** @todo: implement IRQs in level-triggered mode */
+        /** @todo implement IRQs in level-triggered mode */
     }
 }