On Tue, 19 Mar 2019 12:47:00 +0200 Alexandru Ardelean <alexandru.ardelean@xxxxxxxxxx> wrote: > From: Dragos Bogdan <dragos.bogdan@xxxxxxxxxx> > > The desired channel has to be selected in order to correctly fill the > buffer with the corresponding data. > The `ad_sd_write_reg()` already does this, but for the > `ad_sd_read_reg_raw()` this was omitted. > > Fixes: af3008485ea03 ("iio:adc: Add common code for ADI Sigma Delta devices") > Signed-off-by: Dragos Bogdan <dragos.bogdan@xxxxxxxxxx> > Signed-off-by: Alexandru Ardelean <alexandru.ardelean@xxxxxxxxxx> Applied to the fixes-togreg branch of iio.git and marked for stable. Thanks, Jonathan > --- > drivers/iio/adc/ad_sigma_delta.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/iio/adc/ad_sigma_delta.c b/drivers/iio/adc/ad_sigma_delta.c > index ff5f2da2e1b1..54d9978b2740 100644 > --- a/drivers/iio/adc/ad_sigma_delta.c > +++ b/drivers/iio/adc/ad_sigma_delta.c > @@ -121,6 +121,7 @@ static int ad_sd_read_reg_raw(struct ad_sigma_delta *sigma_delta, > if (sigma_delta->info->has_registers) { > data[0] = reg << sigma_delta->info->addr_shift; > data[0] |= sigma_delta->info->read_mask; > + data[0] |= sigma_delta->comm; > spi_message_add_tail(&t[0], &m); > } > spi_message_add_tail(&t[1], &m);