From: Sherry Sun <[email protected]>

Add inline ECC support for lpddr4 on imx8mp-evk. And add a config which
can enable/disable inline ECC feature for lpddr4 on imx8mp-evk board.

Signed-off-by: Sherry Sun <[email protected]>
Signed-off-by: Peng Fan <[email protected]>
---
 arch/arm/include/asm/arch-imx8m/ddr.h      |  7 +++
 board/freescale/imx8mp_evk/lpddr4_timing.c | 27 +++++++++++
 drivers/ddr/imx/imx8m/Kconfig              |  7 +++
 drivers/ddr/imx/imx8m/ddr_init.c           | 72 ++++++++++++++++++++++++++++++
 4 files changed, 113 insertions(+)

diff --git a/arch/arm/include/asm/arch-imx8m/ddr.h 
b/arch/arm/include/asm/arch-imx8m/ddr.h
index 7a2a2d8edc..04c9c962cf 100644
--- a/arch/arm/include/asm/arch-imx8m/ddr.h
+++ b/arch/arm/include/asm/arch-imx8m/ddr.h
@@ -529,6 +529,8 @@ enum msg_response {
 #define DDRC_SBRWDATA0(X)        (DDRC_IPS_BASE_ADDR(X) + 0xf2c)
 #define DDRC_SBRWDATA1(X)        (DDRC_IPS_BASE_ADDR(X) + 0xf30)
 #define DDRC_PDCH(X)             (DDRC_IPS_BASE_ADDR(X) + 0xf34)
+#define DDRC_SBRSTART0(X)        (DDRC_IPS_BASE_ADDR(X) + 0xf38)
+#define DDRC_SBRRANGE0(X)        (DDRC_IPS_BASE_ADDR(X) + 0xf40)
 
 #define DDRC_FREQ1_DERATEEN(X)         (DDRC_IPS_BASE_ADDR(X) + 0x2020)
 #define DDRC_FREQ1_DERATEINT(X)        (DDRC_IPS_BASE_ADDR(X) + 0x2024)
@@ -708,6 +710,11 @@ int ddr_cfg_phy(struct dram_timing_info *timing_info);
 void load_lpddr4_phy_pie(void);
 void ddrphy_trained_csr_save(struct dram_cfg_param *param, unsigned int num);
 void dram_config_save(struct dram_timing_info *info, unsigned long base);
+void board_dram_ecc_scrub(void);
+void ddrc_inline_ecc_scrub(unsigned int start_address,
+                          unsigned int range_address);
+void ddrc_inline_ecc_scrub_end(unsigned int start_address,
+                              unsigned int range_address);
 
 /* utils function for ddr phy training */
 int wait_ddrphy_training_complete(void);
diff --git a/board/freescale/imx8mp_evk/lpddr4_timing.c 
b/board/freescale/imx8mp_evk/lpddr4_timing.c
index 14542490bc..4e5ebf9424 100644
--- a/board/freescale/imx8mp_evk/lpddr4_timing.c
+++ b/board/freescale/imx8mp_evk/lpddr4_timing.c
@@ -14,6 +14,9 @@ struct dram_cfg_param ddr_ddrc_cfg[] = {
        { 0x3d400020, 0x323 },
        { 0x3d400024, 0x1e84800 },
        { 0x3d400064, 0x7a0118 },
+#ifdef CONFIG_IMX8M_DRAM_INLINE_ECC
+       { 0x3d400070, 0x01027f44 },
+#endif
        { 0x3d4000d0, 0xc00307a3 },
        { 0x3d4000d4, 0xc50000 },
        { 0x3d4000dc, 0xf4003f },
@@ -45,12 +48,21 @@ struct dram_cfg_param ddr_ddrc_cfg[] = {
        { 0x3d4001c4, 0x1 },
        { 0x3d4000f4, 0xc99 },
        { 0x3d400108, 0x9121c1c },
+#ifdef CONFIG_IMX8M_DRAM_INLINE_ECC
+       { 0x3d400200, 0x13 },
+       { 0x3d40020c, 0x13131300 },
+       { 0x3d400210, 0x1f1f },
+       { 0x3d400204, 0x50505 },
+       { 0x3d400214, 0x4040404 },
+       { 0x3d400218, 0x68040404 },
+#else
        { 0x3d400200, 0x16 },
        { 0x3d40020c, 0x0 },
        { 0x3d400210, 0x1f1f },
        { 0x3d400204, 0x80808 },
        { 0x3d400214, 0x7070707 },
        { 0x3d400218, 0x68070707 },
+#endif
        { 0x3d40021c, 0xf08 },
        { 0x3d400250, 0x29001701 },
        { 0x3d400254, 0x2c },
@@ -1845,3 +1857,18 @@ struct dram_timing_info dram_timing = {
        .ddrphy_pie_num = ARRAY_SIZE(ddr_phy_pie),
        .fsp_table = { 4000, 400, 100, },
 };
+
+#ifdef CONFIG_IMX8M_DRAM_INLINE_ECC
+void board_dram_ecc_scrub(void)
+{
+       /* add inline scrb function MPlus spcific */
+       /* scrub 0-1.75G */
+       ddrc_inline_ecc_scrub(0x0, 0x1bffffff);
+       /* scrub 2-3.75G */
+       ddrc_inline_ecc_scrub(0x20000000, 0x3bffffff);
+       /* scrub 4-5.75G */
+       ddrc_inline_ecc_scrub(0x40000000, 0x5bffffff);
+       /* set scruber read range 0-6G */
+       ddrc_inline_ecc_scrub_end(0x0, 0x5fffffff);
+}
+#endif
diff --git a/drivers/ddr/imx/imx8m/Kconfig b/drivers/ddr/imx/imx8m/Kconfig
index 5bf61eb258..a5f5524fbe 100644
--- a/drivers/ddr/imx/imx8m/Kconfig
+++ b/drivers/ddr/imx/imx8m/Kconfig
@@ -29,4 +29,11 @@ config SAVED_DRAM_TIMING_BASE
          info into memory for low power use. OCRAM_S is used for this
          purpose on i.MX8MM.
        default 0x180000
+
+config IMX8M_DRAM_INLINE_ECC
+       bool "imx8mp inline ECC"
+       depends on IMX8MP && IMX8M_LPDDR4
+       help
+         Select this config if you want to use inline ecc feature for
+         imx8mp-evk board.
 endmenu
diff --git a/drivers/ddr/imx/imx8m/ddr_init.c b/drivers/ddr/imx/imx8m/ddr_init.c
index 06b4341b11..f573a778d9 100644
--- a/drivers/ddr/imx/imx8m/ddr_init.c
+++ b/drivers/ddr/imx/imx8m/ddr_init.c
@@ -20,6 +20,76 @@ void ddr_cfg_umctl2(struct dram_cfg_param *ddrc_cfg, int num)
        }
 }
 
+#ifdef CONFIG_IMX8M_DRAM_INLINE_ECC
+void ddrc_inline_ecc_scrub(unsigned int start_address,
+                          unsigned int range_address)
+{
+       unsigned int tmp;
+
+       /* Step1: Enable quasi-dynamic programming */
+       reg32_write(DDRC_SWCTL(0), 0x00000000);
+       /* Step2: Set ECCCFG1.ecc_parity_region_lock to 1 */
+       reg32setbit(DDRC_ECCCFG1(0), 0x4);
+       /* Step3: Block the AXI ports from taking the transaction */
+       reg32_write(DDRC_PCTRL_0(0), 0x0);
+       /* Step4: Set scrub start address */
+       reg32_write(DDRC_SBRSTART0(0), start_address);
+       /* Step5: Set scrub range address */
+       reg32_write(DDRC_SBRRANGE0(0), range_address);
+       /* Step6: Set scrub_mode to write */
+       reg32_write(DDRC_SBRCTL(0), 0x00000014);
+       /* Step7: Set the desired pattern through SBRWDATA0 registers */
+       reg32_write(DDRC_SBRWDATA0(0), 0x55aa55aa);
+       /* Step8: Enable the SBR by programming SBRCTL.scrub_en=1 */
+       reg32setbit(DDRC_SBRCTL(0), 0x0);
+       /* Step9: Poll SBRSTAT.scrub_done=1 */
+       tmp = reg32_read(DDRC_SBRSTAT(0));
+       while (tmp != 0x00000002)
+               tmp = reg32_read(DDRC_SBRSTAT(0)) & 0x2;
+       /* Step10: Poll SBRSTAT.scrub_busy=0 */
+       tmp = reg32_read(DDRC_SBRSTAT(0));
+       while (tmp != 0x0)
+               tmp = reg32_read(DDRC_SBRSTAT(0)) & 0x1;
+       /* Step11: Disable SBR by programming SBRCTL.scrub_en=0 */
+       clrbits_le32(DDRC_SBRCTL(0), 0x1);
+       /* Step12: Prepare for normal scrub operation(Read) and set 
scrub_interval*/
+       reg32_write(DDRC_SBRCTL(0), 0x100);
+       /* Step13: Enable the SBR by programming SBRCTL.scrub_en=1 */
+       reg32_write(DDRC_SBRCTL(0), 0x101);
+       /* Step14: Enable AXI ports by programming */
+       reg32_write(DDRC_PCTRL_0(0), 0x1);
+       /* Step15: Disable quasi-dynamic programming */
+       reg32_write(DDRC_SWCTL(0), 0x00000001);
+}
+
+void ddrc_inline_ecc_scrub_end(unsigned int start_address,
+                              unsigned int range_address)
+{
+       /* Step1: Enable quasi-dynamic programming */
+       reg32_write(DDRC_SWCTL(0), 0x00000000);
+       /* Step2: Block the AXI ports from taking the transaction */
+       reg32_write(DDRC_PCTRL_0(0), 0x0);
+       /* Step3: Set scrub start address */
+       reg32_write(DDRC_SBRSTART0(0), start_address);
+       /* Step4: Set scrub range address */
+       reg32_write(DDRC_SBRRANGE0(0), range_address);
+       /* Step5: Disable SBR by programming SBRCTL.scrub_en=0 */
+       clrbits_le32(DDRC_SBRCTL(0), 0x1);
+       /* Step6: Prepare for normal scrub operation(Read) and set 
scrub_interval */
+       reg32_write(DDRC_SBRCTL(0), 0x100);
+       /* Step7: Enable the SBR by programming SBRCTL.scrub_en=1 */
+       reg32_write(DDRC_SBRCTL(0), 0x101);
+       /* Step8: Enable AXI ports by programming */
+       reg32_write(DDRC_PCTRL_0(0), 0x1);
+       /* Step9: Disable quasi-dynamic programming */
+       reg32_write(DDRC_SWCTL(0), 0x00000001);
+}
+#endif
+
+void __weak board_dram_ecc_scrub(void)
+{
+}
+
 int ddr_init(struct dram_timing_info *dram_timing)
 {
        unsigned int tmp, initial_drate, target_freq;
@@ -168,6 +238,8 @@ int ddr_init(struct dram_timing_info *dram_timing)
        reg32_write(DDRC_PCTRL_0(0), 0x00000001);
        debug("DDRINFO: ddrmix config done\n");
 
+       board_dram_ecc_scrub();
+
        /* save the dram timing config into memory */
        dram_config_save(dram_timing, CONFIG_SAVED_DRAM_TIMING_BASE);
 
-- 
2.16.4

Reply via email to