diff options
| author | Miroslav Rezanina <mrezanin@redhat.com> | 2022-11-09 10:57:12 -0500 |
|---|---|---|
| committer | Thomas Huth <thuth@redhat.com> | 2022-11-11 09:12:10 +0100 |
| commit | 6083dcad80743718620a3f8a72fb76ea8b7c28ca (patch) | |
| tree | eff3c1eef38618849711475d1219a3aebfa5c4c3 | |
| parent | 7d7238c72b983cff5064734349d2d45be9c6282c (diff) | |
| download | focaccia-qemu-6083dcad80743718620a3f8a72fb76ea8b7c28ca.tar.gz focaccia-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>
| -rw-r--r-- | hw/net/tulip.c | 4 |
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) { |