On 01/09/16 10:44, Linus Walleij wrote: > All the scaling of the KXSD9 involves multiplication with a > fraction number < 1. > > However the scaling value returned from IIO_INFO_SCALE was > unpredictable as only the micros of the value was assigned, and > not the integer part, resulting in scaling like this: > > $cat in_accel_scale > -1057462640.011978 > > Fix this by assigning zero to the integer part. > > Cc: stable@xxxxxxxxxxxxxxx > Tested-by: Jonathan Cameron <jic23@xxxxxxxxxx> > Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx> Applied to the fixes-togreg branch of iio.git. Thanks, Jonathan > --- > ChangeLog v1->v2: > - Move this first in the series as Jonathan's request. > --- > drivers/iio/accel/kxsd9.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/iio/accel/kxsd9.c b/drivers/iio/accel/kxsd9.c > index da5fb67ecb34..9d72d4bcf5e9 100644 > --- a/drivers/iio/accel/kxsd9.c > +++ b/drivers/iio/accel/kxsd9.c > @@ -166,6 +166,7 @@ static int kxsd9_read_raw(struct iio_dev *indio_dev, > ret = spi_w8r8(st->us, KXSD9_READ(KXSD9_REG_CTRL_C)); > if (ret < 0) > goto error_ret; > + *val = 0; > *val2 = kxsd9_micro_scales[ret & KXSD9_FS_MASK]; > ret = IIO_VAL_INT_PLUS_MICRO; > break; > -- 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
