mx6sabresd: Make SPL DDR configuration to match the DCD table

PATCH New Fabio Estevam 3 participants 10 messages Sep 26, 2016 – Apr 5, 2017

lore

Share & tools

lore

Fabio Estevam

author

2016-09-26 12:14 UTC #0 lore raw

PATCH v3 1/2 mx6sabresd: Make SPL DDR configuration to match the DCD table

From: Fabio Estevam <[email protected]>

When using SPL on i.mx6 we frequently notice some DDR initialization mismatches between the SPL code and the non-SPL code.

This causes stability issues like the ones reported at 7dbda25ecd6d7c ("mx6ul_14x14_evk: Pass refsel and refr fields to avoid hang") and also:
http://lists.denx.de/pipermail/u-boot/2016-September/266355.html .

As the non-SPL code have been tested for long time and proves to be reliable, let's configure the DDR in the exact same way as the non-SPL case.

The idea is simple: just use the DCD table and write directly to the DDR registers.

Retrieved the DCD tables from:
board/freescale/mx6sabresd/mx6q_4x_mt41j128.cfg
and
board/freescale/mx6sabresd/mx6qp.cfg
(NXP U-Boot branch imx_v2015.04_4.1.15_1.0.0_ga)

This method makes it easier for people converting from non-SPL to SPL code.

Other benefit is that the SPL binary size is reduced from 44 kB to 39.9 kB.

Signed-off-by Fabio Estevam <[email protected]>

---
Changes since v1:
- Use the correct mx6qp dcd table
- Create ddr_init() to write the DCD values
- Specify where the DCD tables come from

 board/freescale/mx6sabresd/mx6sabresd.c | 351 ++++++++++++++++++--------------
 1 file changed, 197 insertions(+), 154 deletions(-)

board/freescale/mx6sabresd/mx6sabresd.c +197 ~~-154~~

index f836ecb..3c36395 100644
@@ -682,125 +682,6 @@ int checkboard(void)
 #include <spl.h>
 #include <libfdt.h>
 
-const struct mx6dq_iomux_ddr_regs mx6_ddr_ioregs = {
-	.dram_sdclk_0 =  0x00020030,
-	.dram_sdclk_1 =  0x00020030,
-	.dram_cas =  0x00020030,
-	.dram_ras =  0x00020030,
-	.dram_reset =  0x00020030,
-	.dram_sdcke0 =  0x00003000,
-	.dram_sdcke1 =  0x00003000,
-	.dram_sdba2 =  0x00000000,
-	.dram_sdodt0 =  0x00003030,
-	.dram_sdodt1 =  0x00003030,
-	.dram_sdqs0 =  0x00000030,
-	.dram_sdqs1 =  0x00000030,
-	.dram_sdqs2 =  0x00000030,
-	.dram_sdqs3 =  0x00000030,
-	.dram_sdqs4 =  0x00000030,
-	.dram_sdqs5 =  0x00000030,
-	.dram_sdqs6 =  0x00000030,
-	.dram_sdqs7 =  0x00000030,
-	.dram_dqm0 =  0x00020030,
-	.dram_dqm1 =  0x00020030,
-	.dram_dqm2 =  0x00020030,
-	.dram_dqm3 =  0x00020030,
-	.dram_dqm4 =  0x00020030,
-	.dram_dqm5 =  0x00020030,
-	.dram_dqm6 =  0x00020030,
-	.dram_dqm7 =  0x00020030,
-};
-
-const struct mx6dq_iomux_ddr_regs mx6dqp_ddr_ioregs = {
-	.dram_sdclk_0 =  0x00000030,
-	.dram_sdclk_1 =  0x00000030,
-	.dram_cas =  0x00000030,
-	.dram_ras =  0x00000030,
-	.dram_reset =  0x00000030,
-	.dram_sdcke0 =  0x00003000,
-	.dram_sdcke1 =  0x00003000,
-	.dram_sdba2 =  0x00000000,
-	.dram_sdodt0 =  0x00003030,
-	.dram_sdodt1 =  0x00003030,
-	.dram_sdqs0 =  0x00000030,
-	.dram_sdqs1 =  0x00000030,
-	.dram_sdqs2 =  0x00000030,
-	.dram_sdqs3 =  0x00000030,
-	.dram_sdqs4 =  0x00000030,
-	.dram_sdqs5 =  0x00000030,
-	.dram_sdqs6 =  0x00000030,
-	.dram_sdqs7 =  0x00000030,
-	.dram_dqm0 =  0x00000030,
-	.dram_dqm1 =  0x00000030,
-	.dram_dqm2 =  0x00000030,
-	.dram_dqm3 =  0x00000030,
-	.dram_dqm4 =  0x00000030,
-	.dram_dqm5 =  0x00000030,
-	.dram_dqm6 =  0x00000030,
-	.dram_dqm7 =  0x00000030,
-};
-
-const struct mx6dq_iomux_grp_regs mx6_grp_ioregs = {
-	.grp_ddr_type =  0x000C0000,
-	.grp_ddrmode_ctl =  0x00020000,
-	.grp_ddrpke =  0x00000000,
-	.grp_addds =  0x00000030,
-	.grp_ctlds =  0x00000030,
-	.grp_ddrmode =  0x00020000,
-	.grp_b0ds =  0x00000030,
-	.grp_b1ds =  0x00000030,
-	.grp_b2ds =  0x00000030,
-	.grp_b3ds =  0x00000030,
-	.grp_b4ds =  0x00000030,
-	.grp_b5ds =  0x00000030,
-	.grp_b6ds =  0x00000030,
-	.grp_b7ds =  0x00000030,
-};
-
-const struct mx6_mmdc_calibration mx6_mmcd_calib = {
-	.p0_mpwldectrl0 =  0x001F001F,
-	.p0_mpwldectrl1 =  0x001F001F,
-	.p1_mpwldectrl0 =  0x00440044,
-	.p1_mpwldectrl1 =  0x00440044,
-	.p0_mpdgctrl0 =  0x434B0350,
-	.p0_mpdgctrl1 =  0x034C0359,
-	.p1_mpdgctrl0 =  0x434B0350,
-	.p1_mpdgctrl1 =  0x03650348,
-	.p0_mprddlctl =  0x4436383B,
-	.p1_mprddlctl =  0x39393341,
-	.p0_mpwrdlctl =  0x35373933,
-	.p1_mpwrdlctl =  0x48254A36,
-};
-
-const struct mx6_mmdc_calibration mx6dqp_mmcd_calib = {
-	.p0_mpwldectrl0 =  0x001B001E,
-	.p0_mpwldectrl1 =  0x002E0029,
-	.p1_mpwldectrl0 =  0x001B002A,
-	.p1_mpwldectrl1 =  0x0019002C,
-	.p0_mpdgctrl0 =  0x43240334,
-	.p0_mpdgctrl1 =  0x0324031A,
-	.p1_mpdgctrl0 =  0x43340344,
-	.p1_mpdgctrl1 =  0x03280276,
-	.p0_mprddlctl =  0x44383A3E,
-	.p1_mprddlctl =  0x3C3C3846,
-	.p0_mpwrdlctl =  0x2E303230,
-	.p1_mpwrdlctl =  0x38283E34,
-};
-
-/* MT41K128M16JT-125 */
-static struct mx6_ddr3_cfg mem_ddr = {
-	.mem_speed = 1600,
-	.density = 2,
-	.width = 16,
-	.banks = 8,
-	.rowaddr = 14,
-	.coladdr = 10,
-	.pagesz = 2,
-	.trcd = 1375,
-	.trcmin = 4875,
-	.trasmin = 3500,
-};
-
 static void ccgr_init(void)
 {
 	struct mxc_ccm_reg *ccm = (struct mxc_ccm_reg *)CCM_BASE_ADDR;

@@ -831,44 +712,209 @@ static void gpr_init(void)
 	}
 }
 
-/*
- * This section requires the differentiation between iMX6 Sabre boards, but
- * for now, it will configure only for the mx6q variant.
- */
-static void spl_dram_init(void)
+static int mx6q_dcd_table[] = {
+	0x020e0798, 0x000C0000,
+	0x020e0758, 0x00000000,
+	0x020e0588, 0x00000030,
+	0x020e0594, 0x00000030,
+	0x020e056c, 0x00000030,
+	0x020e0578, 0x00000030,
+	0x020e074c, 0x00000030,
+	0x020e057c, 0x00000030,
+	0x020e058c, 0x00000000,
+	0x020e059c, 0x00000030,
+	0x020e05a0, 0x00000030,
+	0x020e078c, 0x00000030,
+	0x020e0750, 0x00020000,
+	0x020e05a8, 0x00000030,
+	0x020e05b0, 0x00000030,
+	0x020e0524, 0x00000030,
+	0x020e051c, 0x00000030,
+	0x020e0518, 0x00000030,
+	0x020e050c, 0x00000030,
+	0x020e05b8, 0x00000030,
+	0x020e05c0, 0x00000030,
+	0x020e0774, 0x00020000,
+	0x020e0784, 0x00000030,
+	0x020e0788, 0x00000030,
+	0x020e0794, 0x00000030,
+	0x020e079c, 0x00000030,
+	0x020e07a0, 0x00000030,
+	0x020e07a4, 0x00000030,
+	0x020e07a8, 0x00000030,
+	0x020e0748, 0x00000030,
+	0x020e05ac, 0x00000030,
+	0x020e05b4, 0x00000030,
+	0x020e0528, 0x00000030,
+	0x020e0520, 0x00000030,
+	0x020e0514, 0x00000030,
+	0x020e0510, 0x00000030,
+	0x020e05bc, 0x00000030,
+	0x020e05c4, 0x00000030,
+	0x021b0800, 0xa1390003,
+	0x021b080c, 0x001F001F,
+	0x021b0810, 0x001F001F,
+	0x021b480c, 0x001F001F,
+	0x021b4810, 0x001F001F,
+	0x021b083c, 0x43270338,
+	0x021b0840, 0x03200314,
+	0x021b483c, 0x431A032F,
+	0x021b4840, 0x03200263,
+	0x021b0848, 0x4B434748,
+	0x021b4848, 0x4445404C,
+	0x021b0850, 0x38444542,
+	0x021b4850, 0x4935493A,
+	0x021b081c, 0x33333333,
+	0x021b0820, 0x33333333,
+	0x021b0824, 0x33333333,
+	0x021b0828, 0x33333333,
+	0x021b481c, 0x33333333,
+	0x021b4820, 0x33333333,
+	0x021b4824, 0x33333333,
+	0x021b4828, 0x33333333,
+	0x021b08b8, 0x00000800,
+	0x021b48b8, 0x00000800,
+	0x021b0004, 0x00020036,
+	0x021b0008, 0x09444040,
+	0x021b000c, 0x555A7975,
+	0x021b0010, 0xFF538F64,
+	0x021b0014, 0x01FF00DB,
+	0x021b0018, 0x00001740,
+	0x021b001c, 0x00008000,
+	0x021b002c, 0x000026d2,
+	0x021b0030, 0x005A1023,
+	0x021b0040, 0x00000027,
+	0x021b0000, 0x831A0000,
+	0x021b001c, 0x04088032,
+	0x021b001c, 0x00008033,
+	0x021b001c, 0x00048031,
+	0x021b001c, 0x09408030,
+	0x021b001c, 0x04008040,
+	0x021b0020, 0x00005800,
+	0x021b0818, 0x00011117,
+	0x021b4818, 0x00011117,
+	0x021b0004, 0x00025576,
+	0x021b0404, 0x00011006,
+	0x021b001c, 0x00000000,
+};
+
+static int mx6qp_dcd_table[] = {
+	0x020e0798, 0x000c0000,
+	0x020e0758, 0x00000000,
+	0x020e0588, 0x00000030,
+	0x020e0594, 0x00000030,
+	0x020e056c, 0x00000030,
+	0x020e0578, 0x00000030,
+	0x020e074c, 0x00000030,
+	0x020e057c, 0x00000030,
+	0x020e058c, 0x00000000,
+	0x020e059c, 0x00000030,
+	0x020e05a0, 0x00000030,
+	0x020e078c, 0x00000030,
+	0x020e0750, 0x00020000,
+	0x020e05a8, 0x00000030,
+	0x020e05b0, 0x00000030,
+	0x020e0524, 0x00000030,
+	0x020e051c, 0x00000030,
+	0x020e0518, 0x00000030,
+	0x020e050c, 0x00000030,
+	0x020e05b8, 0x00000030,
+	0x020e05c0, 0x00000030,
+	0x020e0774, 0x00020000,
+	0x020e0784, 0x00000030,
+	0x020e0788, 0x00000030,
+	0x020e0794, 0x00000030,
+	0x020e079c, 0x00000030,
+	0x020e07a0, 0x00000030,
+	0x020e07a4, 0x00000030,
+	0x020e07a8, 0x00000030,
+	0x020e0748, 0x00000030,
+	0x020e05ac, 0x00000030,
+	0x020e05b4, 0x00000030,
+	0x020e0528, 0x00000030,
+	0x020e0520, 0x00000030,
+	0x020e0514, 0x00000030,
+	0x020e0510, 0x00000030,
+	0x020e05bc, 0x00000030,
+	0x020e05c4, 0x00000030,
+	0x021b0800, 0xa1390003,
+	0x021b080c, 0x001b001e,
+	0x021b0810, 0x002e0029,
+	0x021b480c, 0x001b002a,
+	0x021b4810, 0x0019002c,
+	0x021b083c, 0x43240334,
+	0x021b0840, 0x0324031a,
+	0x021b483c, 0x43340344,
+	0x021b4840, 0x03280276,
+	0x021b0848, 0x44383A3E,
+	0x021b4848, 0x3C3C3846,
+	0x021b0850, 0x2e303230,
+	0x021b4850, 0x38283E34,
+	0x021b081c, 0x33333333,
+	0x021b0820, 0x33333333,
+	0x021b0824, 0x33333333,
+	0x021b0828, 0x33333333,
+	0x021b481c, 0x33333333,
+	0x021b4820, 0x33333333,
+	0x021b4824, 0x33333333,
+	0x021b4828, 0x33333333,
+	0x021b08c0, 0x24912249,
+	0x021b48c0, 0x24914289,
+	0x021b08b8, 0x00000800,
+	0x021b48b8, 0x00000800,
+	0x021b0004, 0x00020036,
+	0x021b0008, 0x24444040,
+	0x021b000c, 0x555A7955,
+	0x021b0010, 0xFF320F64,
+	0x021b0014, 0x01ff00db,
+	0x021b0018, 0x00001740,
+	0x021b001c, 0x00008000,
+	0x021b002c, 0x000026d2,
+	0x021b0030, 0x005A1023,
+	0x021b0040, 0x00000027,
+	0x021b0400, 0x14420000,
+	0x021b0000, 0x831A0000,
+	0x021b0890, 0x00400C58,
+	0x00bb0008, 0x00000000,
+	0x00bb000c, 0x2891E41A,
+	0x00bb0038, 0x00000564,
+	0x00bb0014, 0x00000040,
+	0x00bb0028, 0x00000020,
+	0x00bb002c, 0x00000020,
+	0x021b001c, 0x04088032,
+	0x021b001c, 0x00008033,
+	0x021b001c, 0x00048031,
+	0x021b001c, 0x09408030,
+	0x021b001c, 0x04008040,
+	0x021b0020, 0x00005800,
+	0x021b0818, 0x00011117,
+	0x021b4818, 0x00011117,
+	0x021b0004, 0x00025576,
+	0x021b0404, 0x00011006,
+	0x021b001c, 0x00000000,
+};
+
+static void ddr_init(int *table, int size)
 {
-	struct mx6_ddr_sysinfo sysinfo = {
-		/* width of data bus:0=16,1=32,2=64 */
-		.dsize = 2,
-		/* config for full 4GB range so that get_mem_size() works */
-		.cs_density = 32, /* 32Gb per CS */
-		/* single chip select */
-		.ncs = 1,
-		.cs1_mirror = 0,
-		.rtt_wr = 1 /*DDR3_RTT_60_OHM*/,	/* RTT_Wr = RZQ/4 */
-		.rtt_nom = 1 /*DDR3_RTT_60_OHM*/,	/* RTT_Nom = RZQ/4 */
-		.walat = 1,	/* Write additional latency */
-		.ralat = 5,	/* Read additional latency */
-		.mif3_mode = 3,	/* Command prediction working mode */
-		.bi_on = 1,	/* Bank interleaving enabled */
-		.sde_to_rst = 0x10,	/* 14 cycles, 200us (JEDEC default) */
-		.rst_to_cke = 0x23,	/* 33 cycles, 500us (JEDEC default) */
-		.ddr_type = DDR_TYPE_DDR3,
-		.refsel = 1,	/* Refresh cycles at 32KHz */
-		.refr = 7,	/* 8 refresh commands per refresh cycle */
-	};
+	int i;
 
-	if (is_mx6dqp()) {
-		mx6dq_dram_iocfg(64, &mx6dqp_ddr_ioregs, &mx6_grp_ioregs);
-		mx6_dram_cfg(&sysinfo, &mx6dqp_mmcd_calib, &mem_ddr);
-	} else {
-		mx6dq_dram_iocfg(64, &mx6_ddr_ioregs, &mx6_grp_ioregs);
-		mx6_dram_cfg(&sysinfo, &mx6_mmcd_calib, &mem_ddr);
-	}
+	for (i = 0; i < size / 2 ; i++)
+		writel(table[2 * i + 1], table[2 * i]);
+}
+
+static void spl_dram_init(void)
+{
+	if (is_mx6dq())
+		ddr_init(mx6q_dcd_table, ARRAY_SIZE(mx6q_dcd_table));
+	else if (is_mx6dqp())
+		ddr_init(mx6qp_dcd_table, ARRAY_SIZE(mx6qp_dcd_table));
 }
 
 void board_init_f(ulong dummy)
 {
+	/* DDR initialization */
+	spl_dram_init();
+
 	/* setup AIPS and disable watchdog */
 	arch_cpu_init();

@@ -884,9 +930,6 @@ void board_init_f(ulong dummy)
 	/* UART clocks enabled and gd valid - init serial console */
 	preloader_console_init();
 
-	/* DDR initialization */
-	spl_dram_init();
-
 	/* Clear the BSS. */
 	memset(__bss_start, 0, __bss_end - __bss_start);

Signature · 1 line

2.7.4

Fabio Estevam

author

2016-09-26 12:14 UTC #1 lore raw

PATCH v3 2/2 mx6sabresd: Add SPL support for the mx6dl variant

From: Fabio Estevam <[email protected]>

Add support for the mx6dlsabresd board in SPL.

Retrieved the DCD table from:
board/freescale/mx6sabresd/mx6dlsabresd.cfg
(NXP U-Boot branch imx_v2015.04_4.1.15_1.0.0_ga)

Signed-off-by Fabio Estevam <[email protected]>

---
Changes since v2:
- Newly introduced in this series

 board/freescale/mx6sabresd/mx6sabresd.c | 88 +++++++++++++++++++++++++++++++++
 1 file changed, 88 insertions(+)

board/freescale/mx6sabresd/mx6sabresd.c +88 -0

index 3c36395..986a82b 100644
@@ -894,6 +894,92 @@ static int mx6qp_dcd_table[] = {
 	0x021b001c, 0x00000000,
 };
 
+static int mx6dl_dcd_table[] = {
+	0x020e0774, 0x000C0000,
+	0x020e0754, 0x00000000,
+	0x020e04ac, 0x00000030,
+	0x020e04b0, 0x00000030,
+	0x020e0464, 0x00000030,
+	0x020e0490, 0x00000030,
+	0x020e074c, 0x00000030,
+	0x020e0494, 0x00000030,
+	0x020e04a0, 0x00000000,
+	0x020e04b4, 0x00000030,
+	0x020e04b8, 0x00000030,
+	0x020e076c, 0x00000030,
+	0x020e0750, 0x00020000,
+	0x020e04bc, 0x00000030,
+	0x020e04c0, 0x00000030,
+	0x020e04c4, 0x00000030,
+	0x020e04c8, 0x00000030,
+	0x020e04cc, 0x00000030,
+	0x020e04d0, 0x00000030,
+	0x020e04d4, 0x00000030,
+	0x020e04d8, 0x00000030,
+	0x020e0760, 0x00020000,
+	0x020e0764, 0x00000030,
+	0x020e0770, 0x00000030,
+	0x020e0778, 0x00000030,
+	0x020e077c, 0x00000030,
+	0x020e0780, 0x00000030,
+	0x020e0784, 0x00000030,
+	0x020e078c, 0x00000030,
+	0x020e0748, 0x00000030,
+	0x020e0470, 0x00000030,
+	0x020e0474, 0x00000030,
+	0x020e0478, 0x00000030,
+	0x020e047c, 0x00000030,
+	0x020e0480, 0x00000030,
+	0x020e0484, 0x00000030,
+	0x020e0488, 0x00000030,
+	0x020e048c, 0x00000030,
+	0x021b0800, 0xa1390003,
+	0x021b080c, 0x001F001F,
+	0x021b0810, 0x001F001F,
+	0x021b480c, 0x001F001F,
+	0x021b4810, 0x001F001F,
+	0x021b083c, 0x4220021F,
+	0x021b0840, 0x0207017E,
+	0x021b483c, 0x4201020C,
+	0x021b4840, 0x01660172,
+	0x021b0848, 0x4A4D4E4D,
+	0x021b4848, 0x4A4F5049,
+	0x021b0850, 0x3F3C3D31,
+	0x021b4850, 0x3238372B,
+	0x021b081c, 0x33333333,
+	0x021b0820, 0x33333333,
+	0x021b0824, 0x33333333,
+	0x021b0828, 0x33333333,
+	0x021b481c, 0x33333333,
+	0x021b4820, 0x33333333,
+	0x021b4824, 0x33333333,
+	0x021b4828, 0x33333333,
+	0x021b08b8, 0x00000800,
+	0x021b48b8, 0x00000800,
+	0x021b0004, 0x0002002D,
+	0x021b0008, 0x00333030,
+	0x021b000c, 0x3F435313,
+	0x021b0010, 0xB66E8B63,
+	0x021b0014, 0x01FF00DB,
+	0x021b0018, 0x00001740,
+	0x021b001c, 0x00008000,
+	0x021b002c, 0x000026d2,
+	0x021b0030, 0x00431023,
+	0x021b0040, 0x00000027,
+	0x021b0000, 0x831A0000,
+	0x021b001c, 0x04008032,
+	0x021b001c, 0x00008033,
+	0x021b001c, 0x00048031,
+	0x021b001c, 0x05208030,
+	0x021b001c, 0x04008040,
+	0x021b0020, 0x00005800,
+	0x021b0818, 0x00011117,
+	0x021b4818, 0x00011117,
+	0x021b0004, 0x0002556D,
+	0x021b0404, 0x00011006,
+	0x021b001c, 0x00000000,
+
+};
 static void ddr_init(int *table, int size)
 {
 	int i;

@@ -908,6 +994,8 @@ static void spl_dram_init(void)
 		ddr_init(mx6q_dcd_table, ARRAY_SIZE(mx6q_dcd_table));
 	else if (is_mx6dqp())
 		ddr_init(mx6qp_dcd_table, ARRAY_SIZE(mx6qp_dcd_table));
+	else if (is_mx6sdl())
+		ddr_init(mx6dl_dcd_table, ARRAY_SIZE(mx6dl_dcd_table));
 }
 
 void board_init_f(ulong dummy)

Signature · 1 line

2.7.4

Fabio Estevam

author

2016-09-28 18:56 UTC #2 lore raw

PATCH v3 2/2 mx6sabresd: Add SPL support for the mx6dl variant

Hi Stefano,

Quoted 9 lines · Fabio Estevam

From: Fabio Estevam <[email protected]>
Add support for the mx6dlsabresd board in SPL.
Retrieved the DCD table from:
board/freescale/mx6sabresd/mx6dlsabresd.cfg
(NXP U-Boot branch imx_v2015.04_4.1.15_1.0.0_ga)
Signed-off-by Fabio Estevam <[email protected]>

Patch 1/2 works fine, but I saw an issue with this one and will rework it. Please do not apply 2/2.

Thanks

Stefano Babic

2016-10-04 08:36 UTC #3 lore raw

PATCH v3 2/2 mx6sabresd: Add SPL support for the mx6dl variant

Hi Fabio,

Quoted 7 lines · Fabio Estevam

Hi Stefano,
Quoted 4 lines · Fabio Estevam
From: Fabio Estevam <[email protected]>
Add support for the mx6dlsabresd board in SPL.

Quoted 9 lines

Quoted 5 lines
Retrieved the DCD table from:
board/freescale/mx6sabresd/mx6dlsabresd.cfg
(NXP U-Boot branch imx_v2015.04_4.1.15_1.0.0_ga)
Signed-off-by Fabio Estevam <[email protected]>
Patch 1/2 works fine, but I saw an issue with this one and will rework it. Please do not apply 2/2.

Ok - I merged 1/2, and drop 2/2.

Regards,
Stefano

Signature · 5 lines

=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

Jagan Teki

2017-03-29 15:15 UTC #4 lore raw

PATCH v3 1/2 mx6sabresd: Make SPL DDR configuration to match the DCD table

In reply to Fabio Estevam

Hi Fabio,

Quoted 249 lines · Fabio Estevam

From: Fabio Estevam <[email protected]>

When using SPL on i.mx6 we frequently notice some DDR initialization mismatches between the SPL code and the non-SPL code.

As the non-SPL code have been tested for long time and proves to be reliable, let's configure the DDR in the exact same way as the non-SPL case.

The idea is simple: just use the DCD table and write directly to the DDR registers.

Retrieved the DCD tables from:
board/freescale/mx6sabresd/mx6q_4x_mt41j128.cfg
and
board/freescale/mx6sabresd/mx6qp.cfg
(NXP U-Boot branch imx_v2015.04_4.1.15_1.0.0_ga)

This method makes it easier for people converting from non-SPL to SPL code.

Other benefit is that the SPL binary size is reduced from 44 kB to 39.9 kB.

Signed-off-by Fabio Estevam <[email protected]>

---
Changes since v1:
- Use the correct mx6qp dcd table
- Create ddr_init() to write the DCD values
- Specify where the DCD tables come from

 board/freescale/mx6sabresd/mx6sabresd.c | 351 ++++++++++++++++++--------------
 1 file changed, 197 insertions(+), 154 deletions(-)

board/freescale/mx6sabresd/mx6sabresd.c +75 ~~-124~~

index f836ecb..3c36395 100644
@@ -682,125 +682,6 @@ int checkboard(void)
 #include <spl.h>
 #include <libfdt.h>
 
-const struct mx6dq_iomux_ddr_regs mx6_ddr_ioregs = {
-       .dram_sdclk_0 =  0x00020030,
-       .dram_sdclk_1 =  0x00020030,
-       .dram_cas =  0x00020030,
-       .dram_ras =  0x00020030,
-       .dram_reset =  0x00020030,
-       .dram_sdcke0 =  0x00003000,
-       .dram_sdcke1 =  0x00003000,
-       .dram_sdba2 =  0x00000000,
-       .dram_sdodt0 =  0x00003030,
-       .dram_sdodt1 =  0x00003030,
-       .dram_sdqs0 =  0x00000030,
-       .dram_sdqs1 =  0x00000030,
-       .dram_sdqs2 =  0x00000030,
-       .dram_sdqs3 =  0x00000030,
-       .dram_sdqs4 =  0x00000030,
-       .dram_sdqs5 =  0x00000030,
-       .dram_sdqs6 =  0x00000030,
-       .dram_sdqs7 =  0x00000030,
-       .dram_dqm0 =  0x00020030,
-       .dram_dqm1 =  0x00020030,
-       .dram_dqm2 =  0x00020030,
-       .dram_dqm3 =  0x00020030,
-       .dram_dqm4 =  0x00020030,
-       .dram_dqm5 =  0x00020030,
-       .dram_dqm6 =  0x00020030,
-       .dram_dqm7 =  0x00020030,
-};
-
-const struct mx6dq_iomux_ddr_regs mx6dqp_ddr_ioregs = {
-       .dram_sdclk_0 =  0x00000030,
-       .dram_sdclk_1 =  0x00000030,
-       .dram_cas =  0x00000030,
-       .dram_ras =  0x00000030,
-       .dram_reset =  0x00000030,
-       .dram_sdcke0 =  0x00003000,
-       .dram_sdcke1 =  0x00003000,
-       .dram_sdba2 =  0x00000000,
-       .dram_sdodt0 =  0x00003030,
-       .dram_sdodt1 =  0x00003030,
-       .dram_sdqs0 =  0x00000030,
-       .dram_sdqs1 =  0x00000030,
-       .dram_sdqs2 =  0x00000030,
-       .dram_sdqs3 =  0x00000030,
-       .dram_sdqs4 =  0x00000030,
-       .dram_sdqs5 =  0x00000030,
-       .dram_sdqs6 =  0x00000030,
-       .dram_sdqs7 =  0x00000030,
-       .dram_dqm0 =  0x00000030,
-       .dram_dqm1 =  0x00000030,
-       .dram_dqm2 =  0x00000030,
-       .dram_dqm3 =  0x00000030,
-       .dram_dqm4 =  0x00000030,
-       .dram_dqm5 =  0x00000030,
-       .dram_dqm6 =  0x00000030,
-       .dram_dqm7 =  0x00000030,
-};
-
-const struct mx6dq_iomux_grp_regs mx6_grp_ioregs = {
-       .grp_ddr_type =  0x000C0000,
-       .grp_ddrmode_ctl =  0x00020000,
-       .grp_ddrpke =  0x00000000,
-       .grp_addds =  0x00000030,
-       .grp_ctlds =  0x00000030,
-       .grp_ddrmode =  0x00020000,
-       .grp_b0ds =  0x00000030,
-       .grp_b1ds =  0x00000030,
-       .grp_b2ds =  0x00000030,
-       .grp_b3ds =  0x00000030,
-       .grp_b4ds =  0x00000030,
-       .grp_b5ds =  0x00000030,
-       .grp_b6ds =  0x00000030,
-       .grp_b7ds =  0x00000030,
-};
-
-const struct mx6_mmdc_calibration mx6_mmcd_calib = {
-       .p0_mpwldectrl0 =  0x001F001F,
-       .p0_mpwldectrl1 =  0x001F001F,
-       .p1_mpwldectrl0 =  0x00440044,
-       .p1_mpwldectrl1 =  0x00440044,
-       .p0_mpdgctrl0 =  0x434B0350,
-       .p0_mpdgctrl1 =  0x034C0359,
-       .p1_mpdgctrl0 =  0x434B0350,
-       .p1_mpdgctrl1 =  0x03650348,
-       .p0_mprddlctl =  0x4436383B,
-       .p1_mprddlctl =  0x39393341,
-       .p0_mpwrdlctl =  0x35373933,
-       .p1_mpwrdlctl =  0x48254A36,
-};
-
-const struct mx6_mmdc_calibration mx6dqp_mmcd_calib = {
-       .p0_mpwldectrl0 =  0x001B001E,
-       .p0_mpwldectrl1 =  0x002E0029,
-       .p1_mpwldectrl0 =  0x001B002A,
-       .p1_mpwldectrl1 =  0x0019002C,
-       .p0_mpdgctrl0 =  0x43240334,
-       .p0_mpdgctrl1 =  0x0324031A,
-       .p1_mpdgctrl0 =  0x43340344,
-       .p1_mpdgctrl1 =  0x03280276,
-       .p0_mprddlctl =  0x44383A3E,
-       .p1_mprddlctl =  0x3C3C3846,
-       .p0_mpwrdlctl =  0x2E303230,
-       .p1_mpwrdlctl =  0x38283E34,
-};
-
-/* MT41K128M16JT-125 */
-static struct mx6_ddr3_cfg mem_ddr = {
-       .mem_speed = 1600,
-       .density = 2,
-       .width = 16,
-       .banks = 8,
-       .rowaddr = 14,
-       .coladdr = 10,
-       .pagesz = 2,
-       .trcd = 1375,
-       .trcmin = 4875,
-       .trasmin = 3500,
-};
-
 static void ccgr_init(void)
 {
        struct mxc_ccm_reg *ccm = (struct mxc_ccm_reg *)CCM_BASE_ADDR;

@@ -831,44 +712,209 @@ static void gpr_init(void)
        }
 }
 
-/*
- * This section requires the differentiation between iMX6 Sabre boards, but
- * for now, it will configure only for the mx6q variant.
- */
-static void spl_dram_init(void)
+static int mx6q_dcd_table[] = {
+       0x020e0798, 0x000C0000,
+       0x020e0758, 0x00000000,
+       0x020e0588, 0x00000030,
+       0x020e0594, 0x00000030,
+       0x020e056c, 0x00000030,
+       0x020e0578, 0x00000030,
+       0x020e074c, 0x00000030,
+       0x020e057c, 0x00000030,
+       0x020e058c, 0x00000000,
+       0x020e059c, 0x00000030,
+       0x020e05a0, 0x00000030,
+       0x020e078c, 0x00000030,
+       0x020e0750, 0x00020000,
+       0x020e05a8, 0x00000030,
+       0x020e05b0, 0x00000030,
+       0x020e0524, 0x00000030,
+       0x020e051c, 0x00000030,
+       0x020e0518, 0x00000030,
+       0x020e050c, 0x00000030,
+       0x020e05b8, 0x00000030,
+       0x020e05c0, 0x00000030,
+       0x020e0774, 0x00020000,
+       0x020e0784, 0x00000030,
+       0x020e0788, 0x00000030,
+       0x020e0794, 0x00000030,
+       0x020e079c, 0x00000030,
+       0x020e07a0, 0x00000030,
+       0x020e07a4, 0x00000030,
+       0x020e07a8, 0x00000030,
+       0x020e0748, 0x00000030,
+       0x020e05ac, 0x00000030,
+       0x020e05b4, 0x00000030,
+       0x020e0528, 0x00000030,
+       0x020e0520, 0x00000030,
+       0x020e0514, 0x00000030,
+       0x020e0510, 0x00000030,
+       0x020e05bc, 0x00000030,
+       0x020e05c4, 0x00000030,
+       0x021b0800, 0xa1390003,
+       0x021b080c, 0x001F001F,
+       0x021b0810, 0x001F001F,
+       0x021b480c, 0x001F001F,
+       0x021b4810, 0x001F001F,
+       0x021b083c, 0x43270338,
+       0x021b0840, 0x03200314,
+       0x021b483c, 0x431A032F,
+       0x021b4840, 0x03200263,
+       0x021b0848, 0x4B434748,
+       0x021b4848, 0x4445404C,
+       0x021b0850, 0x38444542,
+       0x021b4850, 0x4935493A,
+       0x021b081c, 0x33333333,
+       0x021b0820, 0x33333333,
+       0x021b0824, 0x33333333,
+       0x021b0828, 0x33333333,
+       0x021b481c, 0x33333333,
+       0x021b4820, 0x33333333,
+       0x021b4824, 0x33333333,
+       0x021b4828, 0x33333333,
+       0x021b08b8, 0x00000800,
+       0x021b48b8, 0x00000800,
+       0x021b0004, 0x00020036,
+       0x021b0008, 0x09444040,
+       0x021b000c, 0x555A7975,
+       0x021b0010, 0xFF538F64,
+       0x021b0014, 0x01FF00DB,
+       0x021b0018, 0x00001740,
+       0x021b001c, 0x00008000,
+       0x021b002c, 0x000026d2,
+       0x021b0030, 0x005A1023,
+       0x021b0040, 0x00000027,
+       0x021b0000, 0x831A0000,
+       0x021b001c, 0x04088032,
+       0x021b001c, 0x00008033,

board/freescale/mx6sabresd/mx6sabresd.c static void gpr_init(void) · line 799

Quoted 10 lines

board/freescale/mx6sabresd/mx6sabresd.c +10 -0

+       0x021b001c, 0x00048031,
+       0x021b001c, 0x09408030,
+       0x021b001c, 0x04008040,
+       0x021b0020, 0x00005800,
+       0x021b0818, 0x00011117,
+       0x021b4818, 0x00011117,
+       0x021b0004, 0x00025576,
+       0x021b0404, 0x00011006,
+       0x021b001c, 0x00000000,
+};

I believe, this is a random reg-pick from mx6q_4x_mt41j128.cfg. Do we need to follow any specific order to assign these dcd reg-init?

thanks!

Signature · 4 lines

Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.

Jagan Teki

2017-04-04 15:47 UTC #5 lore raw

PATCH v3 1/2 mx6sabresd: Make SPL DDR configuration to match the DCD table

Any help?

Quoted 272 lines · Jagan Teki

Hi Fabio,

Quoted 249 lines · Fabio Estevam

From: Fabio Estevam <[email protected]>

When using SPL on i.mx6 we frequently notice some DDR initialization mismatches between the SPL code and the non-SPL code.

As the non-SPL code have been tested for long time and proves to be reliable, let's configure the DDR in the exact same way as the non-SPL case.

The idea is simple: just use the DCD table and write directly to the DDR registers.

Retrieved the DCD tables from:
board/freescale/mx6sabresd/mx6q_4x_mt41j128.cfg
and
board/freescale/mx6sabresd/mx6qp.cfg
(NXP U-Boot branch imx_v2015.04_4.1.15_1.0.0_ga)

This method makes it easier for people converting from non-SPL to SPL code.

Other benefit is that the SPL binary size is reduced from 44 kB to 39.9 kB.

Signed-off-by Fabio Estevam <[email protected]>

---
Changes since v1:
- Use the correct mx6qp dcd table
- Create ddr_init() to write the DCD values
- Specify where the DCD tables come from

 board/freescale/mx6sabresd/mx6sabresd.c | 351 ++++++++++++++++++--------------
 1 file changed, 197 insertions(+), 154 deletions(-)

board/freescale/mx6sabresd/mx6sabresd.c +75 ~~-124~~

index f836ecb..3c36395 100644
@@ -682,125 +682,6 @@ int checkboard(void)
 #include <spl.h>
 #include <libfdt.h>
 
-const struct mx6dq_iomux_ddr_regs mx6_ddr_ioregs = {
-       .dram_sdclk_0 =  0x00020030,
-       .dram_sdclk_1 =  0x00020030,
-       .dram_cas =  0x00020030,
-       .dram_ras =  0x00020030,
-       .dram_reset =  0x00020030,
-       .dram_sdcke0 =  0x00003000,
-       .dram_sdcke1 =  0x00003000,
-       .dram_sdba2 =  0x00000000,
-       .dram_sdodt0 =  0x00003030,
-       .dram_sdodt1 =  0x00003030,
-       .dram_sdqs0 =  0x00000030,
-       .dram_sdqs1 =  0x00000030,
-       .dram_sdqs2 =  0x00000030,
-       .dram_sdqs3 =  0x00000030,
-       .dram_sdqs4 =  0x00000030,
-       .dram_sdqs5 =  0x00000030,
-       .dram_sdqs6 =  0x00000030,
-       .dram_sdqs7 =  0x00000030,
-       .dram_dqm0 =  0x00020030,
-       .dram_dqm1 =  0x00020030,
-       .dram_dqm2 =  0x00020030,
-       .dram_dqm3 =  0x00020030,
-       .dram_dqm4 =  0x00020030,
-       .dram_dqm5 =  0x00020030,
-       .dram_dqm6 =  0x00020030,
-       .dram_dqm7 =  0x00020030,
-};
-
-const struct mx6dq_iomux_ddr_regs mx6dqp_ddr_ioregs = {
-       .dram_sdclk_0 =  0x00000030,
-       .dram_sdclk_1 =  0x00000030,
-       .dram_cas =  0x00000030,
-       .dram_ras =  0x00000030,
-       .dram_reset =  0x00000030,
-       .dram_sdcke0 =  0x00003000,
-       .dram_sdcke1 =  0x00003000,
-       .dram_sdba2 =  0x00000000,
-       .dram_sdodt0 =  0x00003030,
-       .dram_sdodt1 =  0x00003030,
-       .dram_sdqs0 =  0x00000030,
-       .dram_sdqs1 =  0x00000030,
-       .dram_sdqs2 =  0x00000030,
-       .dram_sdqs3 =  0x00000030,
-       .dram_sdqs4 =  0x00000030,
-       .dram_sdqs5 =  0x00000030,
-       .dram_sdqs6 =  0x00000030,
-       .dram_sdqs7 =  0x00000030,
-       .dram_dqm0 =  0x00000030,
-       .dram_dqm1 =  0x00000030,
-       .dram_dqm2 =  0x00000030,
-       .dram_dqm3 =  0x00000030,
-       .dram_dqm4 =  0x00000030,
-       .dram_dqm5 =  0x00000030,
-       .dram_dqm6 =  0x00000030,
-       .dram_dqm7 =  0x00000030,
-};
-
-const struct mx6dq_iomux_grp_regs mx6_grp_ioregs = {
-       .grp_ddr_type =  0x000C0000,
-       .grp_ddrmode_ctl =  0x00020000,
-       .grp_ddrpke =  0x00000000,
-       .grp_addds =  0x00000030,
-       .grp_ctlds =  0x00000030,
-       .grp_ddrmode =  0x00020000,
-       .grp_b0ds =  0x00000030,
-       .grp_b1ds =  0x00000030,
-       .grp_b2ds =  0x00000030,
-       .grp_b3ds =  0x00000030,
-       .grp_b4ds =  0x00000030,
-       .grp_b5ds =  0x00000030,
-       .grp_b6ds =  0x00000030,
-       .grp_b7ds =  0x00000030,
-};
-
-const struct mx6_mmdc_calibration mx6_mmcd_calib = {
-       .p0_mpwldectrl0 =  0x001F001F,
-       .p0_mpwldectrl1 =  0x001F001F,
-       .p1_mpwldectrl0 =  0x00440044,
-       .p1_mpwldectrl1 =  0x00440044,
-       .p0_mpdgctrl0 =  0x434B0350,
-       .p0_mpdgctrl1 =  0x034C0359,
-       .p1_mpdgctrl0 =  0x434B0350,
-       .p1_mpdgctrl1 =  0x03650348,
-       .p0_mprddlctl =  0x4436383B,
-       .p1_mprddlctl =  0x39393341,
-       .p0_mpwrdlctl =  0x35373933,
-       .p1_mpwrdlctl =  0x48254A36,
-};
-
-const struct mx6_mmdc_calibration mx6dqp_mmcd_calib = {
-       .p0_mpwldectrl0 =  0x001B001E,
-       .p0_mpwldectrl1 =  0x002E0029,
-       .p1_mpwldectrl0 =  0x001B002A,
-       .p1_mpwldectrl1 =  0x0019002C,
-       .p0_mpdgctrl0 =  0x43240334,
-       .p0_mpdgctrl1 =  0x0324031A,
-       .p1_mpdgctrl0 =  0x43340344,
-       .p1_mpdgctrl1 =  0x03280276,
-       .p0_mprddlctl =  0x44383A3E,
-       .p1_mprddlctl =  0x3C3C3846,
-       .p0_mpwrdlctl =  0x2E303230,
-       .p1_mpwrdlctl =  0x38283E34,
-};
-
-/* MT41K128M16JT-125 */
-static struct mx6_ddr3_cfg mem_ddr = {
-       .mem_speed = 1600,
-       .density = 2,
-       .width = 16,
-       .banks = 8,
-       .rowaddr = 14,
-       .coladdr = 10,
-       .pagesz = 2,
-       .trcd = 1375,
-       .trcmin = 4875,
-       .trasmin = 3500,
-};
-
 static void ccgr_init(void)
 {
        struct mxc_ccm_reg *ccm = (struct mxc_ccm_reg *)CCM_BASE_ADDR;

@@ -831,44 +712,209 @@ static void gpr_init(void)
        }
 }
 
-/*
- * This section requires the differentiation between iMX6 Sabre boards, but
- * for now, it will configure only for the mx6q variant.
- */
-static void spl_dram_init(void)
+static int mx6q_dcd_table[] = {
+       0x020e0798, 0x000C0000,
+       0x020e0758, 0x00000000,
+       0x020e0588, 0x00000030,
+       0x020e0594, 0x00000030,
+       0x020e056c, 0x00000030,
+       0x020e0578, 0x00000030,
+       0x020e074c, 0x00000030,
+       0x020e057c, 0x00000030,
+       0x020e058c, 0x00000000,
+       0x020e059c, 0x00000030,
+       0x020e05a0, 0x00000030,
+       0x020e078c, 0x00000030,
+       0x020e0750, 0x00020000,
+       0x020e05a8, 0x00000030,
+       0x020e05b0, 0x00000030,
+       0x020e0524, 0x00000030,
+       0x020e051c, 0x00000030,
+       0x020e0518, 0x00000030,
+       0x020e050c, 0x00000030,
+       0x020e05b8, 0x00000030,
+       0x020e05c0, 0x00000030,
+       0x020e0774, 0x00020000,
+       0x020e0784, 0x00000030,
+       0x020e0788, 0x00000030,
+       0x020e0794, 0x00000030,
+       0x020e079c, 0x00000030,
+       0x020e07a0, 0x00000030,
+       0x020e07a4, 0x00000030,
+       0x020e07a8, 0x00000030,
+       0x020e0748, 0x00000030,
+       0x020e05ac, 0x00000030,
+       0x020e05b4, 0x00000030,
+       0x020e0528, 0x00000030,
+       0x020e0520, 0x00000030,
+       0x020e0514, 0x00000030,
+       0x020e0510, 0x00000030,
+       0x020e05bc, 0x00000030,
+       0x020e05c4, 0x00000030,
+       0x021b0800, 0xa1390003,
+       0x021b080c, 0x001F001F,
+       0x021b0810, 0x001F001F,
+       0x021b480c, 0x001F001F,
+       0x021b4810, 0x001F001F,
+       0x021b083c, 0x43270338,
+       0x021b0840, 0x03200314,
+       0x021b483c, 0x431A032F,
+       0x021b4840, 0x03200263,
+       0x021b0848, 0x4B434748,
+       0x021b4848, 0x4445404C,
+       0x021b0850, 0x38444542,
+       0x021b4850, 0x4935493A,
+       0x021b081c, 0x33333333,
+       0x021b0820, 0x33333333,
+       0x021b0824, 0x33333333,
+       0x021b0828, 0x33333333,
+       0x021b481c, 0x33333333,
+       0x021b4820, 0x33333333,
+       0x021b4824, 0x33333333,
+       0x021b4828, 0x33333333,
+       0x021b08b8, 0x00000800,
+       0x021b48b8, 0x00000800,
+       0x021b0004, 0x00020036,
+       0x021b0008, 0x09444040,
+       0x021b000c, 0x555A7975,
+       0x021b0010, 0xFF538F64,
+       0x021b0014, 0x01FF00DB,
+       0x021b0018, 0x00001740,
+       0x021b001c, 0x00008000,
+       0x021b002c, 0x000026d2,
+       0x021b0030, 0x005A1023,
+       0x021b0040, 0x00000027,
+       0x021b0000, 0x831A0000,
+       0x021b001c, 0x04088032,
+       0x021b001c, 0x00008033,

board/freescale/mx6sabresd/mx6sabresd.c static void gpr_init(void) · line 799

Quoted 10 lines

board/freescale/mx6sabresd/mx6sabresd.c +10 -0

+       0x021b001c, 0x00048031,
+       0x021b001c, 0x09408030,
+       0x021b001c, 0x04008040,
+       0x021b0020, 0x00005800,
+       0x021b0818, 0x00011117,
+       0x021b4818, 0x00011117,
+       0x021b0004, 0x00025576,
+       0x021b0404, 0x00011006,
+       0x021b001c, 0x00000000,
+};

I believe, this is a random reg-pick from mx6q_4x_mt41j128.cfg. Do we need to follow any specific order to assign these dcd reg-init?

thanks!
--
Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.

Signature · 4 lines

Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.

Fabio Estevam

author

2017-04-04 15:50 UTC #6 lore raw

PATCH v3 1/2 mx6sabresd: Make SPL DDR configuration to match the DCD table

Quoted 1 line · Jagan Teki

Any help?

Sorry, didn't have time to look at this issue.

At least SPL U-Boot boots fine in this board. Can you also try U-Boot from NXP and try to debug it?

Jagan Teki

2017-04-04 17:27 UTC #7 lore raw

PATCH v3 1/2 mx6sabresd: Make SPL DDR configuration to match the DCD table

Quoted 7 lines · Fabio Estevam

Quoted 1 line · Jagan Teki
Any help?
Sorry, didn't have time to look at this issue.
At least SPL U-Boot boots fine in this board. Can you also try U-Boot from NXP and try to debug it?

No, this isn't issue, just want to understand how you create reg init in SPL in mx6q_dcd_table, means did you follow any order. because I'm planning move *dl.cfg DCD to SPL

thanks!

Signature · 4 lines

Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.

Fabio Estevam

author

2017-04-04 23:59 UTC #8 lore raw

PATCH v3 1/2 mx6sabresd: Make SPL DDR configuration to match the DCD table

Hi Jagan,

Quoted 3 lines · Jagan Teki

No, this isn't issue, just want to understand how you create reg init in SPL in mx6q_dcd_table, means did you follow any order. because I'm planning move *dl.cfg DCD to SPL

Ok, got it. I thought you were asking about mx6qp sabreauto not booting.

On this patch I just did a 1:1 translation from DCD to SPL code.

Jagan Teki

2017-04-05 17:33 UTC #9 lore raw

PATCH v3 1/2 mx6sabresd: Make SPL DDR configuration to match the DCD table

Quoted 11 lines · Fabio Estevam

Hi Jagan,
Quoted 3 lines · Jagan Teki
No, this isn't issue, just want to understand how you create reg init in SPL in mx6q_dcd_table, means did you follow any order. because I'm planning move *dl.cfg DCD to SPL
Ok, got it. I thought you were asking about mx6qp sabreauto not booting.
On this patch I just did a 1:1 translation from DCD to SPL code.

I saw the reg with respective values same from *.cfg but the sequence of registers are not direct compared to DCD from *.cfg? am I correct?

thanks!

Signature · 4 lines

Jagan Teki
Free Software Engineer | www.openedev.com
U-Boot, Linux | Upstream Maintainer
Hyderabad, India.