2023-07-24 09:30:32

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH] dt-bindings: arm: msm: kpss-acc: Revert "dt-bindings: arm: msm: kpss-acc: Make the optional reg truly optional"

This reverts commit 981be238e1d28e156aa9da2a8722f86f02fd0453 because it
was totally bogus and duplicated existing minItems:

ruamel.yaml.constructor.DuplicateKeyError: while constructing a mapping
Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml: ignoring, error parsing file

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml | 1 -
1 file changed, 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml b/Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml
index facaafefb441..202a5d51ee88 100644
--- a/Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml
+++ b/Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml
@@ -21,7 +21,6 @@ properties:
const: qcom,kpss-acc-v2

reg:
- minItems: 1
items:
- description: Base address and size of the register region
- description: Optional base address and size of the alias register region
--
2.34.1



2023-07-24 11:15:45

by Konrad Dybcio

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: arm: msm: kpss-acc: Revert "dt-bindings: arm: msm: kpss-acc: Make the optional reg truly optional"

On 24.07.2023 11:01, Krzysztof Kozlowski wrote:
> This reverts commit 981be238e1d28e156aa9da2a8722f86f02fd0453 because it
> was totally bogus and duplicated existing minItems:
>
> ruamel.yaml.constructor.DuplicateKeyError: while constructing a mapping
> Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml: ignoring, error parsing file
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
My bad.

Acked-by: Konrad Dybcio <[email protected]>

Konrad

2023-07-26 16:59:01

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: arm: msm: kpss-acc: Revert "dt-bindings: arm: msm: kpss-acc: Make the optional reg truly optional"


On Mon, 24 Jul 2023 11:01:09 +0200, Krzysztof Kozlowski wrote:
> This reverts commit 981be238e1d28e156aa9da2a8722f86f02fd0453 because it
> was totally bogus and duplicated existing minItems:
>
> ruamel.yaml.constructor.DuplicateKeyError: while constructing a mapping
> Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml: ignoring, error parsing file
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml | 1 -
> 1 file changed, 1 deletion(-)
>

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


2023-07-28 04:36:29

by Bjorn Andersson

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: arm: msm: kpss-acc: Revert "dt-bindings: arm: msm: kpss-acc: Make the optional reg truly optional"


On Mon, 24 Jul 2023 11:01:09 +0200, Krzysztof Kozlowski wrote:
> This reverts commit 981be238e1d28e156aa9da2a8722f86f02fd0453 because it
> was totally bogus and duplicated existing minItems:
>
> ruamel.yaml.constructor.DuplicateKeyError: while constructing a mapping
> Documentation/devicetree/bindings/power/qcom,kpss-acc-v2.yaml: ignoring, error parsing file
>
>
> [...]

Applied, thanks!

[1/1] dt-bindings: arm: msm: kpss-acc: Revert "dt-bindings: arm: msm: kpss-acc: Make the optional reg truly optional"
commit: 2f17bea81aff53155689de776e52328f82bf603c

Best regards,
--
Bjorn Andersson <[email protected]>