client: debounce

diff --git a/client/config.c b/client/config.c
index a8e58440dff1b24d52829f6f8989d9afc0b332ac..c5a38b44ad6afbf1b96531ec7d3eee7109da9826 100644 (file)
--- a/client/config.c
+++ b/client/config.c
@@ -17,3 +17,7 @@ u16_t this_tattler_identity = 1;
  * https://projects.raspberrypi.org/en/projects/introduction-to-the-pico/10
  * recommends pins 18, 22, or 28. */
 uint button_pin = 18;
+
+/* Don't bother reporting each separate button press when it is pressed many
+ * times in short succession.  (We also use this to de-bounce. :) */
+u32_t minimum_microseconds_between_button_presses = 1000000;

diff --git a/client/config.h b/client/config.h
index 95c5651b27ec5881b3d5ebaedd2e0181359f6fb6..5ce0baf9708f009a7069f3aca2ae738f614dd973 100644 (file)
--- a/client/config.h
+++ b/client/config.h
@@ -21,4 +21,8 @@ extern u16_t this_tattler_identity;
  * recommends pins 18, 22, or 28. */
 extern uint button_pin;
 
+/* Don't bother reporting each separate button press when it is pressed many
+ * times in short succession.  (We also use this to de-bounce. :) */
+extern u32_t minimum_microseconds_between_button_presses;
+
 #endif

diff --git a/client/tattlekey.c b/client/tattlekey.c
index 45059f6ed2e832b8457c64f4696640d0a6c44e1a..ff61f430bb2b978a298bfa31238f3ffe22fd5850 100644 (file)
--- a/client/tattlekey.c
+++ b/client/tattlekey.c
@@ -11,10 +11,16 @@ queue_t queue;
 
 static void button_pressed() {
   /* This runs in interrupt context; don't linger.  */
-  char zero = '\0';
-  /* We don't check for failure (full queue) here because there's not much to be
-   * done about it. */
-  queue_try_add(&queue, &zero);
+  static uint64_t last_button_press_time = 0;
+  uint64_t now = time_us_64();
+  uint64_t time_since_last_press = now - last_button_press_time;
+  if (time_since_last_press > minimum_microseconds_between_button_presses) {
+    last_button_press_time = now;
+    char zero = '\0';
+    /* We don't check for failure (full queue) here because there's not much to
+     * be done about it. */
+    queue_try_add(&queue, &zero);
+  }
 }
 
 int main() {