hwmon: (smsc47b397) Fix checkpatch errors
authorJean Delvare <khali@linux-fr.org>
Fri, 4 Nov 2011 11:00:47 +0000 (12:00 +0100)
committerJean Delvare <khali@endymion.delvare>
Fri, 4 Nov 2011 11:00:47 +0000 (12:00 +0100)
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Acked-by: Guenter Roeck <guenter.roeck@ericsson.com>
drivers/hwmon/smsc47b397.c

index 9fb7516e6f451284419559fdf4faf3e8fffbeaf9..65c88ff5645ac6d63efb383536a0169e92635f7b 100644 (file)
@@ -113,7 +113,7 @@ struct smsc47b397_data {
        u8 temp[4];
 };
 
-static int smsc47b397_read_value(struct smsc47b397_datadata, 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);