mirrors/qemu
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

hw/ppc: Clean up local variable shadowing in _FDT helper routine

Browse Source 
this fixes numerous warnings of this type :

  In file included from ../hw/ppc/spapr_pci.c:43:
  ../hw/ppc/spapr_pci.c: In function ‘spapr_dt_phb’:
  ../include/hw/ppc/fdt.h:18:13: warning: declaration of ‘ret’ shadows a previous local [-Wshadow=compatible-local]
     18 |         int ret = (exp);                                           \
        |             ^~~
  ../hw/ppc/spapr_pci.c:2355:5: note: in expansion of macro ‘_FDT’
   2355 |     _FDT(bus_off = fdt_add_subnode(fdt, 0, phb->dtbusname));
        |     ^~~~
  ../hw/ppc/spapr_pci.c:2311:24: note: shadowed declaration is here
   2311 |     int bus_off, i, j, ret;
        |                        ^~~

Signed-off-by: Cédric Le Goater <clg@kaod.org>
Message-ID: <20230918145850.241074-2-clg@kaod.org>
Reviewed-by: Harsh Prateek Bora <harshpb@linux.ibm.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
This commit is contained in:
Cédric Le Goater 2023-09-18 16:58:43 +02:00 committed by Markus Armbruster
parent 1cc0c5dd38
commit 90231ce1a3
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
include/hw/ppc/fdt.h
View File

@ -15,10 +15,10 @@
#define _FDT(exp) \ #define _FDT(exp) \
do { \ do { \
int ret = (exp); \ int _ret = (exp); \
if (ret < 0) { \ if (_ret < 0) { \
error_report("error creating device tree: %s: %s", \ error_report("error creating device tree: %s: %s", \
#exp, fdt_strerror(ret)); \ #exp, fdt_strerror(_ret)); \
exit(1); \ exit(1); \
} \ } \
} while (0) } while (0)