From 6cf0a91ebe11cee1df871619af4eb58c9af32666 Mon Sep 17 00:00:00 2001 From: Jean Delvare Date: Fri, 4 Apr 2014 18:01:34 +0200 Subject: 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 Reviewed-by: Guenter Roeck --- drivers/hwmon/w83l785ts.c | 4 ---- 1 file changed, 4 deletions(-) (limited to 'drivers/hwmon/w83l785ts.c') 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. -- cgit v1.2.3-59-g8ed1b