| Message ID | 20180807151909.24433-3-ikegami@allied-telesis.co.jp (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | hwmon: (adt7475) Add error handling for update function | expand |
diff --git a/drivers/hwmon/adt7475.c b/drivers/hwmon/adt7475.c index 42f48c6843c5..680fc9cecb7b 100644 --- a/drivers/hwmon/adt7475.c +++ b/drivers/hwmon/adt7475.c @@ -195,7 +195,7 @@ struct adt7475_data { unsigned long measure_updated; bool limits_updated; - char valid; + bool valid; u8 config4; u8 config5; @@ -1780,6 +1780,7 @@ static struct adt7475_data *adt7475_update_device(struct device *dev) !data->valid) { adt7475_update_measure(dev); data->measure_updated = jiffies; + data->valid = true; } /* Limits and settings, should never change update more than once */
Currently the valid paramter is char type but the type is not required. So change the parameter to boot type correctly. Signed-off-by: Tokunori Ikegami <ikegami@allied-telesis.co.jp> Cc: Guenter Roeck <linux@roeck-us.net> Cc: Chris Packham <chris.packham@alliedtelesis.co.nz> Cc: linux-hwmon@vger.kernel.org --- Changes since v1: - Change to rebase on the v2 patch 1. drivers/hwmon/adt7475.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)