Skip to contentSkip to conversation

HMI1001: add support for MPC5200 Rev. B processors.

PATCH New Wolfgang Denk 2 participants 3 messages Mar 13, 2008

lore
Share & tools
lore

Wolfgang Denk

author
#0 lore raw

PATCH 1/1 HMI1001: add support for MPC5200 Rev. B processors.

Signed-off-by Wolfgang Denk <[email protected]>
---
 board/hmi1001/hmi1001.c |   18 ++++++++++++++++++
 1 files changed, 18 insertions(+), 0 deletions(-)
board/hmi1001/hmi1001.c +18 -0
index 9fa0e74..3ecb74a 100644
@@ -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;
 }
Signature · 1 line
1.5.4.2
#1 lore raw

PATCH 1/1 HMI1001: add support for MPC5200 Rev. B processors.

Quoted 22 lines · Wolfgang Denk
Signed-off-by Wolfgang Denk <[email protected]>
---
 board/hmi1001/hmi1001.c |   18 ++++++++++++++++++
 1 files changed, 18 insertions(+), 0 deletions(-)
board/hmi1001/hmi1001.c +9 -0
index 9fa0e74..3ecb74a 100644
@@ -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."
+	 */

board/hmi1001/hmi1001.c long int initdram (int board_type) · line 168

Quoted 10 lines
board/hmi1001/hmi1001.c +9 -0
+	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?

board/hmi1001/hmi1001.c long int initdram (int board_type) · line 170

Quoted 2 lines
board/hmi1001/hmi1001.c +0 -0
 	return dramsize;
 }

Best Regards,
J.

author
#2 lore raw

PATCH 1/1 HMI1001: add support for MPC5200 Rev. B processors.

Quoted 3 lines · In message <[email protected]> you wrote:
Quoted 2 lines

/* 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

Signature · 6 lines
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