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 99eba524f6ddbe063d2cf2ac359491287177d10b..cb45fa314a92e3ac2e5ca2f2d03de1918b8cda5c 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;