On 08/10/2016 09:55 PM, Shengzhou Liu wrote:
> This patch adds basic support for Freescale MMDC(Multi Mode DDR Controller).
> Currently MMDC is integrated on ARMv8 LS1012A SoC for DDR3L, there will be a
> update to this driver to support more flexible configuration if new features
> (DDR4, multiple controllers/chip selections, etc) are implimented in future.
>
> Signed-off-by: Shengzhou Liu <shengzhou....@nxp.com>
> ---
>  Makefile                                          |   1 +
>  arch/arm/include/asm/arch-fsl-layerscape/config.h |   2 +
>  drivers/ddr/fsl/Makefile                          |   1 +
>  drivers/ddr/fsl/fsl_mmdc.c                        | 152 
> ++++++++++++++++++++++
>  include/fsl_mmdc.h                                |  83 +++++-------
>  5 files changed, 192 insertions(+), 47 deletions(-)
>  create mode 100644 drivers/ddr/fsl/fsl_mmdc.c
>
> diff --git a/Makefile b/Makefile
> index 99cc8cf..1bf6c6a 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -647,6 +647,7 @@ libs-y += drivers/power/ \
>  libs-y += drivers/spi/
>  libs-$(CONFIG_FMAN_ENET) += drivers/net/fm/
>  libs-$(CONFIG_SYS_FSL_DDR) += drivers/ddr/fsl/
> +libs-$(CONFIG_SYS_FSL_MMDC) += drivers/ddr/fsl/
>  libs-$(CONFIG_ALTERA_SDRAM) += drivers/ddr/altera/
>  libs-y += drivers/serial/
>  libs-y += drivers/usb/dwc3/
> diff --git a/arch/arm/include/asm/arch-fsl-layerscape/config.h 
> b/arch/arm/include/asm/arch-fsl-layerscape/config.h
> index b0ad4b4..478b7ab 100644
> --- a/arch/arm/include/asm/arch-fsl-layerscape/config.h
> +++ b/arch/arm/include/asm/arch-fsl-layerscape/config.h
> @@ -18,6 +18,8 @@
>  #ifndef CONFIG_LS1012A
>  #define CONFIG_SYS_FSL_DDR           /* Freescale DDR driver */
>  #define CONFIG_SYS_FSL_DDR_VER               FSL_DDR_VER_5_0
> +#else
> +#define CONFIG_SYS_FSL_MMDC          /* Freescale MMDC driver */
>  #endif

Let's revert this logic, shall we?

>
>  /*
> diff --git a/drivers/ddr/fsl/Makefile b/drivers/ddr/fsl/Makefile
> index 01ea862..00dea42 100644
> --- a/drivers/ddr/fsl/Makefile
> +++ b/drivers/ddr/fsl/Makefile
> @@ -33,3 +33,4 @@ obj-$(CONFIG_SYS_FSL_DDRC_GEN3)     += mpc85xx_ddr_gen3.o
>  obj-$(CONFIG_SYS_FSL_DDR_86XX)               += mpc86xx_ddr.o
>  obj-$(CONFIG_SYS_FSL_DDRC_ARM_GEN3)  += arm_ddr_gen3.o
>  obj-$(CONFIG_SYS_FSL_DDRC_GEN4) += fsl_ddr_gen4.o
> +obj-$(CONFIG_SYS_FSL_MMDC) += fsl_mmdc.o
> diff --git a/drivers/ddr/fsl/fsl_mmdc.c b/drivers/ddr/fsl/fsl_mmdc.c
> new file mode 100644
> index 0000000..eb36cea
> --- /dev/null
> +++ b/drivers/ddr/fsl/fsl_mmdc.c
> @@ -0,0 +1,152 @@
> +/*
> + * Copyright 2016 Freescale Semiconductor, Inc.

I think we should use NXP now.

> + *
> + * SPDX-License-Identifier:     GPL-2.0+
> + */
> +
> +/*
> + * Generic driver for Freescale MMDC(Multi Mode DDR Controller).
> + */
> +
> +#include <common.h>
> +#include <fsl_mmdc.h>
> +#include <asm/io.h>
> +
> +static void set_wait_for_bits_clear(void *ptr, u32 value, u32 bits)
> +{
> +     int timeout = 1000;
> +
> +     out_be32(ptr, value);
> +
> +     while (in_be32(ptr) & bits) {
> +             udelay(100);
> +             timeout--;
> +     }
> +     if (timeout <= 0)
> +             printf("Error: %p wait for clear timeout.\n", ptr);
> +}
> +
> +void mmdc_init(void)
> +{
> +     struct mmdc_regs *mmdc = (struct mmdc_regs *)CONFIG_SYS_FSL_DDR_ADDR;
> +     unsigned int tmp;
> +
> +     /* 1. set configuration request */
> +     out_be32(&mmdc->mdscr, MDSCR_ENABLE_CON_REQ);
> +
> +     /* 2. configure the desired timing parameters */
> +     out_be32(&mmdc->mdotc,  CONFIG_MMDC_MDOTC);
> +     out_be32(&mmdc->mdcfg0, CONFIG_MMDC_MDCFG0);
> +     out_be32(&mmdc->mdcfg1, CONFIG_MMDC_MDCFG1);
> +     out_be32(&mmdc->mdcfg2, CONFIG_MMDC_MDCFG2);
> +
> +     /* 3. configure DDR type and other miscellaneous parameters */
> +     out_be32(&mmdc->mdmisc, CONFIG_MMDC_MDMISC);
> +     out_be32(&mmdc->mpmur0, MMDC_MPMUR0_FRC_MSR);
> +     out_be32(&mmdc->mdrwd,  CONFIG_MMDC_MDRWD);
> +     out_be32(&mmdc->mpodtctrl, CONFIG_MMDC_MPODTCTRL);
> +
> +     /* 4. configure the required delay while leaving reset */
> +     out_be32(&mmdc->mdor,  CONFIG_MMDC_MDOR);
> +
> +     /* 5. configure DDR physical parameters */
> +     /* set row/column address width, burst length, data bus width */
> +     tmp = CONFIG_MMDC_MDCTL & ~(MDCTL_SDE0 | MDCTL_SDE1);
> +     out_be32(&mmdc->mdctl, tmp);
> +     /* configure address space partition */
> +     out_be32(&mmdc->mdasp, CONFIG_MMDC_MDASP);
> +
> +     /* 6. perform a ZQ calibration - not needed here, doing in #8b */
> +
> +     /* 7. enable MMDC with the desired chip select */
> +#if (CONFIG_CHIP_SELECTS_PER_CTRL == 1)
> +             out_be32(&mmdc->mdctl, tmp | MDCTL_SDE0);
> +#elif (CONFIG_CHIP_SELECTS_PER_CTRL == 2)
> +             out_be32(&mmdc->mdctl, tmp | MDCTL_SDE0 | MDCTL_SDE1);
> +#endif
> +
> +     /* 8a. dram init sequence: update MRs for ZQ, ODT, PRE, etc */
> +     out_be32(&mmdc->mdscr,  CMD_ADDR_LSB_MR_ADDR(8) | MDSCR_ENABLE_CON_REQ |
> +                             CMD_LOAD_MODE_REG | CMD_BANK_ADDR_2);
> +
> +     out_be32(&mmdc->mdscr,  CMD_ADDR_LSB_MR_ADDR(0) | MDSCR_ENABLE_CON_REQ |
> +                             CMD_LOAD_MODE_REG | CMD_BANK_ADDR_3);
> +
> +     out_be32(&mmdc->mdscr,  CMD_ADDR_LSB_MR_ADDR(4) | MDSCR_ENABLE_CON_REQ |
> +                             CMD_LOAD_MODE_REG | CMD_BANK_ADDR_1);
> +
> +     out_be32(&mmdc->mdscr,  CMD_ADDR_MSB_MR_OP(0x19) |
> +                             CMD_ADDR_LSB_MR_ADDR(0x30) |
> +                             MDSCR_ENABLE_CON_REQ |
> +                             CMD_LOAD_MODE_REG | CMD_BANK_ADDR_0);
> +
> +     /* 8b. ZQ calibration */
> +     out_be32(&mmdc->mdscr,  CMD_ADDR_MSB_MR_OP(0x4) | MDSCR_ENABLE_CON_REQ |
> +                             CMD_ZQ_CALIBRATION | CMD_BANK_ADDR_0);
> +
> +     set_wait_for_bits_clear(&mmdc->mpzqhwctrl, CONFIG_MMDC_MPZQHWCTRL,
> +                             MPZQHWCTRL_ZQ_HW_FORCE);
> +
> +     /* 9a. calibrations now, wr lvl */
> +     out_be32(&mmdc->mdscr,  CMD_ADDR_LSB_MR_ADDR(0x84) |
> +                             MDSCR_ENABLE_CON_REQ |
> +                             CMD_LOAD_MODE_REG | CMD_BANK_ADDR_1);
> +
> +     out_be32(&mmdc->mdscr,  MDSCR_ENABLE_CON_REQ | MDSCR_WL_EN |
> +                             CMD_NORMAL);
> +
> +     set_wait_for_bits_clear(&mmdc->mpwlgcr, MPWLGCR_HW_WL_EN,
> +                             MPWLGCR_HW_WL_EN);
> +
> +     mdelay(1);
> +
> +     out_be32(&mmdc->mdscr,  CMD_ADDR_LSB_MR_ADDR(4) | MDSCR_ENABLE_CON_REQ |
> +                             CMD_LOAD_MODE_REG | CMD_BANK_ADDR_1);
> +     out_be32(&mmdc->mdscr, MDSCR_ENABLE_CON_REQ);
> +
> +     mdelay(1);
> +
> +     /* 9b. read DQS gating calibration */
> +     out_be32(&mmdc->mdscr,  CMD_ADDR_MSB_MR_OP(4) | MDSCR_ENABLE_CON_REQ |
> +                             CMD_PRECHARGE_BANK_OPEN | CMD_BANK_ADDR_0);
> +
> +     out_be32(&mmdc->mdscr,  CMD_ADDR_LSB_MR_ADDR(4) | MDSCR_ENABLE_CON_REQ |
> +                             CMD_LOAD_MODE_REG | CMD_BANK_ADDR_3);
> +
> +     out_be32(&mmdc->mppdcmpr2, MPPDCMPR2_MPR_COMPARE_EN);
> +     /* set absolute read delay offset for bytes */
> +     out_be32(&mmdc->mprddlctl, CONFIG_MMDC_MPRDDLCTL);
> +
> +     set_wait_for_bits_clear(&mmdc->mpdgctrl0,
> +                             AUTO_RD_DQS_GATING_CALIBRATION_EN,
> +                             AUTO_RD_DQS_GATING_CALIBRATION_EN);
> +
> +     out_be32(&mmdc->mdscr, (MDSCR_ENABLE_CON_REQ | CMD_LOAD_MODE_REG |
> +                             CMD_BANK_ADDR_3));
> +
> +     /* 9c. read calibration */
> +     out_be32(&mmdc->mdscr,  CMD_ADDR_MSB_MR_OP(4) | MDSCR_ENABLE_CON_REQ |
> +                             CMD_PRECHARGE_BANK_OPEN | CMD_BANK_ADDR_0);
> +     out_be32(&mmdc->mdscr,  CMD_ADDR_LSB_MR_ADDR(4) | MDSCR_ENABLE_CON_REQ |
> +                             CMD_LOAD_MODE_REG | CMD_BANK_ADDR_3);
> +     out_be32(&mmdc->mppdcmpr2,  MPPDCMPR2_MPR_COMPARE_EN);
> +     set_wait_for_bits_clear(&mmdc->mprddlhwctl,
> +                             MPRDDLHWCTL_AUTO_RD_CALIBRATION_EN,
> +                             MPRDDLHWCTL_AUTO_RD_CALIBRATION_EN);
> +
> +     out_be32(&mmdc->mdscr, (MDSCR_ENABLE_CON_REQ | CMD_LOAD_MODE_REG |
> +                             CMD_BANK_ADDR_3));
> +
> +     /* 10. configure power-down, self-refresh entry, exit parameters */
> +     out_be32(&mmdc->mdpdc, CONFIG_MMDC_MDPDC);
> +     out_be32(&mmdc->mapsr, MMDC_MAPSR_PWR_SAV_CTRL_STAT);
> +
> +     /* 11. ZQ config again? do nothing here */
> +
> +     /* 12. refresh scheme */
> +     set_wait_for_bits_clear(&mmdc->mdref, CONFIG_MMDC_MDREF,
> +                             MDREF_START_REFRESH);
> +
> +     /* 13. disable CON_REQ */
> +     out_be32(&mmdc->mdscr, MDSCR_DISABLE_CFG_REQ);
> +}
> diff --git a/include/fsl_mmdc.h b/include/fsl_mmdc.h
> index a939d89..a10e40c 100644
> --- a/include/fsl_mmdc.h
> +++ b/include/fsl_mmdc.h
> @@ -7,63 +7,39 @@
>  #ifndef FSL_MMDC_H
>  #define FSL_MMDC_H
>
> -#define CONFIG_SYS_MMDC_CORE_ODT_TIMING              0x12554000
> -#define CONFIG_SYS_MMDC_CORE_TIMING_CFG_0    0xbabf7954
> -#define CONFIG_SYS_MMDC_CORE_TIMING_CFG_1    0xff328f64
> -#define CONFIG_SYS_MMDC_CORE_TIMING_CFG_2    0x01ff00db
> -
> -#define CONFIG_SYS_MMDC_CORE_MISC            0x00001680
> -#define CONFIG_SYS_MMDC_PHY_MEASURE_UNIT     0x00000800
> -#define CONFIG_SYS_MMDC_CORE_RDWR_CMD_DELAY  0x00002000
> -#define CONFIG_SYS_MMDC_PHY_ODT_CTRL         0x0000022a
> -
> -#define CONFIG_SYS_MMDC_CORE_OUT_OF_RESET_DELAY      0x00bf1023
> -
> -#define CONFIG_SYS_MMDC_CORE_ADDR_PARTITION  0x0000007f
> -
> -#define CONFIG_SYS_MMDC_PHY_ZQ_HW_CTRL               0xa1390003
> -
> -#define      FORCE_ZQ_AUTO_CALIBRATION               (0x1 << 16)
> -
> -/* PHY Write Leveling Configuration and Error Status (MPWLGCR) */
> -#define WR_LVL_HW_EN                         0x00000001
> +/* PHY Write Leveling Configuration and Error Status Register (MPWLGCR) */
> +#define MPWLGCR_HW_WL_EN             (1 << 0)
>
>  /* PHY Pre-defined Compare and CA delay-line Configuration (MPPDCMPR2) */
> -#define MPR_COMPARE_EN                               0x00000001
> +#define MPPDCMPR2_MPR_COMPARE_EN     (1 << 0)
>
> -#define CONFIG_SYS_MMDC_PHY_RD_DLY_LINES_CFG 0x40404040
>
>  /* MMDC PHY Read DQS gating control register 0 (MPDGCTRL0) */
> -#define AUTO_RD_DQS_GATING_CALIBRATION_EN    0x10000000
> +#define AUTO_RD_DQS_GATING_CALIBRATION_EN    (1 << 28)
>
>  /* MMDC PHY Read Delay HW Calibration Control Register (MPRDDLHWCTL) */
> -#define AUTO_RD_CALIBRATION_EN                       0x00000010
> -
> -#define CONFIG_SYS_MMDC_CORE_PWR_DOWN_CTRL   0x00030035
> +#define MPRDDLHWCTL_AUTO_RD_CALIBRATION_EN   (1 << 4)
>
> -#define CONFIG_SYS_MMDC_CORE_PWR_SAV_CTRL_STAT       0x00001067
> +/* MMDC Core Power Saving Control and Status Register (MMDC_MAPSR) */
> +#define MMDC_MAPSR_PWR_SAV_CTRL_STAT 0x00001067
>
> -#define CONFIG_SYS_MMDC_CORE_REFRESH_CTL     0x0f3c8000
> -
> -#define START_REFRESH                                0x00000001
> +/* MMDC Core Refresh Control Register (MMDC_MDREF) */
> +#define MDREF_START_REFRESH  (1 << 0)
>
>  /* MMDC Core Special Command Register (MDSCR) */
> -#define CMD_ADDR_MSB_MR_OP(x)   (x << 24)
> -
> -#define  CMD_ADDR_LSB_MR_ADDR(x)    (x << 16)
> -
> -#define DISABLE_CFG_REQ              0x0
> -#define CONFIGURATION_REQ    (0x1  << 15)
> -#define WL_EN                        (0x1  << 9)
> -
> -#define      CMD_NORMAL              (0x0 << 4)
> -#define      CMD_PRECHARGE           (0x1 << 4)
> -#define      CMD_AUTO_REFRESH        (0x2 << 4)
> -#define      CMD_LOAD_MODE_REG       (0x3 << 4)
> -#define      CMD_ZQ_CALIBRATION      (0x4 << 4)
> -#define      CMD_PRECHARGE_BANK_OPEN (0x5 << 4)
> -#define      CMD_MRR                 (0x6 << 4)
> -
> +#define CMD_ADDR_MSB_MR_OP(x)        (x << 24)
> +#define CMD_ADDR_LSB_MR_ADDR(x) (x << 16)
> +#define MDSCR_DISABLE_CFG_REQ   (0 << 15)
> +#define MDSCR_ENABLE_CON_REQ (1 << 15)
> +#define MDSCR_CON_ACK                (1 << 14)
> +#define MDSCR_WL_EN          (1 << 9)
> +#define      CMD_NORMAL              (0 << 4)
> +#define      CMD_PRECHARGE           (1 << 4)
> +#define      CMD_AUTO_REFRESH        (2 << 4)
> +#define      CMD_LOAD_MODE_REG       (3 << 4)
> +#define      CMD_ZQ_CALIBRATION      (4 << 4)
> +#define      CMD_PRECHARGE_BANK_OPEN (5 << 4)
> +#define      CMD_MRR                 (6 << 4)
>  #define CMD_BANK_ADDR_0              0x0
>  #define CMD_BANK_ADDR_1              0x1
>  #define CMD_BANK_ADDR_2              0x2

Removing those macros causes compiling error for ls1012afrdm_qspi 
ls1012aqds_qspi ls1012ardb_qspi. Please rearrange the change.

York

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to