Re: [PATCH] iio: potentiometer: tpl0102: remove unneeded i2c check functionality test

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



No isn't breaking any hardware that I know of.

Thanks,

Matt

On Sat, May 14, 2016 at 9:58 AM, Jonathan Cameron <jic23@xxxxxxxxxx> wrote:
> On 12/05/16 05:47, Matt Ranostay wrote:
>> Actually I2C_FUNC_SMBUS_WORD_DATA isn't need for this device, and regmap
>> handles all single byte reads transparently.
>>
>> Signed-off-by: Matt Ranostay <mranostay@xxxxxxxxx>
> Applied to the togreg branch of iio.git.  If you need it quicker I'm happy to push
> this trivial patch as a fix but only if is preventing known hardware from working
> (i.e. we have users complaining)
>
> thanks
>
> Jonathan
>
>> ---
>>  drivers/iio/potentiometer/tpl0102.c | 4 ----
>>  1 file changed, 4 deletions(-)
>>
>> diff --git a/drivers/iio/potentiometer/tpl0102.c b/drivers/iio/potentiometer/tpl0102.c
>> index 5c304d4..7b6b545 100644
>> --- a/drivers/iio/potentiometer/tpl0102.c
>> +++ b/drivers/iio/potentiometer/tpl0102.c
>> @@ -116,10 +116,6 @@ static int tpl0102_probe(struct i2c_client *client,
>>       struct tpl0102_data *data;
>>       struct iio_dev *indio_dev;
>>
>> -     if (!i2c_check_functionality(client->adapter,
>> -                                  I2C_FUNC_SMBUS_WORD_DATA))
>> -             return -EOPNOTSUPP;
>> -
>>       indio_dev = devm_iio_device_alloc(dev, sizeof(*data));
>>       if (!indio_dev)
>>               return -ENOMEM;
>>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux