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

Reply via email to