Browse Source

drivers: gpio: rzt2m: properly handle error from rzt2m_gpio_get_pin_irq

For the negative errno rzt2m_gpio_get_pin_irq may return to be properly
handled, irq variable needs to use signed type.

Signed-off-by: Benjamin Cabé <benjamin@zephyrproject.org>
pull/92206/head
Benjamin Cabé 3 weeks ago committed by Dan Kalowsky
parent
commit
5b0808fa4a
  1. 7
      drivers/gpio/gpio_rzt2m.c

7
drivers/gpio/gpio_rzt2m.c

@ -318,18 +318,17 @@ static int rzt2m_gpio_pin_interrupt_configure(const struct device *dev, gpio_pin @@ -318,18 +318,17 @@ static int rzt2m_gpio_pin_interrupt_configure(const struct device *dev, gpio_pin
return -ENOTSUP;
}
uint8_t irq = rzt2m_gpio_get_pin_irq(dev, pin);
bool irq_used_by_other = rzt2m_gpio_is_irq_used_by_other_pin(dev, pin, irq);
int irq = rzt2m_gpio_get_pin_irq(dev, pin);
if (irq < 0) {
return -ENOTSUP;
}
/* secure range - currently not supported*/
if (irq >= NS_IRQ_COUNT) {
return -ENOSYS;
}
bool irq_used_by_other = rzt2m_gpio_is_irq_used_by_other_pin(dev, pin, irq);
if (mode == GPIO_INT_MODE_DISABLED) {
rzt2m_gpio_unlock();
WRITE_BIT(*pmc_reg, pin, 0);

Loading…
Cancel
Save