Hi Rob, On Fri, May 29, 2020 at 5:54 PM Rob Herring <robh+dt@xxxxxxxxxx> wrote: > On Thu, May 28, 2020 at 7:23 AM Geert Uytterhoeven > <geert+renesas@xxxxxxxxx> wrote: > > When an interrupt controller has an "interrupt-map" property, an "is > > valid under each of" error is triggered. > > > > Fix this by allowing "interrupt-controller" and "interrupt-map" to > > coexist, in both the interrrupts meta-schema and the > > interrupt-controller schema. > > But both should not be present. If 'interrupt-controller' is present, Why not? > the Linux irq parsing code will ignore 'interrupt-map'. Seems like > that's backwards, but this parsing code is older than dirt and we'd > probably break some 1990s machine changing it. That's fine. rza1_irqc_parse_map() parses the interrupt-map itself, to map from downstream to upstream interrupts. Cfr. the original bindings at https://lore.kernel.org/linux-devicetree/20190502123220.3016-2-geert+renesas@xxxxxxxxx/ Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
