mbox series

[v2,0/6] mmc: sdhci-acpi: A few cleanups

Message ID 20241101101441.3518612-1-andriy.shevchenko@linux.intel.com
Headers show
Series mmc: sdhci-acpi: A few cleanups | expand

Message

Andy Shevchenko Nov. 1, 2024, 10:11 a.m. UTC
A few almost independent cleanups for the driver because of
new available APIs.

In v2:
- added patch 1 to solve compilation error (LKP)
- split patch 3 out of (previous version of) patch 4 (Christophe)
- added patches 5 and 6

Andy Shevchenko (6):
  mmc: sdhci: Use EXPORT_PM_FN_NS_GPL() for exporting PM functions
  mmc: sdhci-acpi: Switch to SYSTEM_SLEEP_PM_OPS()/RUNTIME_PM_OPS() and
    pm_ptr()
  mmc: sdhci-acpi: Remove not so useful error message
  mmc: sdhci-acpi: Use devm_platform_ioremap_resource()
  mmc: sdhci-acpi: Tidy up ACPI ID table
  mmc: sdhci-acpi: Don't use "proxy" headers

 drivers/mmc/host/sdhci-acpi.c | 92 ++++++++++++++---------------------
 drivers/mmc/host/sdhci.c      | 14 ++----
 drivers/mmc/host/sdhci.h      |  2 -
 3 files changed, 40 insertions(+), 68 deletions(-)

Comments

Ulf Hansson Dec. 2, 2024, 3:24 p.m. UTC | #1
On Fri, 1 Nov 2024 at 11:14, Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
>
> A few almost independent cleanups for the driver because of
> new available APIs.
>
> In v2:
> - added patch 1 to solve compilation error (LKP)
> - split patch 3 out of (previous version of) patch 4 (Christophe)
> - added patches 5 and 6
>
> Andy Shevchenko (6):
>   mmc: sdhci: Use EXPORT_PM_FN_NS_GPL() for exporting PM functions
>   mmc: sdhci-acpi: Switch to SYSTEM_SLEEP_PM_OPS()/RUNTIME_PM_OPS() and
>     pm_ptr()
>   mmc: sdhci-acpi: Remove not so useful error message
>   mmc: sdhci-acpi: Use devm_platform_ioremap_resource()
>   mmc: sdhci-acpi: Tidy up ACPI ID table
>   mmc: sdhci-acpi: Don't use "proxy" headers
>
>  drivers/mmc/host/sdhci-acpi.c | 92 ++++++++++++++---------------------
>  drivers/mmc/host/sdhci.c      | 14 ++----
>  drivers/mmc/host/sdhci.h      |  2 -
>  3 files changed, 40 insertions(+), 68 deletions(-)
>
> --

This looks good to me, but deferring to apply it a few more days to
allow Adrian to share his opinion.

Kind regards
Uffe
Andy Shevchenko Dec. 2, 2024, 3:31 p.m. UTC | #2
On Mon, Dec 02, 2024 at 04:24:00PM +0100, Ulf Hansson wrote:
> On Fri, 1 Nov 2024 at 11:14, Andy Shevchenko
> <andriy.shevchenko@linux.intel.com> wrote:
> >
> > A few almost independent cleanups for the driver because of
> > new available APIs.
> >
> > In v2:
> > - added patch 1 to solve compilation error (LKP)
> > - split patch 3 out of (previous version of) patch 4 (Christophe)
> > - added patches 5 and 6

> This looks good to me, but deferring to apply it a few more days to
> allow Adrian to share his opinion.

Sure, no hurry.
Ulf Hansson Dec. 18, 2024, 3:55 p.m. UTC | #3
On Fri, 1 Nov 2024 at 11:14, Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
>
> A few almost independent cleanups for the driver because of
> new available APIs.
>
> In v2:
> - added patch 1 to solve compilation error (LKP)
> - split patch 3 out of (previous version of) patch 4 (Christophe)
> - added patches 5 and 6
>
> Andy Shevchenko (6):
>   mmc: sdhci: Use EXPORT_PM_FN_NS_GPL() for exporting PM functions
>   mmc: sdhci-acpi: Switch to SYSTEM_SLEEP_PM_OPS()/RUNTIME_PM_OPS() and
>     pm_ptr()
>   mmc: sdhci-acpi: Remove not so useful error message
>   mmc: sdhci-acpi: Use devm_platform_ioremap_resource()
>   mmc: sdhci-acpi: Tidy up ACPI ID table
>   mmc: sdhci-acpi: Don't use "proxy" headers
>
>  drivers/mmc/host/sdhci-acpi.c | 92 ++++++++++++++---------------------
>  drivers/mmc/host/sdhci.c      | 14 ++----
>  drivers/mmc/host/sdhci.h      |  2 -
>  3 files changed, 40 insertions(+), 68 deletions(-)
>

Patch 3 and 4 applied for next, deferring the others.

Kind regards
Uffe