michaelh/zephyr
1
0
Fork 0
Code Releases Activity

drivers: amg88xx: check i2c write return value

Browse source 
Check i2c write return value.

Fixes: #20498
Coverity-CID: 205628

Signed-off-by: Johann Fischer <j.fischer@phytec.de>
This commit is contained in:
Johann Fischer 2019-11-21 12:01:53 +01:00 • committed by Maureen Helm
commit dfe57ca9e1
1 changed files with 11 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

15
drivers/sensor/amg88xx/amg88xx_trigger.c
View file

@ -131,8 +131,11 @@ int amg88xx_trigger_set(struct device *dev,
struct amg88xx_data *drv_data = dev->driver_data;
const struct amg88xx_config *config = dev->config->config_info;
i2c_reg_write_byte(drv_data->i2c, config->i2c_address,
AMG88XX_INTC, AMG88XX_INTC_DISABLED);
if (i2c_reg_write_byte(drv_data->i2c, config->i2c_address,
AMG88XX_INTC, AMG88XX_INTC_DISABLED)) {
return -EIO;
}
gpio_pin_disable_callback(drv_data->gpio, config->gpio_pin);
if (trig->type == SENSOR_TRIG_THRESHOLD) {
@ -144,8 +147,12 @@ int amg88xx_trigger_set(struct device *dev,
}
gpio_pin_enable_callback(drv_data->gpio, config->gpio_pin);
i2c_reg_write_byte(drv_data->i2c, config->i2c_address,
AMG88XX_INTC, AMG88XX_INTC_ABS_MODE);
if (i2c_reg_write_byte(drv_data->i2c, config->i2c_address,
AMG88XX_INTC, AMG88XX_INTC_ABS_MODE)) {
return -EIO;
}
return 0;
}