iio: convert to common i2c_check_functionality() return value
Subsystems (5)
PATCH 1/1 iio: convert to common i2c_check_functionality() return value
Previously most drivers that used a i2c_check_functionality() check condition required various error codes on failure. This patchset converts to a standard of -EOPNOTSUPP
--- drivers/iio/adc/mcp3422.c | 2 +- drivers/iio/adc/ti-adc081c.c | 2 +- drivers/iio/chemical/vz89x.c | 2 +- drivers/iio/humidity/hdc100x.c | 2 +- drivers/iio/humidity/htu21.c | 2 +- drivers/iio/humidity/si7005.c | 2 +- drivers/iio/humidity/si7020.c | 2 +- drivers/iio/imu/inv_mpu6050/inv_mpu_i2c.c | 2 +- drivers/iio/light/bh1750.c | 2 +- drivers/iio/light/jsa1212.c | 2 +- drivers/iio/potentiometer/mcp4531.c | 2 +- drivers/iio/pressure/mpl115_i2c.c | 2 +- drivers/iio/pressure/ms5611_i2c.c | 2 +- drivers/iio/pressure/ms5637.c | 2 +- drivers/iio/pressure/t5403.c | 2 +- drivers/iio/proximity/pulsedlight-lidar-lite-v2.c | 2 +- drivers/iio/temperature/mlx90614.c | 2 +- drivers/iio/temperature/tmp006.c | 2 +- drivers/iio/temperature/tsys01.c | 2 +- drivers/iio/temperature/tsys02d.c | 2 +- 20 files changed, 20 insertions(+), 20 deletions(-)
drivers/iio/adc/mcp3422.c +1 -1
@@ -339,7 +339,7 @@ static int mcp3422_probe(struct i2c_client *client,u8 config;if (!i2c_check_functionality (client->adapter,I2C_FUNC_I2C )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*adc));if (!indio_dev)
drivers/iio/adc/ti-adc081c.c +1 -1
@@ -73,7 +73,7 @@ static int adc081c_probe(struct i2c_client *client,int err;if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ; iio =devm_iio_device_alloc (&client->dev,sizeof (*adc));if (!iio)
drivers/iio/chemical/vz89x.c +1 -1
@@ -249,7 +249,7 @@ static int vz89x_probe(struct i2c_client *client, I2C_FUNC_SMBUS_WORD_DATA | I2C_FUNC_SMBUS_BYTE)) data->xfer = vz89x_smbus_xfer; else - return -ENOTSUPP; + return -EOPNOTSUPP;i2c_set_clientdata (client, indio_dev); data->client = client;
drivers/iio/humidity/hdc100x.c +1 -1
@@ -274,7 +274,7 @@ static int hdc100x_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA |I2C_FUNC_SMBUS_BYTE )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*data));if (!indio_dev)
drivers/iio/humidity/htu21.c +1 -1
@@ -192,7 +192,7 @@ static int htu21_probe(struct i2c_client *client, I2C_FUNC_SMBUS_READ_I2C_BLOCK)) {dev_err (&client->dev,"Adapter does not support some i2c transaction\n" ); -return -ENODEV ; +return -EOPNOTSUPP ; } indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*dev_data));
drivers/iio/humidity/si7005.c +1 -1
@@ -135,7 +135,7 @@ static int si7005_probe(struct i2c_client *client,int ret;if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*data));if (!indio_dev)
drivers/iio/humidity/si7020.c +1 -1
@@ -121,7 +121,7 @@ static int si7020_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WRITE_BYTE |I2C_FUNC_SMBUS_READ_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ;/* Reset device, loads default settings. */ ret =i2c_smbus_write_byte (client,SI7020CMD_RESET );
drivers/iio/imu/inv_mpu6050/inv_mpu_i2c.c +1 -1
@@ -120,7 +120,7 @@ static int inv_mpu_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_I2C_BLOCK )) -return -ENOSYS ; +return -EOPNOTSUPP ; regmap =devm_regmap_init_i2c (client, &inv_mpu_regmap_config);if (IS_ERR (regmap)) {
drivers/iio/light/bh1750.c +1 -1
@@ -241,7 +241,7 @@ static int bh1750_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_I2C |I2C_FUNC_SMBUS_WRITE_BYTE )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*data));if (!indio_dev)
drivers/iio/light/jsa1212.c +1 -1
@@ -326,7 +326,7 @@ static int jsa1212_probe(struct i2c_client *client,int ret;if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_BYTE_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*data));if (!indio_dev)
drivers/iio/potentiometer/mcp4531.c +1 -1
@@ -159,7 +159,7 @@ static int mcp4531_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) {dev_err (dev,"SMBUS Word Data not supported\n" ); -return -EIO ; +return -EOPNOTSUPP ; } indio_dev =devm_iio_device_alloc (dev,sizeof (*data));
drivers/iio/pressure/mpl115_i2c.c +1 -1
@@ -42,7 +42,7 @@ static int mpl115_i2c_probe(struct i2c_client *client, const struct i2c_device_id *id) {if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ;return mpl115_probe (&client->dev, id->name, &mpl115_i2c_ops); }
drivers/iio/pressure/ms5611_i2c.c +1 -1
@@ -92,7 +92,7 @@ static int ms5611_i2c_probe(struct i2c_client *client,I2C_FUNC_SMBUS_WRITE_BYTE |I2C_FUNC_SMBUS_READ_WORD_DATA | I2C_FUNC_SMBUS_READ_I2C_BLOCK)) - return -ENODEV; + return -EOPNOTSUPP; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*st));if (!indio_dev)
drivers/iio/pressure/ms5637.c +1 -1
@@ -136,7 +136,7 @@ static int ms5637_probe(struct i2c_client *client, I2C_FUNC_SMBUS_READ_I2C_BLOCK)) {dev_err (&client->dev,"Adapter does not support some i2c transaction\n" ); -return -ENODEV ; +return -EOPNOTSUPP ; } indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*dev_data));
drivers/iio/pressure/t5403.c +1 -1
@@ -221,7 +221,7 @@ static int t5403_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA |I2C_FUNC_SMBUS_I2C_BLOCK )) -return -ENODEV ; +return -EOPNOTSUPP ; ret =i2c_smbus_read_byte_data (client,T5403_SLAVE_ADDR );if (ret <0 )
drivers/iio/proximity/pulsedlight-lidar-lite-v2.c +1 -1
@@ -278,7 +278,7 @@ static int lidar_probe(struct i2c_client *client, I2C_FUNC_SMBUS_WORD_DATA | I2C_FUNC_SMBUS_BYTE)) data->xfer = lidar_smbus_xfer; else - return -ENOTSUPP; + return -EOPNOTSUPP; indio_dev->info = &lidar_info; indio_dev->name =LIDAR_DRV_NAME ;
drivers/iio/temperature/mlx90614.c +1 -1
@@ -516,7 +516,7 @@ static int mlx90614_probe(struct i2c_client *client,int ret;if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*data));if (!indio_dev)
drivers/iio/temperature/tmp006.c +1 -1
@@ -205,7 +205,7 @@ static int tmp006_probe(struct i2c_client *client,int ret;if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ;if (!tmp006_check_identification (client)) {dev_err (&client->dev,"no TMP006 sensor\n" );
drivers/iio/temperature/tsys01.c +1 -1
@@ -190,7 +190,7 @@ static int tsys01_i2c_probe(struct i2c_client *client, I2C_FUNC_SMBUS_READ_I2C_BLOCK)) {dev_err (&client->dev,"Adapter does not support some i2c transaction\n" ); -return -ENODEV ; +return -EOPNOTSUPP ; } indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*dev_data));
drivers/iio/temperature/tsys02d.c +1 -1
@@ -137,7 +137,7 @@ static int tsys02d_probe(struct i2c_client *client, I2C_FUNC_SMBUS_READ_I2C_BLOCK)) {dev_err (&client->dev,"Adapter does not support some i2c transaction\n" ); -return -ENODEV ; +return -EOPNOTSUPP ; } indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*dev_data));
Signature · 1 line
2.7.0
Quoted 5 lines · Matt Ranostay
Previously most drivers that used a
i2c_check_functionality()check condition required various error codes on failure. This patchset converts to a standard of-EOPNOTSUPPSigned-off-by Matt Ranostay <[email protected]>
Applied.
Thanks,
Jonathan
Quoted 284 lines
--- drivers/iio/adc/mcp3422.c | 2 +- drivers/iio/adc/ti-adc081c.c | 2 +- drivers/iio/chemical/vz89x.c | 2 +- drivers/iio/humidity/hdc100x.c | 2 +- drivers/iio/humidity/htu21.c | 2 +- drivers/iio/humidity/si7005.c | 2 +- drivers/iio/humidity/si7020.c | 2 +- drivers/iio/imu/inv_mpu6050/inv_mpu_i2c.c | 2 +- drivers/iio/light/bh1750.c | 2 +- drivers/iio/light/jsa1212.c | 2 +- drivers/iio/potentiometer/mcp4531.c | 2 +- drivers/iio/pressure/mpl115_i2c.c | 2 +- drivers/iio/pressure/ms5611_i2c.c | 2 +- drivers/iio/pressure/ms5637.c | 2 +- drivers/iio/pressure/t5403.c | 2 +- drivers/iio/proximity/pulsedlight-lidar-lite-v2.c | 2 +- drivers/iio/temperature/mlx90614.c | 2 +- drivers/iio/temperature/tmp006.c | 2 +- drivers/iio/temperature/tsys01.c | 2 +- drivers/iio/temperature/tsys02d.c | 2 +- 20 files changed, 20 insertions(+), 20 deletions(-)drivers/iio/adc/mcp3422.c +1
-1@@ -339,7 +339,7 @@ static int mcp3422_probe(struct i2c_client *client,u8 config;if (!i2c_check_functionality (client->adapter,I2C_FUNC_I2C )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*adc));if (!indio_dev)drivers/iio/adc/ti-adc081c.c +1
-1@@ -73,7 +73,7 @@ static int adc081c_probe(struct i2c_client *client,int err;if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ; iio =devm_iio_device_alloc (&client->dev,sizeof (*adc));if (!iio)drivers/iio/chemical/vz89x.c +1
-1@@ -249,7 +249,7 @@ static int vz89x_probe(struct i2c_client *client, I2C_FUNC_SMBUS_WORD_DATA | I2C_FUNC_SMBUS_BYTE)) data->xfer = vz89x_smbus_xfer; else - return -ENOTSUPP; + return -EOPNOTSUPP;i2c_set_clientdata (client, indio_dev); data->client = client;drivers/iio/humidity/hdc100x.c +1
-1@@ -274,7 +274,7 @@ static int hdc100x_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA |I2C_FUNC_SMBUS_BYTE )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*data));if (!indio_dev)drivers/iio/humidity/htu21.c +1
-1@@ -192,7 +192,7 @@ static int htu21_probe(struct i2c_client *client, I2C_FUNC_SMBUS_READ_I2C_BLOCK)) {dev_err (&client->dev,"Adapter does not support some i2c transaction\n" ); -return -ENODEV ; +return -EOPNOTSUPP ; } indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*dev_data));drivers/iio/humidity/si7005.c +1
-1@@ -135,7 +135,7 @@ static int si7005_probe(struct i2c_client *client,int ret;if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*data));if (!indio_dev)drivers/iio/humidity/si7020.c +1
-1@@ -121,7 +121,7 @@ static int si7020_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WRITE_BYTE |I2C_FUNC_SMBUS_READ_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ;/* Reset device, loads default settings. */ ret =i2c_smbus_write_byte (client,SI7020CMD_RESET );drivers/iio/imu/inv_mpu6050/inv_mpu_i2c.c +1
-1@@ -120,7 +120,7 @@ static int inv_mpu_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_I2C_BLOCK )) -return -ENOSYS ; +return -EOPNOTSUPP ; regmap =devm_regmap_init_i2c (client, &inv_mpu_regmap_config);if (IS_ERR (regmap)) {drivers/iio/light/bh1750.c +1
-1@@ -241,7 +241,7 @@ static int bh1750_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_I2C |I2C_FUNC_SMBUS_WRITE_BYTE )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*data));if (!indio_dev)drivers/iio/light/jsa1212.c +1
-1@@ -326,7 +326,7 @@ static int jsa1212_probe(struct i2c_client *client,int ret;if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_BYTE_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*data));if (!indio_dev)drivers/iio/potentiometer/mcp4531.c +1
-1@@ -159,7 +159,7 @@ static int mcp4531_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) {dev_err (dev,"SMBUS Word Data not supported\n" ); -return -EIO ; +return -EOPNOTSUPP ; } indio_dev =devm_iio_device_alloc (dev,sizeof (*data));drivers/iio/pressure/mpl115_i2c.c +1
-1@@ -42,7 +42,7 @@ static int mpl115_i2c_probe(struct i2c_client *client, const struct i2c_device_id *id) {if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ;return mpl115_probe (&client->dev, id->name, &mpl115_i2c_ops); }drivers/iio/pressure/ms5611_i2c.c +1
-1@@ -92,7 +92,7 @@ static int ms5611_i2c_probe(struct i2c_client *client,I2C_FUNC_SMBUS_WRITE_BYTE |I2C_FUNC_SMBUS_READ_WORD_DATA | I2C_FUNC_SMBUS_READ_I2C_BLOCK)) - return -ENODEV; + return -EOPNOTSUPP; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*st));if (!indio_dev)drivers/iio/pressure/ms5637.c +1
-1@@ -136,7 +136,7 @@ static int ms5637_probe(struct i2c_client *client, I2C_FUNC_SMBUS_READ_I2C_BLOCK)) {dev_err (&client->dev,"Adapter does not support some i2c transaction\n" ); -return -ENODEV ; +return -EOPNOTSUPP ; } indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*dev_data));drivers/iio/pressure/t5403.c +1
-1@@ -221,7 +221,7 @@ static int t5403_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA |I2C_FUNC_SMBUS_I2C_BLOCK )) -return -ENODEV ; +return -EOPNOTSUPP ; ret =i2c_smbus_read_byte_data (client,T5403_SLAVE_ADDR );if (ret <0 )drivers/iio/proximity/pulsedlight-lidar-lite-v2.c +1
-1@@ -278,7 +278,7 @@ static int lidar_probe(struct i2c_client *client, I2C_FUNC_SMBUS_WORD_DATA | I2C_FUNC_SMBUS_BYTE)) data->xfer = lidar_smbus_xfer; else - return -ENOTSUPP; + return -EOPNOTSUPP; indio_dev->info = &lidar_info; indio_dev->name =LIDAR_DRV_NAME ;drivers/iio/temperature/mlx90614.c +1
-1@@ -516,7 +516,7 @@ static int mlx90614_probe(struct i2c_client *client,int ret;if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*data));if (!indio_dev)drivers/iio/temperature/tmp006.c +1
-1@@ -205,7 +205,7 @@ static int tmp006_probe(struct i2c_client *client,int ret;if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ;if (!tmp006_check_identification (client)) {dev_err (&client->dev,"no TMP006 sensor\n" );drivers/iio/temperature/tsys01.c +1
-1@@ -190,7 +190,7 @@ static int tsys01_i2c_probe(struct i2c_client *client, I2C_FUNC_SMBUS_READ_I2C_BLOCK)) {dev_err (&client->dev,"Adapter does not support some i2c transaction\n" ); -return -ENODEV ; +return -EOPNOTSUPP ; } indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*dev_data));drivers/iio/temperature/tsys02d.c +1
-1@@ -137,7 +137,7 @@ static int tsys02d_probe(struct i2c_client *client, I2C_FUNC_SMBUS_READ_I2C_BLOCK)) {dev_err (&client->dev,"Adapter does not support some i2c transaction\n" ); -return -ENODEV ; +return -EOPNOTSUPP ; } indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*dev_data));
I hope it is not too late to add:
Quoted 301 lines · Jonathan Cameron
Quoted 5 lines · Matt Ranostay
Previously most drivers that used a
i2c_check_functionality()check condition required various error codes on failure. This patchset converts to a standard of-EOPNOTSUPPSigned-off-by Matt Ranostay <[email protected]>Applied.
Thanks,
Jonathan
Quoted 276 lines
--- drivers/iio/adc/mcp3422.c | 2 +- drivers/iio/adc/ti-adc081c.c | 2 +- drivers/iio/chemical/vz89x.c | 2 +- drivers/iio/humidity/hdc100x.c | 2 +- drivers/iio/humidity/htu21.c | 2 +- drivers/iio/humidity/si7005.c | 2 +- drivers/iio/humidity/si7020.c | 2 +- drivers/iio/imu/inv_mpu6050/inv_mpu_i2c.c | 2 +- drivers/iio/light/bh1750.c | 2 +- drivers/iio/light/jsa1212.c | 2 +- drivers/iio/potentiometer/mcp4531.c | 2 +- drivers/iio/pressure/mpl115_i2c.c | 2 +- drivers/iio/pressure/ms5611_i2c.c | 2 +- drivers/iio/pressure/ms5637.c | 2 +- drivers/iio/pressure/t5403.c | 2 +- drivers/iio/proximity/pulsedlight-lidar-lite-v2.c | 2 +- drivers/iio/temperature/mlx90614.c | 2 +- drivers/iio/temperature/tmp006.c | 2 +- drivers/iio/temperature/tsys01.c | 2 +- drivers/iio/temperature/tsys02d.c | 2 +- 20 files changed, 20 insertions(+), 20 deletions(-)drivers/iio/adc/mcp3422.c +1
-1@@ -339,7 +339,7 @@ static int mcp3422_probe(struct i2c_client *client,u8 config;if (!i2c_check_functionality (client->adapter,I2C_FUNC_I2C )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*adc));if (!indio_dev)drivers/iio/adc/ti-adc081c.c +1
-1@@ -73,7 +73,7 @@ static int adc081c_probe(struct i2c_client *client,int err;if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ; iio =devm_iio_device_alloc (&client->dev,sizeof (*adc));if (!iio)drivers/iio/chemical/vz89x.c +1
-1@@ -249,7 +249,7 @@ static int vz89x_probe(struct i2c_client *client, I2C_FUNC_SMBUS_WORD_DATA | I2C_FUNC_SMBUS_BYTE)) data->xfer = vz89x_smbus_xfer; else - return -ENOTSUPP; + return -EOPNOTSUPP;i2c_set_clientdata (client, indio_dev); data->client = client;drivers/iio/humidity/hdc100x.c +1
-1@@ -274,7 +274,7 @@ static int hdc100x_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA |I2C_FUNC_SMBUS_BYTE )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*data));if (!indio_dev)drivers/iio/humidity/htu21.c +1
-1@@ -192,7 +192,7 @@ static int htu21_probe(struct i2c_client *client, I2C_FUNC_SMBUS_READ_I2C_BLOCK)) {dev_err (&client->dev,"Adapter does not support some i2c transaction\n" ); -return -ENODEV ; +return -EOPNOTSUPP ; } indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*dev_data));drivers/iio/humidity/si7005.c +1
-1@@ -135,7 +135,7 @@ static int si7005_probe(struct i2c_client *client,int ret;if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*data));if (!indio_dev)drivers/iio/humidity/si7020.c +1
-1@@ -121,7 +121,7 @@ static int si7020_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WRITE_BYTE |I2C_FUNC_SMBUS_READ_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ;/* Reset device, loads default settings. */ ret =i2c_smbus_write_byte (client,SI7020CMD_RESET );drivers/iio/imu/inv_mpu6050/inv_mpu_i2c.c +1
-1@@ -120,7 +120,7 @@ static int inv_mpu_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_I2C_BLOCK )) -return -ENOSYS ; +return -EOPNOTSUPP ; regmap =devm_regmap_init_i2c (client, &inv_mpu_regmap_config);if (IS_ERR (regmap)) {drivers/iio/light/bh1750.c +1
-1@@ -241,7 +241,7 @@ static int bh1750_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_I2C |I2C_FUNC_SMBUS_WRITE_BYTE )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*data));if (!indio_dev)drivers/iio/light/jsa1212.c +1
-1@@ -326,7 +326,7 @@ static int jsa1212_probe(struct i2c_client *client,int ret;if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_BYTE_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*data));if (!indio_dev)drivers/iio/potentiometer/mcp4531.c +1
-1@@ -159,7 +159,7 @@ static int mcp4531_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) {dev_err (dev,"SMBUS Word Data not supported\n" ); -return -EIO ; +return -EOPNOTSUPP ; } indio_dev =devm_iio_device_alloc (dev,sizeof (*data));drivers/iio/pressure/mpl115_i2c.c +1
-1@@ -42,7 +42,7 @@ static int mpl115_i2c_probe(struct i2c_client *client, const struct i2c_device_id *id) {if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ;return mpl115_probe (&client->dev, id->name, &mpl115_i2c_ops); }drivers/iio/pressure/ms5611_i2c.c +1
-1@@ -92,7 +92,7 @@ static int ms5611_i2c_probe(struct i2c_client *client,I2C_FUNC_SMBUS_WRITE_BYTE |I2C_FUNC_SMBUS_READ_WORD_DATA | I2C_FUNC_SMBUS_READ_I2C_BLOCK)) - return -ENODEV; + return -EOPNOTSUPP; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*st));if (!indio_dev)drivers/iio/pressure/ms5637.c +1
-1@@ -136,7 +136,7 @@ static int ms5637_probe(struct i2c_client *client, I2C_FUNC_SMBUS_READ_I2C_BLOCK)) {dev_err (&client->dev,"Adapter does not support some i2c transaction\n" ); -return -ENODEV ; +return -EOPNOTSUPP ; } indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*dev_data));drivers/iio/pressure/t5403.c +1
-1@@ -221,7 +221,7 @@ static int t5403_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA |I2C_FUNC_SMBUS_I2C_BLOCK )) -return -ENODEV ; +return -EOPNOTSUPP ; ret =i2c_smbus_read_byte_data (client,T5403_SLAVE_ADDR );if (ret <0 )drivers/iio/proximity/pulsedlight-lidar-lite-v2.c +1
-1@@ -278,7 +278,7 @@ static int lidar_probe(struct i2c_client *client, I2C_FUNC_SMBUS_WORD_DATA | I2C_FUNC_SMBUS_BYTE)) data->xfer = lidar_smbus_xfer; else - return -ENOTSUPP; + return -EOPNOTSUPP; indio_dev->info = &lidar_info; indio_dev->name =LIDAR_DRV_NAME ;drivers/iio/temperature/mlx90614.c +1
-1@@ -516,7 +516,7 @@ static int mlx90614_probe(struct i2c_client *client,int ret;if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*data));if (!indio_dev)drivers/iio/temperature/tmp006.c +1
-1@@ -205,7 +205,7 @@ static int tmp006_probe(struct i2c_client *client,int ret;if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ;if (!tmp006_check_identification (client)) {dev_err (&client->dev,"no TMP006 sensor\n" );drivers/iio/temperature/tsys01.c +1
-1@@ -190,7 +190,7 @@ static int tsys01_i2c_probe(struct i2c_client *client, I2C_FUNC_SMBUS_READ_I2C_BLOCK)) {dev_err (&client->dev,"Adapter does not support some i2c transaction\n" ); -return -ENODEV ; +return -EOPNOTSUPP ; } indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*dev_data));drivers/iio/temperature/tsys02d.c +0
-0drivers/iio/temperature/tsys02d.c static int tsys02d_probe(struct i2c_client *client, · line 143
Quoted 8 lines
drivers/iio/temperature/tsys02d.c +1
-1@@ -137,7 +137,7 @@ static int tsys02d_probe(struct i2c_client *client, I2C_FUNC_SMBUS_READ_I2C_BLOCK)) {dev_err (&client->dev,"Adapter does not support some i2c transaction\n" ); -return -ENODEV ; +return -EOPNOTSUPP ; } indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*dev_data));--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Quoted 2 lines · Crt Mori
I hope it is not too late to add:
Acked-by Crt Mori <[email protected]>
Sorry, pull request went out yesterday so can't add it now. I wanted to sneak this into the current cycle to make our lives easier long term!
Jonathan
Quoted 303 lines · Jonathan Cameron
Quoted 301 lines · Matt Ranostay
Quoted 5 lines
Previously most drivers that used a
i2c_check_functionality()check condition required various error codes on failure. This patchset converts to a standard of-EOPNOTSUPPSigned-off-by Matt Ranostay <[email protected]>Applied.
Thanks,
Jonathan
Quoted 276 lines
--- drivers/iio/adc/mcp3422.c | 2 +- drivers/iio/adc/ti-adc081c.c | 2 +- drivers/iio/chemical/vz89x.c | 2 +- drivers/iio/humidity/hdc100x.c | 2 +- drivers/iio/humidity/htu21.c | 2 +- drivers/iio/humidity/si7005.c | 2 +- drivers/iio/humidity/si7020.c | 2 +- drivers/iio/imu/inv_mpu6050/inv_mpu_i2c.c | 2 +- drivers/iio/light/bh1750.c | 2 +- drivers/iio/light/jsa1212.c | 2 +- drivers/iio/potentiometer/mcp4531.c | 2 +- drivers/iio/pressure/mpl115_i2c.c | 2 +- drivers/iio/pressure/ms5611_i2c.c | 2 +- drivers/iio/pressure/ms5637.c | 2 +- drivers/iio/pressure/t5403.c | 2 +- drivers/iio/proximity/pulsedlight-lidar-lite-v2.c | 2 +- drivers/iio/temperature/mlx90614.c | 2 +- drivers/iio/temperature/tmp006.c | 2 +- drivers/iio/temperature/tsys01.c | 2 +- drivers/iio/temperature/tsys02d.c | 2 +- 20 files changed, 20 insertions(+), 20 deletions(-)drivers/iio/adc/mcp3422.c +1
-1@@ -339,7 +339,7 @@ static int mcp3422_probe(struct i2c_client *client,u8 config;if (!i2c_check_functionality (client->adapter,I2C_FUNC_I2C )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*adc));if (!indio_dev)drivers/iio/adc/ti-adc081c.c +1
-1@@ -73,7 +73,7 @@ static int adc081c_probe(struct i2c_client *client,int err;if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ; iio =devm_iio_device_alloc (&client->dev,sizeof (*adc));if (!iio)drivers/iio/chemical/vz89x.c +1
-1@@ -249,7 +249,7 @@ static int vz89x_probe(struct i2c_client *client, I2C_FUNC_SMBUS_WORD_DATA | I2C_FUNC_SMBUS_BYTE)) data->xfer = vz89x_smbus_xfer; else - return -ENOTSUPP; + return -EOPNOTSUPP;i2c_set_clientdata (client, indio_dev); data->client = client;drivers/iio/humidity/hdc100x.c +1
-1@@ -274,7 +274,7 @@ static int hdc100x_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA |I2C_FUNC_SMBUS_BYTE )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*data));if (!indio_dev)drivers/iio/humidity/htu21.c +1
-1@@ -192,7 +192,7 @@ static int htu21_probe(struct i2c_client *client, I2C_FUNC_SMBUS_READ_I2C_BLOCK)) {dev_err (&client->dev,"Adapter does not support some i2c transaction\n" ); -return -ENODEV ; +return -EOPNOTSUPP ; } indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*dev_data));drivers/iio/humidity/si7005.c +1
-1@@ -135,7 +135,7 @@ static int si7005_probe(struct i2c_client *client,int ret;if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*data));if (!indio_dev)drivers/iio/humidity/si7020.c +1
-1@@ -121,7 +121,7 @@ static int si7020_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WRITE_BYTE |I2C_FUNC_SMBUS_READ_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ;/* Reset device, loads default settings. */ ret =i2c_smbus_write_byte (client,SI7020CMD_RESET );drivers/iio/imu/inv_mpu6050/inv_mpu_i2c.c +1
-1@@ -120,7 +120,7 @@ static int inv_mpu_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_I2C_BLOCK )) -return -ENOSYS ; +return -EOPNOTSUPP ; regmap =devm_regmap_init_i2c (client, &inv_mpu_regmap_config);if (IS_ERR (regmap)) {drivers/iio/light/bh1750.c +1
-1@@ -241,7 +241,7 @@ static int bh1750_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_I2C |I2C_FUNC_SMBUS_WRITE_BYTE )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*data));if (!indio_dev)drivers/iio/light/jsa1212.c +1
-1@@ -326,7 +326,7 @@ static int jsa1212_probe(struct i2c_client *client,int ret;if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_BYTE_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*data));if (!indio_dev)drivers/iio/potentiometer/mcp4531.c +1
-1@@ -159,7 +159,7 @@ static int mcp4531_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) {dev_err (dev,"SMBUS Word Data not supported\n" ); -return -EIO ; +return -EOPNOTSUPP ; } indio_dev =devm_iio_device_alloc (dev,sizeof (*data));drivers/iio/pressure/mpl115_i2c.c +1
-1@@ -42,7 +42,7 @@ static int mpl115_i2c_probe(struct i2c_client *client, const struct i2c_device_id *id) {if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ;return mpl115_probe (&client->dev, id->name, &mpl115_i2c_ops); }drivers/iio/pressure/ms5611_i2c.c +1
-1@@ -92,7 +92,7 @@ static int ms5611_i2c_probe(struct i2c_client *client,I2C_FUNC_SMBUS_WRITE_BYTE |I2C_FUNC_SMBUS_READ_WORD_DATA | I2C_FUNC_SMBUS_READ_I2C_BLOCK)) - return -ENODEV; + return -EOPNOTSUPP; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*st));if (!indio_dev)drivers/iio/pressure/ms5637.c +1
-1@@ -136,7 +136,7 @@ static int ms5637_probe(struct i2c_client *client, I2C_FUNC_SMBUS_READ_I2C_BLOCK)) {dev_err (&client->dev,"Adapter does not support some i2c transaction\n" ); -return -ENODEV ; +return -EOPNOTSUPP ; } indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*dev_data));drivers/iio/pressure/t5403.c +1
-1@@ -221,7 +221,7 @@ static int t5403_probe(struct i2c_client *client,if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA |I2C_FUNC_SMBUS_I2C_BLOCK )) -return -ENODEV ; +return -EOPNOTSUPP ; ret =i2c_smbus_read_byte_data (client,T5403_SLAVE_ADDR );if (ret <0 )drivers/iio/proximity/pulsedlight-lidar-lite-v2.c +1
-1@@ -278,7 +278,7 @@ static int lidar_probe(struct i2c_client *client, I2C_FUNC_SMBUS_WORD_DATA | I2C_FUNC_SMBUS_BYTE)) data->xfer = lidar_smbus_xfer; else - return -ENOTSUPP; + return -EOPNOTSUPP; indio_dev->info = &lidar_info; indio_dev->name =LIDAR_DRV_NAME ;drivers/iio/temperature/mlx90614.c +1
-1@@ -516,7 +516,7 @@ static int mlx90614_probe(struct i2c_client *client,int ret;if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ; indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*data));if (!indio_dev)drivers/iio/temperature/tmp006.c +1
-1@@ -205,7 +205,7 @@ static int tmp006_probe(struct i2c_client *client,int ret;if (!i2c_check_functionality (client->adapter,I2C_FUNC_SMBUS_WORD_DATA )) -return -ENODEV ; +return -EOPNOTSUPP ;if (!tmp006_check_identification (client)) {dev_err (&client->dev,"no TMP006 sensor\n" );drivers/iio/temperature/tsys01.c +1
-1@@ -190,7 +190,7 @@ static int tsys01_i2c_probe(struct i2c_client *client, I2C_FUNC_SMBUS_READ_I2C_BLOCK)) {dev_err (&client->dev,"Adapter does not support some i2c transaction\n" ); -return -ENODEV ; +return -EOPNOTSUPP ; } indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*dev_data));drivers/iio/temperature/tsys02d.c +0
-0drivers/iio/temperature/tsys02d.c static int tsys02d_probe(struct i2c_client *client, · line 143
Quoted 8 lines
drivers/iio/temperature/tsys02d.c +1
-1@@ -137,7 +137,7 @@ static int tsys02d_probe(struct i2c_client *client, I2C_FUNC_SMBUS_READ_I2C_BLOCK)) {dev_err (&client->dev,"Adapter does not support some i2c transaction\n" ); -return -ENODEV ; +return -EOPNOTSUPP ; } indio_dev =devm_iio_device_alloc (&client->dev,sizeof (*dev_data));--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html