diff options
| author | Jean Delvare <jdelvare@suse.de> | 2014-04-04 18:01:34 +0200 |
|---|---|---|
| committer | Jean Delvare <jdelvare@suse.de> | 2014-04-04 18:01:34 +0200 |
| commit | 6cf0a91ebe11cee1df871619af4eb58c9af32666 (patch) | |
| tree | ace57336a6a44dd606466c9ae79b7998c1a2f2ec /drivers/hwmon/w83l785ts.c | |
| parent | 5b58157598f93089ccda90aabd98f27865b344da (diff) | |
| download | linux-6cf0a91ebe11cee1df871619af4eb58c9af32666.tar.bz2 | |
hwmon: Avoid initializing the same field twice
All hwmon drivers allocate their data structure with some form of
kzalloc, so setting data fields to zero explicitly is a waste of time.
Signed-off-by: Jean Delvare <jdelvare@suse.de>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Diffstat (limited to 'drivers/hwmon/w83l785ts.c')
| -rw-r--r-- | drivers/hwmon/w83l785ts.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/drivers/hwmon/w83l785ts.c b/drivers/hwmon/w83l785ts.c index 6384b268f590..ac3043122011 100644 --- a/drivers/hwmon/w83l785ts.c +++ b/drivers/hwmon/w83l785ts.c @@ -188,12 +188,8 @@ static int w83l785ts_probe(struct i2c_client *client, return -ENOMEM; i2c_set_clientdata(client, data); - data->valid = 0; mutex_init(&data->update_lock); - /* Default values in case the first read fails (unlikely). */ - data->temp[1] = data->temp[0] = 0; - /* * Initialize the W83L785TS chip * Nothing yet, assume it is already started. |