On Thu, May 02, 2013 at 09:05:38PM +0200, Sebastian Hesselbarth wrote:
> >>+static struct of_device_id orion_irq_dt_ids[] __initconst = {
> >>+ { .compatible = "marvell,orion-mpic", .data = orion_of_init },
> >>+ { }
> >
> >Is there a strong reason to change the compatible string? Looks to me
> >like either the new driver or the old driver will bind depending on
> >what is in the machine description. No need for a new string?
>
> The reason for a new compatible string is, that we will also need an
> secondary irq controller for bridge irqs. That could be called
> marvell,orion-spic. Dove is again a little bit different than the
> others and this will require timer and especially rtc not to share
> bridge irqs here. RTC irq is located in PMU regs on Dove instead of
> bridge regs.
As Arnd mentioned, I would keep the old name then..
The bridge controller can be called marvell,orion-intc-bridge, and if
Dove needs a pmu controller, marvell,dove-intc-pmu ?
> >.. which lets this go away, use the generic irqchip_init instead of
> >orion_init_irq.
>
> Same as above.
I have kirkwood HW but I haven't had time to make newer kernels run on
it, otherwise I'd test it too :(
Jason
_______________________________________________
devicetree-discuss mailing list
devicetree-discuss@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/devicetree-discuss
