diff options
| author |   Jonathan Cameron <jic23@kernel.org> | 2013-12-11 18:45:00 +0000 |
|---|---|---|
| committer | Jonathan Cameron <jic23@kernel.org> | 2013-12-17 20:34:18 +0000 |
| commit | 3425c0f7ac61f2fcfb7f2728e9b7ba7e27aec429 (patch) | |
| tree | 5cd42fd7e59a93ec378c2ff2aaa8997ffbf0b250 /drivers/iio | |
| parent | staging:iio:mag:hmc5843 fix incorrect endianness of channel as a result of missuse of the IIO_ST macro. (diff) | |
| download | linux-dev-3425c0f7ac61f2fcfb7f2728e9b7ba7e27aec429.tar.xz linux-dev-3425c0f7ac61f2fcfb7f2728e9b7ba7e27aec429.zip | |
iio:imu:adis16400 fix pressure channel scan type
A single channel in this driver was using the IIO_ST macro.
This does not provide a parameter for setting the endianness of
the channel. Thus this channel will have been reported as whatever
is the native endianness of the cpu rather than big endian. This
means it would be incorrect on little endian platforms.
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Acked-by: Lars-Peter Clausen <lars@metafoo.de>
Cc: stable@vger.kernel.org
Diffstat (limited to 'drivers/iio')
| -rw-r--r-- | drivers/iio/imu/adis16400_core.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/drivers/iio/imu/adis16400_core.c b/drivers/iio/imu/adis16400_core.c index 3fb7757a1028..368660dfe135 100644 --- a/drivers/iio/imu/adis16400_core.c +++ b/drivers/iio/imu/adis16400_core.c @@ -651,7 +651,12 @@ static const struct iio_chan_spec adis16448_channels[] = { .info_mask_shared_by_type = BIT(IIO_CHAN_INFO_SCALE), .address = ADIS16448_BARO_OUT, .scan_index = ADIS16400_SCAN_BARO, - .scan_type = IIO_ST('s', 16, 16, 0), + .scan_type = { + .sign = 's', + .realbits = 16, + .storagebits = 16, + .endianness = IIO_BE, + }, }, ADIS16400_TEMP_CHAN(ADIS16448_TEMP_OUT, 12), IIO_CHAN_SOFT_TIMESTAMP(11) |