Re: [PATCH RFC] gpio: move intel drivers into their own directory

From: Alan Borzeszkowski

Date: Tue Sep 29 2026 - 08:26:28 EST


On Mon, 2026-09-28 at 16:13 +0300, Andy Shevchenko wrote:
> On Mon, Sep 28, 2026 at 3:34 PM Mika Westerberg
> <mika.westerberg@xxxxxxxxxxxxxxx> wrote:
> > On Mon, Sep 28, 2026 at 02:34:44PM +0300, Andy Shevchenko wrote:
> > > On Mon, Sep 28, 2026 at 2:13 PM Bartosz Golaszewski
> > > <brgl@xxxxxxxxxx> wrote:
> > > > On Sat, 26 Sep 2026 12:07:27 +0200, Mika Westerberg
> > > > <mika.westerberg@xxxxxxxxxxxxxxx> said:
> > > > > On Fri, Sep 25, 2026 at 05:34:53PM +0300, Andy Shevchenko
> > > > > wrote:
> > > > > > On Fri, Sep 25, 2026 at 02:54:33PM +0200, Mika Westerberg
> > > > > > wrote:
> > > > > > > On Fri, Sep 25, 2026 at 01:48:05PM +0300, Andy Shevchenko
> > > > > > > wrote:
> > > > > > > > On Fri, Sep 25, 2026 at 12:22:15PM +0200, Bartosz
> > > > > > > > Golaszewski wrote:
>
> ...
>
> > > > > > > > >  drivers/gpio/{ => intel}/gpio-graniterapids.c   |  0
> > > > > > > > >  drivers/gpio/{ => intel}/gpio-novalake-events.c |  0
> > > > > > > >
> > > > > > > > These are interesting drivers, not sure where to put
> > > > > > > > them
> > > > > > >
> > > > > > > IMHO they should be there too. We are maintaining them
> > > > > > > both.
> > > > > >
> > > > > > Do you send PRs to Bart or you just rely on him to take the
> > > > > > patches?
> > > > >
> > > > > I don't expect these two drives to get too much attention,
> > > > > and they could
> > > > > be contained nicely inside intel folder and that would help
> > > > > Bart to
> > > > > identify that these are Intel drivers. So your bandwidth
> > > > > would not suffer
> > > > > even if these two are added there. Your call.
> > > >
> > > > I'd like to avoid a situation where we have Andy's Intel
> > > > drivers under
> > > > drivers/gpio/intel/ and other Intel drivers under
> > > > drivers/gpio/. I'd prefer
> > > > to move all of intel GPIO drivers into drivers/gpio/intel/ and
> > > > have Andy
> > > > gather changes to them if he agrees.
> > >
> > > Okay, we have two parts missing:
> > > - Nova Lake driver (I would need Alan or Mika to step in as at
> > > least a
> > > designated reviewer for that)
> >
> > I suggest Alan to be the reviewer if that's okay for him :)
>
> If he agrees, he can provide a patch on top of my series to move Nova
> Lake and add himself to the MAINTAINERS at the same time.

I agree to be the reviewer. I have the patch ready, I can send once
your series is applied.