media: cx231xx: remove redundant assignment to variable status
authorColin Ian King <colin.king@canonical.com>
Tue, 3 Sep 2019 17:06:39 +0000 (14:06 -0300)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Tue, 5 Nov 2019 11:29:13 +0000 (08:29 -0300)
Variable status is being initialized with a value that is never read
and is being re-assigned a later on. The assignment is redundant and
hence can be removed.

Addresses-Coverity: ("Unused value")

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/media/usb/cx231xx/cx231xx-avcore.c

index d417b5fe40930ebdc5fece0ad9ac845dc3bfd7a4..0974965e848f7e86fe30184ac3da788033842069 100644 (file)
@@ -1240,7 +1240,7 @@ int cx231xx_init_ctrl_pin_status(struct cx231xx *dev)
 int cx231xx_set_agc_analog_digital_mux_select(struct cx231xx *dev,
                                              u8 analog_or_digital)
 {
-       int status = 0;
+       int status;
 
        /* first set the direction to output */
        status = cx231xx_set_gpio_direction(dev,