Verify EEPROM has changed before update

[rgblamp.git] / main.c

diff --git a/main.c b/main.c
index dc14185632af7ab30b90057d32ece16513376556..bfc6b1e6d367348390eec8bd17575e71a1d3617c 100644 (file)
--- a/main.c
+++ b/main.c
@@ -65,19 +65,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 +94,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 +121,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 +256,6 @@ void turnOff()
 void pb_clicked()
 {
   if (on) {
-    unsigned char omode = mode;
-
     if (mode == MODE_SOLID) {
       if (++color == COLOR_COUNT) {
         color = 0;
@@ -247,8 +266,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 +285,7 @@ void pb_held()
     else
       bright--;
   }
+  tmr_start(TMR_CFG, CFG_DELAY);
   leds_set();
 }
 
@@ -360,7 +379,7 @@ void user_boot()
 {
   dbgpin_high();
   lfsr_init((adc_random() << 8) + adc_random());
-  cfg_read(mode, color);
+  cfg_read();
   rs_task();
 }
 
@@ -389,7 +408,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;
     }
   }