On 20/06/2022 13:07, Dmitry Baryshkov wrote: > Add pxo/cxo clocks to the GCC device tree node. > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> > --- > arch/arm/boot/dts/qcom-msm8660.dtsi | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/arch/arm/boot/dts/qcom-msm8660.dtsi b/arch/arm/boot/dts/qcom-msm8660.dtsi > index 47b97daecef1..61e3ab0ebfd3 100644 > --- a/arch/arm/boot/dts/qcom-msm8660.dtsi > +++ b/arch/arm/boot/dts/qcom-msm8660.dtsi > @@ -50,13 +50,13 @@ cpu-pmu { > }; > > clocks { > - cxo_board { > + cxo_board: cxo_board { If you touch this line, please correct the naming for node - no underscores and preferred some prefix or suffix, so: cxo-board-clk clk-cxo-board > compatible = "fixed-clock"; > #clock-cells = <0>; > clock-frequency = <19200000>; > }; > > - pxo_board { > + pxo_board: pxo_board { The same > compatible = "fixed-clock"; > #clock-cells = <0>; > clock-frequency = <27000000>; > @@ -129,6 +129,8 @@ gcc: clock-controller@900000 { > #power-domain-cells = <1>; > #reset-cells = <1>; > reg = <0x900000 0x4000>; > + clocks = <&pxo_board>, <&cxo_board>; > + clock-names = "pxo", "cxo"; > }; > > gsbi6: gsbi@16500000 { Best regards, Krzysztof