2020-02-14 09:30:00

by Alexandru Tachici

[permalink] [raw]
Subject: [PATCH 2/5] iio: accel: adxl372: Set iio_chan BE

Data stored in the iio-buffer is BE and this
should be specified in the iio_chan_spec struct.

Signed-off-by: Alexandru Tachici <[email protected]>
---
drivers/iio/accel/adxl372.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/drivers/iio/accel/adxl372.c b/drivers/iio/accel/adxl372.c
index bb6c2bf1a457..538e5053a946 100644
--- a/drivers/iio/accel/adxl372.c
+++ b/drivers/iio/accel/adxl372.c
@@ -237,6 +237,7 @@ static const struct adxl372_axis_lookup adxl372_axis_lookup_table[] = {
.realbits = 12, \
.storagebits = 16, \
.shift = 4, \
+ .endianness = IIO_BE, \
}, \
}

--
2.20.1


2020-02-15 15:37:19

by Jonathan Cameron

[permalink] [raw]
Subject: Re: [PATCH 2/5] iio: accel: adxl372: Set iio_chan BE

On Fri, 14 Feb 2020 11:29:16 +0200
Alexandru Tachici <[email protected]> wrote:

> Data stored in the iio-buffer is BE and this
> should be specified in the iio_chan_spec struct.
>
> Signed-off-by: Alexandru Tachici <[email protected]>
To a quick glance this looks to me like a fix we should be backporting.
Please add a fixes tag if so and I'll get this lined up to go in during
the RCs.

Thanks,

Jonathan

> ---
> drivers/iio/accel/adxl372.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/iio/accel/adxl372.c b/drivers/iio/accel/adxl372.c
> index bb6c2bf1a457..538e5053a946 100644
> --- a/drivers/iio/accel/adxl372.c
> +++ b/drivers/iio/accel/adxl372.c
> @@ -237,6 +237,7 @@ static const struct adxl372_axis_lookup adxl372_axis_lookup_table[] = {
> .realbits = 12, \
> .storagebits = 16, \
> .shift = 4, \
> + .endianness = IIO_BE, \
> }, \
> }
>