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