On 07/17/13 15:44, Lars-Peter Clausen wrote: > Signed-off-by: Lars-Peter Clausen <lars@xxxxxxxxxx> I wonder how many more of these there are out there? Anyhow, applied to the togreg branch of iio.git. Thanks, Jonathan > --- > drivers/staging/iio/gyro/adis16260_core.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/staging/iio/gyro/adis16260_core.c b/drivers/staging/iio/gyro/adis16260_core.c > index 55e6795..f060452 100644 > --- a/drivers/staging/iio/gyro/adis16260_core.c > +++ b/drivers/staging/iio/gyro/adis16260_core.c > @@ -277,9 +277,7 @@ static int adis16260_read_raw(struct iio_dev *indio_dev, > mutex_unlock(&indio_dev->mlock); > return ret; > } > - val16 &= (1 << bits) - 1; > - val16 = (s16)(val16 << (16 - bits)) >> (16 - bits); > - *val = val16; > + *val = sign_extend32(val16, bits - 1); > mutex_unlock(&indio_dev->mlock); > return IIO_VAL_INT; > case IIO_CHAN_INFO_CALIBSCALE: > ? -- 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