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