Re: [PATCH v4 05/18] dt-bindings: timer: arm,arch_timer: Add interrupt-names support

2021-04-06 Thread Rob Herring
On Fri, 02 Apr 2021 18:05:29 +0900, Hector Martin wrote:
> Not all platforms provide the same set of timers/interrupts, and Linux
> only needs one (plus kvm/guest ones); some platforms are working around
> this by using dummy fake interrupts. Implementing interrupt-names allows
> the devicetree to specify an arbitrary set of available interrupts, so
> the timer code can pick the right one.
> 
> This also adds the hyp-virt timer/interrupt, which was previously not
> expressed in the fixed 4-interrupt form.
> 
> Reviewed-by: Linus Walleij 
> Acked-by: Marc Zyngier 
> Reviewed-by: Tony Lindgren 
> Signed-off-by: Hector Martin 
> ---
>  .../bindings/timer/arm,arch_timer.yaml| 19 +++
>  1 file changed, 19 insertions(+)
> 

Reviewed-by: Rob Herring 


[PATCH v4 05/18] dt-bindings: timer: arm,arch_timer: Add interrupt-names support

2021-04-02 Thread Hector Martin
Not all platforms provide the same set of timers/interrupts, and Linux
only needs one (plus kvm/guest ones); some platforms are working around
this by using dummy fake interrupts. Implementing interrupt-names allows
the devicetree to specify an arbitrary set of available interrupts, so
the timer code can pick the right one.

This also adds the hyp-virt timer/interrupt, which was previously not
expressed in the fixed 4-interrupt form.

Reviewed-by: Linus Walleij 
Acked-by: Marc Zyngier 
Reviewed-by: Tony Lindgren 
Signed-off-by: Hector Martin 
---
 .../bindings/timer/arm,arch_timer.yaml| 19 +++
 1 file changed, 19 insertions(+)

diff --git a/Documentation/devicetree/bindings/timer/arm,arch_timer.yaml 
b/Documentation/devicetree/bindings/timer/arm,arch_timer.yaml
index 2c75105c1398..7f5e3af58255 100644
--- a/Documentation/devicetree/bindings/timer/arm,arch_timer.yaml
+++ b/Documentation/devicetree/bindings/timer/arm,arch_timer.yaml
@@ -34,11 +34,30 @@ properties:
   - arm,armv8-timer
 
   interrupts:
+minItems: 1
+maxItems: 5
 items:
   - description: secure timer irq
   - description: non-secure timer irq
   - description: virtual timer irq
   - description: hypervisor timer irq
+  - description: hypervisor virtual timer irq
+
+  interrupt-names:
+oneOf:
+  - minItems: 2
+items:
+  - const: phys
+  - const: virt
+  - const: hyp-phys
+  - const: hyp-virt
+  - minItems: 3
+items:
+  - const: sec-phys
+  - const: phys
+  - const: virt
+  - const: hyp-phys
+  - const: hyp-virt
 
   clock-frequency:
 description: The frequency of the main counter, in Hz. Should be present
-- 
2.30.0