On Fri, May 30, 2014 at 12:40 AM, Rob Herring <robherring2@xxxxxxxxx> wrote: > From: Rob Herring <robh@xxxxxxxxxx> > > The secondary controller on ARM Versatile AB and PB is similar to > other ARM platforms, but has a pass-thru register to connect some > interrupts directly to interrupt inputs on the primary interrupt > controller. The PIC_ENABLES register needs to be configured for > proper operation when the matching node is arm,versatile-sic. Add the > the necessary IRQCHIP_DECLARE as well. > > Signed-off-by: Rob Herring <robh@xxxxxxxxxx> > Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx> > Cc: Jason Cooper <jason@xxxxxxxxxxxxxx> > Cc: Arnd Bergmann <arnd@xxxxxxxx> > --- > v2: > - Rework to use the compatible string for pass-thru register setup. Nice! > + /* > + * On Versatile AB/PB, some secondary interrupts have a direct > + * pass-thru to the primary controller which need to be enabled. > + */ > + if (of_device_is_compatible(node, "arm,versatile-sic")) > + writel(0xffd00000, base + PIC_ENABLES); Maybe mention that we pass thru IRQs 20 and 22 thru 31 and why exactly these IRQs? Anyway: Acked-by: Linus Walleij <linus.walleij@xxxxxxxxxx> Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html