All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH 0/2] OMAP3:DDR timing cleanup series
       [not found] <OMAP3 DDR Fix patches (was Re: [U-Boot] [PATCH 1/5 v2] OMAP3: Fix SDRC init)>
@ 2009-10-21  2:01 ` Nishanth Menon
  2009-10-21  2:01   ` [U-Boot] [PATCH 1/2] OMAP3:SDRC: Cleanup references to SDP Nishanth Menon
  0 siblings, 1 reply; 13+ messages in thread
From: Nishanth Menon @ 2009-10-21  2:01 UTC (permalink / raw)
  To: u-boot

Hi,
in continuation of the discussion on:
http://www.nabble.com/forum/Permalink.jtp?root=25779518&post=25959734&page=y

the following patch set should introduce DDR
timing changes for MICRON and INFINEON DDRs
and allow both classes of boards to live
happily together

Tested ON:
SDP3430 - type INFENINION
BeagleBoard - type MICRON
	tested with mtest 0x82000000 0x83000000 1
	from u-boot prompt

Nishanth Menon (2):
  OMAP3:SDRC: Cleanup references to SDP
  OMAP3:SDRC: introduce DDR types

 cpu/arm_cortexa8/omap3/mem.c      |    2 +-
 cpu/arm_cortexa8/omap3/sys_info.c |    2 +-
 include/asm-arm/arch-omap3/mem.h  |   97 +++++++++++++++++++++++++++----------
 include/configs/devkit8000.h      |    3 +
 include/configs/omap3_beagle.h    |    3 +
 include/configs/omap3_evm.h       |    3 +
 include/configs/omap3_overo.h     |    3 +
 include/configs/omap3_pandora.h   |    3 +
 include/configs/omap3_sdp3430.h   |    3 +
 include/configs/omap3_zoom1.h     |    3 +
 include/configs/omap3_zoom2.h     |    3 +
 11 files changed, 97 insertions(+), 28 deletions(-)

^ permalink raw reply	[flat|nested] 13+ messages in thread

* [U-Boot] [PATCH 1/2] OMAP3:SDRC: Cleanup references to SDP
  2009-10-21  2:01 ` [U-Boot] [PATCH 0/2] OMAP3:DDR timing cleanup series Nishanth Menon
@ 2009-10-21  2:01   ` Nishanth Menon
  2009-10-21  2:01     ` [U-Boot] [PATCH 2/2] OMAP3:SDRC: introduce DDR types Nishanth Menon
  2009-11-07 16:08     ` [U-Boot] [PATCH 1/2] OMAP3:SDRC: Cleanup references to SDP Paulraj, Sandeep
  0 siblings, 2 replies; 13+ messages in thread
From: Nishanth Menon @ 2009-10-21  2:01 UTC (permalink / raw)
  To: u-boot

Remove SDP referenced unused defines

Signed-off-by: Nishanth Menon <nm@ti.com>
---
 cpu/arm_cortexa8/omap3/mem.c      |    2 +-
 cpu/arm_cortexa8/omap3/sys_info.c |    2 +-
 include/asm-arm/arch-omap3/mem.h  |   11 ++---------
 3 files changed, 4 insertions(+), 11 deletions(-)

diff --git a/cpu/arm_cortexa8/omap3/mem.c b/cpu/arm_cortexa8/omap3/mem.c
index 5e6d542..dfb7e4c 100644
--- a/cpu/arm_cortexa8/omap3/mem.c
+++ b/cpu/arm_cortexa8/omap3/mem.c
@@ -161,7 +161,7 @@ void do_sdrc_init(u32 cs, u32 early)
 		writel(0, &sdrc_base->sysconfig);
 
 		/* setup sdrc to ball mux */
-		writel(SDP_SDRC_SHARING, &sdrc_base->sharing);
+		writel(SDRC_SHARING, &sdrc_base->sharing);
 
 		/* Disable Power Down of CKE cuz of 1 CKE on combo part */
 		writel(WAKEUPPROC | PWDNEN | SRFRONRESET | PAGEPOLICY_HIGH,
diff --git a/cpu/arm_cortexa8/omap3/sys_info.c b/cpu/arm_cortexa8/omap3/sys_info.c
index 31b2003..08fb32e 100644
--- a/cpu/arm_cortexa8/omap3/sys_info.c
+++ b/cpu/arm_cortexa8/omap3/sys_info.c
@@ -109,7 +109,7 @@ u32 get_cpu_rev(void)
  ****************************************************/
 u32 is_mem_sdr(void)
 {
-	if (readl(&sdrc_base->cs[CS0].mr) == SDP_SDRC_MR_0_SDR)
+	if (readl(&sdrc_base->cs[CS0].mr) == SDRC_MR_0_SDR)
 		return 1;
 	return 0;
 }
diff --git a/include/asm-arm/arch-omap3/mem.h b/include/asm-arm/arch-omap3/mem.h
index 31cbdef..3ef8478 100644
--- a/include/asm-arm/arch-omap3/mem.h
+++ b/include/asm-arm/arch-omap3/mem.h
@@ -40,11 +40,8 @@ enum {
 #define EARLY_INIT	1
 
 /* Slower full frequency range default timings for x32 operation*/
-#define SDP_SDRC_SHARING	0x00000100
-#define SDP_SDRC_MR_0_SDR	0x00000031
-
-/* optimized timings good for current shipping parts */
-#define SDP_3430_SDRC_RFR_CTRL_165MHz	0x0004e201 /* 7.8us/6ns - 50=0x4e2 */
+#define SDRC_SHARING	0x00000100
+#define SDRC_MR_0_SDR	0x00000031
 
 #define DLL_OFFSET		0
 #define DLL_WRITEDDRCLKX2DIS	1
@@ -91,10 +88,6 @@ enum {
 #define V_ACTIMB_165	(((TCKE_165 << 12) | (XSR_165 << 0)) |	\
 			(TXP_165 << 8) | (TWTR_165 << 16))
 
-#define SDP_SDRC_ACTIM_CTRLA_0	V_ACTIMA_165
-#define SDP_SDRC_ACTIM_CTRLB_0	V_ACTIMB_165
-#define SDP_SDRC_RFR_CTRL	SDP_3430_SDRC_RFR_CTRL_165MHz
-
 /*
  * GPMC settings -
  * Definitions is as per the following format
-- 
1.6.3.3

^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [U-Boot] [PATCH 2/2] OMAP3:SDRC: introduce DDR types
  2009-10-21  2:01   ` [U-Boot] [PATCH 1/2] OMAP3:SDRC: Cleanup references to SDP Nishanth Menon
@ 2009-10-21  2:01     ` Nishanth Menon
  2009-11-07 16:11       ` Paulraj, Sandeep
  2009-11-07 16:08     ` [U-Boot] [PATCH 1/2] OMAP3:SDRC: Cleanup references to SDP Paulraj, Sandeep
  1 sibling, 1 reply; 13+ messages in thread
From: Nishanth Menon @ 2009-10-21  2:01 UTC (permalink / raw)
  To: u-boot

Micron DDR timings based on:
http://www.sakoman.net/cgi-bin/gitweb.cgi?p=x-load-omap3.git;a=blob;f=include/asm/arch-omap3/mem.h;h=e6fbfe3947f5d0d85fea776e30821d4017316d86;hb=HEAD

Introduce Micron DDR timings and provide
CONFIG_OMAP3_INFINEON_DDR and CONFIG_OMAP3_MICRON_DDR config
options to allow for platform files to setup their timings as
per the type of DDR selected

Reported-by: Steve Sakoman in http://www.nabble.com/forum/Permalink.jtp?root=25779518&post=25959734&page=y

Signed-off-by: Nishanth Menon <nm@ti.com>
---
 include/asm-arm/arch-omap3/mem.h |   88 ++++++++++++++++++++++++++++++--------
 include/configs/devkit8000.h     |    3 +
 include/configs/omap3_beagle.h   |    3 +
 include/configs/omap3_evm.h      |    3 +
 include/configs/omap3_overo.h    |    3 +
 include/configs/omap3_pandora.h  |    3 +
 include/configs/omap3_sdp3430.h  |    3 +
 include/configs/omap3_zoom1.h    |    3 +
 include/configs/omap3_zoom2.h    |    3 +
 9 files changed, 94 insertions(+), 18 deletions(-)

diff --git a/include/asm-arm/arch-omap3/mem.h b/include/asm-arm/arch-omap3/mem.h
index 3ef8478..ae06a14 100644
--- a/include/asm-arm/arch-omap3/mem.h
+++ b/include/asm-arm/arch-omap3/mem.h
@@ -68,26 +68,78 @@ enum {
  *	TCKE = 2
  *	XSR = 120/6 = 20
  */
-#define TDAL_165	6
-#define TDPL_165	3
-#define TRRD_165	2
-#define TRCD_165	3
-#define TRP_165		3
-#define TRAS_165	7
-#define TRC_165		10
-#define TRFC_165	12
-#define V_ACTIMA_165	((TRFC_165 << 27) | (TRC_165 << 22) | \
-			(TRAS_165 << 18) | (TRP_165 << 15) |  \
-			(TRCD_165 << 12) | (TRRD_165 << 9) |  \
-			(TDPL_165 << 6) | (TDAL_165))
-
-#define TWTR_165	1
-#define TCKE_165	2
-#define TXP_165		2
-#define XSR_165		20
-#define V_ACTIMB_165	(((TCKE_165 << 12) | (XSR_165 << 0)) |	\
-			(TXP_165 << 8) | (TWTR_165 << 16))
+#define INFINEON_TDAL_165	6
+#define INFINEON_TDPL_165	3
+#define INFINEON_TRRD_165	2
+#define INFINEON_TRCD_165	3
+#define INFINEON_TRP_165	3
+#define INFINEON_TRAS_165	7
+#define INFINEON_TRC_165	10
+#define INFINEON_TRFC_165	12
+#define INFINEON_V_ACTIMA_165	((INFINEON_TRFC_165 << 27) |		\
+		(INFINEON_TRC_165 << 22) | (INFINEON_TRAS_165 << 18) |	\
+		(INFINEON_TRP_165 << 15) | (INFINEON_TRCD_165 << 12) |	\
+		(INFINEON_TRRD_165 << 9) | (INFINEON_TDPL_165 << 6) |	\
+		(INFINEON_TDAL_165))
 
+#define INFINEON_TWTR_165	1
+#define INFINEON_TCKE_165	2
+#define INFINEON_TXP_165	2
+#define INFINEON_XSR_165	20
+#define INFINEON_V_ACTIMB_165	((INFINEON_TCKE_165 << 12) |		\
+		(INFINEON_XSR_165 << 0) | (INFINEON_TXP_165 << 8) |	\
+		(INFINEON_TWTR_165 << 16))
+
+/* Micron part of 3430 EVM (165MHz optimized) 6.06ns
+ * ACTIMA
+ *	TDAL = Twr/Tck + Trp/tck= 15/6 + 18 /6 = 2.5 + 3 = 5.5 -> 6
+ *	TDPL (Twr)	= 15/6	= 2.5 -> 3
+ *	TRRD		= 12/6	= 2
+ *	TRCD		= 18/6	= 3
+ *	TRP		= 18/6	= 3
+ *	TRAS		= 42/6	= 7
+ *	TRC		= 60/6	= 10
+ *	TRFC		= 125/6	= 21
+ * ACTIMB
+ *	TWTR		= 1
+ *	TCKE		= 1
+ *	TXSR		= 138/6	= 23
+ *	TXP		= 25/6	= 4.1 ~5
+ */
+#define MICRON_TDAL_165		6
+#define MICRON_TDPL_165		3
+#define MICRON_TRRD_165		2
+#define MICRON_TRCD_165		3
+#define MICRON_TRP_165		3
+#define MICRON_TRAS_165		7
+#define MICRON_TRC_165		10
+#define MICRON_TRFC_165		21
+#define MICRON_V_ACTIMA_165 ((MICRON_TRFC_165 << 27) |			\
+		(MICRON_TRC_165 << 22) | (MICRON_TRAS_165 << 18) |	\
+		(MICRON_TRP_165 << 15) | (MICRON_TRCD_165 << 12) |	\
+		(MICRON_TRRD_165 << 9) | (MICRON_TDPL_165 << 6) |	\
+		(MICRON_TDAL_165))
+
+#define MICRON_TWTR_165		1
+#define MICRON_TCKE_165		1
+#define MICRON_XSR_165		23
+#define MICRON_TXP_165		5
+#define MICRON_V_ACTIMB_165 ((MICRON_TCKE_165 << 12) |			\
+		(MICRON_XSR_165 << 0) | (MICRON_TXP_165 << 8) |	\
+		(MICRON_TWTR_165 << 16))
+
+#ifdef CONFIG_OMAP3_INFINEON_DDR
+#define V_ACTIMA_165 INFINEON_V_ACTIMA_165
+#define V_ACTIMB_165 INFINEON_V_ACTIMB_165
+#endif
+#ifdef CONFIG_OMAP3_MICRON_DDR
+#define V_ACTIMA_165 MICRON_V_ACTIMA_165
+#define V_ACTIMB_165 MICRON_V_ACTIMB_165
+#endif
+
+#if !defined(V_ACTIMA_165) || !defined(V_ACTIMB_165)
+#error "Please choose the right DDR type in config header"
+#endif
 /*
  * GPMC settings -
  * Definitions is as per the following format
diff --git a/include/configs/devkit8000.h b/include/configs/devkit8000.h
index bd5037e..1011770 100644
--- a/include/configs/devkit8000.h
+++ b/include/configs/devkit8000.h
@@ -66,6 +66,9 @@
 
 /* Hardware drivers */
 
+/* DDR - I use Micron DDR */
+#define CONFIG_OMAP3_MICRON_DDR		1
+
 /* DM9000 */
 #define CONFIG_NET_MULTI		1
 #define CONFIG_NET_RETRY_COUNT		20
diff --git a/include/configs/omap3_beagle.h b/include/configs/omap3_beagle.h
index 19a5ec9..024b9b8 100644
--- a/include/configs/omap3_beagle.h
+++ b/include/configs/omap3_beagle.h
@@ -97,6 +97,9 @@
 #define CONFIG_OMAP3_MMC		1
 #define CONFIG_DOS_PARTITION		1
 
+/* DDR - I use Micron DDR */
+#define CONFIG_OMAP3_MICRON_DDR		1
+
 /* commands to include */
 #include <config_cmd_default.h>
 
diff --git a/include/configs/omap3_evm.h b/include/configs/omap3_evm.h
index a5514ae..6709edc 100644
--- a/include/configs/omap3_evm.h
+++ b/include/configs/omap3_evm.h
@@ -101,6 +101,9 @@
 #define CONFIG_OMAP3_MMC		1
 #define CONFIG_DOS_PARTITION		1
 
+/* DDR - I use Micron DDR */
+#define CONFIG_OMAP3_MICRON_DDR		1
+
 /* commands to include */
 #include <config_cmd_default.h>
 
diff --git a/include/configs/omap3_overo.h b/include/configs/omap3_overo.h
index ffb515d..0f812a7 100644
--- a/include/configs/omap3_overo.h
+++ b/include/configs/omap3_overo.h
@@ -89,6 +89,9 @@
 #define CONFIG_OMAP3_MMC		1
 #define CONFIG_DOS_PARTITION		1
 
+/* DDR - I use Micron DDR */
+#define CONFIG_OMAP3_MICRON_DDR		1
+
 /* commands to include */
 #include <config_cmd_default.h>
 
diff --git a/include/configs/omap3_pandora.h b/include/configs/omap3_pandora.h
index 6f21af3..0cafeb8 100644
--- a/include/configs/omap3_pandora.h
+++ b/include/configs/omap3_pandora.h
@@ -92,6 +92,9 @@
 #define CONFIG_OMAP3_MMC		1
 #define CONFIG_DOS_PARTITION		1
 
+/* DDR - I use Micron DDR */
+#define CONFIG_OMAP3_MICRON_DDR		1
+
 /* commands to include */
 #include <config_cmd_default.h>
 
diff --git a/include/configs/omap3_sdp3430.h b/include/configs/omap3_sdp3430.h
index 229dc5e..d91c8ff 100644
--- a/include/configs/omap3_sdp3430.h
+++ b/include/configs/omap3_sdp3430.h
@@ -137,6 +137,9 @@
 #define CONFIG_SYS_I2C_BUS_SELECT	1
 #define CONFIG_DRIVER_OMAP34XX_I2C	1
 
+/* DDR - I use Infineon DDR */
+#define CONFIG_OMAP3_INFINEON_DDR	1
+
 /* OMITTED:  single 1 Gbit MT29F1G NAND flash */
 
 /*
diff --git a/include/configs/omap3_zoom1.h b/include/configs/omap3_zoom1.h
index da4b677..2aef973 100644
--- a/include/configs/omap3_zoom1.h
+++ b/include/configs/omap3_zoom1.h
@@ -98,6 +98,9 @@
 #define CONFIG_OMAP3_MMC		1
 #define CONFIG_DOS_PARTITION		1
 
+/* DDR - I use Micron DDR */
+#define CONFIG_OMAP3_MICRON_DDR		1
+
 /* commands to include */
 #include <config_cmd_default.h>
 
diff --git a/include/configs/omap3_zoom2.h b/include/configs/omap3_zoom2.h
index 32cd6fd..5b03fb6 100644
--- a/include/configs/omap3_zoom2.h
+++ b/include/configs/omap3_zoom2.h
@@ -98,6 +98,9 @@
 #define CONFIG_OMAP3_MMC		1
 #define CONFIG_DOS_PARTITION		1
 
+/* DDR - I use Micron DDR */
+#define CONFIG_OMAP3_MICRON_DDR		1
+
 /* Status LED */
 #define CONFIG_STATUS_LED		1 /* Status LED enabled	*/
 #define CONFIG_BOARD_SPECIFIC_LED	1
-- 
1.6.3.3

^ permalink raw reply related	[flat|nested] 13+ messages in thread

* [U-Boot] [PATCH 1/2] OMAP3:SDRC: Cleanup references to SDP
  2009-10-21  2:01   ` [U-Boot] [PATCH 1/2] OMAP3:SDRC: Cleanup references to SDP Nishanth Menon
  2009-10-21  2:01     ` [U-Boot] [PATCH 2/2] OMAP3:SDRC: introduce DDR types Nishanth Menon
@ 2009-11-07 16:08     ` Paulraj, Sandeep
  1 sibling, 0 replies; 13+ messages in thread
From: Paulraj, Sandeep @ 2009-11-07 16:08 UTC (permalink / raw)
  To: u-boot



> Remove SDP referenced unused defines
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
>  cpu/arm_cortexa8/omap3/mem.c      |    2 +-
>  cpu/arm_cortexa8/omap3/sys_info.c |    2 +-
>  include/asm-arm/arch-omap3/mem.h  |   11 ++---------
>  3 files changed, 4 insertions(+), 11 deletions(-)


Pushed to u-boot-ti/next

Thanks,
Sandeep

^ permalink raw reply	[flat|nested] 13+ messages in thread

* [U-Boot] [PATCH 2/2] OMAP3:SDRC: introduce DDR types
  2009-10-21  2:01     ` [U-Boot] [PATCH 2/2] OMAP3:SDRC: introduce DDR types Nishanth Menon
@ 2009-11-07 16:11       ` Paulraj, Sandeep
  2009-11-07 17:31         ` Tom
  0 siblings, 1 reply; 13+ messages in thread
From: Paulraj, Sandeep @ 2009-11-07 16:11 UTC (permalink / raw)
  To: u-boot


> Micron DDR timings based on:
> http://www.sakoman.net/cgi-bin/gitweb.cgi?p=x-load-
> omap3.git;a=blob;f=include/asm/arch-
> omap3/mem.h;h=e6fbfe3947f5d0d85fea776e30821d4017316d86;hb=HEAD
> 
> Introduce Micron DDR timings and provide
> CONFIG_OMAP3_INFINEON_DDR and CONFIG_OMAP3_MICRON_DDR config
> options to allow for platform files to setup their timings as
> per the type of DDR selected
> 
> Reported-by: Steve Sakoman in
> http://www.nabble.com/forum/Permalink.jtp?root=25779518&post=25959734&page
> =y
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
>  include/asm-arm/arch-omap3/mem.h |   88 ++++++++++++++++++++++++++++++---
> -----
>  include/configs/devkit8000.h     |    3 +
>  include/configs/omap3_beagle.h   |    3 +
>  include/configs/omap3_evm.h      |    3 +
>  include/configs/omap3_overo.h    |    3 +
>  include/configs/omap3_pandora.h  |    3 +
>  include/configs/omap3_sdp3430.h  |    3 +
>  include/configs/omap3_zoom1.h    |    3 +
>  include/configs/omap3_zoom2.h    |    3 +
>  9 files changed, 94 insertions(+), 18 deletions(-)
> 

Pushed to u-boot-ti/next

I think this series from Nishanth needs some runtime tests.
I believe Steve had some issues with his EVM not booting everytime due to issues with DDR init.

Thanks,
Sandeep

^ permalink raw reply	[flat|nested] 13+ messages in thread

* [U-Boot] [PATCH 2/2] OMAP3:SDRC: introduce DDR types
  2009-11-07 16:11       ` Paulraj, Sandeep
@ 2009-11-07 17:31         ` Tom
  2009-11-07 18:15           ` Paulraj, Sandeep
  2009-11-12 16:48           ` Paulraj, Sandeep
  0 siblings, 2 replies; 13+ messages in thread
From: Tom @ 2009-11-07 17:31 UTC (permalink / raw)
  To: u-boot

Paulraj, Sandeep wrote:
>> Micron DDR timings based on:
>> http://www.sakoman.net/cgi-bin/gitweb.cgi?p=x-load-
>> omap3.git;a=blob;f=include/asm/arch-
>> omap3/mem.h;h=e6fbfe3947f5d0d85fea776e30821d4017316d86;hb=HEAD
>>
>> Introduce Micron DDR timings and provide
>> CONFIG_OMAP3_INFINEON_DDR and CONFIG_OMAP3_MICRON_DDR config
>> options to allow for platform files to setup their timings as
>> per the type of DDR selected
>>
>> Reported-by: Steve Sakoman in
>> http://www.nabble.com/forum/Permalink.jtp?root=25779518&post=25959734&page
>> =y
>>
>> Signed-off-by: Nishanth Menon <nm@ti.com>
>> ---
>>  include/asm-arm/arch-omap3/mem.h |   88 ++++++++++++++++++++++++++++++---
>> -----
>>  include/configs/devkit8000.h     |    3 +
>>  include/configs/omap3_beagle.h   |    3 +
>>  include/configs/omap3_evm.h      |    3 +
>>  include/configs/omap3_overo.h    |    3 +
>>  include/configs/omap3_pandora.h  |    3 +
>>  include/configs/omap3_sdp3430.h  |    3 +
>>  include/configs/omap3_zoom1.h    |    3 +
>>  include/configs/omap3_zoom2.h    |    3 +
>>  9 files changed, 94 insertions(+), 18 deletions(-)
>>
> 
> Pushed to u-boot-ti/next
> 
> I think this series from Nishanth needs some runtime tests.
> I believe Steve had some issues with his EVM not booting everytime due to issues with DDR init.
> 
> Thanks,
> Sandeep

Maybe it should go on the testing branch?

I will give zoom2 a try..

Tom

^ permalink raw reply	[flat|nested] 13+ messages in thread

* [U-Boot] [PATCH 2/2] OMAP3:SDRC: introduce DDR types
  2009-11-07 17:31         ` Tom
@ 2009-11-07 18:15           ` Paulraj, Sandeep
  2009-11-07 20:31             ` Steve Sakoman
  2009-11-12 16:48           ` Paulraj, Sandeep
  1 sibling, 1 reply; 13+ messages in thread
From: Paulraj, Sandeep @ 2009-11-07 18:15 UTC (permalink / raw)
  To: u-boot


> >>
> >> Introduce Micron DDR timings and provide
> >> CONFIG_OMAP3_INFINEON_DDR and CONFIG_OMAP3_MICRON_DDR config
> >> options to allow for platform files to setup their timings as
> >> per the type of DDR selected
> >>
> >> Reported-by: Steve Sakoman in
> >>
> http://www.nabble.com/forum/Permalink.jtp?root=25779518&post=25959734&page
> >> =y
> >>
> >> Signed-off-by: Nishanth Menon <nm@ti.com>
> >> ---
> >>  include/asm-arm/arch-omap3/mem.h |   88
> ++++++++++++++++++++++++++++++---
> >> -----
> >>  include/configs/devkit8000.h     |    3 +
> >>  include/configs/omap3_beagle.h   |    3 +
> >>  include/configs/omap3_evm.h      |    3 +
> >>  include/configs/omap3_overo.h    |    3 +
> >>  include/configs/omap3_pandora.h  |    3 +
> >>  include/configs/omap3_sdp3430.h  |    3 +
> >>  include/configs/omap3_zoom1.h    |    3 +
> >>  include/configs/omap3_zoom2.h    |    3 +
> >>  9 files changed, 94 insertions(+), 18 deletions(-)
> >>
> >
> > Pushed to u-boot-ti/next
> >
> > I think this series from Nishanth needs some runtime tests.
> > I believe Steve had some issues with his EVM not booting everytime due
> to issues with DDR init.
> >
> > Thanks,
> > Sandeep
> 
> Maybe it should go on the testing branch?
> 
> I will give zoom2 a try..
> 
> Tom
> 
Nishanth has already tried on SDP 3430. 
I think the board that needs to be tried is overo.


Sandeep

^ permalink raw reply	[flat|nested] 13+ messages in thread

* [U-Boot] [PATCH 2/2] OMAP3:SDRC: introduce DDR types
  2009-11-07 18:15           ` Paulraj, Sandeep
@ 2009-11-07 20:31             ` Steve Sakoman
  2009-11-09 13:13               ` Tom
  0 siblings, 1 reply; 13+ messages in thread
From: Steve Sakoman @ 2009-11-07 20:31 UTC (permalink / raw)
  To: u-boot

On Sat, Nov 7, 2009 at 10:15 AM, Paulraj, Sandeep <s-paulraj@ti.com> wrote:
>
>> >>
>> >> Introduce Micron DDR timings and provide
>> >> CONFIG_OMAP3_INFINEON_DDR and CONFIG_OMAP3_MICRON_DDR config
>> >> options to allow for platform files to setup their timings as
>> >> per the type of DDR selected
>> >>
>> >> Reported-by: Steve Sakoman in
>> >>
>> http://www.nabble.com/forum/Permalink.jtp?root=25779518&post=25959734&page
>> >> =y
>> >>
>> >> Signed-off-by: Nishanth Menon <nm@ti.com>
>> >> ---
>> >> ?include/asm-arm/arch-omap3/mem.h | ? 88
>> ++++++++++++++++++++++++++++++---
>> >> -----
>> >> ?include/configs/devkit8000.h ? ? | ? ?3 +
>> >> ?include/configs/omap3_beagle.h ? | ? ?3 +
>> >> ?include/configs/omap3_evm.h ? ? ?| ? ?3 +
>> >> ?include/configs/omap3_overo.h ? ?| ? ?3 +
>> >> ?include/configs/omap3_pandora.h ?| ? ?3 +
>> >> ?include/configs/omap3_sdp3430.h ?| ? ?3 +
>> >> ?include/configs/omap3_zoom1.h ? ?| ? ?3 +
>> >> ?include/configs/omap3_zoom2.h ? ?| ? ?3 +
>> >> ?9 files changed, 94 insertions(+), 18 deletions(-)
>> >>
>> >
>> > Pushed to u-boot-ti/next
>> >
>> > I think this series from Nishanth needs some runtime tests.
>> > I believe Steve had some issues with his EVM not booting everytime due
>> to issues with DDR init.
>> >
>> > Thanks,
>> > Sandeep
>>
>> Maybe it should go on the testing branch?
>>
>> I will give zoom2 a try..
>>
>> Tom
>>
> Nishanth has already tried on SDP 3430.
> I think the board that needs to be tried is overo.

Correct, it was Overo, though I suspect Beagle might have the same
issue since they use the same Micron POP memory.  I'll try to test
this weekend.

Steve

^ permalink raw reply	[flat|nested] 13+ messages in thread

* [U-Boot] [PATCH 2/2] OMAP3:SDRC: introduce DDR types
  2009-11-07 20:31             ` Steve Sakoman
@ 2009-11-09 13:13               ` Tom
  2009-11-09 15:15                 ` Steve Sakoman
  0 siblings, 1 reply; 13+ messages in thread
From: Tom @ 2009-11-09 13:13 UTC (permalink / raw)
  To: u-boot

Steve Sakoman wrote:
> On Sat, Nov 7, 2009 at 10:15 AM, Paulraj, Sandeep <s-paulraj@ti.com> wrote:
>>>>> Introduce Micron DDR timings and provide
>>>>> CONFIG_OMAP3_INFINEON_DDR and CONFIG_OMAP3_MICRON_DDR config
>>>>> options to allow for platform files to setup their timings as
>>>>> per the type of DDR selected
>>>>>
>>>>> Reported-by: Steve Sakoman in
>>>>>
>>> http://www.nabble.com/forum/Permalink.jtp?root=25779518&post=25959734&page
>>>>> =y
>>>>>
>>>>> Signed-off-by: Nishanth Menon <nm@ti.com>
>>>>> ---
>>>>>  include/asm-arm/arch-omap3/mem.h |   88
>>> ++++++++++++++++++++++++++++++---
>>>>> -----
>>>>>  include/configs/devkit8000.h     |    3 +
>>>>>  include/configs/omap3_beagle.h   |    3 +
>>>>>  include/configs/omap3_evm.h      |    3 +
>>>>>  include/configs/omap3_overo.h    |    3 +
>>>>>  include/configs/omap3_pandora.h  |    3 +
>>>>>  include/configs/omap3_sdp3430.h  |    3 +
>>>>>  include/configs/omap3_zoom1.h    |    3 +
>>>>>  include/configs/omap3_zoom2.h    |    3 +
>>>>>  9 files changed, 94 insertions(+), 18 deletions(-)
>>>>>
>>>> Pushed to u-boot-ti/next
>>>>
>>>> I think this series from Nishanth needs some runtime tests.
>>>> I believe Steve had some issues with his EVM not booting everytime due
>>> to issues with DDR init.
>>>> Thanks,
>>>> Sandeep
>>> Maybe it should go on the testing branch?
>>>
>>> I will give zoom2 a try..
>>>
>>> Tom
>>>
>> Nishanth has already tried on SDP 3430.
>> I think the board that needs to be tried is overo.
> 
> Correct, it was Overo, though I suspect Beagle might have the same
> issue since they use the same Micron POP memory.  I'll try to test
> this weekend.
> 
> Steve

I can try that too.

Did you see the problem on warm or cold booting?

Tom

^ permalink raw reply	[flat|nested] 13+ messages in thread

* [U-Boot] [PATCH 2/2] OMAP3:SDRC: introduce DDR types
  2009-11-09 13:13               ` Tom
@ 2009-11-09 15:15                 ` Steve Sakoman
  0 siblings, 0 replies; 13+ messages in thread
From: Steve Sakoman @ 2009-11-09 15:15 UTC (permalink / raw)
  To: u-boot

On Mon, Nov 9, 2009 at 5:13 AM, Tom <Tom.Rix@windriver.com> wrote:
> Steve Sakoman wrote:
>>
>> On Sat, Nov 7, 2009 at 10:15 AM, Paulraj, Sandeep <s-paulraj@ti.com>
>> wrote:
>>>>>>
>>>>>> Introduce Micron DDR timings and provide
>>>>>> CONFIG_OMAP3_INFINEON_DDR and CONFIG_OMAP3_MICRON_DDR config
>>>>>> options to allow for platform files to setup their timings as
>>>>>> per the type of DDR selected
>>>>>>
>>>>>> Reported-by: Steve Sakoman in
>>>>>>
>>>>
>>>> http://www.nabble.com/forum/Permalink.jtp?root=25779518&post=25959734&page
>>>>>>
>>>>>> =y
>>>>>>
>>>>>> Signed-off-by: Nishanth Menon <nm@ti.com>
>>>>>> ---
>>>>>> ?include/asm-arm/arch-omap3/mem.h | ? 88
>>>>
>>>> ++++++++++++++++++++++++++++++---
>>>>>>
>>>>>> -----
>>>>>> ?include/configs/devkit8000.h ? ? | ? ?3 +
>>>>>> ?include/configs/omap3_beagle.h ? | ? ?3 +
>>>>>> ?include/configs/omap3_evm.h ? ? ?| ? ?3 +
>>>>>> ?include/configs/omap3_overo.h ? ?| ? ?3 +
>>>>>> ?include/configs/omap3_pandora.h ?| ? ?3 +
>>>>>> ?include/configs/omap3_sdp3430.h ?| ? ?3 +
>>>>>> ?include/configs/omap3_zoom1.h ? ?| ? ?3 +
>>>>>> ?include/configs/omap3_zoom2.h ? ?| ? ?3 +
>>>>>> ?9 files changed, 94 insertions(+), 18 deletions(-)
>>>>>>
>>>>> Pushed to u-boot-ti/next
>>>>>
>>>>> I think this series from Nishanth needs some runtime tests.
>>>>> I believe Steve had some issues with his EVM not booting everytime due
>>>>
>>>> to issues with DDR init.
>>>>>
>>>>> Thanks,
>>>>> Sandeep
>>>>
>>>> Maybe it should go on the testing branch?
>>>>
>>>> I will give zoom2 a try..
>>>>
>>>> Tom
>>>>
>>> Nishanth has already tried on SDP 3430.
>>> I think the board that needs to be tried is overo.
>>
>> Correct, it was Overo, though I suspect Beagle might have the same
>> issue since they use the same Micron POP memory. ?I'll try to test
>> this weekend.
>>
>> Steve
>
> I can try that too.
>
> Did you see the problem on warm or cold booting?

IIRC, it was an issue with both warm and cold boots.

Steve

^ permalink raw reply	[flat|nested] 13+ messages in thread

* [U-Boot] [PATCH 2/2] OMAP3:SDRC: introduce DDR types
  2009-11-07 17:31         ` Tom
  2009-11-07 18:15           ` Paulraj, Sandeep
@ 2009-11-12 16:48           ` Paulraj, Sandeep
  2009-11-12 17:29             ` Tom
  1 sibling, 1 reply; 13+ messages in thread
From: Paulraj, Sandeep @ 2009-11-12 16:48 UTC (permalink / raw)
  To: u-boot


> >> Introduce Micron DDR timings and provide
> >> CONFIG_OMAP3_INFINEON_DDR and CONFIG_OMAP3_MICRON_DDR config
> >> options to allow for platform files to setup their timings as
> >> per the type of DDR selected
> >>
> >> Reported-by: Steve Sakoman in
> >>
> http://www.nabble.com/forum/Permalink.jtp?root=25779518&post=25959734&page
> >> =y
> >>
> >> Signed-off-by: Nishanth Menon <nm@ti.com>
> >> ---
> >>  include/asm-arm/arch-omap3/mem.h |   88
> ++++++++++++++++++++++++++++++---
> >> -----
> >>  include/configs/devkit8000.h     |    3 +
> >>  include/configs/omap3_beagle.h   |    3 +
> >>  include/configs/omap3_evm.h      |    3 +
> >>  include/configs/omap3_overo.h    |    3 +
> >>  include/configs/omap3_pandora.h  |    3 +
> >>  include/configs/omap3_sdp3430.h  |    3 +
> >>  include/configs/omap3_zoom1.h    |    3 +
> >>  include/configs/omap3_zoom2.h    |    3 +
> >>  9 files changed, 94 insertions(+), 18 deletions(-)
> >>
> >
> > Pushed to u-boot-ti/next
> >
> > I think this series from Nishanth needs some runtime tests.
> > I believe Steve had some issues with his EVM not booting everytime due
> to issues with DDR init.
> >
> > Thanks,
> > Sandeep
> 
> Maybe it should go on the testing branch?
> 
> I will give zoom2 a try..
> 
> Tom
> 

Tom, Dirk, Steve,

Did u get a chance to try?

I am seeing a lot of OMAP3 patches which seem to be touching the above mentioned headers files.

Thanks,
Sandeep

^ permalink raw reply	[flat|nested] 13+ messages in thread

* [U-Boot] [PATCH 2/2] OMAP3:SDRC: introduce DDR types
  2009-11-12 16:48           ` Paulraj, Sandeep
@ 2009-11-12 17:29             ` Tom
  2009-11-12 17:41               ` Menon, Nishanth
  0 siblings, 1 reply; 13+ messages in thread
From: Tom @ 2009-11-12 17:29 UTC (permalink / raw)
  To: u-boot

Paulraj, Sandeep wrote:
>>>> Introduce Micron DDR timings and provide
>>>> CONFIG_OMAP3_INFINEON_DDR and CONFIG_OMAP3_MICRON_DDR config
>>>> options to allow for platform files to setup their timings as
>>>> per the type of DDR selected
>>>>
>>>> Reported-by: Steve Sakoman in
>>>>
>> http://www.nabble.com/forum/Permalink.jtp?root=25779518&post=25959734&page
>>>> =y
>>>>
>>>> Signed-off-by: Nishanth Menon <nm@ti.com>
>>>> ---
>>>>  include/asm-arm/arch-omap3/mem.h |   88
>> ++++++++++++++++++++++++++++++---
>>>> -----
>>>>  include/configs/devkit8000.h     |    3 +
>>>>  include/configs/omap3_beagle.h   |    3 +
>>>>  include/configs/omap3_evm.h      |    3 +
>>>>  include/configs/omap3_overo.h    |    3 +
>>>>  include/configs/omap3_pandora.h  |    3 +
>>>>  include/configs/omap3_sdp3430.h  |    3 +
>>>>  include/configs/omap3_zoom1.h    |    3 +
>>>>  include/configs/omap3_zoom2.h    |    3 +
>>>>  9 files changed, 94 insertions(+), 18 deletions(-)
>>>>
>>> Pushed to u-boot-ti/next
>>>
>>> I think this series from Nishanth needs some runtime tests.
>>> I believe Steve had some issues with his EVM not booting everytime due
>> to issues with DDR init.
>>> Thanks,
>>> Sandeep
>> Maybe it should go on the testing branch?
>>
>> I will give zoom2 a try..
>>
>> Tom
>>
> 
> Tom, Dirk, Steve,
> 
> Did u get a chance to try?
> 
> I am seeing a lot of OMAP3 patches which seem to be touching the above mentioned headers files.
> 
> Thanks,
> Sandeep
> 
> 
> 
For zoom2.
I warm booted the kernel with ti/next.
So it looks ok for zoom2
Tom

^ permalink raw reply	[flat|nested] 13+ messages in thread

* [U-Boot] [PATCH 2/2] OMAP3:SDRC: introduce DDR types
  2009-11-12 17:29             ` Tom
@ 2009-11-12 17:41               ` Menon, Nishanth
  0 siblings, 0 replies; 13+ messages in thread
From: Menon, Nishanth @ 2009-11-12 17:41 UTC (permalink / raw)
  To: u-boot

> From: Tom [mailto:Tom.Rix at windriver.com]
> Sent: Thursday, November 12, 2009 11:30 AM
> 
> Paulraj, Sandeep wrote:
> >>>> Introduce Micron DDR timings and provide
> >>>> CONFIG_OMAP3_INFINEON_DDR and CONFIG_OMAP3_MICRON_DDR config
> >>>> options to allow for platform files to setup their timings as
> >>>> per the type of DDR selected
> >>>>
> >>>> Reported-by: Steve Sakoman in
> >>>>
> >>
> http://www.nabble.com/forum/Permalink.jtp?root=25779518&post=25959734&page
> >>>> =y
> >>>>
> >>>> Signed-off-by: Nishanth Menon <nm@ti.com>
> >>>> ---
> >>>>  include/asm-arm/arch-omap3/mem.h |   88
> >> ++++++++++++++++++++++++++++++---
> >>>> -----
> >>>>  include/configs/devkit8000.h     |    3 +
> >>>>  include/configs/omap3_beagle.h   |    3 +
> >>>>  include/configs/omap3_evm.h      |    3 +
> >>>>  include/configs/omap3_overo.h    |    3 +
> >>>>  include/configs/omap3_pandora.h  |    3 +
> >>>>  include/configs/omap3_sdp3430.h  |    3 +
> >>>>  include/configs/omap3_zoom1.h    |    3 +
> >>>>  include/configs/omap3_zoom2.h    |    3 +
> >>>>  9 files changed, 94 insertions(+), 18 deletions(-)
> >>>>
> >>> Pushed to u-boot-ti/next
> >>>
> >>> I think this series from Nishanth needs some runtime tests.
> >>> I believe Steve had some issues with his EVM not booting everytime due
> >> to issues with DDR init.
> >>> Thanks,
> >>> Sandeep
> >> Maybe it should go on the testing branch?
> >>
> >> I will give zoom2 a try..
> >>
> >> Tom
> >>
> >
> > Tom, Dirk, Steve,
> >
> > Did u get a chance to try?
> >
> > I am seeing a lot of OMAP3 patches which seem to be touching the above
> mentioned headers files.
> >
> For zoom2.
> I warm booted the kernel with ti/next.
> So it looks ok for zoom2
u-boot-ti Ok for SDP3430.

Regards,
Nishanth Menon

^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2009-11-12 17:41 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <OMAP3 DDR Fix patches (was Re: [U-Boot] [PATCH 1/5 v2] OMAP3: Fix SDRC init)>
2009-10-21  2:01 ` [U-Boot] [PATCH 0/2] OMAP3:DDR timing cleanup series Nishanth Menon
2009-10-21  2:01   ` [U-Boot] [PATCH 1/2] OMAP3:SDRC: Cleanup references to SDP Nishanth Menon
2009-10-21  2:01     ` [U-Boot] [PATCH 2/2] OMAP3:SDRC: introduce DDR types Nishanth Menon
2009-11-07 16:11       ` Paulraj, Sandeep
2009-11-07 17:31         ` Tom
2009-11-07 18:15           ` Paulraj, Sandeep
2009-11-07 20:31             ` Steve Sakoman
2009-11-09 13:13               ` Tom
2009-11-09 15:15                 ` Steve Sakoman
2009-11-12 16:48           ` Paulraj, Sandeep
2009-11-12 17:29             ` Tom
2009-11-12 17:41               ` Menon, Nishanth
2009-11-07 16:08     ` [U-Boot] [PATCH 1/2] OMAP3:SDRC: Cleanup references to SDP Paulraj, Sandeep

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.