Re: [PATCH v1 0/5] pinctrl: baytrail: Clean up (part 3)

[Hans de Goede <hdegoede@redhat.com>]

Hi Andy,

On 12-12-2019 11:25, Andy Shevchenko wrote:
> This is a part 3 of clean up pin control driver for Intel Baytrail.
> After this applied the driver will use all available data structures
> from pinctrl-intel.h header.
> 
> Note, that patch which exposes common pin control data structrure for drivers
> will be used by Lynxpoint as well.
> 
> This has been tested on the tablet with SD card detection and buttons pressed.
> 
> Based on our pinctrl/intel for-next branch.
> 
> Andy Shevchenko (5):
>    pinctrl: baytrail: Move IRQ valid mask initialization to a dedicated
>      callback
>    pinctrl: intel: Share struct intel_pinctrl for wider use
>    pinctrl: baytrail: Keep pointer to struct device instead of its
>      container
>    pinctrl: baytrail: Use local variable to keep device pointer
>    pinctrl: baytrail: Reuse struct intel_pinctrl in the driver
> 
>   drivers/pinctrl/intel/pinctrl-baytrail.c | 272 +++++++++++------------
>   drivers/pinctrl/intel/pinctrl-intel.c    |  35 +--
>   drivers/pinctrl/intel/pinctrl-intel.h    |  44 ++++
>   3 files changed, 173 insertions(+), 178 deletions(-)
> 
> 
> base-commit: ab68b220e81fd03383c0d9e1a87b51f9bbe4db77

 From a quick look this series looks good to me:

Acked-by: Hans de Goede <hdegoede@redhat.com>

I've also added this to my personal tree for testing purposes, I will let
you know if I hit any problems.

Regards,

Hans