[U-Boot] [PATCH] ARM: socfpga: Unreset NAND in SPL on Gen5

[Tan, Ley Foon]

> -----Original Message-----
> From: Marek Vasut <marex at denx.de>
> Sent: Thursday, November 21, 2019 5:36 AM
> To: u-boot at lists.denx.de
> Cc: Marek Vasut <marex at denx.de>; See, Chin Liang
> <chin.liang.see at intel.com>; Dalon Westergreen <dwesterg at gmail.com>;
> Dinh Nguyen <dinguyen at kernel.org>; Tan, Ley Foon
> <ley.foon.tan at intel.com>; Simon Goldschmidt
> <simon.k.r.goldschmidt at gmail.com>; Chee, Tien Fong
> <tien.fong.chee at intel.com>
> Subject: [PATCH] ARM: socfpga: Unreset NAND in SPL on Gen5
> 
> In case the SPL on Gen5 loads U-Boot from NAND, unreset the NAND IP
> explicitly in the platform code as the denali-spl driver is not aware of DM at
> all.
> 
> Signed-off-by: Marek Vasut <marex at denx.de>
> Cc: Chin Liang See <chin.liang.see at intel.com>
> Cc: Dalon Westergreen <dwesterg at gmail.com>
> Cc: Dinh Nguyen <dinguyen at kernel.org>
> Cc: Ley Foon Tan <ley.foon.tan at intel.com>
> Cc: Simon Goldschmidt <simon.k.r.goldschmidt at gmail.com>
> Cc: Tien Fong Chee <tien.fong.chee at intel.com>
> ---
>  arch/arm/mach-socfpga/spl_gen5.c | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/arch/arm/mach-socfpga/spl_gen5.c b/arch/arm/mach-
> socfpga/spl_gen5.c
> index 47e63709ad..408e409375 100644
> --- a/arch/arm/mach-socfpga/spl_gen5.c
> +++ b/arch/arm/mach-socfpga/spl_gen5.c
> @@ -138,6 +138,13 @@ void board_init_f(ulong dummy)
>  	if (ret)
>  		debug("Reset init failed: %d\n", ret);
> 
> +#ifdef CONFIG_SPL_NAND_DENALI
> +	struct socfpga_reset_manager *reset_manager_base =
> +		(struct socfpga_reset_manager
> *)SOCFPGA_RSTMGR_ADDRESS;
> +
> +	clrbits_le32(&reset_manager_base->per_mod_reset, BIT(4)); #endif
> +
Normal Denali Nand driver (denali_dt_probe()) is not running in SPL Gen5?
I am enabling NAND for Agilex recently, but didn't notice need to de-assert NAND reset outside of denali nand driver.

Regards
Ley Foon