Hello, On Fri, 8 Jul 2016 00:37:46 +0200, Gregory CLEMENT wrote: > +gpio1: gpio@13800 { > + compatible = "marvell,mvebu-gpio-3700", "syscon", "simple-mfd"; I find this compatible string not very consistent with what we do for other drivers, it should have been: marvell,armada-3700-gpio or something like that. > + xtalclk: xtal-clk { > + compatible = "marvell,armada-3700-xtal-clock"; See here for example. Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com -- 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