new button lib

include/button.h

@@ -1,7 +1,7 @@
 #ifndef _BUTTON_H
 #define _BUTTON_H
 
-#include <SimpleButton.h>
+#include <OneButton.h>
 #include "irqhandler.h"
 #include "senddata.h"
 #include "display.h"
@@ -11,7 +11,13 @@
 #define BUTTON_ACTIVEHIGH 0
 #endif
 
-void button_init(int pin);
-void readButton();
+#ifndef BUTTON_PULLUP
+#define BUTTON_PULLUP 0
+#endif
+
+extern TaskHandle_t buttonLoopTask;
+
+void button_init(void);
+void IRAM_ATTR readButton(void);
 
 #endif

include/cyclic.h

@@ -13,6 +13,7 @@
 #include "reset.h"
 #include "led.h"
 #include "power.h"
+#include "button.h"
 
 extern Ticker cyclicTimer;
 

include/irqhandler.h

@@ -36,12 +36,4 @@ void IRAM_ATTR DisplayIRQ();
 void IRAM_ATTR MatrixDisplayIRQ();
 #endif
 
-#ifdef HAS_BUTTON
-void IRAM_ATTR ButtonIRQ();
-#endif
-
-#ifdef HAS_PMU
-void IRAM_ATTR PMUIRQ();
-#endif
-
 #endif

platformio_orig.ini

@@ -81,7 +81,7 @@ lib_deps_basic =
     https://github.com/SukkoPera/Arduino-Rokkit-Hash.git
     bblanchon/ArduinoJson @ ^6
     makuna/RTC @ ^2.3.5
-    spacehuhn/SimpleButton
+    mathertel/OneButton @ ^2.0.3
     lewisxhe/XPowersLib @ ^0.1.4
     256dpi/MQTT @ ^2.5.0
 lib_deps_all =

platformio_orig_s3.ini

@@ -29,7 +29,7 @@ lib_deps_all =
     https://github.com/cyberman54/libpax.git
     https://github.com/SukkoPera/Arduino-Rokkit-Hash.git
     bblanchon/ArduinoJson @ ^6
-    spacehuhn/SimpleButton
+    mathertel/OneButton @ ^2.0.3
     256dpi/MQTT @ ^2.5.0
     ricmoo/QRCode @ ^0.0.1
 build_flags_basic =

src/button.cpp

@@ -3,42 +3,49 @@
 #include "globals.h"
 #include "button.h"
 
-using namespace simplebutton;
-
 // Local logging tag
 static const char TAG[] = __FILE__;
 
-static Button *b = NULL;
+OneButton button(HAS_BUTTON, !BUTTON_ACTIVEHIGH, !!BUTTON_PULLUP);
+TaskHandle_t buttonLoopTask;
 
-void button_init(int pin) {
-#ifdef BUTTON_PULLUP
-  b = new ButtonPullup(pin);
-#else
-  b = new Button(pin, !BUTTON_ACTIVEHIGH);
-#endif
+void IRAM_ATTR readButton(void) { button.tick(); }
 
-  // attach events to the button
-
-  b->setOnDoubleClicked([]() {});
-
-  b->setOnClicked([]() {
+void singleClick(void) {
 #ifdef HAS_DISPLAY
   dp_refresh(true); // switch to next display page
 #endif
 #ifdef HAS_MATRIX_DISPLAY
   refreshTheMatrixDisplay(true); // switch to next display page
 #endif
-  });
+}
 
-  b->setOnHolding([]() {
+void longPressStart(void) {
   payload.reset();
   payload.addButton(0x01);
   SendPayload(BUTTONPORT);
-  });
-
-  // attach interrupt to the button
-  attachInterrupt(digitalPinToInterrupt(pin), ButtonIRQ, CHANGE);
 }
 
-void readButton() { b->update(); }
+void buttonLoop(void *parameter) {
+  while (1) {
+    doIRQ(BUTTON_IRQ);
+    delay(20);
+  }
+}
+
+void button_init(void) {
+  ESP_LOGI(TAG, "Starting button Controller...");
+  xTaskCreatePinnedToCore(buttonLoop,      // task function
+                          "buttonloop",    // name of task
+                          2048,            // stack size of task
+                          (void *)1,       // parameter of the task
+                          2,               // priority of the task
+                          &buttonLoopTask, // task handle
+                          1);              // CPU core
+
+  button.setPressTicks(1000);
+  button.attachClick(singleClick);
+  button.attachLongPressStart(longPressStart);
+};
+
 #endif

src/cyclic.cpp

@@ -71,6 +71,13 @@ void doHousekeeping() {
              eTaskGetState(ledLoopTask));
 #endif
 
+#ifdef HAS_BUTTON
+  if (buttonLoopTask != NULL)
+    ESP_LOGD(TAG, "Buttonloop %d bytes left | Taskstate = %d",
+             uxTaskGetStackHighWaterMark(buttonLoopTask),
+             eTaskGetState(buttonLoopTask));
+#endif
+
 // read battery voltage into global variable
 #if (defined BAT_MEASURE_ADC || defined HAS_PMU || defined HAS_IP5306)
   batt_level = read_battlevel();

src/irqhandler.cpp

@@ -102,14 +102,6 @@ void IRAM_ATTR DisplayIRQ() { doIRQ(DISPLAY_IRQ); }
 void IRAM_ATTR MatrixDisplayIRQ() { doIRQ(MATRIX_DISPLAY_IRQ); }
 #endif
 
-#ifdef HAS_BUTTON
-void IRAM_ATTR ButtonIRQ() { doIRQ(BUTTON_IRQ); }
-#endif
-
-#ifdef HAS_PMU
-void IRAM_ATTR PMUIRQ() { doIRQ(PMU_IRQ); }
-#endif
-
 void mask_user_IRQ() { xTaskNotify(irqHandlerTask, MASK_IRQ, eSetBits); }
 
 void unmask_user_IRQ() { xTaskNotify(irqHandlerTask, UNMASK_IRQ, eSetBits); }

src/main.cpp

@@ -28,6 +28,7 @@ licenses. Refer to LICENSE.txt file in repository for more details.
 Task          	Core  Prio  Purpose
 -------------------------------------------------------------------------------
 ledloop*      	1     1    blinks LEDs
+buttonloop*     1     2    reads button
 spiloop#      	0     2    reads/writes data on spi interface
 lmictask*     	1     2    MCCI LMiC LORAWAN stack
 clockloop#    	1     6    generates realtime telegrams for external clock
@@ -468,7 +469,7 @@ void setup() {
 #else
   strcat_P(features, "PD");
 #endif // BUTTON_PULLUP
-  button_init(HAS_BUTTON);
+  button_init();
 #endif // HAS_BUTTON
 
 // only if we have a timesource we do timesync

src/power.cpp

@@ -25,6 +25,8 @@ static const adc_unit_t unit = ADC_UNIT_1;
 #ifdef HAS_PMU
 XPowersPMU pmu;
 
+void IRAM_ATTR PMUIRQ() { doIRQ(PMU_IRQ); }
+
 void AXP192_powerevent_IRQ(void) {
   pmu.getIrqStatus();