Re: [PATCH v2 1/6] dt-bindings: clock: add DT binding for the Xtal clock on Armada 3700

From: Thomas Petazzoni
Date: Fri Jul 08 2016 - 03:32:40 EST


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