michaelh/zephyr
1
0
Fork 0
Code Releases Activity

drivers: sensor: fxos8700: check gpio calls return code

Browse source 
Some GPIO related calls were not being checked for errors.

This patch fixes coverity issue 236650.

Signed-off-by: Gerard Marull-Paretas <gerard.marull@nordicsemi.no>
This commit is contained in:
Gerard Marull-Paretas 2021-08-26 08:43:05 +02:00 • committed by Anas Nashif
commit 18732a5c74
1 changed files with 15 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

16
drivers/sensor/fxos8700/fxos8700_trigger.c
View file

@ -408,6 +408,7 @@ int fxos8700_trigger_init(const struct device *dev)
const struct fxos8700_config *config = dev->config;
struct fxos8700_data *data = dev->data;
uint8_t ctrl_reg5;
int ret;
data->dev = dev;
@ -469,16 +470,25 @@ int fxos8700_trigger_init(const struct device *dev)
data->gpio_pin = config->gpio_pin;
gpio_pin_configure(data->gpio, config->gpio_pin,
ret = gpio_pin_configure(data->gpio, config->gpio_pin,
GPIO_INPUT | config->gpio_flags);
if (ret < 0) {
return ret;
}
gpio_init_callback(&data->gpio_cb, fxos8700_gpio_callback,
BIT(config->gpio_pin));
gpio_add_callback(data->gpio, &data->gpio_cb);
ret = gpio_add_callback(data->gpio, &data->gpio_cb);
if (ret < 0) {
return ret;
}
gpio_pin_interrupt_configure(data->gpio, config->gpio_pin,
ret = gpio_pin_interrupt_configure(data->gpio, config->gpio_pin,
GPIO_INT_EDGE_TO_ACTIVE);
if (ret < 0) {
return ret;
}
return 0;
}