On Tue, Jun 14, 2011 at 3:59 PM, Sourav Poddar <[email protected]> wrote: > From: Anand Gadiyar <[email protected]> > > The OMAP HSMMC driver uses an ocr_mask to determine the > list of voltages supported by the card. It populates this > mask based on the list of voltages supported by the > regulator that supplies the voltage. > > Commit 64be97822b (omap4 hsmmc: Update ocr mask for MMC2 for > regulator to use) passed a fixed ocr_mask from the OMAP4 SDP > board file to limit the voltage to 2.9-3.0 Volts, and updated > the driver to use this mask if provided, instead of using the > regulator's supported voltages. > > However the commit is buggy - the ocr_mask is overridden > by the regulator's capabilities anyway. Fix this. > > (The bug shows up when a system-wide suspend is attempted > on the OMAP4 SDP/Blaze platforms. The eMMC card comes up > at 3V, but drops to 1.65V after the system resumes). > > Signed-off-by: Anand Gadiyar <[email protected]> > Acked-by: Balaji T K <[email protected]> > Tested-by: Kishore Kadiyala <[email protected]> > Signed-off-by: Sourav Poddar <[email protected]> > --- > v2: Rebase to latest kernel(Linux 3.0-rc3) > v1: https://patchwork.kernel.org/patch/576871/ > > drivers/mmc/host/omap_hsmmc.c | 1 - > 1 files changed, 0 insertions(+), 1 deletions(-) > > diff --git a/drivers/mmc/host/omap_hsmmc.c b/drivers/mmc/host/omap_hsmmc.c > index 5b2e215..8707bcd 100644 > --- a/drivers/mmc/host/omap_hsmmc.c > +++ b/drivers/mmc/host/omap_hsmmc.c > @@ -429,7 +429,6 @@ static int omap_hsmmc_reg_get(struct omap_hsmmc_host > *host) > return -EINVAL; > } > } > - mmc_slot(host).ocr_mask = mmc_regulator_get_ocrmask(reg); > > /* Allow an aux regulator */ > reg = regulator_get(host->dev, "vmmc_aux"); > -- Acked-by: Venkatraman S <[email protected]>
Hi Chris, Can you please queue it for the next rc ? Thanks and regards, Venkat. -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
