#include "lights.h"
#define WAKEUP_LIMIT 5 // times 100 ms
+#define SHORT_PRESS_MIN 2 // in jiffies (16 Hz ticks)
+#define SHORT_PRESS_MAX 5
+#define LONG_PRESS_MIN 10
+
static uint16_t button_start[3];
union {
unsigned char all;
unsigned char btn1 : 1;
unsigned char btn2 : 1;
unsigned char brake : 1;
+ unsigned char brake_working : 1;
+ unsigned char brake_reported : 1;
};
} button_state, prev_state;
} else if (cur && prev) { // --- is still pressed ---
uint16_t duration = jiffies - button_start[button];
- if (duration > 160) {
+ if (duration >= LONG_PRESS_MIN) {
set_status_led(button, on1_pattern);
// acknowledge long press
}
} else if (!cur && prev) { // --- just released ---
uint16_t duration = jiffies - button_start[button];
- if (duration > 6 && duration < 60) {
+ if (duration >= SHORT_PRESS_MIN && duration < SHORT_PRESS_MAX) {
short_press(button);
- } else if (duration > 160) {
+ } else if (duration > LONG_PRESS_MIN) {
set_status_led(button, NULL);
long_press(button);
}
{
if (cur && !prev) { // --- just pressed ---
button_start[2] = jiffies;
- } else if (cur && prev) { // --- is still pressed ---
+ } else if (!cur && prev) { // --- just released ---
+ button_start[2] = jiffies;
+ } else { // --- no change ---
uint16_t duration = jiffies - button_start[2];
if (duration > 6) {
- brake_on();
+ 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;
+ brake_off();
+ }
+ }
button_start[2] = jiffies - 7; // avoid overflow
}
- } else if (!cur && prev) { // --- just released ---
- brake_off();
}
}
void timer_check_buttons()
{
handle_button(0, button_state.btn1, prev_state.btn1);
- handle_button(1, button_state.btn2, prev_state.btn2);
- handle_brake(button_state.brake, prev_state.brake);
- prev_state.all = button_state.all;
+ // when button 1 is on, all other states are unreadable
+ if (button_state.btn1) {
+ prev_state.btn1 = button_state.btn1;
+ } else {
+ handle_button(1, button_state.btn2, prev_state.btn2);
+ handle_brake(button_state.brake, prev_state.brake);
+ prev_state.all = button_state.all;
+ }
if (user_params_state && jiffies - user_params_starttime > 1000) {
user_params_state = 0;
PORTA &= ~_BV(PA3); // disable +5V to the hall probe
// adc.c will re-enable it again
- button_state.all = 0;
+ button_state.btn1 = 0;
+ button_state.btn2 = 0;
+ button_state.brake = 0;
+
if (adcval == 0x3FF) {
button_state.brake = 1;
} else if (adcval > 0x240) {