summaryrefslogtreecommitdiffstats
path: root/drivers/hwmon/w83l785ts.c
diff options
context:
space:
mode:
authorJean Delvare <jdelvare@suse.de>2014-04-04 18:01:34 +0200
committerJean Delvare <jdelvare@suse.de>2014-04-04 18:01:34 +0200
commit6cf0a91ebe11cee1df871619af4eb58c9af32666 (patch)
treeace57336a6a44dd606466c9ae79b7998c1a2f2ec /drivers/hwmon/w83l785ts.c
parenthwmon: (pc87360) Avoid initializing the same field twice (diff)
downloadlinux-6cf0a91ebe11cee1df871619af4eb58c9af32666.tar.xz
linux-6cf0a91ebe11cee1df871619af4eb58c9af32666.zip
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 '')
-rw-r--r--drivers/hwmon/w83l785ts.c4
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.