From 6ec675ca2fecfeba457b3ca8aee6fd88d0e27591 Mon Sep 17 00:00:00 2001 From: Peter Bigot Date: Wed, 11 Mar 2020 05:53:35 -0500 Subject: [PATCH] tests: drivers: gpio_basic_api: silence coverity warning In most cases gpio_pin_configure()'s return value is checked in this application; Coverity noted a case where it is not checked. Add a check to make it happy. Signed-off-by: Peter Bigot --- tests/drivers/gpio/gpio_basic_api/src/test_deprecated.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/tests/drivers/gpio/gpio_basic_api/src/test_deprecated.c b/tests/drivers/gpio/gpio_basic_api/src/test_deprecated.c index c237dbd7f09..e4a28677c8a 100644 --- a/tests/drivers/gpio/gpio_basic_api/src/test_deprecated.c +++ b/tests/drivers/gpio/gpio_basic_api/src/test_deprecated.c @@ -21,6 +21,7 @@ static int cb_cnt; static void callback(struct device *dev, struct gpio_callback *gpio_cb, u32_t pins) { + int rc; const struct drv_data *dd = CONTAINER_OF(gpio_cb, struct drv_data, gpio_cb); @@ -43,8 +44,10 @@ static void callback(struct device *dev, * level interrupts to repeat forever. To prevent hangs * it's necessary to explicitly disable the interrupt. */ - gpio_pin_configure(dev, PIN_IN, GPIO_DIR_IN - | GPIO_INT_DISABLE); + rc = gpio_pin_configure(dev, PIN_IN, GPIO_DIR_IN + | GPIO_INT_DISABLE); + zassert_equal(rc, 0, + "disable interrupts failed: %d", rc); } }