button support and binding procedure updated

Author: fabiankuffer

.vscode/settings.json

@@ -12,6 +12,8 @@
         "console.h": "c",
         "esp_nimble_hci.h": "c",
         "stdio.h": "c",
-        "freertosconfig.h": "c"
+        "freertosconfig.h": "c",
+        "inttypes.h": "c",
+        "glue.h": "c"
     }
 }

components/ble/CMakeLists.txt

@@ -2,5 +2,5 @@ set(srcs "gap.c" "gatt.c" "ble.c")
 
 idf_component_register(SRCS "${srcs}"
                     INCLUDE_DIRS "include"
-                    REQUIRES bt ssd1306 crsf nvs_flash battery
+                    REQUIRES bt ssd1306 crsf nvs_flash battery button
                     PRIV_REQUIRES )

components/ble/gap.c

@@ -194,18 +194,24 @@ int bleGAPEevent(struct ble_gap_event *event, void *arg) {
         ESP_LOGI(tag_GAP, "PASSKEY_ACTION_EVENT started \n");
 
         struct ble_sm_io pkey = {0};
-        int key = 0;
+        BUTTON selectedButton = RIGHTBUTTON;
 
         if (event->passkey.params.action == BLE_SM_IOACT_DISP) {
             //BLE_SM_IO_CAP_DISP_ONLY option
             pkey.action = event->passkey.params.action;
-            pkey.passkey = 123456;
 
+            pkey.passkey = esp_random();
+	        /* Max value is 999999 */
+	        pkey.passkey %= 1000000;
+            
+            char pinStr[12];
+            snprintf(pinStr, 12,"%06d", (int)pkey.passkey);
+
             ssd1306_clear();
             ssd1306_setString("PIN",48,0);
-            ssd1306_setString("123456",31,18);
+            ssd1306_setString(pinStr,31,18);
             ssd1306_display();
-            //display passkey for the other side 123456
+            //display passkey for the other side
 
             rc = ble_sm_inject_io(event->passkey.conn_handle, &pkey);
             ESP_LOGI(tag_GAP, "ble_sm_inject_io result: %d\n", rc);
@@ -215,22 +221,35 @@ int bleGAPEevent(struct ble_gap_event *event, void *arg) {
             //gegenseite kann auch mit yes oder no bestätigen den pin
             //verify with buttons Yes or No --> add timeout after x seconds
             char pinStr[12];
-            snprintf(pinStr, 12,"%d", (int)event->passkey.params.numcmp);
+            snprintf(pinStr, 12,"%06d", (int)event->passkey.params.numcmp);
             ssd1306_clear();
             ssd1306_setString(pinStr,0,0);
             ssd1306_setString("ja",0,18);
             ssd1306_setString("nein",128-43,18);
             ssd1306_display();
 
             pkey.action = event->passkey.params.action;
-            if (true) {
-                pkey.numcmp_accept = true;
+
+            clearStoredButtons();
+            if (getButton(&selectedButton)) {
+                if(selectedButton == LEFTBUTTON){
+                    ESP_LOGI(tag_GAP,"LEFT BUTTON");
+                    pkey.numcmp_accept = true;
+                } else {
+                    pkey.numcmp_accept = false;
+                }
             } else {
-                pkey.numcmp_accept = 0;
+                pkey.numcmp_accept = false;
                 ESP_LOGE(tag_GAP, "Timeout! Rejecting the key");
             }
 
+            ESP_LOGI(tag_GAP, "NUMCP_ACCEPT: %d", pkey.numcmp_accept);
             rc = ble_sm_inject_io(event->passkey.conn_handle, &pkey);
+            if(rc != 0){
+                //https://github.com/espressif/esp-idf/issues/8555#issuecomment-1066591071
+                // 0x05 = Authentication Failure error code
+                ble_gap_terminate(conn_handle, 0x05);
+            }
             ESP_LOGI(tag_GAP, "ble_sm_inject_io result: %d\n", rc);
         }
         return 0;

components/ble/include/gap.h

@@ -6,10 +6,12 @@
 #include "host/ble_hs.h"
 
 #include "esp_log.h"
+#include "esp_random.h"
 
 #include "gatt.h"
 #include "ble.h"
 #include "ssd1306.h"
+#include "button.h"
 
 static const char *tag_GAP = "SimLinkModule_GAP";
 static const ble_uuid16_t hid_service_uuid = BLE_UUID16_INIT(0x1812);

components/button/CMakeLists.txt

@@ -0,0 +1,5 @@
+set(srcs "button.c")
+
+idf_component_register(SRCS "${srcs}"
+                    INCLUDE_DIRS "include"
+                    REQUIRES driver esp_rom)

components/button/button.c

@@ -0,0 +1,37 @@
+#include "button.h"
+
+void initButtons(){
+    //setup queue
+    buttonQueue = xQueueCreate(1, sizeof(BUTTON));
+
+    gpio_install_isr_service(0);
+
+    //left button setup
+    gpio_set_direction(GPIO_NUM_26,GPIO_MODE_INPUT);
+    gpio_set_intr_type(GPIO_NUM_26,GPIO_INTR_NEGEDGE);
+    gpio_isr_handler_add(GPIO_NUM_26, buttonISRHandler, (void *) LEFTBUTTON);
+    gpio_set_pull_mode(GPIO_NUM_26,GPIO_PULLUP_ONLY);
+
+    //right button setup
+    gpio_set_direction(GPIO_NUM_25,GPIO_MODE_INPUT);
+    gpio_set_intr_type(GPIO_NUM_25,GPIO_INTR_NEGEDGE);
+    gpio_isr_handler_add(GPIO_NUM_25, buttonISRHandler, (void *) RIGHTBUTTON);
+    gpio_set_pull_mode(GPIO_NUM_25,GPIO_PULLUP_ONLY);
+}
+
+static void IRAM_ATTR buttonISRHandler(void* arg)
+{
+    BUTTON pressedButton = (BUTTON) arg;
+    //ets_printf("BUTTON %d", pressedButton);
+    xQueueSendFromISR(buttonQueue, &pressedButton, NULL);
+}
+
+int getButton(BUTTON *selectedButton)
+{
+    //timeout after 30 seconds
+    return xQueueReceive(buttonQueue, selectedButton, 30000 / portTICK_PERIOD_MS);
+}
+
+void clearStoredButtons(){
+    xQueueReset(buttonQueue);
+}

components/button/include/button.h

@@ -0,0 +1,23 @@
+#ifndef BUTTON_H
+#define BUTTON_H
+
+#include "freertos/FreeRTOS.h"
+#include "freertos/queue.h"
+#include "driver/gpio.h"
+
+#include "esp_log.h"
+//#include "rom/ets_sys.h"
+
+typedef enum BUTTON {
+    LEFTBUTTON,
+    RIGHTBUTTON,
+} BUTTON;
+
+static QueueHandle_t buttonQueue = NULL;
+
+void initButtons();
+static void IRAM_ATTR buttonISRHandler(void* arg);
+int getButton(BUTTON *selectedButton);
+void clearStoredButtons();
+
+#endif

docs/Pinout.md

@@ -6,8 +6,13 @@
 | 16  | UART-Eingang                |
 | 21  | SDA                         |
 | 22  | SCK                         |
+| 25  | rechter Button              |
+| 26  | linker Button               |
 
-**Anmerkung:** Wenn der ADC für die Ermittlung der Spannung verwendet werden soll. (Wenn ein Pin für die Batteriespannung vorhanden ist) Dann muss ein Spannungsteiler verwendet werden, welche die Maximalspannung der Batterie auf 3.3V herunterregelt. Des Weiteren muss noch die Formel in der Batteriekomponente angepasst werden. Im Testaufbau war R1 650 Ohm und R2 2000 Ohm.
+**Anmerkung:**
+
+- Wenn der ADC für die Ermittlung der Spannung verwendet werden soll. (Wenn ein Pin für die Batteriespannung vorhanden ist) Dann muss ein Spannungsteiler verwendet werden, welche die Maximalspannung der Batterie auf 3.3V herunterregelt. Des Weiteren muss noch die Formel in der Batteriekomponente angepasst werden. Im Testaufbau war R1 650 Ohm und R2 2000 Ohm.
+- Die Buttons sind jeweils mit GND verbunden. Ein interner Pullup-Widerstand im ESP zieht den Pin im Ruhezustand auf 3.3V.
 
 # OLED-Display
 | Pin | Anmerkung |

docs/todo

@@ -1,7 +1,6 @@
-- buttons hinzufügen
 - LEDs hinzufügen
-- BLE Verbindung mit ausgabe von einem PIN und ja nein antworten hinzufügen --> gap.c 212
 
+- includes, variablen und funktionen durchschauen, wo die liegen müssen und ob sie static, extern oder so sein müssen
 - kommentieren und in .md-Datein dokumentieren, wo was zu finden ist
 
 - Bei einem erneuten verbindungsaufbau wird das gestoppte updaten der daten nicht wieder gestartet

main/main.c

@@ -5,6 +5,7 @@
 #include "ssd1306.h"
 #include "ble.h"
 #include "battery.h"
+#include "button.h"
 
 //#include "freertos/projdefs.h"
 #include "freertos/FreeRTOSConfig.h"
@@ -40,6 +41,9 @@ void app_main(void)
     //task to read crsf uart data
     xTaskCreate(crsf_get_ChannelData_task, "crsf_task", 4096, NULL, 10, NULL);
 
+    //setup buttons
+    initButtons();
+
     /*##########################################################
     ############################################################
     # nicht verwendet, da keine Batteriespannung vorhanden ist #