public inbox for linux-rockchip@lists.infradead.org
 help / color / mirror / Atom feed
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(-)
> 


  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