hwmon: move from strlcpy with unused retval to strscpy

Follow the advice of the below link and prefer 'strscpy' in this
subsystem. Conversion is 1:1 because the return value is not used.
Generated by a coccinelle script.

Link: https://lore.kernel.org/r/CAHk-=wgfRnXz0W3D37d01q3JFkr_i_uTL=V6A6G1oUZcprmknw@mail.gmail.com/
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Link: https://lore.kernel.org/r/20220818210014.6769-1-wsa+renesas@sang-engineering.com
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
This commit is contained in:
Wolfram Sang 2022-08-18 23:00:11 +02:00 committed by Guenter Roeck
parent eb12f54876
commit f2f394db4b
56 changed files with 63 additions and 63 deletions

View File

@ -384,7 +384,7 @@ static int adc128_detect(struct i2c_client *client, struct i2c_board_info *info)
if (i2c_smbus_read_byte_data(client, ADC128_REG_BUSY_STATUS) & 0xfc)
return -ENODEV;
strlcpy(info->type, "adc128d818", I2C_NAME_SIZE);
strscpy(info->type, "adc128d818", I2C_NAME_SIZE);
return 0;
}

View File

@ -426,7 +426,7 @@ static int adm1021_detect(struct i2c_client *client,
pr_debug("Detected chip %s at adapter %d, address 0x%02x.\n",
type_name, i2c_adapter_id(adapter), client->addr);
strlcpy(info->type, type_name, I2C_NAME_SIZE);
strscpy(info->type, type_name, I2C_NAME_SIZE);
return 0;
}

View File

@ -470,7 +470,7 @@ static int adm1025_detect(struct i2c_client *client,
else
return -ENODEV;
strlcpy(info->type, name, I2C_NAME_SIZE);
strscpy(info->type, name, I2C_NAME_SIZE);
return 0;
}

View File

@ -1610,7 +1610,7 @@ static int adm1026_detect(struct i2c_client *client,
return -ENODEV;
}
strlcpy(info->type, "adm1026", I2C_NAME_SIZE);
strscpy(info->type, "adm1026", I2C_NAME_SIZE);
return 0;
}

View File

@ -329,7 +329,7 @@ static int adm1029_detect(struct i2c_client *client,
return -ENODEV;
}
strlcpy(info->type, "adm1029", I2C_NAME_SIZE);
strscpy(info->type, "adm1029", I2C_NAME_SIZE);
return 0;
}

View File

@ -985,7 +985,7 @@ static int adm1031_detect(struct i2c_client *client,
return -ENODEV;
name = (id == 0x30) ? "adm1030" : "adm1031";
strlcpy(info->type, name, I2C_NAME_SIZE);
strscpy(info->type, name, I2C_NAME_SIZE);
return 0;
}

View File

@ -590,7 +590,7 @@ static int adt7411_detect(struct i2c_client *client,
return -ENODEV;
}
strlcpy(info->type, "adt7411", I2C_NAME_SIZE);
strscpy(info->type, "adt7411", I2C_NAME_SIZE);
return 0;
}

View File

@ -1782,7 +1782,7 @@ static int adt7462_detect(struct i2c_client *client,
if (revision != ADT7462_REVISION)
return -ENODEV;
strlcpy(info->type, "adt7462", I2C_NAME_SIZE);
strscpy(info->type, "adt7462", I2C_NAME_SIZE);
return 0;
}

View File

@ -1342,7 +1342,7 @@ static int adt7475_detect(struct i2c_client *client,
return -ENODEV;
}
strlcpy(info->type, name, I2C_NAME_SIZE);
strscpy(info->type, name, I2C_NAME_SIZE);
return 0;
}

View File

@ -809,7 +809,7 @@ static int amc6821_detect(
}
dev_info(&adapter->dev, "amc6821: chip found at 0x%02x.\n", address);
strlcpy(info->type, "amc6821", I2C_NAME_SIZE);
strscpy(info->type, "amc6821", I2C_NAME_SIZE);
return 0;
}

View File

@ -769,7 +769,7 @@ static int asb100_detect(struct i2c_client *client,
if (val1 != 0x31 || val2 != 0x06)
return -ENODEV;
strlcpy(info->type, "asb100", I2C_NAME_SIZE);
strscpy(info->type, "asb100", I2C_NAME_SIZE);
return 0;
}

View File

@ -1153,7 +1153,7 @@ static int asc7621_detect(struct i2c_client *client,
if (company == asc7621_chips[chip_index].company_id &&
verstep == asc7621_chips[chip_index].verstep_id) {
strlcpy(info->type, asc7621_chips[chip_index].name,
strscpy(info->type, asc7621_chips[chip_index].name,
I2C_NAME_SIZE);
dev_info(&adapter->dev, "Matched %s at 0x%02x\n",

View File

@ -2456,7 +2456,7 @@ static int dme1737_i2c_detect(struct i2c_client *client,
dev_info(dev, "Found a %s chip at 0x%02x (rev 0x%02x).\n",
verstep == SCH5027_VERSTEP ? "SCH5027" : "DME1737",
client->addr, verstep);
strlcpy(info->type, name, I2C_NAME_SIZE);
strscpy(info->type, name, I2C_NAME_SIZE);
return 0;
}

View File

@ -329,22 +329,22 @@ static int emc1403_detect(struct i2c_client *client,
id = i2c_smbus_read_byte_data(client, THERMAL_PID_REG);
switch (id) {
case 0x20:
strlcpy(info->type, "emc1402", I2C_NAME_SIZE);
strscpy(info->type, "emc1402", I2C_NAME_SIZE);
break;
case 0x21:
strlcpy(info->type, "emc1403", I2C_NAME_SIZE);
strscpy(info->type, "emc1403", I2C_NAME_SIZE);
break;
case 0x22:
strlcpy(info->type, "emc1422", I2C_NAME_SIZE);
strscpy(info->type, "emc1422", I2C_NAME_SIZE);
break;
case 0x23:
strlcpy(info->type, "emc1423", I2C_NAME_SIZE);
strscpy(info->type, "emc1423", I2C_NAME_SIZE);
break;
case 0x25:
strlcpy(info->type, "emc1404", I2C_NAME_SIZE);
strscpy(info->type, "emc1404", I2C_NAME_SIZE);
break;
case 0x27:
strlcpy(info->type, "emc1424", I2C_NAME_SIZE);
strscpy(info->type, "emc1424", I2C_NAME_SIZE);
break;
default:
return -ENODEV;

View File

@ -643,7 +643,7 @@ emc2103_detect(struct i2c_client *new_client, struct i2c_board_info *info)
if ((product != 0x24) && (product != 0x26))
return -ENODEV;
strlcpy(info->type, "emc2103", I2C_NAME_SIZE);
strscpy(info->type, "emc2103", I2C_NAME_SIZE);
return 0;
}

View File

@ -439,7 +439,7 @@ static int emc6w201_detect(struct i2c_client *client,
return -ENODEV;
}
strlcpy(info->type, "emc6w201", I2C_NAME_SIZE);
strscpy(info->type, "emc6w201", I2C_NAME_SIZE);
return 0;
}

View File

@ -897,7 +897,7 @@ static int f75375_detect(struct i2c_client *client,
version = f75375_read8(client, F75375_REG_VERSION);
dev_info(&adapter->dev, "found %s version: %02X\n", name, version);
strlcpy(info->type, name, I2C_NAME_SIZE);
strscpy(info->type, name, I2C_NAME_SIZE);
return 0;
}

View File

@ -1075,7 +1075,7 @@ static int fschmd_detect(struct i2c_client *client,
else
return -ENODEV;
strlcpy(info->type, fschmd_id[kind].name, I2C_NAME_SIZE);
strscpy(info->type, fschmd_id[kind].name, I2C_NAME_SIZE);
return 0;
}

View File

@ -739,7 +739,7 @@ static int fts_detect(struct i2c_client *client,
if (val != 0x11)
return -ENODEV;
strlcpy(info->type, fts_id[0].name, I2C_NAME_SIZE);
strscpy(info->type, fts_id[0].name, I2C_NAME_SIZE);
info->flags = 0;
return 0;
}

View File

@ -586,7 +586,7 @@ static int gl518_detect(struct i2c_client *client, struct i2c_board_info *info)
if (rev != 0x00 && rev != 0x80)
return -ENODEV;
strlcpy(info->type, "gl518sm", I2C_NAME_SIZE);
strscpy(info->type, "gl518sm", I2C_NAME_SIZE);
return 0;
}

View File

@ -811,7 +811,7 @@ static int gl520_detect(struct i2c_client *client, struct i2c_board_info *info)
return -ENODEV;
}
strlcpy(info->type, "gl520sm", I2C_NAME_SIZE);
strscpy(info->type, "gl520sm", I2C_NAME_SIZE);
return 0;
}

View File

@ -441,7 +441,7 @@ static int jc42_detect(struct i2c_client *client, struct i2c_board_info *info)
struct jc42_chips *chip = &jc42_chips[i];
if (manid == chip->manid &&
(devid & chip->devid_mask) == chip->devid) {
strlcpy(info->type, "jc42", I2C_NAME_SIZE);
strscpy(info->type, "jc42", I2C_NAME_SIZE);
return 0;
}
}

View File

@ -996,11 +996,11 @@ static int lm63_detect(struct i2c_client *client,
}
if (chip_id == 0x41 && address == 0x4c)
strlcpy(info->type, "lm63", I2C_NAME_SIZE);
strscpy(info->type, "lm63", I2C_NAME_SIZE);
else if (chip_id == 0x51 && (address == 0x18 || address == 0x4e))
strlcpy(info->type, "lm64", I2C_NAME_SIZE);
strscpy(info->type, "lm64", I2C_NAME_SIZE);
else if (chip_id == 0x49 && address == 0x4c)
strlcpy(info->type, "lm96163", I2C_NAME_SIZE);
strscpy(info->type, "lm96163", I2C_NAME_SIZE);
else
return -ENODEV;

View File

@ -257,7 +257,7 @@ static int lm73_detect(struct i2c_client *new_client,
if (id < 0 || id != LM73_ID)
return -ENODEV;
strlcpy(info->type, "lm73", I2C_NAME_SIZE);
strscpy(info->type, "lm73", I2C_NAME_SIZE);
return 0;
}

View File

@ -893,7 +893,7 @@ static int lm75_detect(struct i2c_client *new_client,
return -ENODEV;
}
strlcpy(info->type, is_lm75a ? "lm75a" : "lm75", I2C_NAME_SIZE);
strscpy(info->type, is_lm75a ? "lm75a" : "lm75", I2C_NAME_SIZE);
return 0;
}

View File

@ -302,7 +302,7 @@ static int lm77_detect(struct i2c_client *client, struct i2c_board_info *info)
|| i2c_smbus_read_word_data(client, 7) != min)
return -ENODEV;
strlcpy(info->type, "lm77", I2C_NAME_SIZE);
strscpy(info->type, "lm77", I2C_NAME_SIZE);
return 0;
}

View File

@ -617,7 +617,7 @@ static int lm78_i2c_detect(struct i2c_client *client,
if (isa)
mutex_unlock(&isa->update_lock);
strlcpy(info->type, client_name, I2C_NAME_SIZE);
strscpy(info->type, client_name, I2C_NAME_SIZE);
return 0;

View File

@ -586,7 +586,7 @@ static int lm80_detect(struct i2c_client *client, struct i2c_board_info *info)
name = "lm80";
}
strlcpy(info->type, name, I2C_NAME_SIZE);
strscpy(info->type, name, I2C_NAME_SIZE);
return 0;
}

View File

@ -412,7 +412,7 @@ static int lm83_detect(struct i2c_client *client,
return -ENODEV;
}
strlcpy(info->type, name, I2C_NAME_SIZE);
strscpy(info->type, name, I2C_NAME_SIZE);
return 0;
}

View File

@ -1539,7 +1539,7 @@ static int lm85_detect(struct i2c_client *client, struct i2c_board_info *info)
if (!type_name)
return -ENODEV;
strlcpy(info->type, type_name, I2C_NAME_SIZE);
strscpy(info->type, type_name, I2C_NAME_SIZE);
return 0;
}

View File

@ -833,7 +833,7 @@ static int lm87_detect(struct i2c_client *client, struct i2c_board_info *info)
return -ENODEV;
}
strlcpy(info->type, name, I2C_NAME_SIZE);
strscpy(info->type, name, I2C_NAME_SIZE);
return 0;
}

View File

@ -2547,7 +2547,7 @@ static int lm90_detect(struct i2c_client *client, struct i2c_board_info *info)
return -ENODEV;
}
strlcpy(info->type, name, I2C_NAME_SIZE);
strscpy(info->type, name, I2C_NAME_SIZE);
return 0;
}

View File

@ -287,7 +287,7 @@ static int lm92_detect(struct i2c_client *new_client,
else
return -ENODEV;
strlcpy(info->type, "lm92", I2C_NAME_SIZE);
strscpy(info->type, "lm92", I2C_NAME_SIZE);
return 0;
}

View File

@ -2575,7 +2575,7 @@ static int lm93_detect(struct i2c_client *client, struct i2c_board_info *info)
return -ENODEV;
}
strlcpy(info->type, name, I2C_NAME_SIZE);
strscpy(info->type, name, I2C_NAME_SIZE);
dev_dbg(&adapter->dev, "loading %s at %d, 0x%02x\n",
client->name, i2c_adapter_id(client->adapter),
client->addr);

View File

@ -644,7 +644,7 @@ static int lm95234_detect(struct i2c_client *client,
if (val & model_mask)
return -ENODEV;
strlcpy(info->type, name, I2C_NAME_SIZE);
strscpy(info->type, name, I2C_NAME_SIZE);
return 0;
}

View File

@ -389,7 +389,7 @@ static int lm95241_detect(struct i2c_client *new_client,
}
/* Fill the i2c board info */
strlcpy(info->type, name, I2C_NAME_SIZE);
strscpy(info->type, name, I2C_NAME_SIZE);
return 0;
}

View File

@ -461,7 +461,7 @@ static int lm95245_detect(struct i2c_client *new_client,
return -ENODEV;
}
strlcpy(info->type, name, I2C_NAME_SIZE);
strscpy(info->type, name, I2C_NAME_SIZE);
return 0;
}

View File

@ -241,7 +241,7 @@ static int max1619_detect(struct i2c_client *client,
return -ENODEV;
}
strlcpy(info->type, "max1619", I2C_NAME_SIZE);
strscpy(info->type, "max1619", I2C_NAME_SIZE);
return 0;
}

View File

@ -386,7 +386,7 @@ static int max1668_detect(struct i2c_client *client,
if (!type_name)
return -ENODEV;
strlcpy(info->type, type_name, I2C_NAME_SIZE);
strscpy(info->type, type_name, I2C_NAME_SIZE);
return 0;
}

View File

@ -399,7 +399,7 @@ static int max31730_detect(struct i2c_client *client,
return -ENODEV;
}
strlcpy(info->type, "max31730", I2C_NAME_SIZE);
strscpy(info->type, "max31730", I2C_NAME_SIZE);
return 0;
}

View File

@ -514,7 +514,7 @@ static int max6639_detect(struct i2c_client *client,
if (dev_id != 0x58 || manu_id != 0x4D)
return -ENODEV;
strlcpy(info->type, "max6639", I2C_NAME_SIZE);
strscpy(info->type, "max6639", I2C_NAME_SIZE);
return 0;
}

View File

@ -148,7 +148,7 @@ static int max6642_detect(struct i2c_client *client,
if ((reg_status & 0x2b) != 0x00)
return -ENODEV;
strlcpy(info->type, "max6642", I2C_NAME_SIZE);
strscpy(info->type, "max6642", I2C_NAME_SIZE);
return 0;
}

View File

@ -1038,7 +1038,7 @@ static int nct7802_detect(struct i2c_client *client,
if (reg < 0 || (reg & 0x3f))
return -ENODEV;
strlcpy(info->type, "nct7802", I2C_NAME_SIZE);
strscpy(info->type, "nct7802", I2C_NAME_SIZE);
return 0;
}

View File

@ -798,7 +798,7 @@ static int nct7904_detect(struct i2c_client *client,
(i2c_smbus_read_byte_data(client, BANK_SEL_REG) & 0xf8) != 0x00)
return -ENODEV;
strlcpy(info->type, "nct7904", I2C_NAME_SIZE);
strscpy(info->type, "nct7904", I2C_NAME_SIZE);
return 0;
}

View File

@ -582,7 +582,7 @@ static int smsc47m192_detect(struct i2c_client *client,
return -ENODEV;
}
strlcpy(info->type, "smsc47m192", I2C_NAME_SIZE);
strscpy(info->type, "smsc47m192", I2C_NAME_SIZE);
return 0;
}

View File

@ -692,7 +692,7 @@ static int stts751_detect(struct i2c_client *new_client,
}
dev_dbg(&new_client->dev, "Chip %s detected", name);
strlcpy(info->type, stts751_id[0].name, I2C_NAME_SIZE);
strscpy(info->type, stts751_id[0].name, I2C_NAME_SIZE);
return 0;
}

View File

@ -352,7 +352,7 @@ static int thmc50_detect(struct i2c_client *client,
pr_debug("thmc50: Detected %s (version %x, revision %x)\n",
type_name, (revision >> 4) - 0xc, revision & 0xf);
strlcpy(info->type, type_name, I2C_NAME_SIZE);
strscpy(info->type, type_name, I2C_NAME_SIZE);
return 0;
}

View File

@ -671,7 +671,7 @@ static int tmp401_detect(struct i2c_client *client,
if (reg > 15)
return -ENODEV;
strlcpy(info->type, tmp401_id[kind].name, I2C_NAME_SIZE);
strscpy(info->type, tmp401_id[kind].name, I2C_NAME_SIZE);
return 0;
}

View File

@ -353,7 +353,7 @@ static int tmp421_detect(struct i2c_client *client,
return -ENODEV;
}
strlcpy(info->type, tmp421_id[kind].name, I2C_NAME_SIZE);
strscpy(info->type, tmp421_id[kind].name, I2C_NAME_SIZE);
dev_info(&adapter->dev, "Detected TI %s chip at 0x%02x\n",
names[kind], client->addr);

View File

@ -1171,7 +1171,7 @@ w83781d_detect(struct i2c_client *client, struct i2c_board_info *info)
if (isa)
mutex_unlock(&isa->update_lock);
strlcpy(info->type, client_name, I2C_NAME_SIZE);
strscpy(info->type, client_name, I2C_NAME_SIZE);
return 0;

View File

@ -1333,7 +1333,7 @@ static int w83791d_detect(struct i2c_client *client,
if (val1 != 0x71 || val2 != 0x5c)
return -ENODEV;
strlcpy(info->type, "w83791d", I2C_NAME_SIZE);
strscpy(info->type, "w83791d", I2C_NAME_SIZE);
return 0;
}

View File

@ -1346,7 +1346,7 @@ w83792d_detect(struct i2c_client *client, struct i2c_board_info *info)
if (val1 != 0x7a || val2 != 0x5c)
return -ENODEV;
strlcpy(info->type, "w83792d", I2C_NAME_SIZE);
strscpy(info->type, "w83792d", I2C_NAME_SIZE);
return 0;
}

View File

@ -1636,7 +1636,7 @@ static int w83793_detect(struct i2c_client *client,
if (chip_id != 0x7b)
return -ENODEV;
strlcpy(info->type, "w83793", I2C_NAME_SIZE);
strscpy(info->type, "w83793", I2C_NAME_SIZE);
return 0;
}

View File

@ -1967,7 +1967,7 @@ static int w83795_detect(struct i2c_client *client,
else
chip_name = "w83795g";
strlcpy(info->type, chip_name, I2C_NAME_SIZE);
strscpy(info->type, chip_name, I2C_NAME_SIZE);
dev_info(&adapter->dev, "Found %s rev. %c at 0x%02hx\n", chip_name,
'A' + (device_id & 0xf), address);

View File

@ -157,7 +157,7 @@ static int w83l785ts_detect(struct i2c_client *client,
return -ENODEV;
}
strlcpy(info->type, "w83l785ts", I2C_NAME_SIZE);
strscpy(info->type, "w83l785ts", I2C_NAME_SIZE);
return 0;
}

View File

@ -687,7 +687,7 @@ w83l786ng_detect(struct i2c_client *client, struct i2c_board_info *info)
return -ENODEV;
}
strlcpy(info->type, "w83l786ng", I2C_NAME_SIZE);
strscpy(info->type, "w83l786ng", I2C_NAME_SIZE);
return 0;
}