From: Michael Hennerich <michael.hennerich@xxxxxxxxxx> RATE and QUADRATURE_CORRECTION data is formatted as a twos complement number, and therefore must be handled as type signed short. TEMP result should be properly shifted. Dynamic Null Correction is a 10-bit signed number. Signed-off-by: Michael Hennerich <michael.hennerich@xxxxxxxxxx> --- drivers/staging/iio/gyro/adxrs450_core.c | 10 +++++----- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/iio/gyro/adxrs450_core.c b/drivers/staging/iio/gyro/adxrs450_core.c index d73acff..e71b91c 100644 --- a/drivers/staging/iio/gyro/adxrs450_core.c +++ b/drivers/staging/iio/gyro/adxrs450_core.c @@ -126,7 +126,7 @@ static int adxrs450_spi_write_reg_16(struct device *dev, * @dev: device associated with child of actual iio_dev * @val: somewhere to pass back the value read **/ -static int adxrs450_spi_sensor_data(struct device *dev, u16 *val) +static int adxrs450_spi_sensor_data(struct device *dev, s16 *val) { struct spi_message msg; struct iio_dev *indio_dev = dev_get_drvdata(dev); @@ -217,7 +217,7 @@ static ssize_t adxrs450_read_temp(struct device *dev, &t); if (ret) return ret; - return sprintf(buf, "%d\n", t); + return sprintf(buf, "%d\n", t >> 7); } static ssize_t adxrs450_read_quad(struct device *dev, @@ -225,7 +225,7 @@ static ssize_t adxrs450_read_quad(struct device *dev, char *buf) { int ret; - u16 t; + s16 t; ret = adxrs450_spi_read_reg_16(dev, ADXRS450_QUAD1, &t); @@ -247,7 +247,7 @@ static ssize_t adxrs450_write_dnc(struct device *dev, goto error_ret; ret = adxrs450_spi_write_reg_16(dev, ADXRS450_DNC1, - val); + val & 0x3FF); error_ret: return ret ? ret : len; } @@ -257,7 +257,7 @@ static ssize_t adxrs450_read_sensor_data(struct device *dev, char *buf) { int ret; - u16 t; + s16 t; ret = adxrs450_spi_sensor_data(dev, &t); if (ret) -- 1.6.0.2 -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html