summaryrefslogtreecommitdiffstats
path: root/drivers/hwmon
diff options
context:
space:
mode:
authorJean Delvare <khali@linux-fr.org>2007-10-25 13:11:01 +0200
committerMark M. Hoffman <mhoffman@lightlink.com>2007-11-08 08:42:47 -0500
commit7768aa76966f06c4add6ac2b0ef397ff34f3dab0 (patch)
treeb1b97cf56996fd848446add7930181eb743a1571 /drivers/hwmon
parentdcbd9f68aea41d22d5bd37468409da33151b4202 (diff)
downloadop-kernel-dev-7768aa76966f06c4add6ac2b0ef397ff34f3dab0.zip
op-kernel-dev-7768aa76966f06c4add6ac2b0ef397ff34f3dab0.tar.gz
hwmon: (w83781d) Add missing curly braces
Missing curly braces cause an if statement to be evaluated when it shouldn't. It happens to be harmless, but that's still worth fixing. Thanks to Riku Voipio for reporting. Signed-off-by: Jean Delvare <khali@linux-fr.org> Acked-By: Riku Voipio <riku.voipio@movial.fi> Signed-off-by: Mark M. Hoffman <mhoffman@lightlink.com>
Diffstat (limited to 'drivers/hwmon')
-rw-r--r--drivers/hwmon/w83781d.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/hwmon/w83781d.c b/drivers/hwmon/w83781d.c
index a6a1edf..e0fa752 100644
--- a/drivers/hwmon/w83781d.c
+++ b/drivers/hwmon/w83781d.c
@@ -1122,12 +1122,13 @@ w83781d_create_files(struct device *dev, int kind, int is_isa)
&sensor_dev_attr_temp3_beep.dev_attr)))
return err;
- if (kind != w83781d)
+ if (kind != w83781d) {
err = sysfs_chmod_file(&dev->kobj,
&sensor_dev_attr_temp3_alarm.dev_attr.attr,
S_IRUGO | S_IWUSR);
if (err)
return err;
+ }
}
if (kind != w83781d && kind != as99127f) {
OpenPOWER on IntegriCloud