On 05/19/2010 04:27 PM, Grant Likely wrote:
On Mon, May 3, 2010 at 10:34 AM, Scott Wood<scottw...@freescale.com>  wrote:
Grant Likely wrote:

+               // IPIC
+               // interrupts cell =<intr #, sense>
+               // sense values match linux IORESOURCE_IRQ_* defines:
+               // sense == 8: Level, low assertion
+               // sense == 2: Edge, high-to-low change
+               //
+               ipic: interrupt-control...@c00 {
+                       compatible = "fsl,mpc5121-ipic", "fsl,ipic";
+                       interrupt-controller;
+                       #address-cells =<0>;

Don't need #address-cells here

#address-cells is required by ePAPR for interrupt controllers if an
interrupt-map is used.

Why?

/me is too lazy to dig out ePAPR and look.

Address cells are part of the interrupt identification. Typically with interrupt maps this is only used on the child end (e.g. to select a particular PCI slot), but if the parent interrupt controller's address cells are non-zero it will be expected in the parent interrupt specifier as well.

I believe the only part of this that is new with ePAPR is that it asks that the interrupt controller address cells be explicitly specified, as it's a bit icky for it to default to 2 in some contexts and 0 in others.

-Scott
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Reply via email to