On Sat, 25 Jun 2022 12:38:52 +0200 Marcus Folkesson <marcus.folkesson@xxxxxxxxx> wrote: > The device supports negative values as well. > > Signed-off-by: Marcus Folkesson <marcus.folkesson@xxxxxxxxx> This looks like a fix to me. So fixes tag and move it to the start of the series. Jonathan > --- > > Notes: > v2: > - No changes > > drivers/iio/adc/mcp3911.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/iio/adc/mcp3911.c b/drivers/iio/adc/mcp3911.c > index a0609d7663e1..a019264e73e3 100644 > --- a/drivers/iio/adc/mcp3911.c > +++ b/drivers/iio/adc/mcp3911.c > @@ -144,6 +144,8 @@ static int mcp3911_read_raw(struct iio_dev *indio_dev, > if (ret) > goto out; > > + *val = sign_extend32(*val, 23); > + > ret = IIO_VAL_INT; > break; >