[PATCH] dt-bindings: clock: fix dt_binding_check error for qcom,gcc-other.yaml

[PATCH] dt-bindings: clock: fix dt_binding_check error for qcom,gcc-other.yaml

[Ansuel Smith]

qcom,gcc-other Documentation lacks a '|' for the description. This cause
dt_binding_check to incorrectly parse "See also:" as a new value.
Add the missing '|' to correctly parse the description.

Fixes: a03965ed1310 ("dt-bindings: clock: split qcom,gcc.yaml to common and specific schema")
Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com>
---
 Documentation/devicetree/bindings/clock/qcom,gcc-other.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/clock/qcom,gcc-other.yaml b/Documentation/devicetree/bindings/clock/qcom,gcc-other.yaml
index 4dc0274dbd6b..6c45e0f85494 100644
--- a/Documentation/devicetree/bindings/clock/qcom,gcc-other.yaml
+++ b/Documentation/devicetree/bindings/clock/qcom,gcc-other.yaml
@@ -10,7 +10,7 @@ maintainers:
   - Stephen Boyd <sboyd@kernel.org>
   - Taniya Das <tdas@codeaurora.org>
 
-description:
+description: |
   Qualcomm global clock control module which supports the clocks, resets and
   power domains.
 
-- 
2.34.1

Re: [PATCH] dt-bindings: clock: fix dt_binding_check error for qcom,gcc-other.yaml

[Krzysztof Kozlowski]

On 23/03/2022 20:42, Ansuel Smith wrote:
> qcom,gcc-other Documentation lacks a '|' for the description. This cause
> dt_binding_check to incorrectly parse "See also:" as a new value.
> Add the missing '|' to correctly parse the description.
> 
> Fixes: a03965ed1310 ("dt-bindings: clock: split qcom,gcc.yaml to common and specific schema")
> Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com>
> ---
>  Documentation/devicetree/bindings/clock/qcom,gcc-other.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Reported-by: Rob Herring <robh@kernel.org>

Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>

Best regards,
Krzysztof

Re: [PATCH] dt-bindings: clock: fix dt_binding_check error for qcom,gcc-other.yaml

[Stephen Boyd]

Quoting Ansuel Smith (2022-03-23 12:42:48)
> qcom,gcc-other Documentation lacks a '|' for the description. This cause
> dt_binding_check to incorrectly parse "See also:" as a new value.
> Add the missing '|' to correctly parse the description.
> 
> Fixes: a03965ed1310 ("dt-bindings: clock: split qcom,gcc.yaml to common and specific schema")
> Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com>
> ---

Applied to clk-next

Re: [PATCH] dt-bindings: clock: fix dt_binding_check error for qcom,gcc-other.yaml

[Krzysztof Kozlowski]

On 25/03/2022 01:49, Stephen Boyd wrote:
> Quoting Ansuel Smith (2022-03-23 12:42:48)
>> qcom,gcc-other Documentation lacks a '|' for the description. This cause
>> dt_binding_check to incorrectly parse "See also:" as a new value.
>> Add the missing '|' to correctly parse the description.
>>
>> Fixes: a03965ed1310 ("dt-bindings: clock: split qcom,gcc.yaml to common and specific schema")
>> Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com>
>> ---

Just to clarify:
This should go with fixed commit (a03965ed1310), so maybe for this merge
window (I don't know the plans for a03965ed1310).


Best regards,
Krzysztof