1 files changed, 8 insertions, 5 deletions

diff --git a/drivers/hwmon/smsc47b397.c b/drivers/hwmon/smsc47b397.c
index 9fb7516e6f45..65c88ff5645a 100644
--- a/drivers/hwmon/smsc47b397.c
+++ b/drivers/hwmon/smsc47b397.c
@@ -113,7 +113,7 @@ struct smsc47b397_data {
         u8 temp[4];
 };
 
-static int smsc47b397_read_value(struct smsc47b397_data* data, u8 reg)
+static int smsc47b397_read_value(struct smsc47b397_data *data, u8 reg)
 {
         int res;
 
@@ -265,7 +265,8 @@ static int __devinit smsc47b397_probe(struct platform_device *pdev)
                 return -EBUSY;
         }
 
-        if (!(data = kzalloc(sizeof(struct smsc47b397_data), GFP_KERNEL))) {
+        data = kzalloc(sizeof(struct smsc47b397_data), GFP_KERNEL);
+        if (!data) {
                 err = -ENOMEM;
                 goto error_release;
         }
@@ -276,7 +277,8 @@ static int __devinit smsc47b397_probe(struct platform_device *pdev)
         mutex_init(&data->update_lock);
         platform_set_drvdata(pdev, data);
 
-        if ((err = sysfs_create_group(&dev->kobj, &smsc47b397_group)))
+        err = sysfs_create_group(&dev->kobj, &smsc47b397_group);
+        if (err)
                 goto error_free;
 
         data->hwmon_dev = hwmon_device_register(dev);
@@ -345,7 +347,7 @@ static int __init smsc47b397_find(unsigned short *addr)
         superio_enter();
         id = force_id ? force_id : superio_inb(SUPERIO_REG_DEVID);
 
-        switch(id) {
+        switch (id) {
         case 0x81:
                 name = "SCH5307-NS";
                 break;
@@ -379,7 +381,8 @@ static int __init smsc47b397_init(void)
         unsigned short address;
         int ret;
 
-        if ((ret = smsc47b397_find(&address)))
+        ret = smsc47b397_find(&address);
+        if (ret)
                 return ret;
 
         ret = platform_driver_register(&smsc47b397_driver);