Commit e6f3799d authored by Colin Ian King's avatar Colin Ian King Committed by Martin K. Petersen

scsi: initio: Remove redundant variable 'rb'

The variable 'rb' is being assigned a value but it isn't being read
afterwards. The assignment is redundant and so 'rb' can be removed.

Cleans up clang scan build warning:

warning: Although the value stored to 'rb' is used in the enclosing
expression, the value is never actually read from 'rb'[deadcode.DeadStores]
Signed-off-by: default avatarColin Ian King <colin.i.king@gmail.com>
Link: https://lore.kernel.org/r/20240116112606.2263738-1-colin.i.king@gmail.comSigned-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent d6b75ba5
...@@ -371,7 +371,6 @@ static u16 initio_se2_rd(unsigned long base, u8 addr) ...@@ -371,7 +371,6 @@ static u16 initio_se2_rd(unsigned long base, u8 addr)
*/ */
static void initio_se2_wr(unsigned long base, u8 addr, u16 val) static void initio_se2_wr(unsigned long base, u8 addr, u16 val)
{ {
u8 rb;
u8 instr; u8 instr;
int i; int i;
...@@ -400,7 +399,7 @@ static void initio_se2_wr(unsigned long base, u8 addr, u16 val) ...@@ -400,7 +399,7 @@ static void initio_se2_wr(unsigned long base, u8 addr, u16 val)
udelay(30); udelay(30);
outb(SE2CS, base + TUL_NVRAM); /* -CLK */ outb(SE2CS, base + TUL_NVRAM); /* -CLK */
udelay(30); udelay(30);
if ((rb = inb(base + TUL_NVRAM)) & SE2DI) if (inb(base + TUL_NVRAM) & SE2DI)
break; /* write complete */ break; /* write complete */
} }
outb(0, base + TUL_NVRAM); /* -CS */ outb(0, base + TUL_NVRAM); /* -CS */
......
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