lkml.org 
[lkml]   [2021]   [Jul]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.13 095/800] iio: ltr501: ltr501_read_ps(): add missing endianness conversion
    Date
    From: Oliver Lang <Oliver.Lang@gossenmetrawatt.com>

    commit 71b33f6f93ef9462c84560e2236ed22209d26a58 upstream.

    The PS ADC Channel data is spread over 2 registers in little-endian
    form. This patch adds the missing endianness conversion.

    Fixes: 2690be905123 ("iio: Add Lite-On ltr501 ambient light / proximity sensor driver")
    Signed-off-by: Oliver Lang <Oliver.Lang@gossenmetrawatt.com>
    Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
    Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
    Tested-by: Nikita Travkin <nikita@trvn.ru> # ltr559
    Link: https://lore.kernel.org/r/20210610134619.2101372-4-mkl@pengutronix.de
    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/light/ltr501.c | 7 ++++---
    1 file changed, 4 insertions(+), 3 deletions(-)

    --- a/drivers/iio/light/ltr501.c
    +++ b/drivers/iio/light/ltr501.c
    @@ -409,18 +409,19 @@ static int ltr501_read_als(const struct

    static int ltr501_read_ps(const struct ltr501_data *data)
    {
    - int ret, status;
    + __le16 status;
    + int ret;

    ret = ltr501_drdy(data, LTR501_STATUS_PS_RDY);
    if (ret < 0)
    return ret;

    ret = regmap_bulk_read(data->regmap, LTR501_PS_DATA,
    - &status, 2);
    + &status, sizeof(status));
    if (ret < 0)
    return ret;

    - return status;
    + return le16_to_cpu(status);
    }

    static int ltr501_read_intr_prst(const struct ltr501_data *data,

    \
     
     \ /
      Last update: 2021-07-12 10:16    [W:3.541 / U:0.432 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site