]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
ti816x: Rework DDR initialization sequence
authorTom Rini <trini@konsulko.com>
Tue, 16 May 2017 18:46:35 +0000 (14:46 -0400)
committerTom Rini <trini@konsulko.com>
Mon, 5 Jun 2017 15:02:23 +0000 (11:02 -0400)
The ti816x/am389x SoC is the first generation in what U-Boot calls the
"am33xx" family.  In the first generation of this family the DDR
initialization sequence is quite different from all of the subsequent
generations.  Whereas with ti814x (second generation) we can easily work
the minor differenced between that and am33xx (third generation), our
attempts to do this for ti816x weren't sufficient.  Rather than add a
large amount of #ifdef logic to make this different sequence work we add
a new file, ti816x_emif4.c to handle the various required undocumented
register writes and sequence and leverage what we can from
arch/arm/mach-omap2/am33xx/ddr.c still.  As DDR2 has similar problems
today but I am unable to test it, we drop the DDR2 defines from the code
rather than imply that it works by leaving it.  We also remove a bunch
of other untested code about changing the speed the DDR runs at.

Signed-off-by: Tom Rini <trini@konsulko.com>
arch/arm/include/asm/arch-am33xx/clocks_am33xx.h
arch/arm/include/asm/arch-am33xx/ddr_defs.h
arch/arm/mach-omap2/am33xx/Makefile
arch/arm/mach-omap2/am33xx/board.c
arch/arm/mach-omap2/am33xx/clock_ti816x.c
arch/arm/mach-omap2/am33xx/ddr.c
arch/arm/mach-omap2/am33xx/emif4.c
arch/arm/mach-omap2/am33xx/ti816x_emif4.c [new file with mode: 0644]
board/ti/ti816x/evm.c
include/configs/ti816x_evm.h
scripts/config_whitelist.txt

index 653ec1b2394f9d7f483f30cd48d7ecad3cfc945d..bc1dab5b72b29ee98489a50bf14cfd55fdecbfef 100644 (file)
@@ -29,6 +29,7 @@
 #define NUM_OPPS       6
 
 extern void enable_dmm_clocks(void);
+extern void enable_emif_clocks(void);
 extern const struct dpll_params dpll_core_opp100;
 extern struct dpll_params dpll_mpu_opp100;
 
index 43e122e261916cf3f0a2e4d917b5a2f95f59c66f..a97ebb557bcd291a353aa349083968c6f4db6d11 100644 (file)
@@ -354,9 +354,15 @@ struct ddr_ctrl {
        unsigned int ddrckectrl;
 };
 
+#ifdef CONFIG_TI816X
+void config_ddr(const struct ddr_data *data, const struct cmd_control *ctrl,
+               const struct emif_regs *regs,
+               const struct dmm_lisa_map_regs *lisa_regs, int nrs);
+#else
 void config_ddr(unsigned int pll, const struct ctrl_ioregs *ioregs,
                const struct ddr_data *data, const struct cmd_control *ctrl,
                const struct emif_regs *regs, int nr);
+#endif
 void emif_get_ext_phy_ctrl_const_regs(const u32 **regs, u32 *size);
 
 #endif  /* _DDR_DEFS_H */
index 05cc8a11c5cbf644bd1d2c7ac5753d1c3559fedb..1e4c04e7a65b8e1fd88ddff47653307b75090b65 100644 (file)
@@ -15,7 +15,10 @@ endif
 obj-$(CONFIG_TI816X)   += clock_ti816x.o
 obj-y  += sys_info.o
 obj-y  += ddr.o
+ifeq ($(CONFIG_TI816X)$(CONFIG_SKIP_LOWLEVEL_INIT),)
 obj-y  += emif4.o
+endif
+obj-$(CONFIG_TI816X)   += ti816x_emif4.o
 obj-y  += board.o
 obj-y  += mux.o
 
index a8b5d13238a220a0ec012afe30f6bbd4d90c9b90..faf7d07a03308c5c7badba97171dea62c500c69a 100644 (file)
 
 DECLARE_GLOBAL_DATA_PTR;
 
+int dram_init(void)
+{
+#ifndef CONFIG_SKIP_LOWLEVEL_INIT
+       sdram_init();
+#endif
+
+       /* dram_init must store complete ramsize in gd->ram_size */
+       gd->ram_size = get_ram_size(
+                       (void *)CONFIG_SYS_SDRAM_BASE,
+                       CONFIG_MAX_RAM_BANK_SIZE);
+       return 0;
+}
+
+int dram_init_banksize(void)
+{
+       gd->bd->bi_dram[0].start = CONFIG_SYS_SDRAM_BASE;
+       gd->bd->bi_dram[0].size = gd->ram_size;
+
+       return 0;
+}
+
 #if !CONFIG_IS_ENABLED(OF_CONTROL)
 static const struct ns16550_platdata am33xx_serial[] = {
        { .base = CONFIG_SYS_NS16550_COM1, .reg_shift = 2,
index 079ddd7eb9fc12afaed8ab4e5138dd114375e681..967623d467b03ff1a4e9dfcc1f48eef30cd17b9b 100644 (file)
 #define MAIN_MDIV7             0x4
 
 /* DDR PLL */
-#if defined(CONFIG_TI816X_DDR_PLL_400) /* 400 MHz */
-#define DDR_N                  59
-#define DDR_P                  0x1
-#define DDR_MDIV1              0x4
-#define DDR_INTFREQ2           0x8
-#define DDR_FRACFREQ2          0xD99999
-#define DDR_MDIV2              0x1E
-#define DDR_INTFREQ3           0x8
-#define DDR_FRACFREQ3          0x0
-#define DDR_MDIV3              0x4
-#define DDR_INTFREQ4           0xE /* Expansion DDR clk */
-#define DDR_FRACFREQ4          0x0
-#define DDR_MDIV4              0x4
-#define DDR_INTFREQ5           0xE /* Expansion DDR clk */
-#define DDR_FRACFREQ5          0x0
-#define DDR_MDIV5              0x4
-#elif defined(CONFIG_TI816X_DDR_PLL_531) /* 531 MHz */
-#define DDR_N                  59
-#define DDR_P                  0x1
-#define DDR_MDIV1              0x3
-#define DDR_INTFREQ2           0x8
-#define DDR_FRACFREQ2          0xD99999
-#define DDR_MDIV2              0x1E
-#define DDR_INTFREQ3           0x8
-#define DDR_FRACFREQ3          0x0
-#define DDR_MDIV3              0x4
-#define DDR_INTFREQ4           0xE /* Expansion DDR clk */
-#define DDR_FRACFREQ4          0x0
-#define DDR_MDIV4              0x4
-#define DDR_INTFREQ5           0xE /* Expansion DDR clk */
-#define DDR_FRACFREQ5          0x0
-#define DDR_MDIV5              0x4
-#elif defined(CONFIG_TI816X_DDR_PLL_675) /* 675 MHz */
-#define DDR_N                  50
-#define DDR_P                  0x1
-#define DDR_MDIV1              0x2
-#define DDR_INTFREQ2           0x9
-#define DDR_FRACFREQ2          0x0
-#define DDR_MDIV2              0x19
-#define DDR_INTFREQ3           0x13
-#define DDR_FRACFREQ3          0x800000
-#define DDR_MDIV3              0x2
-#define DDR_INTFREQ4           0xE /* Expansion DDR clk */
-#define DDR_FRACFREQ4          0x0
-#define DDR_MDIV4              0x4
-#define DDR_INTFREQ5           0xE /* Expansion DDR clk */
-#define DDR_FRACFREQ5          0x0
-#define DDR_MDIV5              0x4
-#elif defined(CONFIG_TI816X_DDR_PLL_796) /* 796 MHz */
 #define DDR_N                  59
 #define DDR_P                  0x1
 #define DDR_MDIV1              0x2
 #define DDR_INTFREQ5           0xE /* Expansion DDR clk */
 #define DDR_FRACFREQ5          0x0
 #define DDR_MDIV5              0x4
-#endif
 
 #define CONTROL_STATUS                 (CTRL_BASE + 0x40)
 #define DDR_RCD                                (CTRL_BASE + 0x070C)
 #define CM_TIMER1_CLKSEL               (PRCM_BASE + 0x390)
-#define DMM_PAT_BASE_ADDR              (DMM_BASE + 0x420)
 #define CM_ALWON_CUST_EFUSE_CLKCTRL    (PRCM_BASE + 0x1628)
 
 #define INTCPS_SYSCONFIG       0x48200010
@@ -187,6 +136,15 @@ const struct wd_timer *wdtimer = (struct wd_timer *)WDT_BASE;
 
 void enable_dmm_clocks(void)
 {
+       writel(PRCM_MOD_EN, &cmdef->dmmclkctrl);
+       /* Wait for dmm to be fully functional, including OCP */
+       while (((readl(&cmdef->dmmclkctrl) >> 17) & 0x3) != 0)
+               ;
+}
+
+void enable_emif_clocks(void)
+{
+       writel(PRCM_MOD_EN, &cmdef->fwclkctrl);
        writel(PRCM_MOD_EN, &cmdef->l3fastclkstctrl);
        writel(PRCM_MOD_EN, &cmdef->emif0clkctrl);
        writel(PRCM_MOD_EN, &cmdef->emif1clkctrl);
@@ -200,14 +158,6 @@ void enable_dmm_clocks(void)
        /* Wait for emif1 to be fully functional, including OCP */
        while (((readl(&cmdef->emif1clkctrl) >> 17) & 0x3) != 0)
                ;
-
-       writel(PRCM_MOD_EN, &cmdef->dmmclkctrl);
-       /* Wait for dmm to be fully functional, including OCP */
-       while (((readl(&cmdef->dmmclkctrl) >> 17) & 0x3) != 0)
-               ;
-
-       /* Enable Tiled Access */
-       writel(0x80000000, DMM_PAT_BASE_ADDR);
 }
 
 /* assume delay is aprox at least 1us */
index 690487e7c38b5a5581eee9479b50c162ed396a17..7bf19ed9665f1b40ab98d2aa997f4a656989fc73 100644 (file)
@@ -163,6 +163,14 @@ void config_sdram_emif4d5(const struct emif_regs *regs, int nr)
  */
 void config_sdram(const struct emif_regs *regs, int nr)
 {
+#ifdef CONFIG_TI816X
+       writel(regs->sdram_config, &emif_reg[nr]->emif_sdram_config);
+       writel(regs->emif_ddr_phy_ctlr_1, &emif_reg[nr]->emif_ddr_phy_ctrl_1);
+       writel(regs->emif_ddr_phy_ctlr_1, &emif_reg[nr]->emif_ddr_phy_ctrl_1_shdw);
+       writel(0x0000613B, &emif_reg[nr]->emif_sdram_ref_ctrl);   /* initially a large refresh period */
+       writel(0x1000613B, &emif_reg[nr]->emif_sdram_ref_ctrl);   /* trigger initialization           */
+       writel(regs->ref_ctrl, &emif_reg[nr]->emif_sdram_ref_ctrl);
+#else
        if (regs->zq_config) {
                writel(regs->zq_config, &emif_reg[nr]->emif_zq_config);
                writel(regs->sdram_config, &cstat->secure_emif_sdram_config);
@@ -184,6 +192,7 @@ void config_sdram(const struct emif_regs *regs, int nr)
        /* Write REG_COS_COUNT_1, REG_COS_COUNT_2, and REG_PR_OLD_COUNT. */
        if (regs->ocp_config)
                writel(regs->ocp_config, &emif_reg[nr]->emif_l3_config);
+#endif
 }
 
 /**
index 3a110f2845707d14c9f5f4f7c39e6f6af1e93a9d..68c7705178710804b16fa4814c7b53b1d5ad8268 100644 (file)
 #include <asm/io.h>
 #include <asm/emif.h>
 
-DECLARE_GLOBAL_DATA_PTR;
-
-int dram_init(void)
-{
-#ifndef CONFIG_SKIP_LOWLEVEL_INIT
-       sdram_init();
-#endif
-
-       /* dram_init must store complete ramsize in gd->ram_size */
-       gd->ram_size = get_ram_size(
-                       (void *)CONFIG_SYS_SDRAM_BASE,
-                       CONFIG_MAX_RAM_BANK_SIZE);
-       return 0;
-}
-
-int dram_init_banksize(void)
-{
-       gd->bd->bi_dram[0].start = CONFIG_SYS_SDRAM_BASE;
-       gd->bd->bi_dram[0].size = gd->ram_size;
-
-       return 0;
-}
-
-
-#ifndef CONFIG_SKIP_LOWLEVEL_INIT
-#ifdef CONFIG_TI81XX
-static struct dmm_lisa_map_regs *hw_lisa_map_regs =
-                               (struct dmm_lisa_map_regs *)DMM_BASE;
-#endif
-#ifndef CONFIG_TI816X
 static struct vtp_reg *vtpreg[2] = {
                                (struct vtp_reg *)VTP0_CTRL_ADDR,
                                (struct vtp_reg *)VTP1_CTRL_ADDR};
-#endif
 #ifdef CONFIG_AM33XX
 static struct ddr_ctrl *ddrctrl = (struct ddr_ctrl *)DDR_CTRL_ADDR;
 #endif
@@ -60,9 +29,12 @@ static struct cm_device_inst *cm_device =
                                (struct cm_device_inst *)CM_DEVICE_INST;
 #endif
 
-#ifdef CONFIG_TI81XX
+#ifdef CONFIG_TI814X
 void config_dmm(const struct dmm_lisa_map_regs *regs)
 {
+       struct dmm_lisa_map_regs *hw_lisa_map_regs =
+                               (struct dmm_lisa_map_regs *)DMM_BASE;
+
        enable_dmm_clocks();
 
        writel(0, &hw_lisa_map_regs->dmm_lisa_map_3);
@@ -77,7 +49,6 @@ void config_dmm(const struct dmm_lisa_map_regs *regs)
 }
 #endif
 
-#ifndef CONFIG_TI816X
 static void config_vtp(int nr)
 {
        writel(readl(&vtpreg[nr]->vtp0ctrlreg) | VTP_CTRL_ENABLE,
@@ -92,7 +63,6 @@ static void config_vtp(int nr)
                        VTP_CTRL_READY)
                ;
 }
-#endif
 
 void __weak ddr_pll_config(unsigned int ddrpll_m)
 {
@@ -103,9 +73,7 @@ void config_ddr(unsigned int pll, const struct ctrl_ioregs *ioregs,
                const struct emif_regs *regs, int nr)
 {
        ddr_pll_config(pll);
-#ifndef CONFIG_TI816X
        config_vtp(nr);
-#endif
        config_cmd_ctrl(ctrl, nr);
 
        config_ddr_data(data, nr);
@@ -139,4 +107,3 @@ void config_ddr(unsigned int pll, const struct ctrl_ioregs *ioregs,
        else
                config_sdram(regs, nr);
 }
-#endif
diff --git a/arch/arm/mach-omap2/am33xx/ti816x_emif4.c b/arch/arm/mach-omap2/am33xx/ti816x_emif4.c
new file mode 100644 (file)
index 0000000..2e7ea90
--- /dev/null
@@ -0,0 +1,165 @@
+/*
+ * ti816x_emif4.c
+ *
+ * TI816x emif4 configuration file
+ *
+ * Copyright (C) 2017, Konsulko Group
+ *
+ * SPDX-License-Identifier:    GPL-2.0+
+ */
+
+#include <common.h>
+#include <asm/arch/cpu.h>
+#include <asm/arch/ddr_defs.h>
+#include <asm/arch/hardware.h>
+#include <asm/arch/clock.h>
+#include <asm/arch/sys_proto.h>
+#include <asm/io.h>
+#include <asm/emif.h>
+
+/*********************************************************************
+ * Init DDR3 on TI816X EVM
+ *********************************************************************/
+static void ddr_init_settings(const struct cmd_control *ctrl, int emif)
+{
+       /*
+        * setup use_rank_delays to 1.  This is only necessary when
+        * multiple ranks are in use.  Though the EVM does not have
+        * multiple ranks, this is a good value to set.
+        */
+       writel(1, DDRPHY_CONFIG_BASE + 0x134); // DATA0_REG_PHY_USE_RANK0_DELAYS
+       writel(1, DDRPHY_CONFIG_BASE + 0x1d8); // DATA1_REG_PHY_USE_RANK0_DELAYS
+       writel(1, DDRPHY_CONFIG_BASE + 0x27c); // DATA2_REG_PHY_USE_RANK0_DELAYS
+       writel(1, DDRPHY_CONFIG_BASE + 0x320); // DATA3_REG_PHY_USE_RANK0_DELAYS
+
+       config_cmd_ctrl(ctrl, emif);
+
+       /* for ddr3 this needs to be set to 1 */
+       writel(0x1, DDRPHY_CONFIG_BASE + 0x0F8); /* init mode */
+       writel(0x1, DDRPHY_CONFIG_BASE + 0x104);
+       writel(0x1, DDRPHY_CONFIG_BASE + 0x19C);
+       writel(0x1, DDRPHY_CONFIG_BASE + 0x1A8);
+       writel(0x1, DDRPHY_CONFIG_BASE + 0x240);
+       writel(0x1, DDRPHY_CONFIG_BASE + 0x24C);
+       writel(0x1, DDRPHY_CONFIG_BASE + 0x2E4);
+       writel(0x1, DDRPHY_CONFIG_BASE + 0x2F0);
+
+       /*
+        * This represents the initial value for the leveling process.  The
+        * value is a ratio - so 0x100 represents one cycle.  The real delay
+        * is determined through the leveling process.
+        *
+        * During the leveling process, 0x20 is subtracted from the value, so
+        * we have added that to the value we want to set.  We also set the
+        * values such that byte3 completes leveling after byte2 and byte1
+        * after byte0.
+        */
+       writel((0x20 << 10) | 0x20, DDRPHY_CONFIG_BASE + 0x0F0); /*  data0 writelvl init ratio */
+       writel(0x0, DDRPHY_CONFIG_BASE + 0x0F4);   /*   */
+       writel((0x20 << 10) | 0x20, DDRPHY_CONFIG_BASE + 0x194); /*  data1 writelvl init ratio */
+       writel(0x0, DDRPHY_CONFIG_BASE + 0x198);   /*   */
+       writel((0x20 << 10) | 0x20, DDRPHY_CONFIG_BASE + 0x238); /*  data2 writelvl init ratio */
+       writel(0x0, DDRPHY_CONFIG_BASE + 0x23c);   /*   */
+       writel((0x20 << 10) | 0x20, DDRPHY_CONFIG_BASE + 0x2dc); /*  data3 writelvl init ratio */
+       writel(0x0, DDRPHY_CONFIG_BASE + 0x2e0);   /*   */
+
+
+       writel((0x20 << 10) | 0x20, DDRPHY_CONFIG_BASE + 0x0FC); /*  data0 gatelvl init ratio */
+       writel(0x0, DDRPHY_CONFIG_BASE + 0x100);
+       writel((0x20 << 10) | 0x20, DDRPHY_CONFIG_BASE + 0x1A0); /*  data1 gatelvl init ratio */
+       writel(0x0, DDRPHY_CONFIG_BASE + 0x1A4);
+       writel((0x20 << 10) | 0x20, DDRPHY_CONFIG_BASE + 0x244); /*  data2 gatelvl init ratio */
+       writel(0x0, DDRPHY_CONFIG_BASE + 0x248);
+       writel((0x20 << 10) | 0x20, DDRPHY_CONFIG_BASE + 0x2E8); /*  data3 gatelvl init ratio */
+       writel(0x0, DDRPHY_CONFIG_BASE + 0x2EC);
+
+       writel(0x5, DDRPHY_CONFIG_BASE + 0x00C);     /* cmd0 io config - output impedance of pad */
+       writel(0x5, DDRPHY_CONFIG_BASE + 0x010);     /* cmd0 io clk config - output impedance of pad */
+       writel(0x5, DDRPHY_CONFIG_BASE + 0x040);     /* cmd1 io config - output impedance of pad */
+       writel(0x5, DDRPHY_CONFIG_BASE + 0x044);     /* cmd1 io clk config - output impedance of pad */
+       writel(0x5, DDRPHY_CONFIG_BASE + 0x074);     /* cmd2 io config - output impedance of pad */
+       writel(0x5, DDRPHY_CONFIG_BASE + 0x078);     /* cmd2 io clk config - output impedance of pad */
+       writel(0x4, DDRPHY_CONFIG_BASE + 0x0A8);     /* data0 io config - output impedance of pad */
+       writel(0x4, DDRPHY_CONFIG_BASE + 0x0AC);     /* data0 io clk config - output impedance of pad */
+       writel(0x4, DDRPHY_CONFIG_BASE + 0x14C);     /* data1 io config - output impedance of pa     */
+       writel(0x4, DDRPHY_CONFIG_BASE + 0x150);     /* data1 io clk config - output impedance of pad */
+       writel(0x4, DDRPHY_CONFIG_BASE + 0x1F0);     /* data2 io config - output impedance of pa */
+       writel(0x4, DDRPHY_CONFIG_BASE + 0x1F4);     /* data2 io clk config - output impedance of pad */
+       writel(0x4, DDRPHY_CONFIG_BASE + 0x294);     /* data3 io config - output impedance of pa */
+       writel(0x4, DDRPHY_CONFIG_BASE + 0x298);     /* data3 io clk config - output impedance of pad */
+}
+
+static void ddr3_sw_levelling(const struct ddr_data *data, int emif)
+{
+       /* Set the correct value to DDR_VTP_CTRL_0 */
+       writel(0x6, (DDRPHY_CONFIG_BASE + 0x358));
+
+       writel(data->datafwsratio0, (DDRPHY_CONFIG_BASE + 0x108));
+       writel(data->datafwsratio0, (DDRPHY_CONFIG_BASE + 0x1AC));
+       writel(data->datafwsratio0, (DDRPHY_CONFIG_BASE + 0x250));
+       writel(data->datafwsratio0, (DDRPHY_CONFIG_BASE + 0x2F4));
+
+       writel(data->datawdsratio0, (DDRPHY_CONFIG_BASE + 0x0DC));
+       writel(data->datawdsratio0, (DDRPHY_CONFIG_BASE + 0x180));
+       writel(data->datawdsratio0, (DDRPHY_CONFIG_BASE + 0x224));
+       writel(data->datawdsratio0, (DDRPHY_CONFIG_BASE + 0x2C8));
+
+       writel(data->datawrsratio0, (DDRPHY_CONFIG_BASE + 0x120));
+       writel(data->datawrsratio0, (DDRPHY_CONFIG_BASE + 0x1C4));
+       writel(data->datawrsratio0, (DDRPHY_CONFIG_BASE + 0x268));
+       writel(data->datawrsratio0, (DDRPHY_CONFIG_BASE + 0x30C));
+
+       writel(data->datardsratio0, (DDRPHY_CONFIG_BASE + 0x0C8));
+       writel(data->datardsratio0, (DDRPHY_CONFIG_BASE + 0x16C));
+       writel(data->datardsratio0, (DDRPHY_CONFIG_BASE + 0x210));
+       writel(data->datardsratio0, (DDRPHY_CONFIG_BASE + 0x2B4));
+}
+
+static struct dmm_lisa_map_regs *hw_lisa_map_regs =
+                               (struct dmm_lisa_map_regs *)DMM_BASE;
+
+#define DMM_PAT_BASE_ADDR              (DMM_BASE + 0x420)
+void config_dmm(const struct dmm_lisa_map_regs *regs)
+{
+       writel(0, &hw_lisa_map_regs->dmm_lisa_map_3);
+       writel(0, &hw_lisa_map_regs->dmm_lisa_map_2);
+       writel(0, &hw_lisa_map_regs->dmm_lisa_map_1);
+       writel(0, &hw_lisa_map_regs->dmm_lisa_map_0);
+
+       writel(regs->dmm_lisa_map_3, &hw_lisa_map_regs->dmm_lisa_map_3);
+       writel(regs->dmm_lisa_map_2, &hw_lisa_map_regs->dmm_lisa_map_2);
+       writel(regs->dmm_lisa_map_1, &hw_lisa_map_regs->dmm_lisa_map_1);
+       writel(regs->dmm_lisa_map_0, &hw_lisa_map_regs->dmm_lisa_map_0);
+
+       /* Enable Tiled Access */
+       writel(0x80000000, DMM_PAT_BASE_ADDR);
+}
+
+void config_ddr(const struct ddr_data *data, const struct cmd_control *ctrl,
+               const struct emif_regs *regs,
+               const struct dmm_lisa_map_regs *lisa_regs, int nrs)
+{
+       int i;
+
+       enable_emif_clocks();
+
+       for (i = 0; i < nrs; i++)
+               ddr_init_settings(ctrl, i);
+
+       enable_dmm_clocks();
+
+       /* Program the DMM to for non-interleaved configuration */
+       config_dmm(lisa_regs);
+
+       /* Program EMIF CFG Registers */
+       for (i = 0; i < nrs; i++) {
+               set_sdram_timings(regs, i);
+               config_sdram(regs, i);
+       }
+
+       udelay(1000);
+       for (i = 0; i < nrs; i++)
+               ddr3_sw_levelling(data, i);
+
+       udelay(50000);  /* Some delay needed */
+}
index b6bf16236f408aadc2003fcc880a747c608c5f6f..ad98e09e2082e5d35cad82f745a01fc29a00e41d 100644 (file)
@@ -41,176 +41,68 @@ static struct module_pin_mux mmc_pin_mux[] = {
        { -1 },
 };
 
-const struct dmm_lisa_map_regs evm_lisa_map_regs = {
-       .dmm_lisa_map_0 = 0x00000000,
-       .dmm_lisa_map_1 = 0x00000000,
-       .dmm_lisa_map_2 = 0x80640300,
-       .dmm_lisa_map_3 = 0xC0640320,
-};
-
-/*
- * DDR2 related definitions
- */
-#ifdef CONFIG_TI816X_EVM_DDR2
-static struct ddr_data ddr2_data = {
-       .datardsratio0          = ((0x40<<10) | (0x40<<0)),
-       .datawdsratio0          = ((0x4A<<10) | (0x4A<<0)),
-       .datawiratio0           = ((0x0<<10) | (0x0<<0)),
-       .datagiratio0           = ((0x0<<10) | (0x0<<0)),
-       .datafwsratio0          = ((0x13A<<10) | (0x13A<<0)),
-       .datawrsratio0          = ((0x8A<<10) | (0x8A<<0)),
-};
-
-static struct cmd_control ddr2_ctrl = {
-       .cmd0csratio    = 0x80,
-       .cmd0iclkout    = 0x00,
-
-       .cmd1csratio    = 0x80,
-       .cmd1iclkout    = 0x00,
-
-       .cmd2csratio    = 0x80,
-       .cmd2iclkout    = 0x00,
-
-};
-
-static struct emif_regs ddr2_emif0_regs = {
-       .sdram_config           = 0x43801A3A,
-       .ref_ctrl               = 0x10000C30,
-       .sdram_tim1             = 0x0AAB15E2,
-       .sdram_tim2             = 0x423631D2,
-       .sdram_tim3             = 0x0080032F,
-       .emif_ddr_phy_ctlr_1    = 0x0, /* depend on cpu rev, set later */
-};
+void set_uart_mux_conf(void) {}
 
-static struct emif_regs ddr2_emif1_regs = {
-       .sdram_config           = 0x43801A3A,
-       .ref_ctrl               = 0x10000C30,
-       .sdram_tim1             = 0x0AAB15E2,
-       .sdram_tim2             = 0x423631D2,
-       .sdram_tim3             = 0x0080032F,
-       .emif_ddr_phy_ctlr_1    = 0x0, /* depend on cpu rev, set later */
-};
-#endif
+void set_mux_conf_regs(void)
+{
+       configure_module_pin_mux(mmc_pin_mux);
+}
 
 /*
- * DDR3 related definitions
+ * EMIF Paramters.  Refer the EMIF register documentation and the
+ * memory datasheet for details.  This is for 796 MHz.
  */
-
-#if defined(CONFIG_TI816X_DDR_PLL_400)
-#define RD_DQS         0x03B
-#define WR_DQS         0x0A6
-#define RD_DQS_GATE    0x12A
-#define EMIF_SDCFG     0x62A41032
-#define EMIF_SDREF     0x10000C30
-#define EMIF_TIM1      0x0CCCE524
-#define EMIF_TIM2      0x30308023
-#define EMIF_TIM3      0x009F82CF
-#define EMIF_PHYCFG    0x0000010B
-#elif defined(CONFIG_TI816X_DDR_PLL_531)
-#define RD_DQS         0x039
-#define WR_DQS         0x0B4
-#define RD_DQS_GATE    0x13D
-#define EMIF_SDCFG     0x62A51832
-#define EMIF_SDREF     0x1000102E
-#define EMIF_TIM1      0x0EF136AC
-#define EMIF_TIM2      0x30408063
-#define EMIF_TIM3      0x009F83AF
-#define EMIF_PHYCFG    0x0000010C
-#elif defined(CONFIG_TI816X_DDR_PLL_675)
-#define RD_DQS         0x039
-#define WR_DQS         0x091
-#define RD_DQS_GATE    0x196
-#define EMIF_SDCFG     0x62A63032
-#define EMIF_SDREF     0x10001491
-#define EMIF_TIM1      0x13358875
-#define EMIF_TIM2      0x5051806C
-#define EMIF_TIM3      0x009F84AF
-#define EMIF_PHYCFG    0x0000010F
-#elif defined(CONFIG_TI816X_DDR_PLL_796)
-#define RD_DQS         0x035
-#define WR_DQS         0x093
-#define RD_DQS_GATE    0x1B3
-#define EMIF_SDCFG     0x62A73832
-#define EMIF_SDREF     0x10001841
-#define EMIF_TIM1      0x1779C9FE
-#define EMIF_TIM2      0x50608074
-#define EMIF_TIM3      0x009F857F
-#define EMIF_PHYCFG    0x00000110
-#endif
-
-static struct ddr_data ddr3_data = {
-       .datardsratio0          = ((RD_DQS<<10) | (RD_DQS<<0)),
-       .datawdsratio0          = ((WR_DQS<<10) | (WR_DQS<<0)),
-       .datawiratio0           = ((0x20<<10) | 0x20<<0),
-       .datagiratio0           = ((0x20<<10) | 0x20<<0),
-       .datafwsratio0          = ((RD_DQS_GATE<<10) | (RD_DQS_GATE<<0)),
-       .datawrsratio0          = (((WR_DQS+0x40)<<10) | ((WR_DQS+0x40)<<0)),
+#define EMIF_TIM1   0x1779C9FE
+#define EMIF_TIM2   0x50608074
+#define EMIF_TIM3   0x009F857F
+#define EMIF_SDREF  0x10001841
+#define EMIF_SDCFG  0x62A73832
+#define EMIF_PHYCFG 0x00000110
+static const struct emif_regs ddr3_emif_regs = {
+       .sdram_config           = EMIF_SDCFG,
+       .ref_ctrl               = EMIF_SDREF,
+       .sdram_tim1             = EMIF_TIM1,
+       .sdram_tim2             = EMIF_TIM2,
+       .sdram_tim3             = EMIF_TIM3,
+       .emif_ddr_phy_ctlr_1    = EMIF_PHYCFG,
 };
 
 static const struct cmd_control ddr3_ctrl = {
        .cmd0csratio    = 0x100,
        .cmd0iclkout    = 0x001,
-
        .cmd1csratio    = 0x100,
        .cmd1iclkout    = 0x001,
-
        .cmd2csratio    = 0x100,
        .cmd2iclkout    = 0x001,
 };
 
-static const struct emif_regs ddr3_emif0_regs = {
-       .sdram_config           = EMIF_SDCFG,
-       .ref_ctrl               = EMIF_SDREF,
-       .sdram_tim1             = EMIF_TIM1,
-       .sdram_tim2             = EMIF_TIM2,
-       .sdram_tim3             = EMIF_TIM3,
-       .emif_ddr_phy_ctlr_1    = EMIF_PHYCFG,
+/* These values are obtained from the CCS app */
+#define RD_DQS_GATE    (0x1B3)
+#define RD_DQS         (0x35)
+#define WR_DQS         (0x93)
+static struct ddr_data ddr3_data = {
+       .datardsratio0          = ((RD_DQS<<10) | (RD_DQS<<0)),
+       .datawdsratio0          = ((WR_DQS<<10) | (WR_DQS<<0)),
+       .datawiratio0           = ((0x20<<10) | 0x20<<0),
+       .datagiratio0           = ((0x20<<10) | 0x20<<0),
+       .datafwsratio0          = ((RD_DQS_GATE<<10) | (RD_DQS_GATE<<0)),
+       .datawrsratio0          = (((WR_DQS+0x40)<<10) | ((WR_DQS+0x40)<<0)),
 };
 
-static const struct emif_regs ddr3_emif1_regs = {
-       .sdram_config           = EMIF_SDCFG,
-       .ref_ctrl               = EMIF_SDREF,
-       .sdram_tim1             = EMIF_TIM1,
-       .sdram_tim2             = EMIF_TIM2,
-       .sdram_tim3             = EMIF_TIM3,
-       .emif_ddr_phy_ctlr_1    = EMIF_PHYCFG,
+static const struct dmm_lisa_map_regs evm_lisa_map_regs = {
+       .dmm_lisa_map_0 = 0x00000000,
+       .dmm_lisa_map_1 = 0x00000000,
+       .dmm_lisa_map_2 = 0x80640300,
+       .dmm_lisa_map_3 = 0xC0640320,
 };
 
-void set_uart_mux_conf(void) {}
-
-void set_mux_conf_regs(void)
-{
-       configure_module_pin_mux(mmc_pin_mux);
-}
-
 void sdram_init(void)
 {
-       config_dmm(&evm_lisa_map_regs);
-
-#ifdef CONFIG_TI816X_EVM_DDR2
-       if (CONFIG_TI816X_USE_EMIF0) {
-               ddr2_emif0_regs.emif_ddr_phy_ctlr_1 =
-                       (get_cpu_rev() == 0x1 ? 0x0000010B : 0x0000030B);
-               config_ddr(0, NULL, &ddr2_data, &ddr2_ctrl, &ddr2_emif0_regs,
-                          0);
-       }
-
-       if (CONFIG_TI816X_USE_EMIF1) {
-               ddr2_emif1_regs.emif_ddr_phy_ctlr_1 =
-                       (get_cpu_rev() == 0x1 ? 0x0000010B : 0x0000030B);
-               config_ddr(1, NULL, &ddr2_data, &ddr2_ctrl, &ddr2_emif1_regs,
-                          1);
-       }
-#endif
-
-#ifdef CONFIG_TI816X_EVM_DDR3
-       if (CONFIG_TI816X_USE_EMIF0)
-               config_ddr(0, NULL, &ddr3_data, &ddr3_ctrl, &ddr3_emif0_regs,
-                          0);
-
-       if (CONFIG_TI816X_USE_EMIF1)
-               config_ddr(1, NULL, &ddr3_data, &ddr3_ctrl, &ddr3_emif1_regs,
-                          1);
-#endif
+       /*
+        * Pass in our DDR3 config information and that we have 2 EMIFs to
+        * configure.
+        */
+       config_ddr(&ddr3_data, &ddr3_ctrl, &ddr3_emif_regs,
+                       &evm_lisa_map_regs, 2);
 }
 #endif /* CONFIG_SPL_BUILD */
index 68eb08f8126f466ee474424ecc799568d34f8a41..f7f6f625a00145a20cd4fddac6d73b700b2d63ff 100644 (file)
 
 #define CONFIG_CMD_ASKENV
 
-/*
- * Only one of the following two options (DDR3/DDR2) should be enabled
- * CONFIG_TI816X_EVM_DDR2
- * CONFIG_TI816X_EVM_DDR3
- */
-#define CONFIG_TI816X_EVM_DDR3
-
-/*
- * Supported values: 400, 531, 675 or 796 MHz
- */
-#define CONFIG_TI816X_DDR_PLL_796
-
-#define CONFIG_TI816X_USE_EMIF0        1
-#define CONFIG_TI816X_USE_EMIF1        1
-
 #define CONFIG_NR_DRAM_BANKS   2               /* we have 2 banks of DRAM */
 #define PHYS_DRAM_1            0x80000000      /* DRAM Bank #1 */
 #define PHYS_DRAM_1_SIZE        0x40000000     /* 1 GB */
index e9531687f39a00700c17190b7c1aa579cbda7506..b2fec8473ba64b9ad58b7b65f90d040ba68d7562 100644 (file)
@@ -6036,11 +6036,6 @@ CONFIG_THUMB2_KERNEL
 CONFIG_THUNDERX
 CONFIG_TI814X
 CONFIG_TI816X
-CONFIG_TI816X_DDR_PLL_796
-CONFIG_TI816X_EVM_DDR2
-CONFIG_TI816X_EVM_DDR3
-CONFIG_TI816X_USE_EMIF0
-CONFIG_TI816X_USE_EMIF1
 CONFIG_TI81XX
 CONFIG_TIMESTAMP
 CONFIG_TIZEN