Per device tree spec, we should use generic name for device node when possible. Let's do that for the fixed-clock nodes, and use property clock-output-names to define the name for clock. While at it, let's drop those unneeded imx specific compatible string for these fixed rate clocks. Signed-off-by: Shawn Guo <shawn.guo@xxxxxxxxxxxxx> --- arch/arm/boot/dts/imx6qdl.dtsi | 18 ++++++++++++------ arch/arm/boot/dts/imx6sl.dtsi | 8 ++++++-- 2 files changed, 18 insertions(+), 8 deletions(-) diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/imx6qdl.dtsi index 2a35b08..400f75c 100644 --- a/arch/arm/boot/dts/imx6qdl.dtsi +++ b/arch/arm/boot/dts/imx6qdl.dtsi @@ -57,22 +57,28 @@ #address-cells = <1>; #size-cells = <0>; - ckil: ckil { - compatible = "fsl,imx-ckil", "fixed-clock"; + ckil: clock@0 { + compatible = "fixed-clock"; + reg = <0>; #clock-cells = <0>; clock-frequency = <32768>; + clock-output-names = "ckil"; }; - ckih1: ckih1 { - compatible = "fsl,imx-ckih1", "fixed-clock"; + ckih1: clock@1 { + compatible = "fixed-clock"; + reg = <1>; #clock-cells = <0>; clock-frequency = <0>; + clock-output-names = "ckih1"; }; - osc: osc { - compatible = "fsl,imx-osc", "fixed-clock"; + osc: clock@2 { + compatible = "fixed-clock"; + reg = <2>; #clock-cells = <0>; clock-frequency = <24000000>; + clock-output-names = "osc"; }; }; diff --git a/arch/arm/boot/dts/imx6sl.dtsi b/arch/arm/boot/dts/imx6sl.dtsi index a6d3afb..275eb30 100644 --- a/arch/arm/boot/dts/imx6sl.dtsi +++ b/arch/arm/boot/dts/imx6sl.dtsi @@ -79,16 +79,20 @@ #address-cells = <1>; #size-cells = <0>; - ckil: ckil { + ckil: clock@0 { compatible = "fixed-clock"; + reg = <0>; #clock-cells = <0>; clock-frequency = <32768>; + clock-output-names = "ckil"; }; - osc: osc { + osc: clock@1 { compatible = "fixed-clock"; + reg = <1>; #clock-cells = <0>; clock-frequency = <24000000>; + clock-output-names = "osc"; }; }; -- 1.8.3.2 -- 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