Skip to content

Commit

Permalink
media: drxj: remove redundant assignments to variable rc
Browse files Browse the repository at this point in the history
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 <[email protected]>
Signed-off-by: Sean Young <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>
  • Loading branch information
Colin Ian King authored and mchehab committed Feb 24, 2020
1 parent f529810 commit ef45b77
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions drivers/media/dvb-frontends/drx39xyj/drxj.c
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down Expand Up @@ -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);
Expand Down

0 comments on commit ef45b77

Please sign in to comment.