Re: [PATCH] driver: platform: Add support for GpioInt() ACPI to platform_get_irq()
From: Andy Shevchenko
Date: Thu Feb 07 2019 - 14:39:40 EST
On Thu, Feb 7, 2019 at 9:04 PM Rafael J. Wysocki <rafael@xxxxxxxxxx> wrote:
>
> +Mika Westerberg
> +Andy Shevchenko
> +Hans de Goede
Thanks, Rafael.
My comments below.
> On Thu, Feb 7, 2019 at 7:59 PM <egranata@xxxxxxxxxx> wrote:
> >
> > From: Enrico Granata <egranata@xxxxxxxxxxxx>
> >
> > ACPI 5 added support for GpioInt resources as a way to provide
> > information about interrupts mediated via a GPIO controller.
> >
> > Several device buses (e.g. SPI, I2C) have support for retrieving
> > an IRQ specified via this type of resource, and providing it
> > directly to the driver as an IRQ number.
> > This is not currently done for the platform drivers, as platform_get_irq()
> > does not try to parse GpioInt() resources.
And why is this a problem?
> > This commit adds that functionality.
How that can override the configuration / BIOS flavour when driver
needs to register an interrupt out of GpioIo() resource?
P.S. Have you looked at drivers/platform/x86/i2c-multi-instantiate.c
and intel_cht_int33fe.c?
They are the only drivers which needs something like this for now and
I would rather say it's a bad BIOS decision to write a table like
that.
--
With Best Regards,
Andy Shevchenko