On Mon, Sep 16, 2019 at 05:47:51PM +0300, Andy Shevchenko wrote: > Keeping the IRQ chip definition static shares it with multiple instances of > the GPIO chip in the system. This is bad and now we get this warning from > GPIO library: > > "detected irqchip that is shared with multiple gpiochips: please fix the driver." > > Hence, move the IRQ chip definition from being driver static into the struct > intel_pinctrl. So a unique IRQ chip is used for each GPIO chip instance. > > Fixes: ee1a6ca43dba ("Add Intel Broxton pin controller support") > Depends-on: 5ff56b015e85 ("Disable GPIO pin interrupts in suspend") > Reported-by: Federico Ricchiuto <fed.ricchiuto@xxxxxxxxx> > Suggested-by: Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx> > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> Applied to intel.git/fixes, thanks!