[PATCH v4 22/35] drivers: misc: irq-uclass: Update irq_get_by_index

[Simon Glass]

On Fri, 20 Sept 2024 at 09:56, Patrick Rudolph
<patrick.rudolph at 9elements.com> wrote:
>
> On Thu, Sep 19, 2024 at 4:09 PM Simon Glass <sjg at chromium.org> wrote:
> >
> > Hi Patrick,
> >
> > On Wed, 18 Sept 2024 at 17:22, Patrick Rudolph
> > <patrick.rudolph at 9elements.com> wrote:
> > >
> > > Support reading the "interrupts" property from the devicetree in case
> > > the "interrupts-extended" property isn't found. As the "interrupts"
> > > property is commonly used, this allows to parse all existing FDT and
> > > makes irq_get_by_index() more useful.
> > >
> > > The "interrupts" property doesn't contain a phandle as "interrupts-extended"
> > > does, so implement a new method to locate the interrupt-parent called
> > > irq_get_interrupt_parent().
> > >
> > > TEST: Read the interrupts from the GIC node for ACPI MADT generation.
> > >
> > > Signed-off-by: Patrick Rudolph <patrick.rudolph at 9elements.com>
> > > ---
> > >  arch/sandbox/dts/test.dts |  3 ++
> > >  drivers/misc/irq-uclass.c | 66 ++++++++++++++++++++++++++++++++++++++-
> > >  include/irq.h             | 14 +++++++++
> > >  test/dm/irq.c             | 15 +++++++++
> > >  4 files changed, 97 insertions(+), 1 deletion(-)
> >
> > Is this a new patch? I don't see a change log so I am wondering if I
> > already reviewed it?
> >
> It's a new patch. I had to extend irq_get_by_index() as it is used to
> parse the DT while
> generating MADT subtables solely from DT.

OK thanks

Reviewed-by: Simon Glass <sjg at chromium.org>