Re: [PATCH v4 05/26] staging: iio: tsl2583: remove redundant write to the control register in taos_probe()

2016-11-13 Thread Jonathan Cameron
On 12/11/16 18:19, Brian Masney wrote: > taos_probe() calls i2c_smbus_write_byte() to select the control > register, however there are no subsequent calls to > i2c_smbus_read_byte(). The write call is unnecessary and is removed by > this patch. > > Verified that the driver still functions

Re: [PATCH v4 05/26] staging: iio: tsl2583: remove redundant write to the control register in taos_probe()

2016-11-13 Thread Jonathan Cameron
On 12/11/16 18:19, Brian Masney wrote: > taos_probe() calls i2c_smbus_write_byte() to select the control > register, however there are no subsequent calls to > i2c_smbus_read_byte(). The write call is unnecessary and is removed by > this patch. > > Verified that the driver still functions

[PATCH v4 05/26] staging: iio: tsl2583: remove redundant write to the control register in taos_probe()

2016-11-12 Thread Brian Masney
taos_probe() calls i2c_smbus_write_byte() to select the control register, however there are no subsequent calls to i2c_smbus_read_byte(). The write call is unnecessary and is removed by this patch. Verified that the driver still functions correctly using a TSL2581 hooked up to a Raspberry Pi 2.

[PATCH v4 05/26] staging: iio: tsl2583: remove redundant write to the control register in taos_probe()

2016-11-12 Thread Brian Masney
taos_probe() calls i2c_smbus_write_byte() to select the control register, however there are no subsequent calls to i2c_smbus_read_byte(). The write call is unnecessary and is removed by this patch. Verified that the driver still functions correctly using a TSL2581 hooked up to a Raspberry Pi 2.