On Mon, Apr 18, 2016 at 10:02 AM, Shawn Guo <shawnguo@xxxxxxxxxx> wrote: > On Wed, Apr 13, 2016 at 08:39:32PM +0000, Stuart Yoder wrote: >> > > @@ -265,6 +265,93 @@ >> > > compatible = "fsl,qoriq-mc"; >> > > reg = <0x00000008 0x0c000000 0 0x40>, /* MC portal base */ >> > > <0x00000000 0x08340000 0 0x40000>; /* MC control reg */ >> > > + msi-parent = <&its>; >> > > + #address-cells = <3>; >> > > + #size-cells = <1>; >> > > + >> > > + /* >> > > + * Region type 0x0 - MC portals >> > > + * Region type 0x1 - QBMAN portals >> > > + */ >> > > + ranges = <0x0 0x0 0x0 0x8 0x0c000000 0x4000000 >> > > + 0x1 0x0 0x0 0x8 0x18000000 0x8000000>; >> > > + >> > > + /* >> > > + * Define the maximum number of MACs present on the SoC. >> > > + * They won't necessarily be all probed, since the >> > > + * Data Path Layout file and the MC firmware can put >> > > + * fewer actual DPMAC objects on the MC bus. >> > > + */ >> > > + dpmacs { >> > > + #address-cells = <1>; >> > > + #size-cells = <0>; >> > > + >> > > + dpmac1: dpmac@1 { >> > > + compatible = "fsl,qoriq-mc-dpmac"; >> > >> > I did not see how this compatible is used by kernel driver. >> >> There is no driver yet, but it will be coming soon. But, does >> it matter? Device trees should be describing hardware >> without regards to some specific kernel use of a node or >> property. > > We need to at least have the bindings documented before we can accept it > in dts files. Ah, sorry. Missed that the bindings are added by patch 1/2. Shawn -- 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