On 25/10/14 19:09, Hartmut Knaack wrote: > In si7020_read_raw() the pointer to the i2c client was obtained as second level > pointer, although a simple pointer is appropriate. > > Signed-off-by: Hartmut Knaack <knaack.h@xxxxxx> What I'd neglected (being a forgetful soul) is that we only just took this driver anyway so the two patches will both end up in togreg whatever. So applied to the togreg branch of iio.git - initially pushed out as testing etc etc. Jonathan > --- > diff --git a/drivers/iio/humidity/si7020.c b/drivers/iio/humidity/si7020.c > index e336af7..62d74d3 100644 > --- a/drivers/iio/humidity/si7020.c > +++ b/drivers/iio/humidity/si7020.c > @@ -45,12 +45,12 @@ static int si7020_read_raw(struct iio_dev *indio_dev, > struct iio_chan_spec const *chan, int *val, > int *val2, long mask) > { > - struct i2c_client **client = iio_priv(indio_dev); > + struct i2c_client *client = iio_priv(indio_dev); > int ret; > > switch (mask) { > case IIO_CHAN_INFO_RAW: > - ret = i2c_smbus_read_word_data(*client, > + ret = i2c_smbus_read_word_data(client, > chan->type == IIO_TEMP ? > SI7020CMD_TEMP_HOLD : > SI7020CMD_RH_HOLD); > > -- > 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 > -- 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