Le 01/04/17 à 19:36, Chris Packham a écrit : > The Marvell 98DX3236, 98DX3336, 98DX4521 and variants are switch ASICs > with integrated CPUs. They are similar to the Armada XP SoCs but have > different I/O interfaces. > > Signed-off-by: Chris Packham <chris.packham@xxxxxxxxxxxxxxxxxxx> > --- > + > + switch { > + packet-processor@0 { > + compatible = "marvell,prestera-98dx4521"; > + }; > + }; This may be a bit premature if you are not providing a binding document for this sub-node, you might as well add it once you also add a corresponding driver (or if this will remain out of tree, at least submitting a separate binding document). Also, if this node's unit address is 0, you would expect at least a reg property whose address cell is 0 to be present. -- Florian -- 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