This patch complements ed919b0 "mmc: sdio: fix runtime PM anomalies by
introducing MMC_CAP_POWER_OFF_CARD" by declaring MMC_CAP_POWER_OFF_CARD
on the ZOOM's wl1271 mmc slot.
This is required in order not to break runtime PM support for the wl1271
sdio driver.
Signed-off-by: Ohad Ben-Cohen <[email protected]>
---
Quick summary:
After adding SDIO runtime PM support, we realized there are some
board/host/card setups that are incapable of powering off the card
after boot. For details, see:
http://thread.gmane.org/gmane.linux.kernel.mmc/4342/focus=4579
As a result, we added MMC_CAP_POWER_OFF_CARD which should be explicitly set
by setups that _do_ support powering off the card.
This ensures we don't break existing functionality: SDIO core will enable
runtime PM for cards only if that cap is set.
As a result, the card will be powered down after boot, and will only
be powered up again when a driver is loaded (and then it's up to the
driver whether power will be kept or not).
To complement that fix (which was just merged upstream), we need this
patch too in 2.6.37, otherwise wl1271_sdio will break.
Pandora/Beagle wl12xx users: you need a similar patch as well, as this one
only takes care of ZOOM (I don't have those other setups and preferred not to
send a patch without testing).
arch/arm/mach-omap2/board-zoom-peripherals.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/arch/arm/mach-omap2/board-zoom-peripherals.c b/arch/arm/mach-omap2/board-zoom-peripherals.c
index 86c9b21..9db9203 100644
--- a/arch/arm/mach-omap2/board-zoom-peripherals.c
+++ b/arch/arm/mach-omap2/board-zoom-peripherals.c
@@ -216,7 +216,7 @@ static struct omap2_hsmmc_info mmc[] __initdata = {
{
.name = "wl1271",
.mmc = 3,
- .caps = MMC_CAP_4_BIT_DATA,
+ .caps = MMC_CAP_4_BIT_DATA | MMC_CAP_POWER_OFF_CARD,
.gpio_wp = -EINVAL,
.gpio_cd = -EINVAL,
.nonremovable = true,
--
1.7.0.4
Hi,
On Sat, Nov 27, 2010 at 01:59:24AM +0200, Ohad Ben-Cohen wrote:
> This patch complements ed919b0 "mmc: sdio: fix runtime PM anomalies by
> introducing MMC_CAP_POWER_OFF_CARD" by declaring MMC_CAP_POWER_OFF_CARD
> on the ZOOM's wl1271 mmc slot.
>
> This is required in order not to break runtime PM support for the wl1271
> sdio driver.
>
> Signed-off-by: Ohad Ben-Cohen <[email protected]>
Looks good. Tony, feel free to take this for .37 through your tree.
Signed-off-by: Chris Ball <[email protected]>
Thanks,
--
Chris Ball <[email protected]> <http://printf.net/>
One Laptop Per Child
* Chris Ball <[email protected]> [101127 18:29]:
> Hi,
>
> On Sat, Nov 27, 2010 at 01:59:24AM +0200, Ohad Ben-Cohen wrote:
> > This patch complements ed919b0 "mmc: sdio: fix runtime PM anomalies by
> > introducing MMC_CAP_POWER_OFF_CARD" by declaring MMC_CAP_POWER_OFF_CARD
> > on the ZOOM's wl1271 mmc slot.
> >
> > This is required in order not to break runtime PM support for the wl1271
> > sdio driver.
> >
> > Signed-off-by: Ohad Ben-Cohen <[email protected]>
>
> Looks good. Tony, feel free to take this for .37 through your tree.
>
> Signed-off-by: Chris Ball <[email protected]>
OK will merge thanks.
Tony