diff mbox series

iio:gyro:mp3050 Suppress 'false' sparse warning about endian assigment.

Message ID 20190422100030.10561-1-jic23@kernel.org (mailing list archive)
State New, archived
Headers show
Series iio:gyro:mp3050 Suppress 'false' sparse warning about endian assigment. | expand

Commit Message

Jonathan Cameron April 22, 2019, 10 a.m. UTC 
From: Jonathan Cameron <Jonathan.Cameron@huawei.com>

In this particular case the value being assigned is 0xAAAA so
is not effected by the endian choice.

It's easy to suppress so we might as well do so to make it easier
to spot real warnings.

Warning is:

CHECK   drivers/iio/gyro/mpu3050-core.c
drivers/iio/gyro/mpu3050-core.c:544:48: warning: incorrect type in assignment (different base types)
drivers/iio/gyro/mpu3050-core.c:544:48:    expected restricted __be16 <noident>
drivers/iio/gyro/mpu3050-core.c:544:48:    got int

Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Cc: Linus Walleij <linus.walleij@linaro.org>
---
 drivers/iio/gyro/mpu3050-core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/drivers/iio/gyro/mpu3050-core.c b/drivers/iio/gyro/mpu3050-core.c
index 496ed4c99ce9..9aacaca145e4 100644
--- a/drivers/iio/gyro/mpu3050-core.c
+++ b/drivers/iio/gyro/mpu3050-core.c
@@ -541,7 +541,7 @@  static irqreturn_t mpu3050_trigger_handler(int irq, void *p)
 				toread = bytes_per_datum;
 				offset = 1;
 				/* Put in some dummy value */
-				fifo_values[0] = 0xAAAA;
+				fifo_values[0] = cpu_to_be16(0xAAAA);
 			}
 
 			ret = regmap_bulk_read(mpu3050->map,