[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/4] aspeed/i2c: Clean up local variable shadowing
From: |
Joel Stanley |
Subject: |
Re: [PATCH 1/4] aspeed/i2c: Clean up local variable shadowing |
Date: |
Wed, 27 Sep 2023 02:38:14 +0000 |
On Fri, 22 Sept 2023 at 15:59, Cédric Le Goater <clg@kaod.org> 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 <clg@kaod.org>
Reviewed-by: Joel Stanley <joel@jms.id.au>
messy.
> ---
> hw/i2c/aspeed_i2c.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/hw/i2c/aspeed_i2c.c b/hw/i2c/aspeed_i2c.c
> index 7275d40749a9..1037c22b2f79 100644
> --- a/hw/i2c/aspeed_i2c.c
> +++ b/hw/i2c/aspeed_i2c.c
> @@ -312,7 +312,6 @@ static void aspeed_i2c_bus_recv(AspeedI2CBus *bus)
> SHARED_ARRAY_FIELD_DP32(bus->regs, reg_pool_ctrl, RX_COUNT, i &
> 0xff);
> SHARED_ARRAY_FIELD_DP32(bus->regs, reg_cmd, RX_BUFF_EN, 0);
> } else if (SHARED_ARRAY_FIELD_EX32(bus->regs, reg_cmd, RX_DMA_EN)) {
> - uint8_t data;
> /* In new mode, clear how many bytes we RXed */
> if (aspeed_i2c_is_new_mode(bus->controller)) {
> ARRAY_FIELD_DP32(bus->regs, I2CM_DMA_LEN_STS, RX_LEN, 0);
> --
> 2.41.0
>
- [PATCH 0/4] aspeed: Clean up local variable shadowing, Cédric Le Goater, 2023/09/22
- [PATCH 1/4] aspeed/i2c: Clean up local variable shadowing, Cédric Le Goater, 2023/09/22
- [PATCH 2/4] aspeed: Clean up local variable shadowing, Cédric Le Goater, 2023/09/22
- [PATCH 3/4] aspeed/i3c: Rename variable shadowing a local, Cédric Le Goater, 2023/09/22
- [PATCH 4/4] aspeed/timer: Clean up local variable shadowing, Cédric Le Goater, 2023/09/22
- Re: [PATCH 0/4] aspeed: Clean up local variable shadowing, Philippe Mathieu-Daudé, 2023/09/22