summaryrefslogtreecommitdiffstats
path: root/hw
diff options
context:
space:
mode:
authorMiroslav Rezanina2022-11-09 16:57:12 +0100
committerThomas Huth2022-11-11 09:12:10 +0100
commit6083dcad80743718620a3f8a72fb76ea8b7c28ca (patch)
treeeff3c1eef38618849711475d1219a3aebfa5c4c3 /hw
parentrtl8139: Remove unused variable (diff)
downloadqemu-6083dcad80743718620a3f8a72fb76ea8b7c28ca.tar.gz
qemu-6083dcad80743718620a3f8a72fb76ea8b7c28ca.tar.xz
qemu-6083dcad80743718620a3f8a72fb76ea8b7c28ca.zip
tulip: Remove unused variable
Variable n used in tulip_idblock_crc function is only incremented but never read. This causes 'Unused but set variable' warning on Clang 15.0.1 compiler. Removing the variable to prevent the warning. Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com> Reviewed-by: Thomas Huth <thuth@redhat.com> Message-Id: <02e1560d115c208df32236df8916fed98429fda1.1668009030.git.mrezanin@redhat.com> Signed-off-by: Thomas Huth <thuth@redhat.com>
Diffstat (limited to 'hw')
-rw-r--r--hw/net/tulip.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/hw/net/tulip.c b/hw/net/tulip.c
index b9e42c322a..c2b3b1bdfa 100644
--- a/hw/net/tulip.c
+++ b/hw/net/tulip.c
@@ -870,11 +870,10 @@ static const MemoryRegionOps tulip_ops = {
static void tulip_idblock_crc(TULIPState *s, uint16_t *srom)
{
- int word, n;
+ int word;
int bit;
unsigned char bitval, crc;
const int len = 9;
- n = 0;
crc = -1;
for (word = 0; word < len; word++) {
@@ -887,7 +886,6 @@ static void tulip_idblock_crc(TULIPState *s, uint16_t *srom)
srom[len - 1] = (srom[len - 1] & 0xff00) | (unsigned short)crc;
break;
}
- n++;
bitval = ((srom[word] >> bit) & 1) ^ ((crc >> 7) & 1);
crc = crc << 1;
if (bitval == 1) {