[U-Boot-Users] [PATCH] HMI1001: add support for MPC5200 Rev. B processors.

[U-Boot-Users] [PATCH] HMI1001: add support for MPC5200 Rev. B processors.

[Wolfgang Denk]

Signed-off-by: Wolfgang Denk <wd@denx.de>
---
 board/hmi1001/hmi1001.c |   18 ++++++++++++++++++
 1 files changed, 18 insertions(+), 0 deletions(-)

diff --git a/board/hmi1001/hmi1001.c b/board/hmi1001/hmi1001.c
index 9fa0e74..3ecb74a 100644
--- a/board/hmi1001/hmi1001.c
+++ b/board/hmi1001/hmi1001.c
@@ -147,6 +147,24 @@ long int initdram (int board_type)
 
 #endif /* CFG_RAMBOOT */
 
+	/*
+	 * On MPC5200B we need to set the special configuration delay in the
+	 * DDR controller. Please refer to Freescale's AN3221 "MPC5200B SDRAM
+	 * Initialization and Configuration", 3.3.1 SDelay--MBAR + 0x0190:
+	 *
+	 * "The SDelay should be written to a value of 0x00000004. It is
+	 * required to account for changes caused by normal wafer processing
+	 * parameters."
+	 */
+	svr = get_svr();
+	pvr = get_pvr();
+	if ((SVR_MJREV(svr) >= 2) &&
+	    (PVR_MAJ(pvr) == 1) && (PVR_MIN(pvr) == 4)) {
+
+		*(vu_long *)MPC5XXX_SDRAM_SDELAY = 0x04;
+		__asm__ volatile ("sync");
+	}
+
 /*	return dramsize + dramsize2; */
 	return dramsize;
 }
-- 
1.5.4.2

[U-Boot-Users] [PATCH] HMI1001: add support for MPC5200 Rev. B processors.

[Jean-Christophe PLAGNIOL-VILLARD]

On 14:33 Thu 13 Mar     , Wolfgang Denk wrote:
> Signed-off-by: Wolfgang Denk <wd@denx.de>
> ---
>  board/hmi1001/hmi1001.c |   18 ++++++++++++++++++
>  1 files changed, 18 insertions(+), 0 deletions(-)
> 
> diff --git a/board/hmi1001/hmi1001.c b/board/hmi1001/hmi1001.c
> index 9fa0e74..3ecb74a 100644
> --- a/board/hmi1001/hmi1001.c
> +++ b/board/hmi1001/hmi1001.c
> @@ -147,6 +147,24 @@ long int initdram (int board_type)
>  
>  #endif /* CFG_RAMBOOT */
>  
> +	/*
> +	 * On MPC5200B we need to set the special configuration delay in the
> +	 * DDR controller. Please refer to Freescale's AN3221 "MPC5200B SDRAM
> +	 * Initialization and Configuration", 3.3.1 SDelay--MBAR + 0x0190:
> +	 *
> +	 * "The SDelay should be written to a value of 0x00000004. It is
> +	 * required to account for changes caused by normal wafer processing
> +	 * parameters."
> +	 */
> +	svr = get_svr();
> +	pvr = get_pvr();
> +	if ((SVR_MJREV(svr) >= 2) &&
> +	    (PVR_MAJ(pvr) == 1) && (PVR_MIN(pvr) == 4)) {
> +
> +		*(vu_long *)MPC5XXX_SDRAM_SDELAY = 0x04;
> +		__asm__ volatile ("sync");
> +	}
> +
>  /*	return dramsize + dramsize2; */
Do we need to keep it?
>  	return dramsize;
>  }
Best Regards,
J.

[U-Boot-Users] [PATCH] HMI1001: add support for MPC5200 Rev. B processors.

[Wolfgang Denk]

In message <20080313155131.GD17306@game.jcrosoft.org> you wrote:
>
> >  /*	return dramsize + dramsize2; */
> Do we need to keep it?

Probably not on the current hardware configuration; maybe the next one
will use two banks of RAM.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
A princess should not be afraid -- not with a brave knight to protect
her.
	-- McCoy, "Shore Leave", stardate 3025.3