diff --git a/main/ble.c b/main/ble.c index 35bd87d..e8c63e2 100644 --- a/main/ble.c +++ b/main/ble.c @@ -51,8 +51,8 @@ static int barback_ble_char_access(uint16_t conn_handle, uint16_t attr_handle, s #define BLE_SERVICE_PUMP_DURATION (0x4350) #define BLE_SERVICE_BARBACK (0x4400) -#define CHAR_POUR (BLE_SERVICE_BARBACK + 1) -#define CHAR_BUTTON (BLE_SERVICE_BARBACK + 2) +#define CHAR_POUR (BLE_SERVICE_BARBACK + 1) // 0x4401 +#define CHAR_BUTTON (BLE_SERVICE_BARBACK + 2) // 0x4402 static barback_ble_access_t barback_ble_pump_access_enabled = { .read = pumps_get_enabled, @@ -139,7 +139,7 @@ static const struct ble_gatt_svc_def service_defs[] = { .val_handle = &svc_handle_button, .flags = BLE_GATT_CHR_F_READ | BLE_GATT_CHR_F_NOTIFY }, - { 0 } + { 0 } } }, { 0 } // no more services @@ -466,7 +466,7 @@ void ble_init(char *name) { memset(&callback_handlers, 0, sizeof(event_callback_handle_t) * CONFIG_CACO_MAX_SERVICES); - ESP_ERROR_CHECK(esp_nimble_hci_and_controller_init()); + //ESP_ERROR_CHECK(esp_nimble_hci_and_controller_init()); nimble_port_init(); diff --git a/main/console.c b/main/console.c index 2e0ded6..c183e7e 100644 --- a/main/console.c +++ b/main/console.c @@ -25,7 +25,6 @@ void console_task(void *args) { .data_bits = UART_DATA_8_BITS, .parity = UART_PARITY_DISABLE, .stop_bits = UART_STOP_BITS_1, - .use_ref_tick = true }; ESP_ERROR_CHECK(uart_param_config(CONFIG_ESP_CONSOLE_UART_NUM, &uart_config)); diff --git a/main/pumps.c b/main/pumps.c index 0ae4636..65bf440 100644 --- a/main/pumps.c +++ b/main/pumps.c @@ -76,7 +76,7 @@ uint8_t pump_disable(int8_t i) { } void pump_timer_done(TimerHandle_t timer) { - uint8_t idx = (pcTimerGetTimerName(timer) - 0x48); + uint8_t idx = (pcTimerGetName(timer) - 0x48); ESP_LOGD(TAG, "pump done: %d", idx); running--; @@ -86,9 +86,12 @@ void pump_timer_done(TimerHandle_t timer) { void pumps_stop() { for(uint8_t i = 0; i < PUMPS; i++) { - if(pump_timers[i] != NULL) + if(pump_timers[i] != NULL) { pump_disable(i); xTimerStop(pump_timers[i], 0); + + i++; + } } } @@ -117,7 +120,7 @@ uint8_t pumps_init() { gpio_set_level(pump_gpio_map[i], 0); } - io_conf.intr_type = GPIO_PIN_INTR_DISABLE; + io_conf.intr_type = GPIO_INTR_DISABLE; io_conf.mode = GPIO_MODE_OUTPUT; io_conf.pin_bit_mask = GPIO_PUMP_PIN_SEL; io_conf.pull_down_en = 1; @@ -126,7 +129,7 @@ uint8_t pumps_init() { // zero out pump enabled, duration memset(&pumps_enabled, 0, sizeof(uint8_t) * PUMPS); - memset(&pumps_duration, 10, sizeof(uint8_t) * PUMPS); + memset(&pumps_duration, 5, sizeof(uint8_t) * PUMPS); size_t size = sizeof(uint8_t) * PUMPS; if(!nvs_get_blob(config_handle, "pumps_duration", &pumps_duration, &size)) { diff --git a/main/user_button.c b/main/user_button.c index 114ad14..8b8001b 100644 --- a/main/user_button.c +++ b/main/user_button.c @@ -7,7 +7,7 @@ #include "pumps.h" #include "ble.h" -#define GPIO_USER_BUTTON (19) +#define GPIO_USER_BUTTON (14) static void IRAM_ATTR gpio_isr_handler(void* arg) { uint32_t gpio_num = (uint32_t) arg; @@ -27,10 +27,10 @@ static void gpio_task(void* arg) { if(level == last_level) continue; last_level = level; - printf("GPIO[%d] intr, val: %d\n", io_num, level); + printf("GPIO[%ld] intr, val: %d\n", io_num, level); if(state == 0) - vTaskDelay(10 / portTICK_RATE_MS); + vTaskDelay(10 / portTICK_PERIOD_MS); pumps_run(); @@ -44,7 +44,7 @@ static void gpio_task(void* arg) { void user_button_init() { gpio_config_t io_conf; - io_conf.intr_type = GPIO_PIN_INTR_ANYEDGE; + io_conf.intr_type = GPIO_INTR_ANYEDGE; io_conf.mode = GPIO_MODE_INPUT; io_conf.pin_bit_mask = (1ULL << GPIO_USER_BUTTON); io_conf.pull_down_en = 0;