Commit 28cbd461 authored by Jean Delvare's avatar Jean Delvare

hwmon: (asc7621) Clean up and improve detect function

* The dev variable is never used.
* Detect functions only need to set info->type, not client->name.
* Include the device address in the log message.
Signed-off-by: default avatarJean Delvare <khali@linux-fr.org>
Cc: George Joseph <george.joseph@fairview5.com>
Cc: Ken Milmore <ken.milmore@googlemail.com>
parent 738e5e05
...@@ -1150,9 +1150,6 @@ static int asc7621_detect(struct i2c_client *client, ...@@ -1150,9 +1150,6 @@ static int asc7621_detect(struct i2c_client *client,
{ {
struct i2c_adapter *adapter = client->adapter; struct i2c_adapter *adapter = client->adapter;
int company, verstep, chip_index; int company, verstep, chip_index;
struct device *dev;
dev = &client->dev;
if (!i2c_check_functionality(adapter, I2C_FUNC_SMBUS_BYTE_DATA)) if (!i2c_check_functionality(adapter, I2C_FUNC_SMBUS_BYTE_DATA))
return -ENODEV; return -ENODEV;
...@@ -1169,13 +1166,11 @@ static int asc7621_detect(struct i2c_client *client, ...@@ -1169,13 +1166,11 @@ static int asc7621_detect(struct i2c_client *client,
if (company == asc7621_chips[chip_index].company_id && if (company == asc7621_chips[chip_index].company_id &&
verstep == asc7621_chips[chip_index].verstep_id) { verstep == asc7621_chips[chip_index].verstep_id) {
strlcpy(client->name, asc7621_chips[chip_index].name,
I2C_NAME_SIZE);
strlcpy(info->type, asc7621_chips[chip_index].name, strlcpy(info->type, asc7621_chips[chip_index].name,
I2C_NAME_SIZE); I2C_NAME_SIZE);
dev_info(&adapter->dev, "Matched %s\n", dev_info(&adapter->dev, "Matched %s at 0x%02x\n",
asc7621_chips[chip_index].name); asc7621_chips[chip_index].name, client->addr);
return 0; return 0;
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment