On 22/9/23 17:59, Cédric Le Goater wrote:
Remove superfluous local 'data' variable and use the one define at the top of the routine. This fixes :../hw/i2c/aspeed_i2c.c: In function ‘aspeed_i2c_bus_recv’: ../hw/i2c/aspeed_i2c.c:315:17: warning: declaration of ‘data’ shadows a previous local [-Wshadow=compatible-local] 315 | uint8_t data; | ^~~~ ../hw/i2c/aspeed_i2c.c:288:13: note: shadowed declaration is here 288 | uint8_t data; | ^~~~ Signed-off-by: Cédric Le Goater <[email protected]> --- hw/i2c/aspeed_i2c.c | 1 - 1 file changed, 1 deletion(-)
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
