ESP32 Forum

Posted: **Sun Sep 05, 2021 6:56 pm**

Hi,

I modified the switch example and enabling two GPIO pins using two switch buttons.
I am using the ESP32 custom board and after loading the program getting the below error.

# switch button
#define BUTTON_GPIO_1 21
#define BUTTON_GPIO_2 17
#define BUTTON_ACTIVE_LEVEL 0

/* This is the GPIO on which the power will be set */
#define OUTPUT_GPIO_1 14
#define OUTPUT_GPIO_2 27

E (887) gpio: gpio_install_isr_service(438): GPIO isr service already installed

: gpio_isr.PNG (11.86 KiB) Viewed 73686 times

can you please help on this.

Thanks,
Nirmesh

Posted: **Mon Sep 06, 2021 11:20 am**

From what you posted I can only see that you have defined GPIO pins. Can you post the code where you configured GPIO?
You need to have something similar with:

Code: Select all

    button_handle_t btn_handle_1 = iot_button_create(BUTTON_GPIO_1, BUTTON_ACTIVE_LEVEL_1);
    button_handle_t btn_handle_2 = iot_button_create(BUTTON_GPIO_2, BUTTON_ACTIVE_LEVEL_2);
    if (btn_handle_1) {
        /* Register a callback for a button tap (short press) event */
        iot_button_set_evt_cb(btn_handle_1, BUTTON_CB_TAP, push_btn_cb_1, NULL);
        /* Register Wi-Fi reset and factory reset functionality on same button */
        app_reset_button_register(btn_handle_1, WIFI_RESET_BUTTON_TIMEOUT, FACTORY_RESET_BUTTON_TIMEOUT);
    }
    if (btn_handle_2) {
        /* Register a callback for a button tap (short press) event */
        iot_button_set_evt_cb(btn_handle_2, BUTTON_CB_TAP, push_btn_cb_2, NULL);
    }
    // Configure the GPIO
    gpio_config_t io_conf = {
        .mode = GPIO_MODE_OUTPUT,
        .pull_up_en = 1,
    };
    uint64_t pin_mask = (((uint64_t)1 << OUTPUT_GPIO_1) | ((uint64_t)1 << OUTPUT_GPIO_2));
    io_conf.pin_bit_mask = pin_mask;
    gpio_config(&io_conf);

I hope I managed to help you.

Posted: **Mon Sep 06, 2021 2:49 pm**

codorstelian wrote: ↑

Mon Sep 06, 2021 11:20 am

From what you posted I can only see that you have defined GPIO pins. Can you post the code where you configured GPIO?
You need to have something similar with:

Code: Select all

    button_handle_t btn_handle_1 = iot_button_create(BUTTON_GPIO_1, BUTTON_ACTIVE_LEVEL_1);
    button_handle_t btn_handle_2 = iot_button_create(BUTTON_GPIO_2, BUTTON_ACTIVE_LEVEL_2);
    if (btn_handle_1) {
        /* Register a callback for a button tap (short press) event */
        iot_button_set_evt_cb(btn_handle_1, BUTTON_CB_TAP, push_btn_cb_1, NULL);
        /* Register Wi-Fi reset and factory reset functionality on same button */
        app_reset_button_register(btn_handle_1, WIFI_RESET_BUTTON_TIMEOUT, FACTORY_RESET_BUTTON_TIMEOUT);
    }
    if (btn_handle_2) {
        /* Register a callback for a button tap (short press) event */
        iot_button_set_evt_cb(btn_handle_2, BUTTON_CB_TAP, push_btn_cb_2, NULL);
    }
    // Configure the GPIO
    gpio_config_t io_conf = {
        .mode = GPIO_MODE_OUTPUT,
        .pull_up_en = 1,
    };
    uint64_t pin_mask = (((uint64_t)1 << OUTPUT_GPIO_1) | ((uint64_t)1 << OUTPUT_GPIO_2));
    io_conf.pin_bit_mask = pin_mask;
    gpio_config(&io_conf);

I hope I managed to help you.

@codorstelian Thanks for the help.
I have defined the gpio like as you suggested but still getting the same error.
E (906) gpio: gpio_install_isr_service(438): GPIO isr service already installed

I am using the below code.
https://github.com/nirmeshru/esp_switch ... p_driver.c

Thanks,
Nirmesh

Posted: **Mon Sep 06, 2021 3:20 pm**

Hi Nirmesh,

If I remember correctly, that is more like a warning rather than an error. Do you face any actual issues while using the GPIO.

Regards,
Piyush

Posted: **Mon Sep 06, 2021 4:15 pm**

I have tried to reproduce the error and it appears when I add a second iot_button so I think this error is caused from the button library located in esp-rainmaker/components.

Posted: **Mon Sep 06, 2021 6:02 pm**

ESP_Piyush wrote: ↑
Mon Sep 06, 2021 3:20 pm
Hi Nirmesh,

If I remember correctly, that is more like a warning rather than an error. Do you face any actual issues while using the GPIO.

Regards,
Piyush

Hi Piyush,

Actually when I tried two switches then there is no functional issue only getting the above error (warning).
but I am working for four switches so whenever I am trying with four switches getting below error and some watchdog task trigger and device start rebooting in a loop.

serial log -

I (867) gpio: GPIO[17]| InputEn: 1| OutputEn: 0| OpenDrain: 0| Pullup: 1| Pulldown: 0| Intr:3
E (867) gpio: gpio_install_isr_service(438): GPIO isr service already installed
I (877) gpio: GPIO[18]| InputEn: 1| OutputEn: 0| OpenDrain: 0| Pullup: 1| Pulldown: 0| Intr:3
E (887) gpio: gpio_install_isr_service(438): GPIO isr service already installed
I (897) gpio: GPIO[19]| InputEn: 1| OutputEn: 0| OpenDrain: 0| Pullup: 1| Pulldown: 0| Intr:3
E (907) gpio: gpio_install_isr_service(438): GPIO isr service already installed

I (E (5777) task_wdt: Task watchdog got triggered. The following tasks did not reset the watchdog in time:
E (5777) task_wdt: - IDLE0 (CPU 0)
E (5777) task_wdt: Tasks currently running:
E (5777) task_wdt: CPU 0: main
E (5777) task_wdt: CPU 1: ipc1
E (5777) task_wdt: Print CPU 0 (current core) backtrace

Thanks,
Nirmesh

Posted: **Tue Sep 07, 2021 6:11 pm**

Hi Nirmesh,

Can you share the code snippet for this new issue?

Posted: **Tue Sep 07, 2021 6:28 pm**

ESP_Piyush wrote: ↑
Tue Sep 07, 2021 6:11 pm
Hi Nirmesh,

Can you share the code snippet for this new issue?

Sure Piyush.
As similar to the 2 switch app_driver.c file, I modified app_driver.c for 4 switches.

Code: Select all

   
/* This is the button that is used for toggling the power */
#define BUTTON_GPIO_1		17
#define BUTTON_GPIO_2       18
#define BUTTON_GPIO_3       19
#define BUTTON_GPIO_4       21
#define BUTTON_ACTIVE_LEVEL  0

/* This is the GPIO on which the power will be set */
#define OUTPUT_GPIO_1		14
#define OUTPUT_GPIO_2   	25 
#define OUTPUT_GPIO_3   	26
#define OUTPUT_GPIO_4   	27
static bool g_power_state = DEFAULT_POWER;
static bool g_power_state3 = DEFAULT_POWER;
static bool g_power_state4 = DEFAULT_POWER;
static bool g_power_state_nirmesh = DEFAULT_POWER;

#define WIFI_RESET_BUTTON_TIMEOUT		3
#define FACTORY_RESET_BUTTON_TIMEOUT    10


static void push_btn_cb1(void *arg)
{
    bool new_state = !g_power_state;
    app_driver_set_state(new_state);
    esp_rmaker_param_update_and_report(
            esp_rmaker_device_get_param_by_name(switch_device1, ESP_RMAKER_DEF_POWER_NAME),
            esp_rmaker_bool(new_state));
}

static void push_btn_cb2(void *arg)
{
    bool new_state_nirmesh = !g_power_state_nirmesh;
    app_driver_set_state2(new_state_nirmesh);
    esp_rmaker_param_update_and_report(
            esp_rmaker_device_get_param_by_name(switch_device2, ESP_RMAKER_DEF_POWER_NAME),
            esp_rmaker_bool(new_state_nirmesh));
}

static void push_btn_cb3(void *arg)
{
    bool new_state3 = !g_power_state3;
    app_driver_set_state3(new_state3);
    esp_rmaker_param_update_and_report(
            esp_rmaker_device_get_param_by_name(switch_device3, ESP_RMAKER_DEF_POWER_NAME),
            esp_rmaker_bool(new_state3));
}

static void push_btn_cb4(void *arg)
{
    bool new_state4 = !g_power_state4;
    app_driver_set_state4(new_state4);
    esp_rmaker_param_update_and_report(
            esp_rmaker_device_get_param_by_name(switch_device4, ESP_RMAKER_DEF_POWER_NAME),
            esp_rmaker_bool(new_state4));
}

static void set_power_state(bool target)
{
    gpio_set_level(OUTPUT_GPIO_1, target);
    
}

static void set_power_state2(bool target)
{
    gpio_set_level(OUTPUT_GPIO_2, target);
    
}

static void set_power_state3(bool target)
{
    gpio_set_level(OUTPUT_GPIO_3, target);
    
}

static void set_power_state4(bool target)
{
    gpio_set_level(OUTPUT_GPIO_4, target);
    
}

void app_driver_init()
{
    button_handle_t btn_handle1 = iot_button_create(BUTTON_GPIO_1, BUTTON_ACTIVE_LEVEL);
	button_handle_t btn_handle2 = iot_button_create(BUTTON_GPIO_2, BUTTON_ACTIVE_LEVEL);
    button_handle_t btn_handle3 = iot_button_create(BUTTON_GPIO_3, BUTTON_ACTIVE_LEVEL);
	button_handle_t btn_handle4 = iot_button_create(BUTTON_GPIO_4, BUTTON_ACTIVE_LEVEL);
	
    if (btn_handle1) {
        /* Register a callback for a button tap (short press) event */
        iot_button_set_evt_cb(btn_handle1, BUTTON_CB_TAP, push_btn_cb1, NULL);
        /* Register Wi-Fi reset and factory reset functionality on same button */
        app_reset_button_register(btn_handle1, WIFI_RESET_BUTTON_TIMEOUT, FACTORY_RESET_BUTTON_TIMEOUT);
	}
	if (btn_handle2) {
        /* Register a callback for a button tap (short press) event */
        iot_button_set_evt_cb(btn_handle2, BUTTON_CB_TAP, push_btn_cb2, NULL);
        /* Register Wi-Fi reset and factory reset functionality on same button */
		
    }

	if (btn_handle3) {
        /* Register a callback for a button tap (short press) event */
        iot_button_set_evt_cb(btn_handle3, BUTTON_CB_TAP, push_btn_cb3, NULL);
        /* Register Wi-Fi reset and factory reset functionality on same button */
		
    }

	if (btn_handle4) {
        /* Register a callback for a button tap (short press) event */
        iot_button_set_evt_cb(btn_handle4, BUTTON_CB_TAP, push_btn_cb4, NULL);
        /* Register Wi-Fi reset and factory reset functionality on same button */
		
    }

    /* Configure power */
    gpio_config_t io_conf = {
        .mode = GPIO_MODE_OUTPUT,
        .pull_up_en = 1,
    };
    io_conf.pin_bit_mask = (((uint64_t)1 << OUTPUT_GPIO_1) | ((uint64_t)1 << OUTPUT_GPIO_2) | ((uint64_t)1 << OUTPUT_GPIO_3) | ((uint64_t)1 << OUTPUT_GPIO_4));
    /* Configure the GPIO */
    gpio_config(&io_conf);
   
}

int IRAM_ATTR app_driver_set_state(bool state)
{
    
    g_power_state = state;
    set_power_state(g_power_state);
    
    return ESP_OK;
}

int IRAM_ATTR app_driver_set_state2(bool state)
{
    
    g_power_state_nirmesh = state;
    set_power_state2(g_power_state_nirmesh);
    
    return ESP_OK;
}

int IRAM_ATTR app_driver_set_state3(bool state)
{
    
    g_power_state3 = state;
    set_power_state3(g_power_state3);
    
    return ESP_OK;
}

int IRAM_ATTR app_driver_set_state4(bool state)
{
    
    g_power_state4 = state;
    set_power_state4(g_power_state4);
    
    return ESP_OK;
}

bool app_driver_get_state(void)
{
    return g_power_state;
}

bool app_driver_get_state2(void)
{
    return g_power_state_nirmesh;
}

bool app_driver_get_state3(void)
{
    return g_power_state3;
}

bool app_driver_get_state4(void)
{
    return g_power_state4;
}

Posted: **Tue Sep 07, 2021 11:41 pm**

ESP_Piyush wrote: ↑
Mon Sep 06, 2021 3:20 pm
Hi Nirmesh,

If I remember correctly, that is more like a warning rather than an error. Do you face any actual issues while using the GPIO.

Regards,
Piyush

"E (906)" indicates it's an error message.
If it is indeed harmless, please convert it to warning message (W).

Posted: **Wed Sep 08, 2021 7:49 am**

"E (906)" indicates it's an error message.
If it is indeed harmless, please convert it to warning message (W).

Fyi, this is indeed an error, caused by a few of our components not working well together. The message you see is the only result of this error, as the GPIO API happens to catch it, but even if it is otherwise harmless, it is an error regardless and we will take measures to fix it.

(I'd like to also state that in general, it's not really our policy to generate error messages like that that you can 'just ignore', even if it happens to work out here.)

ESP32 Forum

Error:GPIO isr service already installed

Error:GPIO isr service already installed

Re: Error:GPIO isr service already installed

Re: Error:GPIO isr service already installed

Re: Error:GPIO isr service already installed

Re: Error:GPIO isr service already installed

Re: Error:GPIO isr service already installed

Re: Error:GPIO isr service already installed

Re: Error:GPIO isr service already installed

Re: Error:GPIO isr service already installed

Re: Error:GPIO isr service already installed