staging: iio: ad2s1210: remove redundant assignment to variable negative
authorColin Ian King <colin.i.king@gmail.com>
Mon, 18 Apr 2022 13:46:03 +0000 (14:46 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 20 Apr 2022 16:45:35 +0000 (18:45 +0200)
Variable negative is being assigned a value that is never read, it is
being re-assigned later. The assignment is redundant and can be removed.

Cleans up clang scan build warning:
drivers/staging/iio/resolver/ad2s1210.c:502:3: warning: Value stored
to 'negative' is never read [deadcode.DeadStores]

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
Link: https://lore.kernel.org/r/20220418134603.81336-1-colin.i.king@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/iio/resolver/ad2s1210.c

index 74adb82..c0b2716 100644 (file)
@@ -499,7 +499,6 @@ static int ad2s1210_read_raw(struct iio_dev *indio_dev,
                ret = IIO_VAL_INT;
                break;
        case IIO_ANGL_VEL:
-               negative = st->rx[0] & 0x80;
                vel = be16_to_cpup((__be16 *)st->rx);
                vel >>= 16 - st->resolution;
                if (vel & 0x8000) {