| Message ID | 20200215205831.22308-1-xypron.glpk@gmx.de |
|---|---|
| State | Accepted |
| Commit | a1dd1fcb6ede14f74d3c7ec1319a6f3f9cd23f6a |
| Delegated to: | Stefan Roese |
| Headers | show |
| Series | [1/1] arm: mvebu: drivers/ddr: remove redundant assignment | expand |
On 15.02.20 21:58, Heinrich Schuchardt wrote: > The value of local variable ecc is immediately overwritten. So we can > remove the first assignment. > > Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de> Reviewed-by: Stefan Roese <sr@denx.de> Thanks, Stefan
diff --git a/drivers/ddr/marvell/axp/ddr3_init.c b/drivers/ddr/marvell/axp/ddr3_init.c index 13df9126e7..30ad5d4151 100644 --- a/drivers/ddr/marvell/axp/ddr3_init.c +++ b/drivers/ddr/marvell/axp/ddr3_init.c @@ -434,10 +434,6 @@ static u32 ddr3_init_main(void) scrub_size = 0; #endif -#if defined(ECC_SUPPORT) && defined(AUTO_DETECTION_SUPPORT) - ecc = DRAM_ECC; -#endif - #if defined(ECC_SUPPORT) && defined(AUTO_DETECTION_SUPPORT) ecc = 0; if (ddr3_check_config(BUS_WIDTH_ECC_TWSI_ADDR, CONFIG_ECC))
The value of local variable ecc is immediately overwritten. So we can remove the first assignment. Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de> --- drivers/ddr/marvell/axp/ddr3_init.c | 4 ---- 1 file changed, 4 deletions(-) -- 2.25.0