iio: adc: ad_sigma_delta: Use {get,put}_unaligned_be24()

This makes the driver code slightly easier to read.

Cc: Lars-Peter Clausen <lars@metafoo.de>
Cc: Michael Hennerich <Michael.Hennerich@analog.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Acked-by: Nuno Sá <nuno.sa@analog.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

diff --git a/drivers/iio/adc/ad_sigma_delta.c b/drivers/iio/adc/ad_sigma_delta.c
index 8115b6de1d6c9fbd70b0b7f5a2d6e1deed673964..dd3d54b3bc8bba19612451e4546a326181fc3512 100644 (file)
--- a/drivers/iio/adc/ad_sigma_delta.c
+++ b/drivers/iio/adc/ad_sigma_delta.c
@@ -70,9 +70,7 @@ int ad_sd_write_reg(struct ad_sigma_delta *sigma_delta, unsigned int reg,
 
        switch (size) {
        case 3:
-               data[1] = val >> 16;
-               data[2] = val >> 8;
-               data[3] = val;
+               put_unaligned_be24(val, &data[1]);
                break;
        case 2:
                put_unaligned_be16(val, &data[1]);
@@ -157,9 +155,7 @@ int ad_sd_read_reg(struct ad_sigma_delta *sigma_delta,
                *val = get_unaligned_be32(sigma_delta->data);
                break;
        case 3:
-               *val = (sigma_delta->data[0] << 16) |
-                       (sigma_delta->data[1] << 8) |
-                       sigma_delta->data[2];
+               *val = get_unaligned_be24(&sigma_delta->data[0]);
                break;
        case 2:
                *val = get_unaligned_be16(sigma_delta->data);