> -----Original Message----- > From: Lukasz Majewski <lu...@denx.de> > Sent: 2019年5月29日 14:40 > To: Y.b. Lu <yangbo...@nxp.com> > Cc: u-boot@lists.denx.de; Stefano Babic <sba...@denx.de>; Fabio Estevam > <feste...@gmail.com>; dl-uboot-imx <uboot-...@nxp.com>; Albert Aribaud > <albert.u.b...@aribaud.net>; Eddy Petrișor <eddy.petri...@gmail.com>; > Akshay Bhat <akshayb...@timesys.com>; Ken Lin > <ken....@advantech.com.tw>; Heiko Schocher <h...@denx.de>; Christian > Gmeiner <christian.gmei...@gmail.com>; Stefan Roese <s...@denx.de>; Patrick > Bruenn <p.bru...@beckhoff.com>; Troy Kisky > <troy.ki...@boundarydevices.com>; Uri Mashiach > <uri.mashi...@compulab.co.il>; Nikita Kiryanov <nik...@compulab.co.il>; > Otavio Salvador <ota...@ossystems.com.br>; Andreas Geisreiter > <ageisrei...@dh-electronics.de>; Ludwig Zenz <lz...@dh-electronics.de>; Eric > Bénard <e...@eukrea.com>; Peng Fan <peng....@nxp.com>; Jason Liu > <jason.hui....@nxp.com>; Ye Li <ye...@nxp.com>; Adrian Alonso > <adrian.alo...@nxp.com>; Alison Wang <alison.w...@nxp.com>; > thar...@gateworks.com; Ian Ray <ian....@ge.com>; Marcin Niestroj > <m.niest...@grinn-global.com>; Andrej Rosano <and...@inversepath.com>; > Marek Vasut <ma...@denx.de>; Adam Ford <aford...@gmail.com>; Olaf > Mandel <o.man...@menlosystems.com>; Martyn Welch > <martyn.we...@collabora.com>; Ingo Schroeck <open-sou...@samtec.de>; > Boris Brezillon <boris.brezil...@free-electrons.com>; Soeren Moch > <sm...@web.de>; Richard Hu <richard...@technexion.com>; Vanessa > Maegima <vanessa.maeg...@nxp.com>; Max Krummenacher > <max.krummenac...@toradex.com>; Stefan Agner > <stefan.ag...@toradex.com>; Markus Niebel <markus.nie...@tq-group.com>; > Breno Matheus Lima <breno.l...@nxp.com>; Francesco Montefoschi > <francesco.montefos...@udoo.org>; Parthiban Nallathambi > <parthi...@gmail.com>; Albert ARIBAUD <albert.arib...@3adev.fr>; Jagan > Teki <ja...@amarulasolutions.com>; Raffaele RECALCATI > <raffaele.recalc...@bticino.it>; Simone CIANNI <simone.cia...@bticino.it>; > Bhaskar Upadhaya <bhaskar.upadh...@nxp.com>; Vinitha V Pillai > <vinitha.pil...@nxp.com>; Prabhakar Kushwaha > <prabhakar.kushw...@nxp.com>; Rajesh Bhagat <rajesh.bha...@nxp.com>; > Antti Mäentausta <antti.maentau...@ge.com>; Sébastien Szymanski > <sebastien.szyman...@armadeus.com>; Lucile Quirion > <lucile.quir...@savoirfairelinux.com>; Alexey Brodkin > <abrod...@synopsys.com>; Trevor Woerner <tre...@toganlabs.com>; > Anatolij Gustschin <ag...@denx.de>; Denis Zalevskiy > <denis.zalevs...@ge.com>; Fabien Lahoudere > <fabien.lahoud...@collabora.com>; Joe Hershberger > <joe.hershber...@ni.com>; Simon Goldschmidt > <simon.k.r.goldschm...@gmail.com>; James Byrne > <james.by...@origamienergy.com>; Angelo Dureghello <ang...@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...@nxp.com> wrote: > > > Dropped i.MX code which couldn't be reused. > > > > Signed-off-by: Yangbo Lu <yangbo...@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(®s->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(®s->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(®s->cmdarg, cmd->cmdarg); -#if > > defined(CONFIG_FSL_USDHC) > > - esdhc_write32(®s->mixctrl, > > - (esdhc_read32(®s->mixctrl) & 0xFFFFFF80) | (xfertyp & > > 0x7F) > > - | (mmc->ddr_mode ? XFERTYP_DDREN : 0)); > > - esdhc_write32(®s->xfertyp, xfertyp & 0xFFFF0000); > > -#else > > esdhc_write32(®s->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(®s->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(®s->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(®s->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(®s->mixctrl) | MIX_CTRL_DDREN, > > ®s->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(®s->vendorspec, VENDORSPEC_CKEN); > > -#else > > esdhc_clrbits32(®s->sysctl, SYSCTL_CKEN); -#endif > > > > esdhc_clrsetbits32(®s->sysctl, SYSCTL_CLOCK_MASK, clk); > > > > udelay(10000); > > > > -#ifdef CONFIG_FSL_USDHC > > - esdhc_setbits32(®s->vendorspec, VENDORSPEC_PEREN | > > VENDORSPEC_CKEN); -#else > > esdhc_setbits32(®s->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(®s->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, > > ®s->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, ®s->strobe_dllctrl); > > - /* wait 1us to make sure strobe dll status register > > stable */ > > - mdelay(1); > > - val = readl(®s->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(®s->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, ®s->mixctrl); > > - break; > > - case MMC_HS_400: > > - mixctrl |= MIX_CTRL_DDREN | MIX_CTRL_HS400_EN; > > - writel(mixctrl, ®s->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, ®s->mixctrl); > > - break; > > - case UHS_DDR50: > > - case MMC_DDR_52: > > - mixctrl |= MIX_CTRL_DDREN; > > - writel(mixctrl, ®s->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(®s->vendorspec, > > ESDHC_VENDORSPEC_VSELECT); > > - if (!(esdhc_read32(®s->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(®s->vendorspec, > > ESDHC_VENDORSPEC_VSELECT); > > - if (esdhc_read32(®s->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(®s->irqstaten); > > - u32 irqsigen = readl(®s->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(®s->autoc12err); > > - mixctrl = readl(®s->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, ®s->autoc12err); > > - writel(mixctrl, ®s->mixctrl); > > - } > > - > > - /* sdhci_writew(host, SDHCI_TRNS_READ, SDHCI_TRANSFER_MODE); > > */ > > - mixctrl = readl(®s->mixctrl); > > - mixctrl = MIX_CTRL_DTDSEL_READ | (mixctrl & > > ~MIX_CTRL_SDHCI_MASK); > > - writel(mixctrl, ®s->mixctrl); > > - > > - writel(IRQSTATEN_BRR, ®s->irqstaten); > > - writel(IRQSTATEN_BRR, ®s->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, ®s->blkattr); > > - else if (mmc->bus_width == 4) > > - writel(0x7040, ®s->blkattr); > > - } else { > > - writel(0x7040, ®s->blkattr); > > - } > > - > > - /* sdhci_writew(host, SDHCI_TRNS_READ, > > SDHCI_TRANSFER_MODE) */ > > - val = readl(®s->mixctrl); > > - val = MIX_CTRL_DTDSEL_READ | (val & > > ~MIX_CTRL_SDHCI_MASK); > > - writel(val, ®s->mixctrl); > > - > > - /* We are using STD tuning, no need to check return > > value */ > > - mmc_send_tuning(mmc, opcode, NULL); > > - > > - ctrl = readl(®s->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, ®s->irqstaten); > > - writel(irqsigen, ®s->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(®s->scr, ESDHCCTL_PCS); > > + esdhc_setbits32(®s->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(®s->vendorspec, VENDORSPEC_CKEN); > > -#else > > - esdhc_clrbits32(®s->sysctl, SYSCTL_CKEN); > > -#endif > > - } else { > > -#ifdef CONFIG_FSL_USDHC > > - esdhc_setbits32(®s->vendorspec, VENDORSPEC_PEREN | > > - VENDORSPEC_CKEN); > > -#else > > - esdhc_setbits32(®s->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(®s->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(®s->mmcboot, 0x0); > > - /* Reset MIX_CTRL and CLK_TUNE_CTRL_STATUS regs to 0 */ > > - esdhc_write32(®s->mixctrl, 0x0); > > - esdhc_write32(®s->clktunectrlstatus, 0x0); > > - > > - /* Put VEND_SPEC to default value */ > > - if (priv->vs18_enable) > > - esdhc_write32(®s->vendorspec, (VENDORSPEC_INIT | > > - ESDHC_VENDORSPEC_VSELECT)); > > - else > > - esdhc_write32(®s->vendorspec, VENDORSPEC_INIT); > > - > > - /* Disable DLL_CTRL delay line */ > > - esdhc_write32(®s->dllctrl, 0x0); > > -#endif > > - > > -#ifndef ARCH_MXC > > /* Enable cache snooping */ > > - esdhc_write32(®s->scr, 0x00000040); > > -#endif > > + esdhc_write32(®s->esdhcctl, 0x00000040); > > > > -#ifndef CONFIG_FSL_USDHC > > esdhc_setbits32(®s->sysctl, SYSCTL_HCKEN | SYSCTL_IPGEN); -#else > > - esdhc_setbits32(®s->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(®s->prsstat) & PRSSTAT_CINS) && > > --timeout) @@ -1190,20 +754,8 @@ static int fsl_esdhc_init(struct > > fsl_esdhc_priv *priv, esdhc_write32(®s->proctl, PROCTL_INIT | > > PROCTL_D3CD); #endif > > > > -#ifndef CONFIG_FSL_USDHC > > - esdhc_setbits32(®s->sysctl, SYSCTL_PEREN | SYSCTL_HCKEN > > - | SYSCTL_IPGEN | SYSCTL_CKEN); > > - /* Clearing tuning bits in case ROM has set it already */ > > - esdhc_write32(®s->mixctrl, 0); > > - esdhc_write32(®s->autoc12err, 0); > > - esdhc_write32(®s->clktunectrlstatus, 0); > > -#else > > - esdhc_setbits32(®s->vendorspec, VENDORSPEC_PEREN | > > - VENDORSPEC_HCKEN | VENDORSPEC_IPGEN | > > VENDORSPEC_CKEN); -#endif > > - > > - if (priv->vs18_enable) > > - esdhc_setbits32(®s->vendorspec, > > ESDHC_VENDORSPEC_VSELECT); > > + esdhc_setbits32(®s->sysctl, SYSCTL_PEREN | SYSCTL_HCKEN | > > + SYSCTL_IPGEN | SYSCTL_CKEN); > > > > writel(SDHCI_IRQ_EN_BITS, ®s->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, ®s->dllctrl); > > - if (priv->flags & ESDHC_FLAG_USDHC) { > > - if (priv->flags & ESDHC_FLAG_STD_TUNING) { > > - u32 val = readl(®s->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, ®s->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: > lu...@denx.de _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot