Remove unused defs

[rgblamp.git] / main.c

diff --git a/main.c b/main.c
index dc14185632af7ab30b90057d32ece16513376556..b09dcc44021597bfa6ac2c37cfbdba3f77734043 100644 (file)
--- a/main.c
+++ b/main.c
@@ -46,14 +46,6 @@
 #include "task.h"
 #include "adc_random.h"
 
-//#define AUTO_OFF_COUNT   150  /* 5 mins on*/
-//#define AUTO_ON_COUNT    300  /* 10 mins off */
-//#define AUTO_OFF_COUNT   450  /* 15 mins on*/
-//#define AUTO_ON_COUNT    1350 /* 45 mins off */
-//#define AUTO_OFF_COUNT   600  /* 20 mins on*/
-//#define AUTO_ON_COUNT    1200 /* 40 mins off */
-//#define AUTO_OFF_COUNT   450  /* 2 hours on*/
-//#define AUTO_ON_COUNT    1350 /* 22 hours off */
 #define AUTO_OFF_COUNT   9000U  /*  5 hrs in 2 sec units */
 #define AUTO_ON_COUNT    34200U /* 19 hrs in 2 sec units */
 #define dbgpin_init()    do { \
@@ -65,19 +57,6 @@
 #define dbgpin_low()  (RA2 = 0)
 #define dbgpin_toggle()  (RA2 = (LATA2 == 0) ? 1 : 0)
 
-#define cfg_write(mode, color) do { \
-    eeprom_write(CFG_MODE_ADDR, (mode)); \
-    eeprom_write(CFG_COLOR_ADDR, (color)); \
-  } while (0)
-
-#define cfg_read(mode, color) do { \
-    unsigned char tmp; \
-    tmp = eeprom_read(CFG_MODE_ADDR); \
-    mode = (tmp < MODE_COUNT) ? tmp : MODE_SOLID; \
-    tmp = eeprom_read(CFG_COLOR_ADDR); \
-    color = (tmp < COLOR_COUNT) ? tmp : 0; \
-  } while (0)
-
 enum {
   /* Operating modes */
   MODE_SOLID = 0,        /* Cycle through colors[][] before next mode */
@@ -107,9 +86,8 @@ enum {
   PARTY_MIN = 8,         /* Min party fade and incolor units */
   PARTY_RANGE = 8,       /* Party fade/incolor range mask for lfsr_get() */
                          /* ... see start_fade() */
-  CFG_MODE_ADDR = 0,     /* EEPROM address of mode variable */
-  CFG_COLOR_ADDR,        /* EEPROM address of color variable */
-  CFG_DELAY = 92,        /* 3 seconds in 32.768 msec units */
+  CFG_ADDR = 0,          /* EEPROM address of start of configuration */
+  CFG_DELAY = 183,        /* 6 seconds in 32.768 msec units */
 };
 
 typedef struct {
@@ -135,16 +113,51 @@ const unsigned char colors[COLOR_COUNT][LED_COUNT] = {
 
 unsigned char mode = MODE_SOLID;
 unsigned char color = COLOR_WHITE;
+signed char bright = BRIGHT_INIT;
+bit bright_up;
 led_t red;
 led_t grn;
 led_t blu;
 led_t wht;
 bit on;
 int fade_steps;
-signed char bright = BRIGHT_INIT;
-bit bright_up;
 signed char pbHeldCount;
 
+void cfg_write()
+{
+  unsigned char addr = CFG_ADDR;
+  unsigned char m, c;
+  signed char b;
+
+  m = eeprom_read(addr++);
+  c = eeprom_read(addr++);
+  b = eeprom_read(addr);
+  if (mode != m || color != c || bright != b) {
+    addr = CFG_ADDR;
+    eeprom_write(addr++, mode);
+    eeprom_write(addr++, color);
+    eeprom_write(addr, bright);
+  }
+}
+
+void cfg_read()
+{
+  unsigned char addr = CFG_ADDR;
+  unsigned char tmp;
+
+  tmp = eeprom_read(addr++);
+  if (tmp < MODE_COUNT)
+    mode = tmp;
+
+  tmp = eeprom_read(addr++);
+  if (tmp < COLOR_COUNT)
+    color = tmp;
+
+  tmp = eeprom_read(addr);
+  if ((signed char)tmp >= BRIGHT_BOTTOM && (signed char)tmp <= BRIGHT_TOP)
+    bright = tmp;
+}
+
 /* Combine LED color value and brightness level to generate an RGB value.
  * bright will be BRIGHT_MIN...BRIGHT_MAX
  *
@@ -235,8 +248,6 @@ void turnOff()
 void pb_clicked()
 {
   if (on) {
-    unsigned char omode = mode;
-
     if (mode == MODE_SOLID) {
       if (++color == COLOR_COUNT) {
         color = 0;
@@ -247,8 +258,7 @@ void pb_clicked()
       mode = 0;
       color = 0;
     }
-    if (mode != omode || mode == MODE_SOLID)
-      tmr_start(TMR_CFG, CFG_DELAY);
+    tmr_start(TMR_CFG, CFG_DELAY);
     start_fade();
   }
 }
@@ -267,6 +277,7 @@ void pb_held()
     else
       bright--;
   }
+  tmr_start(TMR_CFG, CFG_DELAY);
   leds_set();
 }
 
@@ -360,7 +371,7 @@ void user_boot()
 {
   dbgpin_high();
   lfsr_init((adc_random() << 8) + adc_random());
-  cfg_read(mode, color);
+  cfg_read();
   rs_task();
 }
 
@@ -389,7 +400,7 @@ void user_tasks(unsigned char block)
         auto_offon_task();
         break;
       case TASK_CFG:            /* Save config to EEPROM event */
-        cfg_write(mode, color);
+        cfg_write();
         break;
     }
   }