Re: [PATCH v3 2/3] docs: firmware-guide: ACPI: Add named interrupt doc

From: Andy Shevchenko
Date: Thu Jan 20 2022 - 10:05:02 EST


On Thu, Jan 20, 2022 at 3:45 PM Akhil R <akhilrajeev@xxxxxxxxxx> wrote:
>
> Added details and example for named interrupts in the ACPI table.

Added details and example for --> Add a detailed example of the

...

> + Interrupt (ResourceConsumer, Level, ActiveHigh, Exclusive) {
> + 0x20,

I would think of splitting this to two separate entries in between of
which the GpioInt() resource is provided. It will explicitly show that
you describe the case only for Interrupt(). Something like

Interrupt (...) { 0x20 }
GpioInt(...) { ... }
Interrupt (...) { 0x24 }

But it's up to you.

> + 0x24
> + }

...

> +The driver can call the function - device_irq_get_byname() with the device
> +and interrupt name as arguments to get the corresponding IRQ number.

Needs switch to fwnode as per comment against the previous patch.

--
With Best Regards,
Andy Shevchenko