Re: [PATCH] mmc: enable runtime PM by default

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sun, Jul 17, 2011 at 6:39 PM, Daniel Drake <dsd@xxxxxxxxxx> wrote:
> Now that we have improved the runtime power management powerup/powerdown
> code, we believe that MMC_CAP_POWER_OFF_CARD is no longer necessary:
> runtime PM should now work everywhere.

Let's also add:

The only hard evidence for introducing MMC_CAP_POWER_OFF_CARD was the
Marvell sd8686 wifi chip, which was believed to require external gpio
manipulation which wasn't supported by some boards.

After further investigation it was realized (and confirmed by Marvell
folks) that sd8686 requirements can be fulfilled by changing the reset
sequence itself, even if no external gpio is manipulated.

For further information, see the following thread:
http://www.mail-archive.com/linux-mmc@xxxxxxxxxxxxxxx/msg04289.html

> Enable this trivially for a release or two. If no problems are reported,
> we will follow up with a more extensive patch to remove this flag
> altogether. If problems are reported, we can look at whitelist/blacklist
> possibilities as before.

This is exactly how I planed to take this too, thanks.

Acked-by: Ohad Ben-Cohen <ohad@xxxxxxxxxx>
--
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


[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux