media: drxj: remove redundant assignments to variable rc
authorColin Ian King <colin.king@canonical.com>
Tue, 7 Jan 2020 17:11:07 +0000 (18:11 +0100)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Mon, 24 Feb 2020 15:26:14 +0000 (16:26 +0100)
The variable rc is being initialized with a value that is never
read and it is being updated later with a new value.  The initialization
is redundant and can be removed.

Addresses-Coverity: ("Unused value")

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Sean Young <sean@mess.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/media/dvb-frontends/drx39xyj/drxj.c

index ac7be872f460baf548a41c08c0c84587e5bf8011..5de016412c42990578b6d1b5b7a911761a37e038 100644 (file)
@@ -2182,7 +2182,7 @@ int drxj_dap_atomic_read_reg32(struct i2c_device_addr *dev_addr,
                                     u32 *data, u32 flags)
 {
        u8 buf[sizeof(*data)] = { 0 };
-       int rc = -EIO;
+       int rc;
        u32 word = 0;
 
        if (!data)
@@ -4229,7 +4229,7 @@ int drxj_dap_scu_atomic_write_reg16(struct i2c_device_addr *dev_addr,
                                          u16 data, u32 flags)
 {
        u8 buf[2];
-       int rc = -EIO;
+       int rc;
 
        buf[0] = (u8) (data & 0xff);
        buf[1] = (u8) ((data >> 8) & 0xff);