From: Jaehoon Chung <jh80.chung@samsung.com>
To: Ulf Hansson <ulf.hansson@linaro.org>,
linux-mmc <linux-mmc@vger.kernel.org>,
Adrian Hunter <adrian.hunter@intel.com>
Cc: Arnd Bergmann <arnd@arndb.de>
Subject: Re: [PATCH 00/11] mmc: sdhci: Cleanup assignment/definition of PM ops
Date: Thu, 28 Jul 2016 19:27:25 +0900 [thread overview]
Message-ID: <5799DE0D.1070205@samsung.com> (raw)
In-Reply-To: <CAPDyKFraYeLZ25Z4FoEcmrz5eYcyXHjtF17q7mAs2N-SCfE+Sw@mail.gmail.com>
Hi Ulf,
On 07/27/2016 09:50 PM, Ulf Hansson wrote:
> On 27 July 2016 at 13:31, Ulf Hansson <ulf.hansson@linaro.org> wrote:
>> The code in various sdhci drivers can be cleaned up in regards how PM ops is
>> being defined and assigned. This series attempts to clean up this code.
>>
>> In the end we want system PM sleep callbacks to be defined within #ifdef
>> CONFIG_PM_SLEEP and not as currently within CONFIG_PM.
It seems that dwmmc controller can also apply the similar cleanup patches.
After checking, i will send the patches..
Best Regards,
Jaehoon Chung
>>
>> Ulf Hansson (11):
>> mmc: sdhci-pci-core: Simplify code by using SET_SYSTEM_SLEEP_PM_OPS
>> mmc: sdhci-acpi: Simplify code by using SET_SYSTEM_SLEEP_PM_OPS
>> mmc: sdhci-of-esdhc: Simplify code by using SIMPLE_DEV_PM_OPS
>> mmc: sdhci-pxav3: Remove non needed #ifdef CONFIG_PM for dev_pm_ops
>> mmc: sdhci-s3c: Remove non needed #ifdef CONFIG_PM for dev_pm_ops
>> mmc: sdhci-sirf: Remove non needed #ifdef CONFIG_PM* for dev_pm_ops
>> mmc: sdhci-esdhc-imx: Assign system PM ops within #ifdef
>> CONFIG_PM_SLEEP
>> mmc: sdhci-esdhc-imx: Use common sdhci_suspend|resume_host()
>> mmc: sdhci-pltfm: Make sdhci_pltfm_suspend|resume() static
>> mmc: sdhci-pltfm: Convert to use the SET_SYSTEM_SLEEP_PM_OPS
>> mmc: sdhci-pltfm: Drop define for SDHCI_PLTFM_PMOPS
>>
>> drivers/mmc/host/sdhci-acpi.c | 8 +-------
>> drivers/mmc/host/sdhci-bcm-kona.c | 2 +-
>> drivers/mmc/host/sdhci-cns3xxx.c | 2 +-
>> drivers/mmc/host/sdhci-dove.c | 2 +-
>> drivers/mmc/host/sdhci-esdhc-imx.c | 10 +++++++---
>> drivers/mmc/host/sdhci-iproc.c | 2 +-
>> drivers/mmc/host/sdhci-of-esdhc.c | 16 ++++++----------
>> drivers/mmc/host/sdhci-of-hlwd.c | 2 +-
>> drivers/mmc/host/sdhci-pci-core.c | 16 +++++-----------
>> drivers/mmc/host/sdhci-pltfm.c | 13 +++++--------
>> drivers/mmc/host/sdhci-pltfm.h | 7 -------
>> drivers/mmc/host/sdhci-pxav2.c | 2 +-
>> drivers/mmc/host/sdhci-pxav3.c | 9 +--------
>> drivers/mmc/host/sdhci-s3c.c | 9 +--------
>> drivers/mmc/host/sdhci-sirf.c | 4 +---
>> drivers/mmc/host/sdhci-tegra.c | 2 +-
>> drivers/mmc/host/sdhci_f_sdh30.c | 2 +-
>> 17 files changed, 35 insertions(+), 73 deletions(-)
>>
>> --
>> 1.9.1
>>
>
> I decided to queue up this for next, as most of these are rather
> trivial changes and to get it tested in next.
>
> Feel free to add acks/reviewed-by, or tell me to drop these changes!
>
> Kind regards
> Uffe
> --
> To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
>
>
prev parent reply other threads:[~2016-07-28 10:27 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-07-27 11:31 [PATCH 00/11] mmc: sdhci: Cleanup assignment/definition of PM ops Ulf Hansson
2016-07-27 11:31 ` [PATCH 01/11] mmc: sdhci-pci-core: Simplify code by using SET_SYSTEM_SLEEP_PM_OPS Ulf Hansson
2016-07-27 11:31 ` [PATCH 02/11] mmc: sdhci-acpi: " Ulf Hansson
2016-07-27 11:31 ` [PATCH 03/11] mmc: sdhci-of-esdhc: Simplify code by using SIMPLE_DEV_PM_OPS Ulf Hansson
2016-07-27 11:32 ` [PATCH 04/11] mmc: sdhci-pxav3: Remove non needed #ifdef CONFIG_PM for dev_pm_ops Ulf Hansson
2016-07-27 11:32 ` [PATCH 05/11] mmc: sdhci-s3c: " Ulf Hansson
2016-07-27 11:32 ` [PATCH 06/11] mmc: sdhci-sirf: Remove non needed #ifdef CONFIG_PM* " Ulf Hansson
2016-07-27 11:32 ` [PATCH 07/11] mmc: sdhci-esdhc-imx: Assign system PM ops within #ifdef CONFIG_PM_SLEEP Ulf Hansson
2016-07-29 8:38 ` Dong Aisheng
2016-07-27 11:32 ` [PATCH 08/11] mmc: sdhci-esdhc-imx: Use common sdhci_suspend|resume_host() Ulf Hansson
2016-07-29 8:39 ` Dong Aisheng
2016-07-27 11:32 ` [PATCH 09/11] mmc: sdhci-pltfm: Make sdhci_pltfm_suspend|resume() static Ulf Hansson
2016-07-27 11:32 ` [PATCH 10/11] mmc: sdhci-pltfm: Convert to use the SET_SYSTEM_SLEEP_PM_OPS Ulf Hansson
2016-07-27 11:32 ` [PATCH 11/11] mmc: sdhci-pltfm: Drop define for SDHCI_PLTFM_PMOPS Ulf Hansson
2016-07-27 12:50 ` [PATCH 00/11] mmc: sdhci: Cleanup assignment/definition of PM ops Ulf Hansson
2016-07-28 10:27 ` Jaehoon Chung [this message]
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=5799DE0D.1070205@samsung.com \
--to=jh80.chung@samsung.com \
--cc=adrian.hunter@intel.com \
--cc=arnd@arndb.de \
--cc=linux-mmc@vger.kernel.org \
--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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.