diff mbox series

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

Message ID 20180220100809.379-1-mosipov@ilbers.de
State Accepted
Commit d247cf50698a590571cf9a6fd4608e702580e40e
Headers show
Series [U-Boot] Revert "mx6: ddr: Allow changing REFSEL and REFR fields" | expand

Commit Message

Maxim Yu. Osipov Feb. 20, 2018, 10:08 a.m. UTC
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@ilbers.de>
---
 board/compulab/cm_fx6/spl.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Stefano Babic Feb. 20, 2018, 2:29 p.m. UTC | #1
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@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@denx.de>

Best regards,
Stefano Babic
diff mbox series

Patch

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 = {