Clarifies minor PIT comments
authorBarret Rhoden <brho@cs.berkeley.edu>
Mon, 9 Dec 2013 17:55:01 +0000 (09:55 -0800)
committerBarret Rhoden <brho@cs.berkeley.edu>
Thu, 16 Jan 2014 21:07:52 +0000 (13:07 -0800)
We only ever use PIT timer 1, which has 0, 1, and 2 counters.  Some of
the comments made it harder to verify that.

kern/arch/x86/apic.c
kern/arch/x86/apic.h

index a55fd11..37939c0 100644 (file)
@@ -262,7 +262,7 @@ static int getpit()
     int high, low;
        // TODO: need a lock to protect access to PIT
 
-    /* Select timer0 and latch counter value. */
+    /* Select counter 0 and latch counter value. */
     outb(TIMER_MODE, TIMER_SEL0 | TIMER_LATCH);
     
     low = inb(TIMER_CNTR0);
index 91aeaf7..c7473a3 100644 (file)
@@ -86,9 +86,9 @@
 #define LAPIC_IRR                                      (LAPIC_BASE + 0x200)
 
 // PIT (Programmable Interval Timer)
-#define        TIMER_REG_CNTR0 0       /* timer 0 counter port */
-#define        TIMER_REG_CNTR1 1       /* timer 1 counter port */
-#define        TIMER_REG_CNTR2 2       /* timer 2 counter port */
+#define        TIMER_REG_CNTR0 0       /* timer counter 0 port */
+#define        TIMER_REG_CNTR1 1       /* timer counter 1 port */
+#define        TIMER_REG_CNTR2 2       /* timer counter 2 port */
 #define        TIMER_REG_MODE  3       /* timer mode port */
 #define        TIMER_SEL0      0x00    /* select counter 0 */
 #define        TIMER_SEL1      0x40    /* select counter 1 */