Kevin Hilman <khilman@xxxxxxxxxxxxxxxxxxx> writes: > This ensures that each HSMMC block is reset so it will not interfere > with retention or OFF-mode. > Sigh... I still didn't get this right for all the cases of MMC built-in, module or disabled. Stay tuned for v3. Kevin > --- > arch/arm/mach-omap2/hsmmc.c | 73 ++++++++++++++++++++++++++++++++++++++++++- > 1 files changed, 72 insertions(+), 1 deletions(-) > > diff --git a/arch/arm/mach-omap2/hsmmc.c b/arch/arm/mach-omap2/hsmmc.c > index 7334d86..00fe354 100644 > --- a/arch/arm/mach-omap2/hsmmc.c > +++ b/arch/arm/mach-omap2/hsmmc.c > @@ -15,7 +15,9 @@ > #include <linux/platform_device.h> > #include <linux/interrupt.h> > #include <linux/delay.h> > +#include <linux/clk.h> > #include <linux/i2c/twl4030.h> > + > #include <mach/hardware.h> > #include <mach/mmc.h> > #include <mach/board.h> > @@ -46,6 +48,74 @@ > #define OMAP2_CONTROL_PBIAS_PWRDNZ (1 << 1) > #define OMAP2_CONTROL_PBIAS_SCTRL (1 << 2) > > +#define MMCHS1 (L4_34XX_BASE + 0x9C000) > +#define MMCHS2 (L4_34XX_BASE + 0xB4000) > +#define MMCHS3 (L4_34XX_BASE + 0xAD000) > +#define MAX_MMC 3 > +#define MMCHS_SYSCONFIG 0x0010 > +#define MMCHS_SYSCONFIG_SWRESET (1 << 1) > +#define MMCHS_SYSSTATUS 0x0014 > +#define MMCHS_SYSSTATUS_RESETDONE (1 << 0) > + > +static struct platform_device dummy_pdev = { > + .dev = { > + .bus = &platform_bus_type, > + }, > +}; > + > +/** > + * hsmmc_reset() - Full reset of each HS-MMC controller > + * > + * Ensure that each MMC controller is fully reset. Controllers > + * left in an unknown state (by bootloaer) may prevent retention > + * or OFF-mode. > + * > + * In order for reset to work, interface, functional and debounce > + * clocks must be enabled. The debounce clock comes from func_32k_clk > + * and is not under SW controle, so we only enable i- and f-clocks. > + **/ > +static void __init hsmmc_reset(void) > +{ > + u32 i, base[MAX_MMC] = {MMCHS1, MMCHS2, MMCHS3}; > + > + for (i = 0; i < MAX_MMC; i++) { > + u32 v; > + struct clk *iclk, *fclk; > + struct device *dev = &dummy_pdev.dev; > + > + dummy_pdev.id = i + 1; > + iclk = clk_get(dev, "mmchs_ick"); > + if (iclk && clk_enable(iclk)) > + iclk = NULL; > + > + fclk = clk_get(dev, "mmchs_fck"); > + if (fclk && clk_enable(fclk)) > + fclk = NULL; > + > + if (!iclk || !fclk) { > + printk(KERN_WARNING > + "%s: Unable to enable clocks for MMC%d, " > + "cannot reset.\n", __func__, i); > + break; > + } > + > + omap_writel(MMCHS_SYSCONFIG_SWRESET, base[i] + MMCHS_SYSCONFIG); > + v = omap_readl(base[i] + MMCHS_SYSSTATUS); > + while (!(omap_readl(base[i] + MMCHS_SYSSTATUS) & > + MMCHS_SYSSTATUS_RESETDONE)) > + cpu_relax(); > + > + if (fclk) { > + clk_disable(fclk); > + clk_put(fclk); > + } > + if (iclk) { > + clk_disable(iclk); > + clk_put(iclk); > + } > + } > +} > + > static int hsmmc_card_detect(int irq) > { > return twl4030_get_gpio_datain(irq - TWL4030_GPIO_IRQ_BASE); > @@ -282,6 +352,7 @@ static struct omap_mmc_platform_data hsmmc_data = { > > void __init hsmmc_init(void) > { > + hsmmc_reset(); > omap2_init_mmc(&hsmmc_data); > } > > @@ -289,7 +360,7 @@ void __init hsmmc_init(void) > > void __init hsmmc_init(void) > { > - > + hsmmc_reset(); > } > > #endif > -- > 1.6.0 -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html