[U-Boot] [PATCH] Revert "mx6: ddr: Allow changing REFSEL and REFR fields"

Stefano Babic sbabic at denx.de
Tue Feb 20 14:29:40 UTC 2018


On 20/02/2018 11:08, Maxim Yu. Osipov wrote:
> This reverts commit edf0093732225c2fd0791c3864e9a3eef1f92f19 for
> cm_fx6 iMX.6 Solo module as it causes frequent (around 10 percent of
> power cycles) board's hangs.
> 
> These hangs happen in SPL when BSS is being initialized in SDRAM -
> it appear that variables from BSS contain trash values which lead to board
> hangs. Looks like that SDRAM doesn't yet finish initialization in these
> cases.
> 
> Signed-off-by: Maxim Yu. Osipov <mosipov at ilbers.de>
> ---
>  board/compulab/cm_fx6/spl.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/board/compulab/cm_fx6/spl.c b/board/compulab/cm_fx6/spl.c
> index 56aac60239..16e5bf8dfa 100644
> --- a/board/compulab/cm_fx6/spl.c
> +++ b/board/compulab/cm_fx6/spl.c
> @@ -107,8 +107,6 @@ static struct mx6_ddr_sysinfo cm_fx6_sysinfo_s = {
>  	.mif3_mode	= 3,
>  	.rst_to_cke	= 0x23,
>  	.sde_to_rst	= 0x10,
> -	.refsel = 1,		/* Refresh cycles at 32KHz */
> -	.refr = 7,		/* 8 refresh commands per refresh cycle */
>  };
>  
>  static struct mx6_ddr3_cfg cm_fx6_ddr3_cfg_s = {
> 

Reviewed-by: Stefano Babic <sbabic at denx.de>

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================


More information about the U-Boot mailing list