RE: nRF5340, gpio1 interrupt not occurring

Håkon Alseth — Tue, 11 Apr 2023 13:05:01 GMT

Hi,

In order to use NRF_P1, you should declare "gpio0" and "gpio1" to use that port in your overlay, like here where P1.08 and P1.09 are used as buttons:

/ {
	buttons {
		compatible = "gpio-keys";
		button0: button_0 {
			gpios = <&gpio1 8 (GPIO_PULL_UP | GPIO_ACTIVE_LOW)>;
			label = "Push button 1";
		};
		button1: button_1 {
			gpios = <&gpio1 9 (GPIO_PULL_UP | GPIO_ACTIVE_LOW)>;
			label = "Push button 2";
		};
	};
	aliases {
		sw0 = &button0;
		sw1 = &button1;
	};	
};

When you define it like this, you can use the ncs\zephyr\samples\basic\button sample directly without any change in the firmware itself.

For your case, I tried your test program, changed to use P1.08 and P1.09 (ie. "gpio1" device on both pins) and that worked:

*** Booting Zephyr OS build v3.2.99-ncs2 ***
name: gpio@842800, pins: 256
name: gpio@842800, pins: 512
name: gpio@842800, pins: 512
name: gpio@842800, pins: 256
name: gpio@842800, pins: 256
name: gpio@842800, pins: 256
name: gpio@842800, pins: 256
name: gpio@842800, pins: 256
name: gpio@842800, pins: 512
name: gpio@842800, pins: 512
name: gpio@842800, pins: 512
name: gpio@842800, pins: 512

At my end, the buttons are active low, so this is my slightly modified firmware with active low and pull-up on the pins:

#include <zephyr/kernel.h>
#include <zephyr/device.h>
#include <zephyr/devicetree.h>
#include <zephyr/drivers/gpio.h>

#include <zephyr/sys/printk.h>

#define SLEEP_TIME_MS   10*60*1000

#define MY_PIN0 8
#define MY_PIN1 9

static const struct device *gpio0 = DEVICE_DT_GET(DT_NODELABEL(gpio0));
static const struct device *gpio1 = DEVICE_DT_GET(DT_NODELABEL(gpio1));

static struct gpio_callback gpio0_cb_data;
static struct gpio_callback gpio1_cb_data;

void cb(const struct device *dev, struct gpio_callback *cb, uint32_t pins)
{
	printk("name: %s, pins: %u\n",dev->name, pins);
}

void main(void)
{
	if (!device_is_ready(gpio0)) {
		return;
	}

	if (!device_is_ready(gpio1)) {
		return;
	}

	gpio_pin_configure(gpio1, MY_PIN0, GPIO_INPUT | GPIO_PULL_UP);
	gpio_pin_configure(gpio1, MY_PIN1, GPIO_INPUT | GPIO_PULL_UP);

	gpio_pin_interrupt_configure(gpio1, MY_PIN0, GPIO_INT_EDGE_FALLING);
	gpio_init_callback(&gpio0_cb_data, cb, BIT(MY_PIN0));
	gpio_add_callback(gpio1, &gpio0_cb_data);

	gpio_pin_interrupt_configure(gpio1, MY_PIN1, GPIO_INT_EDGE_FALLING);
	gpio_init_callback(&gpio1_cb_data, cb, BIT(MY_PIN1));
	gpio_add_callback(gpio1, &gpio1_cb_data);	

	while (1) {
        k_msleep(SLEEP_TIME_MS);
	}
}

Kind regards,

Håkon