Quoting Vladimir Zapolskiy (2022-06-10 03:49:10) > diff --git a/Documentation/devicetree/bindings/clock/qcom,sm8450-camcc.yaml b/Documentation/devicetree/bindings/clock/qcom,sm8450-camcc.yaml > new file mode 100644 > index 000000000000..6e774e59f213 > --- /dev/null > +++ b/Documentation/devicetree/bindings/clock/qcom,sm8450-camcc.yaml > @@ -0,0 +1,94 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) [..] > + > +examples: > + - | > + #include <dt-bindings/clock/qcom,gcc-sm8450.h> > + #include <dt-bindings/clock/qcom,rpmh.h> > + #include <dt-bindings/power/qcom-rpmpd.h> > + soc { > + #address-cells = <2>; > + #size-cells = <2>; I'd prefer to not have container node and just change the reg property of the clock-controller node instead. > + > + clock-controller@ade0000 { > + compatible = "qcom,sm8450-camcc"; > + reg = <0 0xade0000 0 0x20000>; > + clocks = <&gcc GCC_CAMERA_AHB_CLK>, > + <&rpmhcc RPMH_CXO_CLK>,