Commit c1a75288 authored by Dan Carpenter's avatar Dan Carpenter Committed by Greg Kroah-Hartman
Browse files

Staging: iio: fix endian conversion in ad7298_scan_direct() 



"tmp" is used to store the output from cpu_to_be16() so it should be
a __be16 bit type.

Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Acked-by: default avatarMichael Hennerich <michael.hennerich@analog.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent 5ae8f440

drivers/staging/iio/adc/ad7298_core.c

+7 −6
Original line number Diff line number Diff line
@@ -69,27 +69,28 @@ static int ad7298_scan_direct(struct ad7298_state *st, unsigned ch) 
static int ad7298_scan_temp(struct ad7298_state *st, int *val)

{

	int tmp, ret;

	__be16 buf;



	tmp = cpu_to_be16(AD7298_WRITE | AD7298_TSENSE |

	buf = cpu_to_be16(AD7298_WRITE | AD7298_TSENSE |

			  AD7298_TAVG | st->ext_ref);



	ret = spi_write(st->spi, (u8 *)&tmp, 2);

	ret = spi_write(st->spi, (u8 *)&buf, 2);

	if (ret)

		return ret;



	tmp = 0;

	buf = cpu_to_be16(0);



	ret = spi_write(st->spi, (u8 *)&tmp, 2);

	ret = spi_write(st->spi, (u8 *)&buf, 2);

	if (ret)

		return ret;



	usleep_range(101, 1000); /* sleep > 100us */



	ret = spi_read(st->spi, (u8 *)&tmp, 2);

	ret = spi_read(st->spi, (u8 *)&buf, 2);

	if (ret)

		return ret;



	tmp = be16_to_cpu(tmp) & RES_MASK(AD7298_BITS);

	tmp = be16_to_cpu(buf) & RES_MASK(AD7298_BITS);



	/*

	 * One LSB of the ADC corresponds to 0.25 deg C.