Subject: Re: [PATCH 2/2] ARM: dts: s3c64xx: specify dependency of clock providers

From: "[email-masked]" <[email-masked]>
Date: Mon, 9 Sep 2019 20:53:43 +0200

On Sat, Sep 07, 2019 at 02:48:12AM +0000, Yao Lihua wrote:
> From: Lihua Yao <[email-masked]>
> 
> fin_pll is the parent of clock-controller@7e00f000, specify
> the dependency to ensure proper initialization order of clock
> providers.
> 
> Fixes: 3f6d439f2022 ("clk: reverse default clk provider initialization order in of_clk_init()")
> Signed-off-by: Lihua Yao <[email-masked]>
> ---
>  arch/arm/boot/dts/s3c6400.dtsi | 1 +
>  arch/arm/boot/dts/s3c6410.dtsi | 1 +
>  2 files changed, 2 insertions(+)

Idea looks good but should go to each of DTS files.

Best regards,
Krzysztof

> 
> diff --git a/arch/arm/boot/dts/s3c6400.dtsi b/arch/arm/boot/dts/s3c6400.dtsi
> index 8c28e8a0c824..ef5a8fa3555c 100644
> --- a/arch/arm/boot/dts/s3c6400.dtsi
> +++ b/arch/arm/boot/dts/s3c6400.dtsi
> @@ -34,5 +34,6 @@
>  		compatible = "samsung,s3c6400-clock";
>  		reg = <0x7e00f000 0x1000>;
>  		#clock-cells = <1>;
> +		clocks = <&fin_pll>;
>  	};
>  };
> diff --git a/arch/arm/boot/dts/s3c6410.dtsi b/arch/arm/boot/dts/s3c6410.dtsi
> index a766d6de696c..b201b71d45b5 100644
> --- a/arch/arm/boot/dts/s3c6410.dtsi
> +++ b/arch/arm/boot/dts/s3c6410.dtsi
> @@ -38,6 +38,7 @@
>  		compatible = "samsung,s3c6410-clock";
>  		reg = <0x7e00f000 0x1000>;
>  		#clock-cells = <1>;
> +		clocks = <&fin_pll>;
>  	};
>  
>  	i2c1: i2c@7f00f000 {
> -- 
> 2.17.1
> 

_______________________________________________
linux-arm-kernel mailing list
[email-masked]
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel