iio: ad9523: Fix displayed phase
authorLars-Peter Clausen <lars@metafoo.de>
Mon, 25 Jun 2018 08:03:07 +0000 (11:03 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 9 Sep 2018 17:55:56 +0000 (19:55 +0200)
commit 5a4e33c1c53ae7d4425f7d94e60e4458a37b349e upstream.

Fix the displayed phase for the ad9523 driver. Currently the most
significant decimal place is dropped and all other digits are shifted one
to the left. This is due to a multiplication by 10, which is not necessary,
so remove it.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Alexandru Ardelean <alexandru.ardelean@analog.com>
Fixes: cd1678f9632 ("iio: frequency: New driver for AD9523 SPI Low Jitter Clock Generator")
Cc: <Stable@vger.kernel.org>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/iio/frequency/ad9523.c

index 99eba52..cb45fa3 100644 (file)
@@ -642,7 +642,7 @@ static int ad9523_read_raw(struct iio_dev *indio_dev,
                code = (AD9523_CLK_DIST_DIV_PHASE_REV(ret) * 3141592) /
                        AD9523_CLK_DIST_DIV_REV(ret);
                *val = code / 1000000;
-               *val2 = (code % 1000000) * 10;
+               *val2 = code % 1000000;
                return IIO_VAL_INT_PLUS_MICRO;
        default:
                return -EINVAL;