On 27/01/17 04:10, Gregory CLEMENT wrote: > Hi Chris, > > On ven., janv. 06 2017, Chris Packham <chris.packham@xxxxxxxxxxxxxxxxxxx> wrote: > >> 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. > > Before sending a new version I have a few remarks: > > [snip] I'll update the dtsi files to use the node labels and correct the commends as requested > > Why the following node is not part of the dtsi? > > Gregory > >> + resume@20980 { >> + compatible = "marvell,98dx3336-resume-ctrl"; >> + reg = <0x20980 0x10>; >> + }; >> + }; >> + }; The 98DX9236 has a single ARMv7 core. As such this resume control isn't present on it. The 98DX3336 and 98DX4521 have dual ARMv7 cores and this is used to boot the second core (SMP support is a little different compared to Armada-XP). In other words {98DX3336, 98DX4521} = 98DX9236 + an additional core. At the switch packet processor level there are more differences but as far as the kernel is concerned the only real difference is the number of cores. -- 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