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;
 
                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;
        }
        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);
        superio_enter();
        id = force_id ? force_id : superio_inb(SUPERIO_REG_DEVID);
 
-       switch(id) {
+       switch (id) {
        case 0x81:
                name = "SCH5307-NS";
                break;
        unsigned short address;
        int ret;
 
-       if ((ret = smsc47b397_find(&address)))
+       ret = smsc47b397_find(&address);
+       if (ret)
                return ret;
 
        ret = platform_driver_register(&smsc47b397_driver);