Re: [PATCH 1/1] riscv: sifive: fu540: redundant initialization

2020-08-04 Thread Rick Chen
> From: Heinrich Schuchardt [mailto:xypron.g...@gmx.de] > Sent: Tuesday, August 04, 2020 5:10 AM > To: Rick Jian-Zhi Chen(陳建志) > Cc: Pragnesh Patel; Bin Meng; u-boot@lists.denx.de; Heinrich Schuchardt > Subject: [PATCH 1/1] riscv: sifive: fu540: redundant initialization &g

RE: [PATCH 1/1] riscv: sifive: fu540: redundant initialization

2020-08-03 Thread Pragnesh Patel
>-Original Message- >From: Heinrich Schuchardt >Sent: 04 August 2020 02:40 >To: Rick Chen >Cc: Pragnesh Patel ; Bin Meng >; u-boot@lists.denx.de; Heinrich Schuchardt > >Subject: [PATCH 1/1] riscv: sifive: fu540: redundant initialization > >[Externa

Re: [PATCH 1/1] riscv: sifive: fu540: redundant initialization

2020-08-03 Thread Bin Meng
On Tue, Aug 4, 2020 at 5:10 AM Heinrich Schuchardt wrote: > > We should not initialize a variable if the value is overwritten before > being read. > > Signed-off-by: Heinrich Schuchardt > --- > arch/riscv/cpu/fu540/cache.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Reviewed-by: B

[PATCH 1/1] riscv: sifive: fu540: redundant initialization

2020-08-03 Thread Heinrich Schuchardt
We should not initialize a variable if the value is overwritten before being read. Signed-off-by: Heinrich Schuchardt --- arch/riscv/cpu/fu540/cache.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/riscv/cpu/fu540/cache.c b/arch/riscv/cpu/fu540/cache.c index 9ee364b509.