On Sat, Oct 19, 2024 at 12:42:35AM +0530, Taniya Das wrote: > Add device tree bindings for global clock controller on QCS615 SoCs. > > Signed-off-by: Taniya Das <quic_tdas@xxxxxxxxxxx> > --- That's 5/12 but rest is missing. Are you sure you used sent it correctly? > .../devicetree/bindings/clock/qcom,qcs615-gcc.yaml | 59 ++++++ > include/dt-bindings/clock/qcom,qcs615-gcc.h | 211 +++++++++++++++++++++ > 2 files changed, 270 insertions(+) > > diff --git a/Documentation/devicetree/bindings/clock/qcom,qcs615-gcc.yaml b/Documentation/devicetree/bindings/clock/qcom,qcs615-gcc.yaml > new file mode 100644 > index 0000000000000000000000000000000000000000..fecc694cd71b8aeb3d420ccea5f5ffba04c8ff9c > --- /dev/null > +++ b/Documentation/devicetree/bindings/clock/qcom,qcs615-gcc.yaml > @@ -0,0 +1,59 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/clock/qcom,qcs615-gcc.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Qualcomm Global Clock & Reset Controller on QCS615 > + > +maintainers: > + - Taniya Das <quic_tdas@xxxxxxxxxxx> > + > +description: | > + Qualcomm global clock control module provides the clocks, resets and power > + domains on QCS615. > + > + See also:: include/dt-bindings/clock/qcom,qcs615-gcc.h s/::/:/ > + > +properties: > + compatible: > + const: qcom,qcs615-gcc > + > + clocks: > + items: > + - description: Board XO source > + - description: Board active XO source > + - description: Sleep clock source > + > + clock-names: > + items: > + - const: bi_tcxo > + - const: bi_tcxo_ao > + - const: sleep_clk That's just "sleep". Why do you need clock-names in the first place? Best regards, Krzysztof