PWM LEDs driven by a single pattern

[bike-lights.git] / firmware / buttons.c

diff --git a/firmware/buttons.c b/firmware/buttons.c
index 80ecce603c65503ad940ad05b6edab7b98bb099e..ac41c06879b46a01c2cbae7ef19a321b271332a2 100644 (file)
--- a/firmware/buttons.c
+++ b/firmware/buttons.c
@@ -37,7 +37,7 @@ static uint16_t user_params_starttime;
 
 static void inline set_status_led(unsigned char n, pattern_t *pattern)
 {
-       led_set_pattern(n + N_PWMLEDS, pattern);
+       led_set_pattern(n + 1, pattern);
 }
 
 unsigned char buttons_setup_in_progress()
@@ -111,7 +111,7 @@ static inline void short_press(unsigned char button)
 static inline void long_press(unsigned char button)
 {
        if (button == 0) {
-               power_down();
+               power_down(0);
                return;
        }
 
@@ -189,30 +189,47 @@ static void handle_button(unsigned char button, unsigned char cur,
 
 static void handle_brake(unsigned char cur, unsigned char prev)
 {
+       uint16_t duration;
+
        if (cur && !prev) {                   // --- just pressed ---
                button_start[2] = jiffies;
+               return;
        } else if (!cur && prev) {            // --- just released ---
                button_start[2] = jiffies;
-       } else {                              // --- no change ---
-               uint16_t duration = jiffies - button_start[2];
-
-               if (duration > 6) {
-                       if (cur) {
-                               if (button_state.brake_working
-                                       && !button_state.brake_reported) {
-                                       button_state.brake_reported = 1;
-                                       brake_on();
-                               }
-                       } else {
-                               button_state.brake_working = 1;
-                               if (button_state.brake_reported) {
-                                       button_state.brake_reported = 0;
+               return;
+       }
+                                             // --- no change ---
+       duration = jiffies - button_start[2];
+
+       if (duration <= 3)
+               return;
+
+       if (cur) {
+               if (button_state.brake_working) {
+                       static unsigned int brake_time;
+                       if (button_state.brake_reported) {
+                               if (brake_time) {
+                                       brake_time--;
+                               } else {
                                        brake_off();
+                                       button_state.brake_working = 0;
+                                       button_state.brake_reported = 0;
                                }
+                       } else {
+                               button_state.brake_reported = 1;
+                               brake_on();
+                               brake_time = 255; // avoid longer than ~16s
                        }
-                       button_start[2] = jiffies - 7; // avoid overflow
+               }
+       } else {
+               button_state.brake_working = 1;
+               if (button_state.brake_reported) {
+                       button_state.brake_reported = 0;
+                       brake_off();
                }
        }
+
+       button_start[2] = jiffies - 7; // avoid overflow
 }
 
 void timer_check_buttons()