Simplify clock device implementation

Two pairs of generic get/set functions now do the role of the previous
pair of custom get/set functions for each timer.

2 files changed, 12 insertions, 55 deletions

diff --git a/arm9/source/devices/clock.c b/arm9/source/devices/clock.c
index 6f3ce85..6ecee1d 100644
--- a/arm9/source/devices/clock.c
+++ b/arm9/source/devices/clock.c
@@ -11,22 +11,10 @@ void uptime_handler(void) { uptime++; }
 // Check if any timer has expired.
 bool check_timers(ClockDevice *clk) {
     bool output = FALSE;
-    if (clk->t1.end && clk->t1.end <= uptime) {
-        clk->t1.end = 0;
-        output = TRUE;
-    }
-    if (clk->t2.end && clk->t2.end <= uptime) {
-        clk->t2.end = 0;
-        output = TRUE;
-    }
-    if (clk->t3.end && clk->t3.end <= uptime) {
-        clk->t3.end = 0;
-        output = TRUE;
-    }
-    if (clk->t4.end && clk->t4.end <= uptime) {
-        clk->t4.end = 0;
-        output = TRUE;
-    }
+    if (clk->t1.end && clk->t1.end <= uptime) { clk->t1.end = 0; output = TRUE; }
+    if (clk->t2.end && clk->t2.end <= uptime) { clk->t2.end = 0; output = TRUE; }
+    if (clk->t3.end && clk->t3.end <= uptime) { clk->t3.end = 0; output = TRUE; }
+    if (clk->t4.end && clk->t4.end <= uptime) { clk->t4.end = 0; output = TRUE; }
     return output;
 }
 
@@ -34,8 +22,7 @@ u8 get_timer_high(ClockTimer *t) {
     if (t->end > uptime) {
         t->read = t->end - uptime;
     } else {
-        t->end = 0;
-        t->read = 0;
+        t->end = 0; t->read = 0;
     }
     return HIGH(t->read);
 }
@@ -56,25 +43,6 @@ void set_timer_low(ClockTimer *t, u8 low) {
     }
 }
 
-u8 get_timer1_high(ClockDevice *clock) { return get_timer_high(&clock->t1); }
-u8 get_timer1_low( ClockDevice *clock) { return get_timer_low( &clock->t1); }
-u8 get_timer2_high(ClockDevice *clock) { return get_timer_high(&clock->t2); }
-u8 get_timer2_low( ClockDevice *clock) { return get_timer_low( &clock->t2); }
-u8 get_timer3_high(ClockDevice *clock) { return get_timer_high(&clock->t3); }
-u8 get_timer3_low( ClockDevice *clock) { return get_timer_low( &clock->t3); }
-u8 get_timer4_high(ClockDevice *clock) { return get_timer_high(&clock->t4); }
-u8 get_timer4_low( ClockDevice *clock) { return get_timer_low( &clock->t4); }
-
-void set_timer1_high(ClockDevice *clock, u8 high) { set_timer_high(&clock->t1, high); }
-void set_timer1_low( ClockDevice *clock, u8  low) { set_timer_low( &clock->t1,  low); }
-void set_timer2_high(ClockDevice *clock, u8 high) { set_timer_high(&clock->t2, high); }
-void set_timer2_low( ClockDevice *clock, u8  low) { set_timer_low( &clock->t2,  low); }
-void set_timer3_high(ClockDevice *clock, u8 high) { set_timer_high(&clock->t3, high); }
-void set_timer3_low( ClockDevice *clock, u8  low) { set_timer_low( &clock->t3,  low); }
-void set_timer4_high(ClockDevice *clock, u8 high) { set_timer_high(&clock->t4, high); }
-void set_timer4_low( ClockDevice *clock, u8  low) { set_timer_low( &clock->t4,  low); }
-
-
 void init_clock(void) {
     // Start a 256Hz timer to increment the uptime value.
     timerStart(0, ClockDivider_1024, TIMER_FREQ_1024(256), uptime_handler);
diff --git a/arm9/source/devices/clock.h b/arm9/source/devices/clock.h
index 892479f..04c4d5d 100644
--- a/arm9/source/devices/clock.h
+++ b/arm9/source/devices/clock.h
@@ -1,8 +1,8 @@
+#include <time.h>
+
 #ifndef CLOCK_H_
     #define CLOCK_H_
 
-    #include <time.h>
-
     typedef struct {
         u32 end;            // real end time
         u16 read, write;    // read write caches
@@ -27,20 +27,9 @@
 
     bool check_timers(ClockDevice *clk);
 
-    u8 get_timer1_high(ClockDevice *clock);
-    u8 get_timer1_low( ClockDevice *clock);
-    u8 get_timer2_high(ClockDevice *clock);
-    u8 get_timer2_low( ClockDevice *clock);
-    u8 get_timer3_high(ClockDevice *clock);
-    u8 get_timer3_low( ClockDevice *clock);
-    u8 get_timer4_high(ClockDevice *clock);
-    u8 get_timer4_low( ClockDevice *clock);
-    void set_timer1_high(ClockDevice *clock, u8 high);
-    void set_timer1_low( ClockDevice *clock, u8  low);
-    void set_timer2_high(ClockDevice *clock, u8 high);
-    void set_timer2_low( ClockDevice *clock, u8  low);
-    void set_timer3_high(ClockDevice *clock, u8 high);
-    void set_timer3_low( ClockDevice *clock, u8  low);
-    void set_timer4_high(ClockDevice *clock, u8 high);
-    void set_timer4_low( ClockDevice *clock, u8  low);
+    u8 get_timer_high(ClockTimer *t);
+    u8 get_timer_low( ClockTimer *t);
+    void set_timer_high(ClockTimer *t, u8 high);
+    void set_timer_low( ClockTimer *t, u8  low);
+
 #endif