From fdb43d131fba3e314f5cb353745a8caeefdff48b Mon Sep 17 00:00:00 2001 From: Colin Ian King Date: Thu, 28 Mar 2024 11:15:57 +0000 Subject: [PATCH] staging: vt6655: remove redundant assignment to variable byData Variable byData is being assigned a value that is never read, it is being re-assigned later on. The assignment is redundant and can be removed. Cleans up clang scan build warning: drivers/staging/vt6655/srom.c:67:2: warning: Value stored to 'byData' is never read [deadcode.DeadStores] Signed-off-by: Colin Ian King Tested-by: Philipp Hortmann Link: https://lore.kernel.org/r/20240328111557.761380-1-colin.i.king@gmail.com Signed-off-by: Greg Kroah-Hartman --- drivers/staging/vt6655/srom.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/vt6655/srom.c b/drivers/staging/vt6655/srom.c index 1b89d401a7ebc..e80556509c58d 100644 --- a/drivers/staging/vt6655/srom.c +++ b/drivers/staging/vt6655/srom.c @@ -64,7 +64,6 @@ unsigned char SROMbyReadEmbedded(void __iomem *iobase, unsigned char byData; unsigned char byOrg; - byData = 0xFF; byOrg = ioread8(iobase + MAC_REG_I2MCFG); /* turn off hardware retry for getting NACK */ iowrite8(byOrg & (~I2MCFG_NORETRY), iobase + MAC_REG_I2MCFG); -- 2.30.2