On 30.10.2022 16:55, Dmitry Baryshkov wrote: > Add clocks and clock-names nodes to the gcc device to bind clocks using > the DT links. > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> > --- > arch/arm/boot/dts/qcom-msm8974.dtsi | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/arch/arm/boot/dts/qcom-msm8974.dtsi b/arch/arm/boot/dts/qcom-msm8974.dtsi > index 7f8b4356ccdb..4e70f51c8750 100644 > --- a/arch/arm/boot/dts/qcom-msm8974.dtsi > +++ b/arch/arm/boot/dts/qcom-msm8974.dtsi > @@ -1054,6 +1054,10 @@ gcc: clock-controller@fc400000 { > #reset-cells = <1>; > #power-domain-cells = <1>; > reg = <0xfc400000 0x4000>; > + > + clock-names = "xo", "sleep_clk"; > + clocks = <&xo_board>, > + <&sleep_clk>; I think it would make sense to either have both properties take up either one or two lines. Other than that though: Reviewed-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxx> Konrad > }; > > rpm_msg_ram: sram@fc428000 {