diff mbox series

[2/3] dt-bindings: interrupt-controller: arm,gic-v3: Make 'interrupts' optional

Message ID 20220822152224.507497-3-jean-philippe@linaro.org
State Accepted, archived
Headers show
Series dt-bindings: Some schema fixes | expand

Checks

Context Check Description
robh/checkpatch success
robh/patch-applied success
robh/dtbs-check warning build log
robh/dt-meta-schema success

Commit Message

Jean-Philippe Brucker Aug. 22, 2022, 3:22 p.m. UTC
A GICv3 implementation without virtualization, such as the base QEMU
virt machine (without -M virtualization=on), does not issue maintenance
interrupts. Therefore its device-tree node does not need an 'interrupts'
property. Currently, validating the QEMU virt device-tree throws a
warning that 'interrupts' is missing. Make it optional.

Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
---
 .../devicetree/bindings/interrupt-controller/arm,gic-v3.yaml     | 1 -
 1 file changed, 1 deletion(-)

Comments

Rob Herring Aug. 22, 2022, 7:46 p.m. UTC | #1
On Mon, Aug 22, 2022 at 10:23 AM Jean-Philippe Brucker
<jean-philippe@linaro.org> wrote:
>
> A GICv3 implementation without virtualization, such as the base QEMU
> virt machine (without -M virtualization=on), does not issue maintenance
> interrupts. Therefore its device-tree node does not need an 'interrupts'
> property. Currently, validating the QEMU virt device-tree throws a
> warning that 'interrupts' is missing. Make it optional.
>
> Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
> ---
>  .../devicetree/bindings/interrupt-controller/arm,gic-v3.yaml     | 1 -
>  1 file changed, 1 deletion(-)

Reviewed-by: Rob Herring <robh@kernel.org>
Marc Zyngier Aug. 22, 2022, 7:47 p.m. UTC | #2
On Mon, 22 Aug 2022 16:22:23 +0100,
Jean-Philippe Brucker <jean-philippe@linaro.org> wrote:
> 
> A GICv3 implementation without virtualization, such as the base QEMU
> virt machine (without -M virtualization=on), does not issue maintenance
> interrupts. Therefore its device-tree node does not need an 'interrupts'
> property. Currently, validating the QEMU virt device-tree throws a
> warning that 'interrupts' is missing. Make it optional.
> 
> Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>

Acked-by: Marc Zyngier <maz@kernel.org>

	M.
Rob Herring Aug. 23, 2022, 3:41 p.m. UTC | #3
On Mon, 22 Aug 2022 16:22:23 +0100, Jean-Philippe Brucker wrote:
> A GICv3 implementation without virtualization, such as the base QEMU
> virt machine (without -M virtualization=on), does not issue maintenance
> interrupts. Therefore its device-tree node does not need an 'interrupts'
> property. Currently, validating the QEMU virt device-tree throws a
> warning that 'interrupts' is missing. Make it optional.
> 
> Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
> ---
>  .../devicetree/bindings/interrupt-controller/arm,gic-v3.yaml     | 1 -
>  1 file changed, 1 deletion(-)
> 

Applied, thanks!
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/interrupt-controller/arm,gic-v3.yaml b/Documentation/devicetree/bindings/interrupt-controller/arm,gic-v3.yaml
index 3912a89162f0..9f7d3e11aacb 100644
--- a/Documentation/devicetree/bindings/interrupt-controller/arm,gic-v3.yaml
+++ b/Documentation/devicetree/bindings/interrupt-controller/arm,gic-v3.yaml
@@ -170,7 +170,6 @@  dependencies:
 
 required:
   - compatible
-  - interrupts
   - reg
 
 patternProperties: