Commit ef45b77b authored by Colin Ian King's avatar Colin Ian King Committed by Mauro Carvalho Chehab

media: drxj: remove redundant assignments to variable rc

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: default avatarColin Ian King <colin.king@canonical.com>
Signed-off-by: default avatarSean Young <sean@mess.org>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+huawei@kernel.org>
parent f5298101
...@@ -2182,7 +2182,7 @@ int drxj_dap_atomic_read_reg32(struct i2c_device_addr *dev_addr, ...@@ -2182,7 +2182,7 @@ int drxj_dap_atomic_read_reg32(struct i2c_device_addr *dev_addr,
u32 *data, u32 flags) u32 *data, u32 flags)
{ {
u8 buf[sizeof(*data)] = { 0 }; u8 buf[sizeof(*data)] = { 0 };
int rc = -EIO; int rc;
u32 word = 0; u32 word = 0;
if (!data) if (!data)
...@@ -4229,7 +4229,7 @@ int drxj_dap_scu_atomic_write_reg16(struct i2c_device_addr *dev_addr, ...@@ -4229,7 +4229,7 @@ int drxj_dap_scu_atomic_write_reg16(struct i2c_device_addr *dev_addr,
u16 data, u32 flags) u16 data, u32 flags)
{ {
u8 buf[2]; u8 buf[2];
int rc = -EIO; int rc;
buf[0] = (u8) (data & 0xff); buf[0] = (u8) (data & 0xff);
buf[1] = (u8) ((data >> 8) & 0xff); buf[1] = (u8) ((data >> 8) & 0xff);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment