Skip to content

Commit

Permalink
tests/rtl8139: Clean up global variable shadowing
Browse files Browse the repository at this point in the history
Rename the variable to fix:

  tests/qtest/rtl8139-test.c:28:33: error: declaration shadows a variable in the global scope [-Werror,-Wshadow]
  static void save_fn(QPCIDevice *dev, int devfn, void *data)
                                  ^
  tests/qtest/rtl8139-test.c:37:17: error: declaration shadows a variable in the global scope [-Werror,-Wshadow]
      QPCIDevice *dev;
                  ^
  tests/qtest/rtl8139-test.c:25:20: note: previous declaration is here
  static QPCIDevice *dev;
                     ^

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Message-ID: <20231009100251.56019-7-philmd@linaro.org>
Signed-off-by: Thomas Huth <thuth@redhat.com>
  • Loading branch information
philmd authored and huth committed Oct 27, 2023
1 parent e33ba60 commit a186fed
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions tests/qtest/rtl8139-test.c
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ static void nop(void)
#define CLK 33333333

static QPCIBus *pcibus;
static QPCIDevice *dev;
static QPCIDevice *pcidev;
static QPCIBar dev_bar;

static void save_fn(QPCIDevice *dev, int devfn, void *data)
Expand All @@ -46,7 +46,7 @@ static QPCIDevice *get_device(void)
#define PORT(name, len, val) \
static unsigned __attribute__((unused)) in_##name(void) \
{ \
unsigned res = qpci_io_read##len(dev, dev_bar, (val)); \
unsigned res = qpci_io_read##len(pcidev, dev_bar, (val)); \
if (verbosity_level >= 2) { \
g_test_message("*%s -> %x", #name, res); \
} \
Expand All @@ -57,7 +57,7 @@ static void out_##name(unsigned v) \
if (verbosity_level >= 2) { \
g_test_message("%x -> *%s", v, #name); \
} \
qpci_io_write##len(dev, dev_bar, (val), v); \
qpci_io_write##len(pcidev, dev_bar, (val), v); \
}

PORT(Timer, l, 0x48)
Expand Down Expand Up @@ -189,11 +189,11 @@ static void test_init(void)
{
uint64_t barsize;

dev = get_device();
pcidev = get_device();

dev_bar = qpci_iomap(dev, 0, &barsize);
dev_bar = qpci_iomap(pcidev, 0, &barsize);

qpci_device_enable(dev);
qpci_device_enable(pcidev);

test_timer();
}
Expand Down

0 comments on commit a186fed

Please sign in to comment.