On 10/8/19 18:59, Krzysztof Kozlowski wrote: > From: Lihua Yao <ylhuajnu@xxxxxxxxxxx> > > 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()") The patch looks good but I'm not sure above tag points to the right commit. That commit is just a regression fix for 1771b10d605d26cc "clk: respect the clock dependencies in of_clk_init" How about picking some commit touching the dts files itself, e.g. a43736deb47d21bd "ARM: dts: Add dts file for S3C6410-based Mini6410 board" ? > Signed-off-by: Lihua Yao <ylhuajnu@xxxxxxxxxxx> > Signed-off-by: Krzysztof Kozlowski <krzk@xxxxxxxxxx> Reviewed-by: Sylwester Nawrocki <s.nawrocki@xxxxxxxxxxx> -- Regards, Sylwester