diff options
author | Jean Delvare <khali@linux-fr.org> | 2007-10-25 07:11:01 -0400 |
---|---|---|
committer | Mark M. Hoffman <mhoffman@lightlink.com> | 2007-11-08 08:42:47 -0500 |
commit | 7768aa76966f06c4add6ac2b0ef397ff34f3dab0 (patch) | |
tree | b1b97cf56996fd848446add7930181eb743a1571 | |
parent | dcbd9f68aea41d22d5bd37468409da33151b4202 (diff) |
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>
-rw-r--r-- | drivers/hwmon/w83781d.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/hwmon/w83781d.c b/drivers/hwmon/w83781d.c index a6a1edfe7614..e0fa7520400d 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) | |||
1122 | &sensor_dev_attr_temp3_beep.dev_attr))) | 1122 | &sensor_dev_attr_temp3_beep.dev_attr))) |
1123 | return err; | 1123 | return err; |
1124 | 1124 | ||
1125 | if (kind != w83781d) | 1125 | if (kind != w83781d) { |
1126 | err = sysfs_chmod_file(&dev->kobj, | 1126 | err = sysfs_chmod_file(&dev->kobj, |
1127 | &sensor_dev_attr_temp3_alarm.dev_attr.attr, | 1127 | &sensor_dev_attr_temp3_alarm.dev_attr.attr, |
1128 | S_IRUGO | S_IWUSR); | 1128 | S_IRUGO | S_IWUSR); |
1129 | if (err) | 1129 | if (err) |
1130 | return err; | 1130 | return err; |
1131 | } | ||
1131 | } | 1132 | } |
1132 | 1133 | ||
1133 | if (kind != w83781d && kind != as99127f) { | 1134 | if (kind != w83781d && kind != as99127f) { |