dt-bindings: power: renesas,apmu: Fix cpus property limits

[PATCH] dt-bindings: power: renesas,apmu: Fix cpus property limits

Posted by Geert Uytterhoeven 4 years, 2 months ago

"make dtbs_check":

    arch/arm/boot/dts/r8a7791-koelsch.dtb: apmu@e6152000: cpus:0: [6, 7] is too long
	    From schema: Documentation/devicetree/bindings/power/renesas,apmu.yaml

Correct the minimum and maximum number of CPUs controlled by a single
APMU instance.

Fixes: 39bd2b6a3783b899 ("dt-bindings: Improve phandle-array schemas")
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 Documentation/devicetree/bindings/power/renesas,apmu.yaml | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/power/renesas,apmu.yaml b/Documentation/devicetree/bindings/power/renesas,apmu.yaml
index 540e6fe1b7430fcd..f5d97d651e828981 100644
--- a/Documentation/devicetree/bindings/power/renesas,apmu.yaml
+++ b/Documentation/devicetree/bindings/power/renesas,apmu.yaml
@@ -36,7 +36,8 @@ properties:
   cpus:
     $ref: /schemas/types.yaml#/definitions/phandle-array
     items:
-      maxItems: 1
+      minItems: 1
+      maxItems: 4
     description: |
       Array of phandles pointing to CPU cores, which should match the order of
       CPU cores used by the WUPCR and PSTR registers in the Advanced Power
-- 
2.25.1

Re: [PATCH] dt-bindings: power: renesas,apmu: Fix cpus property limits

Posted by Krzysztof Kozlowski 4 years, 2 months ago

On 30/03/2022 15:04, Geert Uytterhoeven wrote:
> "make dtbs_check":
> 
>     arch/arm/boot/dts/r8a7791-koelsch.dtb: apmu@e6152000: cpus:0: [6, 7] is too long
> 	    From schema: Documentation/devicetree/bindings/power/renesas,apmu.yaml
> 
> Correct the minimum and maximum number of CPUs controlled by a single
> APMU instance.
> 
> Fixes: 39bd2b6a3783b899 ("dt-bindings: Improve phandle-array schemas")
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
>  Documentation/devicetree/bindings/power/renesas,apmu.yaml | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 


Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>


Best regards,
Krzysztof

Re: [PATCH] dt-bindings: power: renesas,apmu: Fix cpus property limits

Posted by Rob Herring 4 years, 2 months ago

On Wed, 30 Mar 2022 15:04:16 +0200, Geert Uytterhoeven wrote:
> "make dtbs_check":
> 
>     arch/arm/boot/dts/r8a7791-koelsch.dtb: apmu@e6152000: cpus:0: [6, 7] is too long
> 	    From schema: Documentation/devicetree/bindings/power/renesas,apmu.yaml
> 
> Correct the minimum and maximum number of CPUs controlled by a single
> APMU instance.
> 
> Fixes: 39bd2b6a3783b899 ("dt-bindings: Improve phandle-array schemas")
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
>  Documentation/devicetree/bindings/power/renesas,apmu.yaml | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 

Applied, thanks!