staging: vt6655: remove redundant assignment to variable byData
authorColin Ian King <colin.i.king@gmail.com>
Thu, 28 Mar 2024 11:15:57 +0000 (11:15 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 9 Apr 2024 15:51:06 +0000 (17:51 +0200)
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 <colin.i.king@gmail.com>
Tested-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
Link: https://lore.kernel.org/r/20240328111557.761380-1-colin.i.king@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/vt6655/srom.c

index 1b89d401a7ebc164043d11e29591d88db11692c0..e80556509c58dde01addf6750044405fd3c960d8 100644 (file)
@@ -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);