Re: [PATCH v3 00/17] pinctrl: intel: Use NOIRQ PM helper

[Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx>]

Hi Andy,

On Mon, Oct 30, 2023 at 02:07:17PM +0200, Andy Shevchenko wrote:
> Intel pin control drivers use NOIRQ variant of the PM callbacks.
> To make them smaller and less error prone against different
> kernel configurations (with possible defined but not used variables)
> switch to use NOIRQ PM helper.
> 
> Changelog v3:
> - dropped applied patches
> - dropped MediaTek patch for now
> - split Intel patch to the series using suggection by Paul
> 
> Changelog v2:
> - rewritten commit message in patch 1 (Rafael)
> - converted non-Intel pin control drivers as well
> - added couple of kinda related patches to use pm_ptr()
> 
> Andy Shevchenko (17):
>   pinctrl: intel: Provide Intel pin control wide PM ops structure
>   pinctrl: alderlake: Switch to use Intel pin control PM ops
>   pinctrl: broxton: Switch to use Intel pin control PM ops
>   pinctrl: cannonlake: Switch to use Intel pin control PM ops
>   pinctrl: cedarfork: Switch to use Intel pin control PM ops
>   pinctrl: denverton: Switch to use Intel pin control PM ops
>   pinctrl: elkhartlake: Switch to use Intel pin control PM ops
>   pinctrl: emmitsburg: Switch to use Intel pin control PM ops
>   pinctrl: geminilake: Switch to use Intel pin control PM ops
>   pinctrl: icelake: Switch to use Intel pin control PM ops
>   pinctrl: jasperlake: Switch to use Intel pin control PM ops
>   pinctrl: lakefield: Switch to use Intel pin control PM ops
>   pinctrl: lewisburg: Switch to use Intel pin control PM ops
>   pinctrl: meteorlake: Switch to use Intel pin control PM ops
>   pinctrl: sunrisepoint: Switch to use Intel pin control PM ops
>   pinctrl: tigerlake: Switch to use Intel pin control PM ops
>   pinctrl: intel: Make PM ops functions static
> 
>  drivers/pinctrl/intel/pinctrl-alderlake.c    |  5 ++---
>  drivers/pinctrl/intel/pinctrl-broxton.c      |  5 ++---
>  drivers/pinctrl/intel/pinctrl-cannonlake.c   |  5 ++---
>  drivers/pinctrl/intel/pinctrl-cedarfork.c    |  5 ++---
>  drivers/pinctrl/intel/pinctrl-denverton.c    |  5 ++---
>  drivers/pinctrl/intel/pinctrl-elkhartlake.c  |  5 ++---
>  drivers/pinctrl/intel/pinctrl-emmitsburg.c   |  5 ++---
>  drivers/pinctrl/intel/pinctrl-geminilake.c   |  5 ++---
>  drivers/pinctrl/intel/pinctrl-icelake.c      |  5 ++---
>  drivers/pinctrl/intel/pinctrl-intel.c        | 14 +++++++-------
>  drivers/pinctrl/intel/pinctrl-intel.h        | 11 +----------
>  drivers/pinctrl/intel/pinctrl-jasperlake.c   |  5 ++---
>  drivers/pinctrl/intel/pinctrl-lakefield.c    |  5 ++---
>  drivers/pinctrl/intel/pinctrl-lewisburg.c    |  5 ++---
>  drivers/pinctrl/intel/pinctrl-meteorlake.c   |  5 ++---
>  drivers/pinctrl/intel/pinctrl-sunrisepoint.c |  5 ++---
>  drivers/pinctrl/intel/pinctrl-tigerlake.c    |  5 ++---
>  17 files changed, 38 insertions(+), 62 deletions(-)

Nice! For the series,

Acked-by: Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx>