Re: [PATCH v4 2/2] gpio: pca953x: Override IRQ for one of the expanders on Galileo Gen 2

[Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>]

On Mon, Jun 01, 2020 at 11:40:23AM +0300, Mika Westerberg wrote:
> On Fri, May 29, 2020 at 04:30:54PM +0300, Andy Shevchenko wrote:
> > ACPI table on Intel Galileo Gen 2 has wrong pin number for IRQ resource
> > of one of the I²C GPIO expanders. Since we know what that number is and
> > luckily have GPIO bases fixed for SoC's controllers, we may use a simple
> > DMI quirk to match the platform and retrieve GpioInt() pin on it for
> > the expander in question.
> > 
> > Suggested-by: Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx>
> 
> I don't think this is needed since I did not suggest you to write this
> patch. I only gave a couple of review comments.

You suggested the way to avoid a quirk which brings us to almost rewritten
version of this. I think it's appropriate to put your tag, but if you insist,
I'll remove it.

> > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> 
> Reviewed-by: Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx>

Thank you!

-- 
With Best Regards,
Andy Shevchenko