On Mon, Oct 24, 2011 at 10:05 AM, Ulf Hansson <ulf.hansson@xxxxxxxxxxxxxx> wrote: > The hole idea with this PM patch is to make sure the vcore regulator and the > clock are disabled in runtime_suspend to be able to save a huge amount of > current in "idle" mode. > > Disabling the vcore regulator will sooner or later (depending on your > regulator tree) mean that that power to the controller is actually cut, > which then means that all registers will be "cleared" including the MCIPWR. I guess since this means writing power-off to bits 1 and 0 of of MCIPower meaning "power off" it should not be done unless the MMC core has first issued mmci_set_ios() with MMC_POWER_OFF set in ios->power_mode. Which means mmc_power_off() must have been called. That in turn is called by mmc_power_save_host() or mmc_suspend_host(). mmc_power_save_host() is in turn called by mmc_runtime_suspend() in drivers/mmc/bus.c! So if that is called *before* the driver-local runtime suspend calls, you're actually on the safe side, and you can proceed to cut the power. So these need to be zero before we enter runtime suspend. This could then be assured by: if (readl(host->base + MMCIPOWER) & 3) return -EBUSY; If this has already been done we are on the green and can runtime suspend the controller I guess? So what this code > + /* Save registers for POWER, CLOCK and IRQMASK0 */ > + host->irqmask0_reg = readl(host->base + MMCIMASK0); > + host->pwr_reg = readl(host->base + MMCIPOWER); > + host->clk_reg = readl(host->base + MMCICLOCK); In the end does is save the contents of the other 30 bits. (MCI_OD, MCI_ROD, and the ST-specific bits) So can we first: - Assure that mmc_power_save_host() has already been done (MMCI_POWER_OFF) - Make this variable a prerequiste for cutting the power in runtime_pm? Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
