* Re: [PATCH] dt-bindings: interrupt-controller: arm,gic: Correct VGIC interrupt description
2024-12-17 6:12 [PATCH] dt-bindings: interrupt-controller: arm,gic: Correct VGIC interrupt description Krzysztof Kozlowski
@ 2024-12-17 8:13 ` Marc Zyngier
2024-12-17 18:23 ` Conor Dooley
2024-12-18 19:29 ` Rob Herring (Arm)
2 siblings, 0 replies; 4+ messages in thread
From: Marc Zyngier @ 2024-12-17 8:13 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Thomas Gleixner, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
linux-arm-kernel, linux-kernel, devicetree
On Tue, 17 Dec 2024 06:12:26 +0000,
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote:
>
> The description of VGIC interrupt referenced obsolete "see below" after
> converting TXT to DT Schema in commit 66ed144f147a ("dt-bindings:
> interrupt-controller: Convert ARM GIC to json-schema"), because there is
> no dedicated "VGIC" chapter anymore below.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
> .../devicetree/bindings/interrupt-controller/arm,gic.yaml | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/interrupt-controller/arm,gic.yaml b/Documentation/devicetree/bindings/interrupt-controller/arm,gic.yaml
> index a2846e493497..7173c4b5a228 100644
> --- a/Documentation/devicetree/bindings/interrupt-controller/arm,gic.yaml
> +++ b/Documentation/devicetree/bindings/interrupt-controller/arm,gic.yaml
> @@ -110,8 +110,8 @@ properties:
>
> interrupts:
> description: Interrupt source of the parent interrupt controller on
> - secondary GICs, or VGIC maintenance interrupt on primary GIC (see
> - below).
> + secondary GICs, or VGIC maintenance interrupt on primary GIC (see "GICv2
> + with virtualization extensions" paragraph in the "reg" property).
> maxItems: 1
>
> cpu-offset:
Acked-by: Marc Zyngier <maz@kernel.org>
M.
--
Without deviation from the norm, progress is not possible.
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] dt-bindings: interrupt-controller: arm,gic: Correct VGIC interrupt description
2024-12-17 6:12 [PATCH] dt-bindings: interrupt-controller: arm,gic: Correct VGIC interrupt description Krzysztof Kozlowski
2024-12-17 8:13 ` Marc Zyngier
@ 2024-12-17 18:23 ` Conor Dooley
2024-12-18 19:29 ` Rob Herring (Arm)
2 siblings, 0 replies; 4+ messages in thread
From: Conor Dooley @ 2024-12-17 18:23 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Marc Zyngier, Thomas Gleixner, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, linux-arm-kernel, linux-kernel, devicetree
[-- Attachment #1: Type: text/plain, Size: 1470 bytes --]
On Tue, Dec 17, 2024 at 07:12:26AM +0100, Krzysztof Kozlowski wrote:
> The description of VGIC interrupt referenced obsolete "see below" after
> converting TXT to DT Schema in commit 66ed144f147a ("dt-bindings:
> interrupt-controller: Convert ARM GIC to json-schema"), because there is
> no dedicated "VGIC" chapter anymore below.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
> .../devicetree/bindings/interrupt-controller/arm,gic.yaml | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/interrupt-controller/arm,gic.yaml b/Documentation/devicetree/bindings/interrupt-controller/arm,gic.yaml
> index a2846e493497..7173c4b5a228 100644
> --- a/Documentation/devicetree/bindings/interrupt-controller/arm,gic.yaml
> +++ b/Documentation/devicetree/bindings/interrupt-controller/arm,gic.yaml
> @@ -110,8 +110,8 @@ properties:
>
> interrupts:
> description: Interrupt source of the parent interrupt controller on
> - secondary GICs, or VGIC maintenance interrupt on primary GIC (see
> - below).
> + secondary GICs, or VGIC maintenance interrupt on primary GIC (see "GICv2
> + with virtualization extensions" paragraph in the "reg" property).
I don't think things like this /should/ need a dt ack, even if they
weren't from a dt maintainer - but in case one's expected
Acked-by: Conor Dooley <conor.dooley@microchip.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] dt-bindings: interrupt-controller: arm,gic: Correct VGIC interrupt description
2024-12-17 6:12 [PATCH] dt-bindings: interrupt-controller: arm,gic: Correct VGIC interrupt description Krzysztof Kozlowski
2024-12-17 8:13 ` Marc Zyngier
2024-12-17 18:23 ` Conor Dooley
@ 2024-12-18 19:29 ` Rob Herring (Arm)
2 siblings, 0 replies; 4+ messages in thread
From: Rob Herring (Arm) @ 2024-12-18 19:29 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: linux-arm-kernel, devicetree, Thomas Gleixner, linux-kernel,
Conor Dooley, Marc Zyngier, Krzysztof Kozlowski
On Tue, 17 Dec 2024 07:12:26 +0100, Krzysztof Kozlowski wrote:
> The description of VGIC interrupt referenced obsolete "see below" after
> converting TXT to DT Schema in commit 66ed144f147a ("dt-bindings:
> interrupt-controller: Convert ARM GIC to json-schema"), because there is
> no dedicated "VGIC" chapter anymore below.
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
> .../devicetree/bindings/interrupt-controller/arm,gic.yaml | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
Applied, thanks!
^ permalink raw reply [flat|nested] 4+ messages in thread