From: Jaehoon Chung <jh80.chung@samsung.com>
To: Shawn Lin <shawn.lin@rock-chips.com>,
Ulf Hansson <ulf.hansson@linaro.org>
Cc: linux-mmc@vger.kernel.org, Doug Anderson <dianders@chromium.org>,
linux-rockchip@lists.infradead.org
Subject: Re: [PATCH v2 0/9] Init runtime PM support for dw_mmc
Date: Tue, 18 Oct 2016 09:24:31 +0900 [thread overview]
Message-ID: <488999ee-acf5-47bd-3ebd-707899224749@samsung.com> (raw)
In-Reply-To: <1476240643-5701-1-git-send-email-shawn.lin@rock-chips.com>
Hi Shawn,
On 10/12/2016 11:50 AM, Shawn Lin wrote:
> Hi Jaehoon and Ulf,
>
> This patch is gonna support runtime PM for dw_mmc.
> It could support to disable ciu_clk by default and disable
> biu_clk if the devices are non-removeable, or removeable
> with gpio-base card detect.
>
> Then I remove the system PM since the runtime PM actually
> does the same thing as it. So I help migrate the dw_mmc variant
> drivers to use runtime PM pairs and pm_runtime_force_*. Note
> that I only enable runtime PM for dw_mmc-rockchip as I will
> leave the decision to the owners of the corresponding drivers.
> I just tested it on my RK3288 platform with linux-next to make
> the runtime PM and system PM work fine for my emmc, sd card and
> sdio. But I don't have hardware to help test other variant drivers.
> But in theory it should work fine as I mentioned that the runtime
> PM does the same thing as system PM except for disabling ciu_clk
> aggressively which should not be related to the variant hosts.
>
> As you could see that I just extend the slot-gpio a bit, so the
> ideal way is Ulf could pick them up with Jaehoon's ack. :)
I have tested with my boards..it's working fine..I will apply on this if there is no special issue.
Thanks!
Best Regards,
Jaehoon Chung
>
>
> Changes in v2:
> - use struct device as argument for runtime callback
> - use dw_mci_runtime_* directly
> - use dw_mci_runtime_* directly
> - minor fix since I change the argument for dw_mci_runtime_*
> - use dw_mci_runtime_* directly
> - use dw_mci_runtime_* directly
>
> Shawn Lin (9):
> mmc: dw_mmc: add runtime PM callback
> mmc: dw_mmc-rockchip: add runtime PM support
> mmc: core: expose the capability of gpio card detect
> mmc: dw_mmc: disable biu clk if possible
> mmc: dw_mmc-k3: deploy runtime PM facilities
> mmc: dw_mmc-exynos: deploy runtime PM facilities
> mmc: dw_mmc-pci: deploy runtime PM facilities
> mmc: dw_mmc-pltfm: deploy runtime PM facilities
> mmc: dw_mmc: remove system PM callback
>
> drivers/mmc/core/slot-gpio.c | 8 +++++++
> drivers/mmc/host/dw_mmc-exynos.c | 24 +++++++++-----------
> drivers/mmc/host/dw_mmc-k3.c | 39 ++++++++------------------------
> drivers/mmc/host/dw_mmc-pci.c | 29 ++++++++----------------
> drivers/mmc/host/dw_mmc-pltfm.c | 28 +++++++----------------
> drivers/mmc/host/dw_mmc-rockchip.c | 42 +++++++++++++++++++++++++++++++---
> drivers/mmc/host/dw_mmc.c | 46 +++++++++++++++++++++++++-------------
> drivers/mmc/host/dw_mmc.h | 6 ++---
> include/linux/mmc/slot-gpio.h | 1 +
> 9 files changed, 117 insertions(+), 106 deletions(-)
>
next prev parent reply other threads:[~2016-10-18 0:24 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <CGME20161012024453epcas1p1bdc2c6bd18d864b5535bd2b067b491af@epcas1p1.samsung.com>
2016-10-12 2:50 ` [PATCH v2 0/9] Init runtime PM support for dw_mmc Shawn Lin
2016-10-12 2:50 ` [PATCH v2 1/9] mmc: dw_mmc: add runtime PM callback Shawn Lin
2016-10-12 2:50 ` [PATCH v2 2/9] mmc: dw_mmc-rockchip: add runtime PM support Shawn Lin
2016-10-12 2:50 ` [PATCH v2 3/9] mmc: core: expose the capability of gpio card detect Shawn Lin
2016-10-12 2:50 ` [PATCH v2 4/9] mmc: dw_mmc: disable biu clk if possible Shawn Lin
[not found] ` <1476240643-5701-1-git-send-email-shawn.lin-TNX95d0MmH7DzftRWevZcw@public.gmane.org>
2016-10-12 2:54 ` [PATCH v2 5/9] mmc: dw_mmc-k3: deploy runtime PM facilities Shawn Lin
2016-10-12 2:55 ` [PATCH v2 6/9] mmc: dw_mmc-exynos: " Shawn Lin
2016-10-12 2:56 ` [PATCH v2 7/9] mmc: dw_mmc-pci: " Shawn Lin
2016-10-12 2:56 ` [PATCH v2 8/9] mmc: dw_mmc-pltfm: " Shawn Lin
2016-10-12 2:56 ` [PATCH v2 9/9] mmc: dw_mmc: remove system PM callback Shawn Lin
2016-10-18 0:24 ` Jaehoon Chung [this message]
2016-10-18 1:04 ` [PATCH v2 0/9] Init runtime PM support for dw_mmc Shawn Lin
2016-10-18 8:46 ` Ulf Hansson
2016-10-18 10:45 ` Jaehoon Chung
[not found] ` <17f821c0-46d3-76e7-06dd-fbb415de37eb-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2016-10-18 11:15 ` Ulf Hansson
2016-10-18 11:35 ` Shawn Lin
2016-10-18 13:26 ` Heiko Stübner
2016-10-21 1:56 ` Jaehoon Chung
2016-10-21 2:43 ` Shawn Lin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=488999ee-acf5-47bd-3ebd-707899224749@samsung.com \
--to=jh80.chung@samsung.com \
--cc=dianders@chromium.org \
--cc=linux-mmc@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=shawn.lin@rock-chips.com \
--cc=ulf.hansson@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox