2022-03-25 02:48:19

by Christian Marangi

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

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 <[email protected]>
---
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 <[email protected]>
- Taniya Das <[email protected]>

-description:
+description: |
Qualcomm global clock control module which supports the clocks, resets and
power domains.

--
2.34.1

2022-03-25 03:58:31

by Krzysztof Kozlowski

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

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 <[email protected]>
> ---
> Documentation/devicetree/bindings/clock/qcom,gcc-other.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>

Reported-by: Rob Herring <[email protected]>

Reviewed-by: Krzysztof Kozlowski <[email protected]>

Best regards,
Krzysztof

2022-03-25 17:35:24

by Stephen Boyd

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

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 <[email protected]>
> ---

Applied to clk-next

2022-03-25 18:41:15

by Krzysztof Kozlowski

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

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 <[email protected]>
>> ---

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