Re: [PATCH 2/2] dt-bindings: clock: Add GCC clock bindings for SM7250

Krzysztof Kozlowski <[email protected]>
Newsgroups org.kernel.vger.linux-clk,org.kernel.vger.linux-arm-msm,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-kernel
Message-ID <[email protected]>
On 16/08/2026 11:07, Sreeshankar K wrote:
> Add device tree bindings for global clock controller on SM7250 SoC.
> 
> Signed-off-by: Sreeshankar K <[email protected]>

Please start using b4. Does this work:

b4 diff '<[email protected]>'

? No.



> ---
>  .../bindings/clock/qcom,gcc-sm7250.yaml       |  60 +++++++
>  include/dt-bindings/clock/qcom,gcc-sm7250.h   | 163 ++++++++++++++++++
>  2 files changed, 223 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/clock/qcom,gcc-sm7250.yaml
>  create mode 100644 include/dt-bindings/clock/qcom,gcc-sm7250.h
> 
> diff --git a/Documentation/devicetree/bindings/clock/qcom,gcc-sm7250.yaml b/Documentation/devicetree/bindings/clock/qcom,gcc-sm7250.yaml
> new file mode 100644
> index 000000000..822df7dc1
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/clock/qcom,gcc-sm7250.yaml
> @@ -0,0 +1,60 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/clock/qcom,gcc-sm7250.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Qualcomm Global Clock & Reset Controller on SM7250
> +
> +maintainers:
> +  - Bjorn Andersson <[email protected]>
> +  - Sreeshankar K <[email protected]>
> +
> +description: |
> +  Qualcomm global clock control module provides the clocks, resets and power
> +  domains on SM7250.
> +
> +  See also: include/dt-bindings/clock/qcom,gcc-sm7250.h
> +
> +properties:
> +  compatible:
> +    const: qcom,gcc-sm7250
> +
> +  clocks:
> +    items:
> +      - description: Board XO source
> +      - description: Board active XO source
> +      - description: Sleep clock source

Why this cannot go to sm7150 or sm6375 files?

Best regards,
Krzysztof
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.