Commit 7576750f authored by Thilo Cestonaro's avatar Thilo Cestonaro Committed by Guenter Roeck

hwmon: (ftsteutates) Fix clearing alarm sysfs entries

sysfs store functions should return the number of bytes written.
Returning zero results in an endless loop.

Fixes: 08426eda ("hwmon: Add driver for FTS BMC chip "Teutates"")
Signed-off-by: default avatarThilo Cestonaro <thilo.cestonaro@ts.fujitsu.com>
[groeck: Clean up documentation change and description]
Signed-off-by: default avatarGuenter Roeck <linux@roeck-us.net>
parent aae4e7a8
...@@ -18,6 +18,10 @@ enhancements. It can monitor up to 4 voltages, 16 temperatures and ...@@ -18,6 +18,10 @@ enhancements. It can monitor up to 4 voltages, 16 temperatures and
8 fans. It also contains an integrated watchdog which is currently 8 fans. It also contains an integrated watchdog which is currently
implemented in this driver. implemented in this driver.
To clear a temperature or fan alarm, execute the following command with the
correct path to the alarm file:
echo 0 >XXXX_alarm
Specification of the chip can be found here: Specification of the chip can be found here:
ftp://ftp.ts.fujitsu.com/pub/Mainboard-OEM-Sales/Services/Software&Tools/Linux_SystemMonitoring&Watchdog&GPIO/BMC-Teutates_Specification_V1.21.pdf ftp://ftp.ts.fujitsu.com/pub/Mainboard-OEM-Sales/Services/Software&Tools/Linux_SystemMonitoring&Watchdog&GPIO/BMC-Teutates_Specification_V1.21.pdf
ftp://ftp.ts.fujitsu.com/pub/Mainboard-OEM-Sales/Services/Software&Tools/Linux_SystemMonitoring&Watchdog&GPIO/Fujitsu_mainboards-1-Sensors_HowTo-en-US.pdf ftp://ftp.ts.fujitsu.com/pub/Mainboard-OEM-Sales/Services/Software&Tools/Linux_SystemMonitoring&Watchdog&GPIO/Fujitsu_mainboards-1-Sensors_HowTo-en-US.pdf
...@@ -435,6 +435,7 @@ clear_temp_alarm(struct device *dev, struct device_attribute *devattr, ...@@ -435,6 +435,7 @@ clear_temp_alarm(struct device *dev, struct device_attribute *devattr,
goto error; goto error;
data->valid = false; data->valid = false;
ret = count;
error: error:
mutex_unlock(&data->update_lock); mutex_unlock(&data->update_lock);
return ret; return ret;
...@@ -508,6 +509,7 @@ clear_fan_alarm(struct device *dev, struct device_attribute *devattr, ...@@ -508,6 +509,7 @@ clear_fan_alarm(struct device *dev, struct device_attribute *devattr,
goto error; goto error;
data->valid = false; data->valid = false;
ret = count;
error: error:
mutex_unlock(&data->update_lock); mutex_unlock(&data->update_lock);
return ret; return ret;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment