2019-08-15 12:16:41

by Colin King

[permalink] [raw]
Subject: [PATCH] iio: dac: ad5380: fix incorrect assignment to val

From: Colin Ian King <[email protected]>

Currently the pointer val is being incorrectly incremented
instead of the value pointed to by val. Fix this by adding
in the missing * indirection operator.

Addresses-Coverity: ("Unused value")
Fixes: c03f2c536818 ("staging:iio:dac: Add AD5380 driver")
Signed-off-by: Colin Ian King <[email protected]>
---
drivers/iio/dac/ad5380.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/iio/dac/ad5380.c b/drivers/iio/dac/ad5380.c
index 4335214800d2..2ebe08326048 100644
--- a/drivers/iio/dac/ad5380.c
+++ b/drivers/iio/dac/ad5380.c
@@ -220,7 +220,7 @@ static int ad5380_read_raw(struct iio_dev *indio_dev,
if (ret)
return ret;
*val >>= chan->scan_type.shift;
- val -= (1 << chan->scan_type.realbits) / 2;
+ *val -= (1 << chan->scan_type.realbits) / 2;
return IIO_VAL_INT;
case IIO_CHAN_INFO_SCALE:
*val = 2 * st->vref;
--
2.20.1


2019-08-16 06:18:36

by Alexandru Ardelean

[permalink] [raw]
Subject: Re: [PATCH] iio: dac: ad5380: fix incorrect assignment to val

On Thu, 2019-08-15 at 12:58 +0100, Colin King wrote:
> [External]
>

Reviewed-by: Alexandru Ardelean <[email protected]>

> From: Colin Ian King <[email protected]>
>
> Currently the pointer val is being incorrectly incremented
> instead of the value pointed to by val. Fix this by adding
> in the missing * indirection operator.
>
> Addresses-Coverity: ("Unused value")
> Fixes: c03f2c536818 ("staging:iio:dac: Add AD5380 driver")
> Signed-off-by: Colin Ian King <[email protected]>
> ---
> drivers/iio/dac/ad5380.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/iio/dac/ad5380.c b/drivers/iio/dac/ad5380.c
> index 4335214800d2..2ebe08326048 100644
> --- a/drivers/iio/dac/ad5380.c
> +++ b/drivers/iio/dac/ad5380.c
> @@ -220,7 +220,7 @@ static int ad5380_read_raw(struct iio_dev *indio_dev,
> if (ret)
> return ret;
> *val >>= chan->scan_type.shift;
> - val -= (1 << chan->scan_type.realbits) / 2;
> + *val -= (1 << chan->scan_type.realbits) / 2;
> return IIO_VAL_INT;
> case IIO_CHAN_INFO_SCALE:
> *val = 2 * st->vref;

2019-08-18 17:56:59

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [PATCH] iio: dac: ad5380: fix incorrect assignment to val

On Fri, 16 Aug 2019 06:16:26 +0000
"Ardelean, Alexandru" <[email protected]> wrote:

> On Thu, 2019-08-15 at 12:58 +0100, Colin King wrote:
> > [External]
> >
>
> Reviewed-by: Alexandru Ardelean <[email protected]>
Applied to the fixes-togreg branch of iio.git.

Thanks,

Jonathan

>
> > From: Colin Ian King <[email protected]>
> >
> > Currently the pointer val is being incorrectly incremented
> > instead of the value pointed to by val. Fix this by adding
> > in the missing * indirection operator.
> >
> > Addresses-Coverity: ("Unused value")
> > Fixes: c03f2c536818 ("staging:iio:dac: Add AD5380 driver")
> > Signed-off-by: Colin Ian King <[email protected]>
> > ---
> > drivers/iio/dac/ad5380.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/iio/dac/ad5380.c b/drivers/iio/dac/ad5380.c
> > index 4335214800d2..2ebe08326048 100644
> > --- a/drivers/iio/dac/ad5380.c
> > +++ b/drivers/iio/dac/ad5380.c
> > @@ -220,7 +220,7 @@ static int ad5380_read_raw(struct iio_dev *indio_dev,
> > if (ret)
> > return ret;
> > *val >>= chan->scan_type.shift;
> > - val -= (1 << chan->scan_type.realbits) / 2;
> > + *val -= (1 << chan->scan_type.realbits) / 2;
> > return IIO_VAL_INT;
> > case IIO_CHAN_INFO_SCALE:
> > *val = 2 * st->vref;

2019-08-27 20:04:35

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [PATCH] iio: dac: ad5380: fix incorrect assignment to val

On Sun, 18 Aug 2019 18:55:44 +0100
Jonathan Cameron <[email protected]> wrote:

> On Fri, 16 Aug 2019 06:16:26 +0000
> "Ardelean, Alexandru" <[email protected]> wrote:
>
> > On Thu, 2019-08-15 at 12:58 +0100, Colin King wrote:
> > > [External]
> > >
> >
> > Reviewed-by: Alexandru Ardelean <[email protected]>
> Applied to the fixes-togreg branch of iio.git.

I changed my mind on this because I've been a bit slow sending a pull
request and we are now at rc6. Hence I've cherry picked it into
togreg so it'll go in during the next merge window and get backported
at that point.

Thanks,

Jonathan

>
> Thanks,
>
> Jonathan
>
> >
> > > From: Colin Ian King <[email protected]>
> > >
> > > Currently the pointer val is being incorrectly incremented
> > > instead of the value pointed to by val. Fix this by adding
> > > in the missing * indirection operator.
> > >
> > > Addresses-Coverity: ("Unused value")
> > > Fixes: c03f2c536818 ("staging:iio:dac: Add AD5380 driver")
> > > Signed-off-by: Colin Ian King <[email protected]>
> > > ---
> > > drivers/iio/dac/ad5380.c | 2 +-
> > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/drivers/iio/dac/ad5380.c b/drivers/iio/dac/ad5380.c
> > > index 4335214800d2..2ebe08326048 100644
> > > --- a/drivers/iio/dac/ad5380.c
> > > +++ b/drivers/iio/dac/ad5380.c
> > > @@ -220,7 +220,7 @@ static int ad5380_read_raw(struct iio_dev *indio_dev,
> > > if (ret)
> > > return ret;
> > > *val >>= chan->scan_type.shift;
> > > - val -= (1 << chan->scan_type.realbits) / 2;
> > > + *val -= (1 << chan->scan_type.realbits) / 2;
> > > return IIO_VAL_INT;
> > > case IIO_CHAN_INFO_SCALE:
> > > *val = 2 * st->vref;
>