2020-05-19 08:07:33

by Geert Uytterhoeven

[permalink] [raw]
Subject: [PATCH] dt-bindings: interrupt-controller: arm,gic: Document resets property

A generic GIC block embedded in an SoC may be connected to an on-SoC
reset controller. Hence allow the DTS writer to describe this relation,
by documenting the optional presence of a "reset" property.

This gets rid of "make dtbs_check" warnings like:

arch/arm/boot/dts/r8a7791-porter.dt.yaml: interrupt-controller@f1001000: 'resets' does not match any of the regexes: '^v2m@[0-9a-f]+$', 'pinctrl-[0-9]+'

Signed-off-by: Geert Uytterhoeven <[email protected]>
---
.../devicetree/bindings/interrupt-controller/arm,gic.yaml | 3 +++
1 file changed, 3 insertions(+)

diff --git a/Documentation/devicetree/bindings/interrupt-controller/arm,gic.yaml b/Documentation/devicetree/bindings/interrupt-controller/arm,gic.yaml
index 9a47820ef34649dd..caefcc50bcf92c92 100644
--- a/Documentation/devicetree/bindings/interrupt-controller/arm,gic.yaml
+++ b/Documentation/devicetree/bindings/interrupt-controller/arm,gic.yaml
@@ -125,6 +125,9 @@ properties:
power-domains:
maxItems: 1

+ resets:
+ maxItems: 1
+
required:
- compatible
- reg
--
2.17.1


2020-05-28 21:16:03

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: interrupt-controller: arm, gic: Document resets property

On Tue, 19 May 2020 10:05:19 +0200, Geert Uytterhoeven wrote:
> A generic GIC block embedded in an SoC may be connected to an on-SoC
> reset controller. Hence allow the DTS writer to describe this relation,
> by documenting the optional presence of a "reset" property.
>
> This gets rid of "make dtbs_check" warnings like:
>
> arch/arm/boot/dts/r8a7791-porter.dt.yaml: interrupt-controller@f1001000: 'resets' does not match any of the regexes: '^v2m@[0-9a-f]+$', 'pinctrl-[0-9]+'
>
> Signed-off-by: Geert Uytterhoeven <[email protected]>
> ---
> .../devicetree/bindings/interrupt-controller/arm,gic.yaml | 3 +++
> 1 file changed, 3 insertions(+)
>

Applied, thanks!