On 22/9/23 17:59, Cédric Le Goater wrote:
Remove superfluous local 'irq' variables and use the one define at the
top of the routine. This fixes warnings in aspeed_soc_ast2600_realize()
such as :

   ../hw/arm/aspeed_ast2600.c: In function ‘aspeed_soc_ast2600_realize’:
   ../hw/arm/aspeed_ast2600.c:420:18: warning: declaration of ‘irq’ shadows a 
previous local [-Wshadow=compatible-local]
     420 |         qemu_irq irq = aspeed_soc_get_irq(s, ASPEED_DEV_TIMER1 + i);
         |                  ^~~
   ../hw/arm/aspeed_ast2600.c:312:14: note: shadowed declaration is here
     312 |     qemu_irq irq;
         |              ^~~

Signed-off-by: Cédric Le Goater <c...@kaod.org>
---
  hw/arm/aspeed_ast2600.c | 10 +++++-----
  1 file changed, 5 insertions(+), 5 deletions(-)

Reviewed-by: Philippe Mathieu-Daudé <phi...@linaro.org>


Reply via email to