[U-Boot] [PATCH 12/17] powerpc/km8360: fix the ODT parameters for CS0

Heiko Schocher hs at denx.de
Mon Nov 16 12:03:48 CET 2015


Hello Valentin,

Am 13.11.2015 um 15:59 schrieb Valentin Longchamp:
> The ODT parameters for km8360 set the ODT_WR_ACS bit in u-boot KM-2011.09
> that is used in the release bootpackage for kmcoge5ne. During the
> transition from the kmeter1 to km8360 and with the migration to
> KM-2012.10 (commit 0f2b721c80fa50c8e09548f0ad1b4210d2197bf9), this
> was changed to ODT_RD_ONLY_CURRENT, which is uncorrect and causes faulty
> RAM accesses at low temperatures.
>
> This is now changed to ODT_WR_ONLY_CURRENT which is the equivalent of
> ODT_WR_ACS.
>
> Signed-off-by: Valentin Longchamp <valentin.longchamp at keymile.com>
> ---
>
>   include/configs/km8360.h | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)


Reviewed-by: Heiko Schocher <hs at denx.de>

bye,
Heiko
>
> diff --git a/include/configs/km8360.h b/include/configs/km8360.h
> index 04cde46..5034fda 100644
> --- a/include/configs/km8360.h
> +++ b/include/configs/km8360.h
> @@ -100,7 +100,7 @@
>   #define CONFIG_SYS_DDR_CS0_CONFIG (\
>   	CSCONFIG_EN | \
>   	CSCONFIG_AP | \
> -	CSCONFIG_ODT_RD_ONLY_CURRENT | \
> +	CSCONFIG_ODT_WR_ONLY_CURRENT | \
>   	CSCONFIG_BANK_BIT_3 | \
>   	CSCONFIG_ROW_BIT_13 | \
>   	CSCONFIG_COL_BIT_10)
> @@ -108,7 +108,7 @@
>   #define CONFIG_SYS_DDR_CS0_CONFIG	(CSCONFIG_EN | CSCONFIG_AP | \
>   					 CSCONFIG_ROW_BIT_13 | \
>   					 CSCONFIG_COL_BIT_10 | \
> -					 CSCONFIG_ODT_RD_ONLY_CURRENT)
> +					 CSCONFIG_ODT_WR_ONLY_CURRENT)
>   #endif
>
>   #define CONFIG_SYS_DDR_CLK_CNTL (\
>

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany


More information about the U-Boot mailing list