Re: [PATCH V3 1/2] dt-bindings: clk: qcom: gcc-sm6125: Document SM6125 GCC driver

From: Konrad Dybcio
Date: Sat Jun 05 2021 - 06:56:18 EST



> Dual license new bindings please:
>
> GPL-2.0-only OR BSD-2-Clause

Ack.


>> +%YAML 1.2
>> +---
>> +$id: http://devicetree.org/schemas/clock/qcom,gcc-sm6125.yaml#
>> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>> +
>> +title: Qualcomm Global Clock & Reset Controller Binding for SM6125
>> +
>> +maintainers:
>> + - Konrad Dybcio <konrad.dybcio@xxxxxxxxxxxxxx>
>> +
>> +description: |
>> + Qualcomm global clock control module which supports the clocks, resets and
>> + power domains on SM6125.
>> +
>> + See also:
>> + - dt-bindings/clock/qcom,gcc-sm6125.h
>> +
>> +properties:
>> + compatible:
>> + const: qcom,gcc-sm6125
> The normal ordering would be 'qcom,sm6125-gcc'

The current one is in line with all the other qcom clk drivers.


>> +
>> +examples:
>> + - |
>> + #include <dt-bindings/clock/qcom,rpmcc.h>
>> + clock-controller@1400000 {
>> + compatible = "qcom,gcc-sm6125";
> Wrong indentation.

Ack.


Konrad