Re: [PATCH] pinctrl: intel: Pass irqchip when adding gpiochip
From: Mika Westerberg
Date: Tue Jan 07 2020 - 08:52:22 EST
On Tue, Jan 07, 2020 at 11:32:54AM +0100, Linus Walleij wrote:
> On Mon, Dec 30, 2019 at 11:20 AM Mika Westerberg
> <mika.westerberg@xxxxxxxxxxxxxxx> wrote:
> > On Sun, Dec 29, 2019 at 02:30:59AM +0100, Linus Walleij wrote:
> > > We need to convert all old gpio irqchips to pass the irqchip
> > > setup along when adding the gpio_chip. For more info see
> > > drivers/gpio/TODO.
> > >
> > > Set up the pin ranges using the new callback.
> > Maybe have this one split as a separate patch? Same what we do for
> > Baytrail and Cherryview.
> I'm afraid to do that since splitting the semantic ordering was
> something that broke a lot of times already, I was under the
> impression that doing the two things (moving to the callback
> and adding along with the gpio_chip) at the same time was
> the only way to preserve the semantic ordering.
Well at least we do the same for others (add the callback in another
patch and then pass irqchip in another) but no strong feelings. I'm fine
with this one as well :)
> But more than anything I want someone to test it ...
I quickly tested this on Whiskey Lake and SD card detection interrupt
still works fine after this patch.