[U-Boot] [v3, 4/5] mmc: fsl_esdhc: drop i.MX code

Y.b. Lu yangbo.lu at nxp.com
Wed May 29 07:34:09 UTC 2019


> -----Original Message-----
> From: Lukasz Majewski <lukma at denx.de>
> Sent: 2019年5月29日 14:40
> To: Y.b. Lu <yangbo.lu at nxp.com>
> Cc: u-boot at lists.denx.de; Stefano Babic <sbabic at denx.de>; Fabio Estevam
> <festevam at gmail.com>; dl-uboot-imx <uboot-imx at nxp.com>; Albert Aribaud
> <albert.u.boot at aribaud.net>; Eddy Petrișor <eddy.petrisor at gmail.com>;
> Akshay Bhat <akshaybhat at timesys.com>; Ken Lin
> <Ken.Lin at advantech.com.tw>; Heiko Schocher <hs at denx.de>; Christian
> Gmeiner <christian.gmeiner at gmail.com>; Stefan Roese <sr at denx.de>; Patrick
> Bruenn <p.bruenn at beckhoff.com>; Troy Kisky
> <troy.kisky at boundarydevices.com>; Uri Mashiach
> <uri.mashiach at compulab.co.il>; Nikita Kiryanov <nikita at compulab.co.il>;
> Otavio Salvador <otavio at ossystems.com.br>; Andreas Geisreiter
> <ageisreiter at dh-electronics.de>; Ludwig Zenz <lzenz at dh-electronics.de>; Eric
> Bénard <eric at eukrea.com>; Peng Fan <peng.fan at nxp.com>; Jason Liu
> <jason.hui.liu at nxp.com>; Ye Li <ye.li at nxp.com>; Adrian Alonso
> <adrian.alonso at nxp.com>; Alison Wang <alison.wang at nxp.com>;
> tharvey at gateworks.com; Ian Ray <ian.ray at ge.com>; Marcin Niestroj
> <m.niestroj at grinn-global.com>; Andrej Rosano <andrej at inversepath.com>;
> Marek Vasut <marex at denx.de>; Adam Ford <aford173 at gmail.com>; Olaf
> Mandel <o.mandel at menlosystems.com>; Martyn Welch
> <martyn.welch at collabora.com>; Ingo Schroeck <open-source at samtec.de>;
> Boris Brezillon <boris.brezillon at free-electrons.com>; Soeren Moch
> <smoch at web.de>; Richard Hu <richard.hu at technexion.com>; Vanessa
> Maegima <vanessa.maegima at nxp.com>; Max Krummenacher
> <max.krummenacher at toradex.com>; Stefan Agner
> <stefan.agner at toradex.com>; Markus Niebel <Markus.Niebel at tq-group.com>;
> Breno Matheus Lima <breno.lima at nxp.com>; Francesco Montefoschi
> <francesco.montefoschi at udoo.org>; Parthiban Nallathambi
> <parthitce at gmail.com>; Albert ARIBAUD <albert.aribaud at 3adev.fr>; Jagan
> Teki <jagan at amarulasolutions.com>; Raffaele RECALCATI
> <raffaele.recalcati at bticino.it>; Simone CIANNI <simone.cianni at bticino.it>;
> Bhaskar Upadhaya <bhaskar.upadhaya at nxp.com>; Vinitha V Pillai
> <vinitha.pillai at nxp.com>; Prabhakar Kushwaha
> <prabhakar.kushwaha at nxp.com>; Rajesh Bhagat <rajesh.bhagat at nxp.com>;
> Antti Mäentausta <antti.maentausta at ge.com>; Sébastien Szymanski
> <sebastien.szymanski at armadeus.com>; Lucile Quirion
> <lucile.quirion at savoirfairelinux.com>; Alexey Brodkin
> <abrodkin at synopsys.com>; Trevor Woerner <trevor at toganlabs.com>;
> Anatolij Gustschin <agust at denx.de>; Denis Zalevskiy
> <denis.zalevskiy at ge.com>; Fabien Lahoudere
> <fabien.lahoudere at collabora.com>; Joe Hershberger
> <joe.hershberger at ni.com>; Simon Goldschmidt
> <simon.k.r.goldschmidt at gmail.com>; James Byrne
> <james.byrne at origamienergy.com>; Angelo Dureghello <angelo at sysam.it>
> Subject: Re: [v3, 4/5] mmc: fsl_esdhc: drop i.MX code
> 
> On Tue, 21 May 2019 08:52:44 +0000
> "Y.b. Lu" <yangbo.lu at nxp.com> wrote:
> 
> > Dropped i.MX code which couldn't be reused.
> >
> > Signed-off-by: Yangbo Lu <yangbo.lu at nxp.com>
> > ---
> > Changes for v2:
> > 	- Added this patch.
> > Changes for v3:
> > 	- Rebased.
> > ---
> >  drivers/mmc/fsl_esdhc.c | 609
> > ++-------------------------------------- include/fsl_esdhc.h     |
> > 57 ---- 2 files changed, 21 insertions(+), 645 deletions(-)
> >
> > diff --git a/drivers/mmc/fsl_esdhc.c b/drivers/mmc/fsl_esdhc.c index
> > 1b7de74a72..3f4f75ae4c 100644
> > --- a/drivers/mmc/fsl_esdhc.c
> > +++ b/drivers/mmc/fsl_esdhc.c
> > @@ -16,14 +16,11 @@
> >  #include <hwconfig.h>
> >  #include <mmc.h>
> >  #include <part.h>
> > -#include <power/regulator.h>
> >  #include <malloc.h>
> >  #include <fsl_esdhc.h>
> >  #include <fdt_support.h>
> >  #include <asm/io.h>
> >  #include <dm.h>
> > -#include <asm-generic/gpio.h>
> > -#include <dm/pinctrl.h>
> >
> >  DECLARE_GLOBAL_DATA_PTR;
> >
> > @@ -33,8 +30,6 @@ DECLARE_GLOBAL_DATA_PTR;
> >  				IRQSTATEN_CIE | IRQSTATEN_DTOE |
> > IRQSTATEN_DCE | \ IRQSTATEN_DEBE | IRQSTATEN_BRR | IRQSTATEN_BWR
> | \
> >  				IRQSTATEN_DINT)
> > -#define MAX_TUNING_LOOP 40
> > -
> >  struct fsl_esdhc {
> >  	uint    dsaddr;		/* SDMA system address
> > register */ uint    blkattr;	/* Block attributes register */
> > @@ -54,37 +49,20 @@ struct fsl_esdhc {
> >  	uint    autoc12err;	/* Auto CMD error status register
> > */ uint    hostcapblt;	/* Host controller capabilities
> > register */ uint    wml;		/* Watermark level register */
> > -	uint    mixctrl;	/* For USDHC */
> > -	char    reserved1[4];	/* reserved */
> > +	char    reserved1[8];	/* reserved */
> >  	uint    fevt;		/* Force event register */
> >  	uint    admaes;		/* ADMA error status register
> > */ uint    adsaddr;	/* ADMA system address register */
> > -	char    reserved2[4];
> > -	uint    dllctrl;
> > -	uint    dllstat;
> > -	uint    clktunectrlstatus;
> > -	char    reserved3[4];
> > -	uint	strobe_dllctrl;
> > -	uint	strobe_dllstat;
> > -	char    reserved4[72];
> > -	uint    vendorspec;
> > -	uint    mmcboot;
> > -	uint    vendorspec2;
> > -	uint    tuning_ctrl;	/* on i.MX6/7/8 */
> > -	char	reserved5[44];
> > +	char    reserved2[160];
> >  	uint    hostver;	/* Host controller version register
> > */
> > -	char    reserved6[4];	/* reserved */
> > +	char    reserved3[4];	/* reserved */
> >  	uint    dmaerraddr;	/* DMA error address register */
> > -	char    reserved7[4];	/* reserved */
> > +	char    reserved4[4];	/* reserved */
> >  	uint    dmaerrattr;	/* DMA error attribute register */
> > -	char    reserved8[4];	/* reserved */
> > +	char    reserved5[4];	/* reserved */
> >  	uint    hostcapblt2;	/* Host controller capabilities
> > register 2 */
> > -	char    reserved9[8];	/* reserved */
> > -	uint    tcr;		/* Tuning control register */
> > -	char    reserved10[28];	/* reserved */
> > -	uint    sddirctl;	/* SD direction control register */
> > -	char    reserved11[712];/* reserved */
> > -	uint    scr;		/* eSDHC control register */
> > +	char    reserved6[756];	/* reserved */
> > +	uint    esdhcctl;	/* eSDHC control register */
> >  };
> >
> >  struct fsl_esdhc_plat {
> > @@ -92,11 +70,6 @@ struct fsl_esdhc_plat {
> >  	struct mmc mmc;
> >  };
> >
> > -struct esdhc_soc_data {
> > -	u32 flags;
> > -	u32 caps;
> > -};
> > -
> >  /**
> >   * struct fsl_esdhc_priv
> >   *
> > @@ -109,13 +82,6 @@ struct esdhc_soc_data {
> >   * @dev: pointer for the device
> >   * @non_removable: 0: removable; 1: non-removable
> >   * @wp_enable: 1: enable checking wp; 0: no check
> > - * @vs18_enable: 1: use 1.8V voltage; 0: use 3.3V
> > - * @flags: ESDHC_FLAG_xx in include/fsl_esdhc.h
> > - * @caps: controller capabilities
> > - * @tuning_step: tuning step setting in tuning_ctrl register
> > - * @start_tuning_tap: the start point for tuning in tuning_ctrl
> > register
> > - * @strobe_dll_delay_target: settings in strobe_dllctrl
> > - * @signal_voltage: indicating the current voltage
> >   * @cd_gpio: gpio for card detection
> >   * @wp_gpio: gpio for write protection
> >   */
> > @@ -124,7 +90,6 @@ struct fsl_esdhc_priv {
> >  	unsigned int sdhc_clk;
> >  	struct clk per_clk;
> >  	unsigned int clock;
> > -	unsigned int mode;
> >  	unsigned int bus_width;
> >  #if !CONFIG_IS_ENABLED(BLK)
> >  	struct mmc *mmc;
> > @@ -132,21 +97,6 @@ struct fsl_esdhc_priv {
> >  	struct udevice *dev;
> >  	int non_removable;
> >  	int wp_enable;
> > -	int vs18_enable;
> > -	u32 flags;
> > -	u32 caps;
> > -	u32 tuning_step;
> > -	u32 tuning_start_tap;
> > -	u32 strobe_dll_delay_target;
> > -	u32 signal_voltage;
> > -#if IS_ENABLED(CONFIG_DM_REGULATOR)
> > -	struct udevice *vqmmc_dev;
> > -	struct udevice *vmmc_dev;
> > -#endif
> > -#ifdef CONFIG_DM_GPIO
> > -	struct gpio_desc cd_gpio;
> > -	struct gpio_desc wp_gpio;
> > -#endif
> 
> I suppose that _all_ 85xx and layerscape/OorIQ boards are not converted to
> DM and they are not using card detect and write protect pins?
> 
> IIRC the 85xx gained recently the support for DM/DTS. Maybe it would be
> worth to re-check if this code would be used?

[Y.b. Lu] All QorIQ platforms were using card detect and write protect pins, whose status could be checked through esdhc_prsstat register.
The cd/wp_gpio definitions were only for i.MX. So dropped such code.

For DM, all QorIQ ARM boards had used DM_MMC. QorIQ PowerPC platforms were in progress to convert to use DM_MMC.

> 
> >  };
> >
> >  /* Return the XFERTYP flags for a given command and data packet */ @@
> > -258,8 +208,7 @@ static int esdhc_setup_data(struct fsl_esdhc_priv
> > *priv, struct mmc *mmc, {
> >  	int timeout;
> >  	struct fsl_esdhc *regs = priv->esdhc_regs; -#if
> > defined(CONFIG_FSL_LAYERSCAPE) || defined(CONFIG_S32V234) || \
> > -	defined(CONFIG_IMX8) || defined(CONFIG_IMX8M)
> > +#if defined(CONFIG_FSL_LAYERSCAPE)
> >  	dma_addr_t addr;
> >  #endif
> >  	uint wml_value;
> > @@ -272,8 +221,7 @@ static int esdhc_setup_data(struct fsl_esdhc_priv
> > *priv, struct mmc *mmc,
> >  		esdhc_clrsetbits32(&regs->wml, WML_RD_WML_MASK, wml_value);
> #ifndef
> > CONFIG_SYS_FSL_ESDHC_USE_PIO -#if defined(CONFIG_FSL_LAYERSCAPE) ||
> > defined(CONFIG_S32V234) || \
> > -	defined(CONFIG_IMX8) || defined(CONFIG_IMX8M)
> > +#if defined(CONFIG_FSL_LAYERSCAPE)
> >  		addr = virt_to_phys((void *)(data->dest));
> >  		if (upper_32_bits(addr))
> >  			printf("Error found for upper 32 bits\n"); @@ -297,20 +245,12
> @@
> > static int esdhc_setup_data(struct fsl_esdhc_priv *priv, struct mmc
> > *mmc, printf("\nThe SD card is locked. Can not write to a locked
> > card.\n\n"); return -ETIMEDOUT;
> >  			}
> > -		} else {
> > -#ifdef CONFIG_DM_GPIO
> > -			if (dm_gpio_is_valid(&priv->wp_gpio) &&
> > dm_gpio_get_value(&priv->wp_gpio)) {
> > -				printf("\nThe SD card is locked. Can
> > not write to a locked card.\n\n");
> > -				return -ETIMEDOUT;
> > -			}
> > -#endif
> >  		}
> >
> >  		esdhc_clrsetbits32(&regs->wml, WML_WR_WML_MASK,
> >  					wml_value << 16);
> >  #ifndef CONFIG_SYS_FSL_ESDHC_USE_PIO
> > -#if defined(CONFIG_FSL_LAYERSCAPE) || defined(CONFIG_S32V234) || \
> > -	defined(CONFIG_IMX8) || defined(CONFIG_IMX8M)
> > +#if defined(CONFIG_FSL_LAYERSCAPE)
> >  		addr = virt_to_phys((void *)(data->src));
> >  		if (upper_32_bits(addr))
> >  			printf("Error found for upper 32 bits\n"); @@ -375,8 +315,7
> @@
> > static void check_and_invalidate_dcache_range
> >  	unsigned end = 0;
> >  	unsigned size = roundup(ARCH_DMA_MINALIGN,
> >  				data->blocks*data->blocksize);
> > -#if defined(CONFIG_FSL_LAYERSCAPE) || defined(CONFIG_S32V234) || \
> > -	defined(CONFIG_IMX8) || defined(CONFIG_IMX8M)
> > +#if defined(CONFIG_FSL_LAYERSCAPE)
> >  	dma_addr_t addr;
> >
> >  	addr = virt_to_phys((void *)(data->dest)); @@ -466,14 +405,7 @@
> > static int esdhc_send_cmd_common(struct fsl_esdhc_priv *priv, struct
> > mmc *mmc,
> >  	/* Send the command */
> >  	esdhc_write32(&regs->cmdarg, cmd->cmdarg); -#if
> > defined(CONFIG_FSL_USDHC)
> > -	esdhc_write32(&regs->mixctrl,
> > -	(esdhc_read32(&regs->mixctrl) & 0xFFFFFF80) | (xfertyp &
> > 0x7F)
> > -			| (mmc->ddr_mode ? XFERTYP_DDREN : 0));
> > -	esdhc_write32(&regs->xfertyp, xfertyp & 0xFFFF0000);
> > -#else
> >  	esdhc_write32(&regs->xfertyp, xfertyp); -#endif
> >
> >  	if ((cmd->cmdidx == MMC_CMD_SEND_TUNING_BLOCK) ||
> >  	    (cmd->cmdidx == MMC_CMD_SEND_TUNING_BLOCK_HS200)) @@
> -500,15
> > +432,6 @@ static int esdhc_send_cmd_common(struct fsl_esdhc_priv
> > *priv, struct mmc *mmc, goto out;
> >  	}
> >
> > -	/* Switch voltage to 1.8V if CMD11 succeeded */
> > -	if (cmd->cmdidx == SD_CMD_SWITCH_UHS18V) {
> > -		esdhc_setbits32(&regs->vendorspec,
> > ESDHC_VENDORSPEC_VSELECT); -
> > -		printf("Run CMD11 1.8V switch\n");
> > -		/* Sleep for 5 ms - max time for card to switch to
> > 1.8V */
> > -		udelay(5000);
> > -	}
> > -
> >  	/* Workaround for ESDHC errata ENGcm03648 */
> >  	if (!data && (cmd->resp_type & MMC_RSP_BUSY)) {
> >  		int timeout = 6000;
> > @@ -596,10 +519,6 @@ out:
> >  			while ((esdhc_read32(&regs->sysctl) &
> > SYSCTL_RSTD)) ;
> >  		}
> > -
> > -		/* If this was CMD11, then notify that power cycle
> > is needed */
> > -		if (cmd->cmdidx == SD_CMD_SWITCH_UHS18V)
> > -			printf("CMD11 to switch to 1.8V mode failed,
> > card requires power cycle.\n"); }
> >
> >  	esdhc_write32(&regs->irqstat, -1);
> > @@ -611,62 +530,32 @@ static void set_sysctl(struct fsl_esdhc_priv
> > *priv, struct mmc *mmc, uint clock) {
> >  	struct fsl_esdhc *regs = priv->esdhc_regs;
> >  	int div = 1;
> > -#ifdef ARCH_MXC
> > -#ifdef CONFIG_MX53
> > -	/* For i.MX53 eSDHCv3, SYSCTL.SDCLKFS may not be set to 0. */
> > -	int pre_div = (regs == (struct fsl_esdhc
> > *)MMC_SDHC3_BASE_ADDR) ? 2 : 1; -#else
> > -	int pre_div = 1;
> > -#endif
> > -#else
> >  	int pre_div = 2;
> > -#endif
> > +	int ddr_pre_div = mmc->ddr_mode ? 2 : 1;
> >  	int sdhc_clk = priv->sdhc_clk;
> >  	uint clk;
> >
> > -	/*
> > -	 * For ddr mode, usdhc need to enable DDR mode first, after
> > select
> > -	 * this DDR mode, usdhc will automatically divide the usdhc
> > clock
> > -	 */
> > -	if (mmc->ddr_mode) {
> > -		writel(readl(&regs->mixctrl) | MIX_CTRL_DDREN,
> > &regs->mixctrl);
> > -		sdhc_clk >>= 1;
> > -	}
> > -
> >  	if (clock < mmc->cfg->f_min)
> >  		clock = mmc->cfg->f_min;
> >
> > -	if (sdhc_clk / 16 > clock) {
> > -		for (; pre_div < 256; pre_div *= 2)
> > -			if ((sdhc_clk / pre_div) <= (clock * 16))
> > -				break;
> > -	} else
> > -		pre_div = 1;
> > +	while (sdhc_clk / (16 * pre_div * ddr_pre_div) > clock &&
> > pre_div < 256)
> > +		pre_div *= 2;
> >
> > -	for (div = 1; div <= 16; div++)
> > -		if ((sdhc_clk / (div * pre_div)) <= clock)
> > -			break;
> > +	while (sdhc_clk / (div * pre_div * ddr_pre_div) > clock &&
> > div < 16)
> > +		div++;
> >
> 
> Have you updated your series to be on top of newest mainline? The above
> code looks like the one, which has been recently reverted as it breaks
> i.MX53 devices.

[Y.b. Lu] The code base was latest when I sent out the patches.

e14d9ca491 (origin/master, origin/HEAD) Merge git://git.denx.de/u-boot-x86
d3d212b624 Merge branch '2019-05-19-master-imports'

Anyway let me rebase and send new version to address all comments.

> 
> 
> >  	pre_div >>= 1;
> >  	div -= 1;
> >
> >  	clk = (pre_div << 8) | (div << 4);
> >
> > -#ifdef CONFIG_FSL_USDHC
> > -	esdhc_clrbits32(&regs->vendorspec, VENDORSPEC_CKEN);
> > -#else
> >  	esdhc_clrbits32(&regs->sysctl, SYSCTL_CKEN); -#endif
> >
> >  	esdhc_clrsetbits32(&regs->sysctl, SYSCTL_CLOCK_MASK, clk);
> >
> >  	udelay(10000);
> >
> > -#ifdef CONFIG_FSL_USDHC
> > -	esdhc_setbits32(&regs->vendorspec, VENDORSPEC_PEREN |
> > VENDORSPEC_CKEN); -#else
> >  	esdhc_setbits32(&regs->sysctl, SYSCTL_PEREN | SYSCTL_CKEN); -#endif
> >
> >  	priv->clock = clock;
> >  }
> > @@ -700,317 +589,20 @@ static void esdhc_clock_control(struct
> > fsl_esdhc_priv *priv, bool enable) }  #endif
> >
> > -#ifdef MMC_SUPPORTS_TUNING
> > -static int esdhc_change_pinstate(struct udevice *dev) -{
> > -	struct fsl_esdhc_priv *priv = dev_get_priv(dev);
> > -	int ret;
> > -
> > -	switch (priv->mode) {
> > -	case UHS_SDR50:
> > -	case UHS_DDR50:
> > -		ret = pinctrl_select_state(dev, "state_100mhz");
> > -		break;
> > -	case UHS_SDR104:
> > -	case MMC_HS_200:
> > -	case MMC_HS_400:
> > -		ret = pinctrl_select_state(dev, "state_200mhz");
> > -		break;
> > -	default:
> > -		ret = pinctrl_select_state(dev, "default");
> > -		break;
> > -	}
> > -
> > -	if (ret)
> > -		printf("%s %d error\n", __func__, priv->mode);
> > -
> > -	return ret;
> > -}
> > -
> > -static void esdhc_reset_tuning(struct mmc *mmc) -{
> > -	struct fsl_esdhc_priv *priv = dev_get_priv(mmc->dev);
> > -	struct fsl_esdhc *regs = priv->esdhc_regs;
> > -
> > -	if (priv->flags & ESDHC_FLAG_USDHC) {
> > -		if (priv->flags & ESDHC_FLAG_STD_TUNING) {
> > -			esdhc_clrbits32(&regs->autoc12err,
> > -					MIX_CTRL_SMPCLK_SEL |
> > -					MIX_CTRL_EXE_TUNE);
> > -		}
> > -	}
> > -}
> > -
> > -static void esdhc_set_strobe_dll(struct mmc *mmc) -{
> > -	struct fsl_esdhc_priv *priv = dev_get_priv(mmc->dev);
> > -	struct fsl_esdhc *regs = priv->esdhc_regs;
> > -	u32 val;
> > -
> > -	if (priv->clock > ESDHC_STROBE_DLL_CLK_FREQ) {
> > -		writel(ESDHC_STROBE_DLL_CTRL_RESET,
> > &regs->strobe_dllctrl); -
> > -		/*
> > -		 * enable strobe dll ctrl and adjust the delay target
> > -		 * for the uSDHC loopback read clock
> > -		 */
> > -		val = ESDHC_STROBE_DLL_CTRL_ENABLE |
> > -			(priv->strobe_dll_delay_target <<
> > -			 ESDHC_STROBE_DLL_CTRL_SLV_DLY_TARGET_SHIFT);
> > -		writel(val, &regs->strobe_dllctrl);
> > -		/* wait 1us to make sure strobe dll status register
> > stable */
> > -		mdelay(1);
> > -		val = readl(&regs->strobe_dllstat);
> > -		if (!(val & ESDHC_STROBE_DLL_STS_REF_LOCK))
> > -			pr_warn("HS400 strobe DLL status REF not
> > lock!\n");
> > -		if (!(val & ESDHC_STROBE_DLL_STS_SLV_LOCK))
> > -			pr_warn("HS400 strobe DLL status SLV not
> > lock!\n");
> > -	}
> > -}
> > -
> > -static int esdhc_set_timing(struct mmc *mmc) -{
> > -	struct fsl_esdhc_priv *priv = dev_get_priv(mmc->dev);
> > -	struct fsl_esdhc *regs = priv->esdhc_regs;
> > -	u32 mixctrl;
> > -
> > -	mixctrl = readl(&regs->mixctrl);
> > -	mixctrl &= ~(MIX_CTRL_DDREN | MIX_CTRL_HS400_EN);
> > -
> > -	switch (mmc->selected_mode) {
> > -	case MMC_LEGACY:
> > -	case SD_LEGACY:
> > -		esdhc_reset_tuning(mmc);
> > -		writel(mixctrl, &regs->mixctrl);
> > -		break;
> > -	case MMC_HS_400:
> > -		mixctrl |= MIX_CTRL_DDREN | MIX_CTRL_HS400_EN;
> > -		writel(mixctrl, &regs->mixctrl);
> > -		esdhc_set_strobe_dll(mmc);
> > -		break;
> > -	case MMC_HS:
> > -	case MMC_HS_52:
> > -	case MMC_HS_200:
> > -	case SD_HS:
> > -	case UHS_SDR12:
> > -	case UHS_SDR25:
> > -	case UHS_SDR50:
> > -	case UHS_SDR104:
> > -		writel(mixctrl, &regs->mixctrl);
> > -		break;
> > -	case UHS_DDR50:
> > -	case MMC_DDR_52:
> > -		mixctrl |= MIX_CTRL_DDREN;
> > -		writel(mixctrl, &regs->mixctrl);
> > -		break;
> > -	default:
> > -		printf("Not supported %d\n", mmc->selected_mode);
> > -		return -EINVAL;
> > -	}
> > -
> > -	priv->mode = mmc->selected_mode;
> > -
> > -	return esdhc_change_pinstate(mmc->dev);
> > -}
> > -
> > -static int esdhc_set_voltage(struct mmc *mmc) -{
> > -	struct fsl_esdhc_priv *priv = dev_get_priv(mmc->dev);
> > -	struct fsl_esdhc *regs = priv->esdhc_regs;
> > -	int ret;
> > -
> > -	priv->signal_voltage = mmc->signal_voltage;
> > -	switch (mmc->signal_voltage) {
> > -	case MMC_SIGNAL_VOLTAGE_330:
> > -		if (priv->vs18_enable)
> > -			return -EIO;
> > -#if CONFIG_IS_ENABLED(DM_REGULATOR)
> > -		if (!IS_ERR_OR_NULL(priv->vqmmc_dev)) {
> > -			ret = regulator_set_value(priv->vqmmc_dev,
> > 3300000);
> > -			if (ret) {
> > -				printf("Setting to 3.3V error");
> > -				return -EIO;
> > -			}
> > -			/* Wait for 5ms */
> > -			mdelay(5);
> > -		}
> > -#endif
> > -
> > -		esdhc_clrbits32(&regs->vendorspec,
> > ESDHC_VENDORSPEC_VSELECT);
> > -		if (!(esdhc_read32(&regs->vendorspec) &
> > -		    ESDHC_VENDORSPEC_VSELECT))
> > -			return 0;
> > -
> > -		return -EAGAIN;
> > -	case MMC_SIGNAL_VOLTAGE_180:
> > -#if CONFIG_IS_ENABLED(DM_REGULATOR)
> > -		if (!IS_ERR_OR_NULL(priv->vqmmc_dev)) {
> > -			ret = regulator_set_value(priv->vqmmc_dev,
> > 1800000);
> > -			if (ret) {
> > -				printf("Setting to 1.8V error");
> > -				return -EIO;
> > -			}
> > -		}
> > -#endif
> > -		esdhc_setbits32(&regs->vendorspec,
> > ESDHC_VENDORSPEC_VSELECT);
> > -		if (esdhc_read32(&regs->vendorspec) &
> > ESDHC_VENDORSPEC_VSELECT)
> > -			return 0;
> > -
> > -		return -EAGAIN;
> > -	case MMC_SIGNAL_VOLTAGE_120:
> > -		return -ENOTSUPP;
> > -	default:
> > -		return 0;
> > -	}
> > -}
> > -
> > -static void esdhc_stop_tuning(struct mmc *mmc) -{
> > -	struct mmc_cmd cmd;
> > -
> > -	cmd.cmdidx = MMC_CMD_STOP_TRANSMISSION;
> > -	cmd.cmdarg = 0;
> > -	cmd.resp_type = MMC_RSP_R1b;
> > -
> > -	dm_mmc_send_cmd(mmc->dev, &cmd, NULL);
> > -}
> > -
> > -static int fsl_esdhc_execute_tuning(struct udevice *dev, uint32_t
> > opcode) -{
> > -	struct fsl_esdhc_plat *plat = dev_get_platdata(dev);
> > -	struct fsl_esdhc_priv *priv = dev_get_priv(dev);
> > -	struct fsl_esdhc *regs = priv->esdhc_regs;
> > -	struct mmc *mmc = &plat->mmc;
> > -	u32 irqstaten = readl(&regs->irqstaten);
> > -	u32 irqsigen = readl(&regs->irqsigen);
> > -	int i, ret = -ETIMEDOUT;
> > -	u32 val, mixctrl;
> > -
> > -	/* clock tuning is not needed for upto 52MHz */
> > -	if (mmc->clock <= 52000000)
> > -		return 0;
> > -
> > -	/* This is readw/writew SDHCI_HOST_CONTROL2 when tuning */
> > -	if (priv->flags & ESDHC_FLAG_STD_TUNING) {
> > -		val = readl(&regs->autoc12err);
> > -		mixctrl = readl(&regs->mixctrl);
> > -		val &= ~MIX_CTRL_SMPCLK_SEL;
> > -		mixctrl &= ~(MIX_CTRL_FBCLK_SEL |
> > MIX_CTRL_AUTO_TUNE_EN); -
> > -		val |= MIX_CTRL_EXE_TUNE;
> > -		mixctrl |= MIX_CTRL_FBCLK_SEL |
> > MIX_CTRL_AUTO_TUNE_EN; -
> > -		writel(val, &regs->autoc12err);
> > -		writel(mixctrl, &regs->mixctrl);
> > -	}
> > -
> > -	/* sdhci_writew(host, SDHCI_TRNS_READ, SDHCI_TRANSFER_MODE);
> > */
> > -	mixctrl = readl(&regs->mixctrl);
> > -	mixctrl = MIX_CTRL_DTDSEL_READ | (mixctrl &
> > ~MIX_CTRL_SDHCI_MASK);
> > -	writel(mixctrl, &regs->mixctrl);
> > -
> > -	writel(IRQSTATEN_BRR, &regs->irqstaten);
> > -	writel(IRQSTATEN_BRR, &regs->irqsigen);
> > -
> > -	/*
> > -	 * Issue opcode repeatedly till Execute Tuning is set to 0
> > or the number
> > -	 * of loops reaches 40 times.
> > -	 */
> > -	for (i = 0; i < MAX_TUNING_LOOP; i++) {
> > -		u32 ctrl;
> > -
> > -		if (opcode == MMC_CMD_SEND_TUNING_BLOCK_HS200) {
> > -			if (mmc->bus_width == 8)
> > -				writel(0x7080, &regs->blkattr);
> > -			else if (mmc->bus_width == 4)
> > -				writel(0x7040, &regs->blkattr);
> > -		} else {
> > -			writel(0x7040, &regs->blkattr);
> > -		}
> > -
> > -		/* sdhci_writew(host, SDHCI_TRNS_READ,
> > SDHCI_TRANSFER_MODE) */
> > -		val = readl(&regs->mixctrl);
> > -		val = MIX_CTRL_DTDSEL_READ | (val &
> > ~MIX_CTRL_SDHCI_MASK);
> > -		writel(val, &regs->mixctrl);
> > -
> > -		/* We are using STD tuning, no need to check return
> > value */
> > -		mmc_send_tuning(mmc, opcode, NULL);
> > -
> > -		ctrl = readl(&regs->autoc12err);
> > -		if ((!(ctrl & MIX_CTRL_EXE_TUNE)) &&
> > -		    (ctrl & MIX_CTRL_SMPCLK_SEL)) {
> > -			/*
> > -			 * need to wait some time, make sure sd/mmc
> > fininsh
> > -			 * send out tuning data, otherwise, the
> > sd/mmc can't
> > -			 * response to any command when the card
> > still out
> > -			 * put the tuning data.
> > -			 */
> > -			mdelay(1);
> > -			ret = 0;
> > -			break;
> > -		}
> > -
> > -		/* Add 1ms delay for SD and eMMC */
> > -		mdelay(1);
> > -	}
> > -
> > -	writel(irqstaten, &regs->irqstaten);
> > -	writel(irqsigen, &regs->irqsigen);
> > -
> > -	esdhc_stop_tuning(mmc);
> > -
> > -	return ret;
> > -}
> > -#endif
> > -
> >  static int esdhc_set_ios_common(struct fsl_esdhc_priv *priv, struct
> > mmc *mmc) {
> >  	struct fsl_esdhc *regs = priv->esdhc_regs;
> > -	int ret __maybe_unused;
> >
> >  #ifdef CONFIG_FSL_ESDHC_USE_PERIPHERAL_CLK
> >  	/* Select to use peripheral clock */
> >  	esdhc_clock_control(priv, false);
> > -	esdhc_setbits32(&regs->scr, ESDHCCTL_PCS);
> > +	esdhc_setbits32(&regs->esdhcctl, ESDHCCTL_PCS);
> >  	esdhc_clock_control(priv, true);
> >  #endif
> >  	/* Set the clock speed */
> >  	if (priv->clock != mmc->clock)
> >  		set_sysctl(priv, mmc, mmc->clock);
> >
> > -#ifdef MMC_SUPPORTS_TUNING
> > -	if (mmc->clk_disable) {
> > -#ifdef CONFIG_FSL_USDHC
> > -		esdhc_clrbits32(&regs->vendorspec, VENDORSPEC_CKEN);
> > -#else
> > -		esdhc_clrbits32(&regs->sysctl, SYSCTL_CKEN);
> > -#endif
> > -	} else {
> > -#ifdef CONFIG_FSL_USDHC
> > -		esdhc_setbits32(&regs->vendorspec, VENDORSPEC_PEREN |
> > -				VENDORSPEC_CKEN);
> > -#else
> > -		esdhc_setbits32(&regs->sysctl, SYSCTL_PEREN |
> > SYSCTL_CKEN); -#endif
> > -	}
> > -
> > -	if (priv->mode != mmc->selected_mode) {
> > -		ret = esdhc_set_timing(mmc);
> > -		if (ret) {
> > -			printf("esdhc_set_timing error %d\n", ret);
> > -			return ret;
> > -		}
> > -	}
> > -
> > -	if (priv->signal_voltage != mmc->signal_voltage) {
> > -		ret = esdhc_set_voltage(mmc);
> > -		if (ret) {
> > -			printf("esdhc_set_voltage error %d\n", ret);
> > -			return ret;
> > -		}
> > -	}
> > -#endif
> > -
> >  	/* Set the bus width */
> >  	esdhc_clrbits32(&regs->proctl, PROCTL_DTW_4 | PROCTL_DTW_8);
> >
> > @@ -1037,34 +629,10 @@ static int esdhc_init_common(struct
> > fsl_esdhc_priv *priv, struct mmc *mmc) return -ETIMEDOUT;
> >  	}
> >
> > -#if defined(CONFIG_FSL_USDHC)
> > -	/* RSTA doesn't reset MMC_BOOT register, so manually reset
> > it */
> > -	esdhc_write32(&regs->mmcboot, 0x0);
> > -	/* Reset MIX_CTRL and CLK_TUNE_CTRL_STATUS regs to 0 */
> > -	esdhc_write32(&regs->mixctrl, 0x0);
> > -	esdhc_write32(&regs->clktunectrlstatus, 0x0);
> > -
> > -	/* Put VEND_SPEC to default value */
> > -	if (priv->vs18_enable)
> > -		esdhc_write32(&regs->vendorspec, (VENDORSPEC_INIT |
> > -			      ESDHC_VENDORSPEC_VSELECT));
> > -	else
> > -		esdhc_write32(&regs->vendorspec, VENDORSPEC_INIT);
> > -
> > -	/* Disable DLL_CTRL delay line */
> > -	esdhc_write32(&regs->dllctrl, 0x0);
> > -#endif
> > -
> > -#ifndef ARCH_MXC
> >  	/* Enable cache snooping */
> > -	esdhc_write32(&regs->scr, 0x00000040);
> > -#endif
> > +	esdhc_write32(&regs->esdhcctl, 0x00000040);
> >
> > -#ifndef CONFIG_FSL_USDHC
> >  	esdhc_setbits32(&regs->sysctl, SYSCTL_HCKEN | SYSCTL_IPGEN); -#else
> > -	esdhc_setbits32(&regs->vendorspec, VENDORSPEC_HCKEN |
> > VENDORSPEC_IPGEN); -#endif
> >
> >  	/* Set the initial clock speed */
> >  	mmc_set_clock(mmc, 400000, MMC_CLK_ENABLE); @@ -1098,10
> +666,6 @@
> > static int esdhc_getcd_common(struct fsl_esdhc_priv *priv) #if
> > CONFIG_IS_ENABLED(DM_MMC)
> >  	if (priv->non_removable)
> >  		return 1;
> > -#ifdef CONFIG_DM_GPIO
> > -	if (dm_gpio_is_valid(&priv->cd_gpio))
> > -		return dm_gpio_get_value(&priv->cd_gpio);
> > -#endif
> >  #endif
> >
> >  	while (!(esdhc_read32(&regs->prsstat) & PRSSTAT_CINS) &&
> > --timeout) @@ -1190,20 +754,8 @@ static int fsl_esdhc_init(struct
> > fsl_esdhc_priv *priv, esdhc_write32(&regs->proctl, PROCTL_INIT |
> > PROCTL_D3CD); #endif
> >
> > -#ifndef CONFIG_FSL_USDHC
> > -	esdhc_setbits32(&regs->sysctl, SYSCTL_PEREN | SYSCTL_HCKEN
> > -				| SYSCTL_IPGEN | SYSCTL_CKEN);
> > -	/* Clearing tuning bits in case ROM has set it already */
> > -	esdhc_write32(&regs->mixctrl, 0);
> > -	esdhc_write32(&regs->autoc12err, 0);
> > -	esdhc_write32(&regs->clktunectrlstatus, 0);
> > -#else
> > -	esdhc_setbits32(&regs->vendorspec, VENDORSPEC_PEREN |
> > -			VENDORSPEC_HCKEN | VENDORSPEC_IPGEN |
> > VENDORSPEC_CKEN); -#endif
> > -
> > -	if (priv->vs18_enable)
> > -		esdhc_setbits32(&regs->vendorspec,
> > ESDHC_VENDORSPEC_VSELECT);
> > +	esdhc_setbits32(&regs->sysctl, SYSCTL_PEREN | SYSCTL_HCKEN |
> > +				       SYSCTL_IPGEN | SYSCTL_CKEN);
> >
> >  	writel(SDHCI_IRQ_EN_BITS, &regs->irqstaten);
> >  	cfg = &plat->cfg;
> > @@ -1279,27 +831,11 @@ static int fsl_esdhc_init(struct fsl_esdhc_priv
> > *priv, cfg->host_caps &= ~MMC_MODE_8BIT;  #endif
> >
> > -	cfg->host_caps |= priv->caps;
> > -
> >  	cfg->f_min = 400000;
> >  	cfg->f_max = min(priv->sdhc_clk, (u32)200000000);
> >
> >  	cfg->b_max = CONFIG_SYS_MMC_MAX_BLK_COUNT;
> >
> > -	writel(0, &regs->dllctrl);
> > -	if (priv->flags & ESDHC_FLAG_USDHC) {
> > -		if (priv->flags & ESDHC_FLAG_STD_TUNING) {
> > -			u32 val = readl(&regs->tuning_ctrl);
> > -
> > -			val |= ESDHC_STD_TUNING_EN;
> > -			val &= ~ESDHC_TUNING_START_TAP_MASK;
> > -			val |= priv->tuning_start_tap;
> > -			val &= ~ESDHC_TUNING_STEP_MASK;
> > -			val |= (priv->tuning_step) <<
> > ESDHC_TUNING_STEP_SHIFT;
> > -			writel(val, &regs->tuning_ctrl);
> > -		}
> > -	}
> > -
> >  	return 0;
> >  }
> >
> > @@ -1314,7 +850,6 @@ static int fsl_esdhc_cfg_to_priv(struct
> > fsl_esdhc_cfg *cfg, priv->bus_width = cfg->max_bus_width;
> >  	priv->sdhc_clk = cfg->sdhc_clk;
> >  	priv->wp_enable  = cfg->wp_enable;
> > -	priv->vs18_enable  = cfg->vs18_enable;
> >
> >  	return 0;
> >  };
> > @@ -1449,22 +984,11 @@ void fdt_fixup_esdhc(void *blob, bd_t *bd)
> >
> >  #if CONFIG_IS_ENABLED(DM_MMC)
> >  #include <asm/arch/clock.h>
> > -__weak void init_clk_usdhc(u32 index) -{ -}
> > -
> >  static int fsl_esdhc_probe(struct udevice *dev)  {
> >  	struct mmc_uclass_priv *upriv = dev_get_uclass_priv(dev);
> >  	struct fsl_esdhc_plat *plat = dev_get_platdata(dev);
> >  	struct fsl_esdhc_priv *priv = dev_get_priv(dev);
> > -	const void *fdt = gd->fdt_blob;
> > -	int node = dev_of_offset(dev);
> > -	struct esdhc_soc_data *data =
> > -		(struct esdhc_soc_data *)dev_get_driver_data(dev);
> > -#if CONFIG_IS_ENABLED(DM_REGULATOR)
> > -	struct udevice *vqmmc_dev;
> > -#endif
> >  	fdt_addr_t addr;
> >  	unsigned int val;
> >  	struct mmc *mmc;
> > @@ -1476,11 +1000,6 @@ static int fsl_esdhc_probe(struct udevice *dev)
> >
> >  	priv->esdhc_regs = (struct fsl_esdhc *)addr;
> >  	priv->dev = dev;
> > -	priv->mode = -1;
> > -	if (data) {
> > -		priv->flags = data->flags;
> > -		priv->caps = data->caps;
> > -	}
> >
> >  	val = dev_read_u32_default(dev, "bus-width", -1);
> >  	if (val == 8)
> > @@ -1490,81 +1009,13 @@ static int fsl_esdhc_probe(struct udevice
> > *dev) else
> >  		priv->bus_width = 1;
> >
> > -	val = fdtdec_get_int(fdt, node, "fsl,tuning-step", 1);
> > -	priv->tuning_step = val;
> > -	val = fdtdec_get_int(fdt, node, "fsl,tuning-start-tap",
> > -			     ESDHC_TUNING_START_TAP_DEFAULT);
> > -	priv->tuning_start_tap = val;
> > -	val = fdtdec_get_int(fdt, node,
> > "fsl,strobe-dll-delay-target",
> > -
> > ESDHC_STROBE_DLL_CTRL_SLV_DLY_TARGET_DEFAULT);
> > -	priv->strobe_dll_delay_target = val;
> > -
> >  	if (dev_read_bool(dev, "non-removable")) {
> >  		priv->non_removable = 1;
> >  	 } else {
> >  		priv->non_removable = 0;
> > -#ifdef CONFIG_DM_GPIO
> > -		gpio_request_by_name(dev, "cd-gpios", 0,
> > &priv->cd_gpio,
> > -				     GPIOD_IS_IN);
> > -#endif
> >  	}
> >
> > -	if (dev_read_prop(dev, "fsl,wp-controller", NULL)) {
> > -		priv->wp_enable = 1;
> > -	} else {
> > -		priv->wp_enable = 0;
> > -#ifdef CONFIG_DM_GPIO
> > -		gpio_request_by_name(dev, "wp-gpios", 0,
> > &priv->wp_gpio,
> > -				   GPIOD_IS_IN);
> > -#endif
> > -	}
> > -
> > -	priv->vs18_enable = 0;
> > -
> > -#if CONFIG_IS_ENABLED(DM_REGULATOR)
> > -	/*
> > -	 * If emmc I/O has a fixed voltage at 1.8V, this must be
> > provided,
> > -	 * otherwise, emmc will work abnormally.
> > -	 */
> > -	ret = device_get_supply_regulator(dev, "vqmmc-supply",
> > &vqmmc_dev);
> > -	if (ret) {
> > -		dev_dbg(dev, "no vqmmc-supply\n");
> > -	} else {
> > -		ret = regulator_set_enable(vqmmc_dev, true);
> > -		if (ret) {
> > -			dev_err(dev, "fail to enable
> > vqmmc-supply\n");
> > -			return ret;
> > -		}
> > -
> > -		if (regulator_get_value(vqmmc_dev) == 1800000)
> > -			priv->vs18_enable = 1;
> > -	}
> > -#endif
> > -
> > -	if (fdt_get_property(fdt, node, "no-1-8-v", NULL))
> > -		priv->caps &= ~(UHS_CAPS | MMC_MODE_HS200 |
> > MMC_MODE_HS400); -
> > -	/*
> > -	 * TODO:
> > -	 * Because lack of clk driver, if SDHC clk is not enabled,
> > -	 * need to enable it first before this driver is invoked.
> > -	 *
> > -	 * we use MXC_ESDHC_CLK to get clk freq.
> > -	 * If one would like to make this function work,
> > -	 * the aliases should be provided in dts as this:
> > -	 *
> > -	 *  aliases {
> > -	 *	mmc0 = &usdhc1;
> > -	 *	mmc1 = &usdhc2;
> > -	 *	mmc2 = &usdhc3;
> > -	 *	mmc3 = &usdhc4;
> > -	 *	};
> > -	 * Then if your board only supports mmc2 and mmc3, but we can
> > -	 * correctly get the seq as 2 and 3, then let mxc_get_clock
> > -	 * work as expected.
> > -	 */
> > -
> > -	init_clk_usdhc(dev->seq);
> > +	priv->wp_enable = 1;
> >
> >  	if (IS_ENABLED(CONFIG_CLK)) {
> >  		/* Assigned clock already set clock */ @@ -1631,28 +1082,10 @@
> > static const struct dm_mmc_ops fsl_esdhc_ops
> > = { .get_cd		= fsl_esdhc_get_cd,
> >  	.send_cmd	= fsl_esdhc_send_cmd,
> >  	.set_ios	= fsl_esdhc_set_ios,
> > -#ifdef MMC_SUPPORTS_TUNING
> > -	.execute_tuning	= fsl_esdhc_execute_tuning,
> > -#endif
> >  };
> >  #endif
> >
> > -static struct esdhc_soc_data usdhc_imx7d_data = {
> > -	.flags = ESDHC_FLAG_USDHC | ESDHC_FLAG_STD_TUNING
> > -			| ESDHC_FLAG_HAVE_CAP1 | ESDHC_FLAG_HS200
> > -			| ESDHC_FLAG_HS400,
> > -	.caps = UHS_CAPS | MMC_MODE_HS200 | MMC_MODE_DDR_52MHz |
> > -		MMC_MODE_HS_52MHz | MMC_MODE_HS,
> > -};
> > -
> >  static const struct udevice_id fsl_esdhc_ids[] = {
> > -	{ .compatible = "fsl,imx53-esdhc", },
> > -	{ .compatible = "fsl,imx6ul-usdhc", },
> > -	{ .compatible = "fsl,imx6sx-usdhc", },
> > -	{ .compatible = "fsl,imx6sl-usdhc", },
> > -	{ .compatible = "fsl,imx6q-usdhc", },
> > -	{ .compatible = "fsl,imx7d-usdhc", .data =
> > (ulong)&usdhc_imx7d_data,},
> > -	{ .compatible = "fsl,imx7ulp-usdhc", },
> >  	{ .compatible = "fsl,esdhc", },
> >  	{ /* sentinel */ }
> >  };
> > diff --git a/include/fsl_esdhc.h b/include/fsl_esdhc.h index
> > 8dbd5249a7..7d7e946ab3 100644
> > --- a/include/fsl_esdhc.h
> > +++ b/include/fsl_esdhc.h
> > @@ -9,7 +9,6 @@
> >  #ifndef  __FSL_ESDHC_H__
> >  #define	__FSL_ESDHC_H__
> >
> > -#include <linux/bitops.h>
> >  #include <linux/errno.h>
> >  #include <asm/byteorder.h>
> >
> > @@ -25,22 +24,14 @@
> >  #define SYSCTL_INITA		0x08000000
> >  #define SYSCTL_TIMEOUT_MASK	0x000f0000
> >  #define SYSCTL_CLOCK_MASK	0x0000fff0
> > -#if !defined(CONFIG_FSL_USDHC)
> >  #define SYSCTL_CKEN		0x00000008
> >  #define SYSCTL_PEREN		0x00000004
> >  #define SYSCTL_HCKEN		0x00000002
> >  #define SYSCTL_IPGEN		0x00000001
> > -#endif
> >  #define SYSCTL_RSTA		0x01000000
> >  #define SYSCTL_RSTC		0x02000000
> >  #define SYSCTL_RSTD		0x04000000
> >
> > -#define VENDORSPEC_CKEN		0x00004000
> > -#define VENDORSPEC_PEREN	0x00002000
> > -#define VENDORSPEC_HCKEN	0x00001000
> > -#define VENDORSPEC_IPGEN	0x00000800
> > -#define VENDORSPEC_INIT		0x20007809
> > -
> >  #define IRQSTAT			0x0002e030
> >  #define IRQSTAT_DMAE		(0x10000000)
> >  #define IRQSTAT_AC12E		(0x01000000)
> > @@ -172,54 +163,6 @@
> >  #define ESDHC_HOSTCAPBLT_DMAS	0x00400000
> >  #define ESDHC_HOSTCAPBLT_HSS	0x00200000
> >
> > -#define ESDHC_VENDORSPEC_VSELECT 0x00000002 /* Use 1.8V */
> > -
> > -/* Imported from Linux Kernel drivers/mmc/host/sdhci-esdhc-imx.c */
> > -#define	MIX_CTRL_DDREN		BIT(3)
> > -#define MIX_CTRL_DTDSEL_READ	BIT(4)
> > -#define	MIX_CTRL_AC23EN		BIT(7)
> > -#define	MIX_CTRL_EXE_TUNE	BIT(22)
> > -#define	MIX_CTRL_SMPCLK_SEL	BIT(23)
> > -#define	MIX_CTRL_AUTO_TUNE_EN	BIT(24)
> > -#define	MIX_CTRL_FBCLK_SEL	BIT(25)
> > -#define	MIX_CTRL_HS400_EN	BIT(26)
> > -#define	MIX_CTRL_HS400_ES	BIT(27)
> > -/* Bits 3 and 6 are not SDHCI standard definitions */
> > -#define	MIX_CTRL_SDHCI_MASK	0xb7
> > -/* Tuning bits */
> > -#define	MIX_CTRL_TUNING_MASK	0x03c00000
> > -
> > -/* strobe dll register */
> > -#define ESDHC_STROBE_DLL_CTRL		0x70
> > -#define ESDHC_STROBE_DLL_CTRL_ENABLE	BIT(0)
> > -#define ESDHC_STROBE_DLL_CTRL_RESET	BIT(1)
> > -#define ESDHC_STROBE_DLL_CTRL_SLV_DLY_TARGET_DEFAULT	0x7
> > -#define ESDHC_STROBE_DLL_CTRL_SLV_DLY_TARGET_SHIFT	3
> > -
> > -#define ESDHC_STROBE_DLL_STATUS		0x74
> > -#define ESDHC_STROBE_DLL_STS_REF_LOCK	BIT(1)
> > -#define ESDHC_STROBE_DLL_STS_SLV_LOCK	0x1
> > -#define ESDHC_STROBE_DLL_CLK_FREQ	100000000
> > -
> > -#define ESDHC_STD_TUNING_EN             BIT(24)
> > -/* NOTE: the minimum valid tuning start tap for mx6sl is 1 */
> > -#define ESDHC_TUNING_START_TAP_DEFAULT	0x1
> > -#define ESDHC_TUNING_START_TAP_MASK	0xff
> > -#define ESDHC_TUNING_STEP_MASK		0x00070000
> > -#define ESDHC_TUNING_STEP_SHIFT		16
> > -
> > -#define	ESDHC_FLAG_MULTIBLK_NO_INT	BIT(1)
> > -#define	ESDHC_FLAG_ENGCM07207		BIT(2)
> > -#define	ESDHC_FLAG_USDHC		BIT(3)
> > -#define	ESDHC_FLAG_MAN_TUNING		BIT(4)
> > -#define	ESDHC_FLAG_STD_TUNING		BIT(5)
> > -#define	ESDHC_FLAG_HAVE_CAP1		BIT(6)
> > -#define	ESDHC_FLAG_ERR004536		BIT(7)
> > -#define	ESDHC_FLAG_HS200		BIT(8)
> > -#define	ESDHC_FLAG_HS400		BIT(9)
> > -#define	ESDHC_FLAG_ERR010450		BIT(10)
> > -#define	ESDHC_FLAG_HS400_ES		BIT(11)
> > -
> >  struct fsl_esdhc_cfg {
> >  	phys_addr_t esdhc_base;
> >  	u32	sdhc_clk;
> 
> 
> 
> 
> Best regards,
> 
> Lukasz Majewski
> 
> --
> 
> DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email:
> lukma at denx.de


More information about the U-Boot mailing list