iio: pressure: dlhl60d: Don't take garbage into consideration when reading data

Both pressure and temperature are 24-bit long. Use proper accessors
instead of overlapping readings.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://lore.kernel.org/r/20220726142048.4494-1-andriy.shevchenko@linux.intel.com
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>

diff --git a/drivers/iio/pressure/dlhl60d.c b/drivers/iio/pressure/dlhl60d.c
index 5f6bb36..f0b0d19 100644 (file)
--- a/drivers/iio/pressure/dlhl60d.c
+++ b/drivers/iio/pressure/dlhl60d.c
@@ -129,9 +129,8 @@ static int dlh_read_direct(struct dlh_state *st,
        if (ret)
                return ret;
 
-       *pressure = get_unaligned_be32(&st->rx_buf[1]) >> 8;
-       *temperature = get_unaligned_be32(&st->rx_buf[3]) &
-               GENMASK(DLH_NUM_TEMP_BITS - 1, 0);
+       *pressure = get_unaligned_be24(&st->rx_buf[1]);
+       *temperature = get_unaligned_be24(&st->rx_buf[4]);
 
        return 0;
 }