On Fri, 20 Sept 2024 at 09:56, Patrick Rudolph <patrick.rudo...@9elements.com> wrote: > > On Thu, Sep 19, 2024 at 4:09 PM Simon Glass <s...@chromium.org> wrote: > > > > Hi Patrick, > > > > On Wed, 18 Sept 2024 at 17:22, Patrick Rudolph > > <patrick.rudo...@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.rudo...@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 <s...@chromium.org>