hwmon: (w83781d) Add missing curly braces
authorJean Delvare <khali@linux-fr.org>
Thu, 25 Oct 2007 11:11:01 +0000 (13:11 +0200)
committerMark M. Hoffman <mhoffman@lightlink.com>
Thu, 8 Nov 2007 13:42:47 +0000 (08:42 -0500)
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>
drivers/hwmon/w83781d.c

index a6a1edfe76141e8a9fb5fb058a13014d90787ac9..e0fa7520400d8c565f0b7c77034a6c0e35cf4a1a 100644 (file)
@@ -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) {