diff options
| author |   Karim Eshapa <karim.eshapa@gmail.com> | 2018-06-12 18:48:38 +0200 |
|---|---|---|
| committer |   Jonathan Cameron <Jonathan.Cameron@huawei.com> | 2018-06-16 19:04:53 +0100 |
| commit | c660abab5c7edbc9ec9cd3b7b8611e7bac01d0ac (patch) | |
| tree | ba95d6788be827df8a48d479b27217dc4d386c10 /drivers/staging/iio | |
| parent | staging: iio: adc: ad7606: fix function pointer parameter names missing. (diff) | |
| download | linux-dev-c660abab5c7edbc9ec9cd3b7b8611e7bac01d0ac.tar.xz linux-dev-c660abab5c7edbc9ec9cd3b7b8611e7bac01d0ac.zip | |
staging:iio:accel:adis16203: sign extend function rather code duplication
Use sign_extend32 kernel function instead of code duplication.
This function is also safe for 16 bits.
Signed-off-by: Karim Eshapa <karim.eshapa@gmail.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Diffstat (limited to 'drivers/staging/iio')
| -rw-r--r-- | drivers/staging/iio/accel/adis16203.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/drivers/staging/iio/accel/adis16203.c b/drivers/staging/iio/accel/adis16203.c index b3e4571340ab..5cc96c8086b5 100644 --- a/drivers/staging/iio/accel/adis16203.c +++ b/drivers/staging/iio/accel/adis16203.c @@ -168,7 +168,6 @@ static int adis16203_read_raw(struct iio_dev *indio_dev, { struct adis *st = iio_priv(indio_dev); int ret; - int bits; u8 addr; s16 val16; @@ -202,14 +201,11 @@ static int adis16203_read_raw(struct iio_dev *indio_dev, *val = 25000 / -470 - 1278; /* 25 C = 1278 */ return IIO_VAL_INT; case IIO_CHAN_INFO_CALIBBIAS: - bits = 14; addr = adis16203_addresses[chan->scan_index]; ret = adis_read_reg_16(st, addr, &val16); if (ret) return ret; - val16 &= (1 << bits) - 1; - val16 = (s16)(val16 << (16 - bits)) >> (16 - bits); - *val = val16; + *val = sign_extend32(val16, 13); return IIO_VAL_INT; default: return -EINVAL; |