Hi Dan,
On Sun, Jul 17 2011, Daniel Drake wrote:
> At
> http://www.mail-archive.com/linux-mmc@vger.kernel.org/msg08371.html
> we found and fixed a bug where mmc's runtime power management functions
> were not being called. We have now also made improvements to the SDIO
> powerup routine which could possibly mask this kind of issue in future.
>
> Add debug messages to the runtime PM hooks so that it is easy to verify
> if and when runtime PM is happening.
>
> Signed-off-by: Daniel Drake <d...@laptop.org>
> ---
> drivers/mmc/core/core.c | 8 ++++++++
> 1 files changed, 8 insertions(+), 0 deletions(-)
>
> For linux-3.1.
>
> diff --git a/drivers/mmc/core/core.c b/drivers/mmc/core/core.c
> index 0a04886..ad0c55a 100644
> --- a/drivers/mmc/core/core.c
> +++ b/drivers/mmc/core/core.c
> @@ -1664,6 +1664,10 @@ int mmc_power_save_host(struct mmc_host *host)
> {
> int ret = 0;
>
> +#ifdef CONFIG_MMC_DEBUG
> + pr_info("%s: %s: powering down\n", mmc_hostname(host), __func__);
> +#endif
> +
Did you consider just:
pr_debug("%s: %s: powering down\n", mmc_hostname(host), __func__);
which gets you the same CONFIG_MMC_DEBUG semantics? (The messages will
be at the debug loglevel rather than info, but that sounds right to me.)
Thanks,
- Chris.
--
Chris Ball <c...@laptop.org> <http://printf.net/>
One Laptop Per Child
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
