Deeper sleep when idle

diff --git a/projects/step-up/adc.c b/projects/step-up/adc.c
index ea309357a429ad988b26960e3e1c381bda7248a9..55cd8610ab441cacc7663ddc8e605dde49869d7c 100644 (file)
--- a/projects/step-up/adc.c
+++ b/projects/step-up/adc.c
@@ -10,8 +10,9 @@
 //#define NUM_ADCS     ZERO_ADC
 #define NUM_ADCS       2
 
-volatile static unsigned char current_adc;;
-volatile unsigned char need_battery_adc;
+volatile static unsigned char current_adc;
+volatile unsigned char adc_enabled;
+volatile unsigned char need_battery_adc, need_pwmled_adc;
 static uint16_t adc_sum, read_zero, drop_count, read_count, n_reads_log;
 volatile uint16_t jiffies;
 
@@ -40,13 +41,23 @@ void start_next_adc()
                drop_count = 1;
                read_count = 1;
                n_reads_log = 0;
-       } else {
+       } else if (need_pwmled_adc) {
                current_adc = 0;
                read_zero = 0;
                drop_count = 1;
                read_count = 1 << PWMLED_ADC_SHIFT;
                n_reads_log = PWMLED_ADC_SHIFT;
+       } else {
+               ADCSRA &= ~_BV(ADEN);
+               power_adc_disable();
+               adc_enabled = 0;
+               return;
+       }
 
+       if (!adc_enabled) {
+               power_adc_enable();
+               ADCSRA |= _BV(ADEN);
+               adc_enabled = 1;
        }
 
        adc_sum = 0;
@@ -95,7 +106,9 @@ static uint16_t read_adc_sync()
 void init_adc()
 {
        need_battery_adc = 0;
+       need_pwmled_adc = 0;
        current_adc = 0;
+       adc_enabled = 1;
 
        power_adc_enable();
        ACSR |= _BV(ACD);       // but disable the analog comparator
@@ -124,6 +137,8 @@ void susp_adc()
 {
        ADCSRA = 0;
        DIDR0 = 0;
+       power_adc_disable();
+       adc_enabled = 0;
 }
 
 ISR(ADC_vect) { // IRQ handler

diff --git a/projects/step-up/battery.c b/projects/step-up/battery.c
index ceb53cb2c1133eecb104b607fd79d67f15cfbbee..0e51cb14e64a1975ad7fbdc0631c35b668c612cb 100644 (file)
--- a/projects/step-up/battery.c
+++ b/projects/step-up/battery.c
@@ -77,8 +77,10 @@ unsigned char battery_gauge()
                rv = 7;
        }
 
+#if 0
        if (rv == 1 && !initial_readings)
                set_error(ERR_BATTERY);
+#endif
 
 #if 0
        log_byte(0xbb);

diff --git a/projects/step-up/lights.h b/projects/step-up/lights.h
index 2980cd3678b06c0eba88c649ebdcea8c6a56f1de..4a0b6b9753fdab7d6e088e18324b40e83187fba4 100644 (file)
--- a/projects/step-up/lights.h
+++ b/projects/step-up/lights.h
@@ -21,13 +21,14 @@ void inline log_word(uint16_t word) { }
 /* adc.c */
 #define PWMLED_ADC_SHIFT 1 /* 1<<1 measurements per single callback */
 extern volatile unsigned char need_battery_adc;
-extern volatile unsigned char adc_running;
+extern volatile unsigned char need_pwmled_adc;
+extern volatile unsigned char adc_enabled;
 void init_adc();
 void susp_adc();
 
 /* pwm.c */
 #define PWM_MAX 0xFF
-extern volatile unsigned char pwm_running;
+extern volatile unsigned char pwm_enabled;
 void init_pwm();
 void susp_pwm();
 void pwm_off();

diff --git a/projects/step-up/main.c b/projects/step-up/main.c
index c1ee381453a334bf43493ddb93e287f9fa2bea62..25d80df070bb4bf7269e89c6ee50792fdb245612 100644 (file)
--- a/projects/step-up/main.c
+++ b/projects/step-up/main.c
@@ -72,13 +72,15 @@ int main(void)
 #if 1
        while (1) {
                cli();
-               if (pwm_running) {
+               if (pwm_enabled) {
                        set_sleep_mode(SLEEP_MODE_IDLE);
-               } else if (adc_running) {
+               } else if (adc_enabled) {
                        set_sleep_mode(SLEEP_MODE_ADC);
                } else {
                        set_sleep_mode(SLEEP_MODE_PWR_DOWN);
                }
+
+               sleep_enable();
                // keep BOD active, no sleep_bod_disable();
                sei();
                sleep_cpu();

diff --git a/projects/step-up/pwm.c b/projects/step-up/pwm.c
index 42ed777357eba68b6d8853e28c92891bd5298e06..b93a4fab8502039d73d811c69b96592725c5bad5 100644 (file)
--- a/projects/step-up/pwm.c
+++ b/projects/step-up/pwm.c
@@ -11,7 +11,7 @@
  * Counts from 0 to 0xFF, without OCR1C compare.
  */
 
-static unsigned char pwm_enabled;
+volatile unsigned char pwm_enabled;
 
 static void inline enable_pll()
 {
@@ -57,6 +57,7 @@ void pwm_off()
        DDRB &= ~_BV(PB4);
 
        PLLCSR &= ~(_BV(PLLE) | _BV(PCKE));
+       power_timer1_disable();
        pwm_enabled = 0;
 }
 
@@ -65,6 +66,7 @@ void pwm_set(uint8_t stride)
        OCR1B = stride;
 
        if (!pwm_enabled) {
+               power_timer1_enable();
                enable_pll();
                DDRB |= _BV(PB4);
                pwm_enabled = 1;

diff --git a/projects/step-up/pwmled.c b/projects/step-up/pwmled.c
index bc604da70063a0c63ec3528294563128c03308db..485d024d8f0719007113f36b82f903189442c660 100644 (file)
--- a/projects/step-up/pwmled.c
+++ b/projects/step-up/pwmled.c
@@ -60,9 +60,11 @@ void pwmled_on_off(unsigned char mode)
        if (mode) {
                state = ST_ON;
                mode_changed = 1;
+               need_pwmled_adc = 1;
                pwm_set(pwm_val);
        } else {
                state = ST_OFF;
+               need_pwmled_adc = 0;
                pwm_off();
        }
 }
@@ -113,6 +115,7 @@ void pwmled_adc(uint16_t adcval)
        if (pwm_val >= PWM_MAX
                || (pwm_val > (2*PWM_MAX/3) && adcval < 0x08)) {
                pwmled_err();
+               need_pwmled_adc = 0;
                return;
        }
 

diff --git a/projects/step-up/wdt.c b/projects/step-up/wdt.c
index 15bfbbb54f1374069ea6bd01d1e4b44b8247fffc..cb94ec8bc5260b054170404846ef50f61ead66ea 100644 (file)
--- a/projects/step-up/wdt.c
+++ b/projects/step-up/wdt.c
@@ -17,10 +17,16 @@ void susp_wdt()
 ISR(WDT_vect) {
        ++jiffies;
 
+       if (pwm_enabled) {
+               need_pwmled_adc = 1;
+       }
+
        if (jiffies & 0x000F) {
                need_battery_adc = 1; // about every 1s
        }
 
        patterns_next_tick();
        timer_check_buttons();
+       if (!adc_enabled)
+               start_next_adc(); // only if not running
 }