michaelh/zephyr
1
0
Fork 0
Code Releases Activity

sensor: mcp9808: Bug fix for setting resolution

Browse source 
Resolution currently only gets set if CONFIG_MCP9808_TRIGGER is also
set. If CONFIG_MCP9808_TRIGGER is not set, resolution gets not to '0'.

This small bug fix moves resolution to before CONFIG_MCP9808_TRIGGER is
checked, so that the resolution is changed as required.

Signed-off-by: Steven Daglish <s.c.daglish@gmail.com>
This commit is contained in:
Steven Daglish 2021-03-21 13:04:38 +00:00 • committed by Carles Cufí
commit 8f13b7860e
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
drivers/sensor/mcp9808/mcp9808.c
View file

@ -132,11 +132,11 @@ static struct mcp9808_data mcp9808_data;
static const struct mcp9808_config mcp9808_cfg = {
.i2c_bus = DT_INST_BUS_LABEL(0),
.i2c_addr = DT_INST_REG_ADDR(0),
.resolution = DT_INST_PROP(0, resolution),
#ifdef CONFIG_MCP9808_TRIGGER
.alert_pin = DT_INST_GPIO_PIN(0, int_gpios),
.alert_flags = DT_INST_GPIO_FLAGS(0, int_gpios),
.alert_controller = DT_INST_GPIO_LABEL(0, int_gpios),
.resolution = DT_INST_PROP(0, resolution),
#endif /* CONFIG_MCP9808_TRIGGER */
};