On Fri, Dec 15, 2023 at 04:58:24PM +0200, Mika Westerberg wrote: > On Fri, Dec 15, 2023 at 04:42:31PM +0200, Andy Shevchenko wrote: > > On Fri, Dec 15, 2023 at 08:44:17AM +0200, Mika Westerberg wrote: > > > On Thu, Dec 14, 2023 at 05:46:53PM +0200, Andy Shevchenko wrote: > > > > This driver supports pinctrl/GPIO hardware found on Intel Meteor Point > > > > (a Meteor Lake PCH) providing users a pinctrl and GPIO interfaces > > > > including GPIO interrupts. > > > > > > This is driver for Meteor Lake-S and related, correct? Why it cannot use > > > pinctrl-meteorlake.c instead? Same way we do for other desktop CPUs > > > whose PCHs have the GPIO/pinctrl block. > > > > Meteor Lake-S may or may not have an additional PCH, so the SoC (on-die) > > pin control IP is serviced by pinctrl-meteorlake.c, here we need to have > > a different file as we would have a name clash. Yes, it means that on > > the end user platform it may be two pin control drivers! > > I see, thanks for clarification! Then this makes sense. > > Acked-by: Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx> Pushed to my review and testing queue, thanks! -- With Best Regards, Andy Shevchenko