On 11.01.2023 20:57, Dmitry Baryshkov wrote: > Add device tree node for the CBF clock. > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> > --- > arch/arm64/boot/dts/qcom/msm8996.dtsi | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/arch/arm64/boot/dts/qcom/msm8996.dtsi b/arch/arm64/boot/dts/qcom/msm8996.dtsi > index 37ebd8a3b2c8..e5a638b85ffc 100644 > --- a/arch/arm64/boot/dts/qcom/msm8996.dtsi > +++ b/arch/arm64/boot/dts/qcom/msm8996.dtsi > @@ -3558,6 +3558,13 @@ saw3: syscon@9a10000 { > reg = <0x09a10000 0x1000>; > }; > > + cbf: clock-controller@9a11000 { > + compatible = "qcom,msm8996-cbf"; > + reg = <0x09a11000 0x10000>; > + clocks = <&rpmcc RPM_SMD_BB_CLK1>, <&apcs_glb>; Shouldn't this be RPM_SMD_XO_A_CLK_SRC? I don't see BB_CLK referenced *anywhere* downstream, other than being defined in the 8996 GCC driver.. Konrad > + #clock-cells = <0>; > + }; > + > intc: interrupt-controller@9bc0000 { > compatible = "qcom,msm8996-gic-v3", "arm,gic-v3"; > #interrupt-cells = <3>;