* [PATCH net-next] dt-bindings: net: dsa: renesas,rzn1-a5psw: add interrupts description
@ 2022-06-29 9:13 Clément Léger
2022-06-30 16:17 ` Rob Herring
2022-06-30 16:20 ` Clément Léger
0 siblings, 2 replies; 3+ messages in thread
From: Clément Léger @ 2022-06-29 9:13 UTC (permalink / raw)
To: Andrew Lunn, Vivien Didelot, Florian Fainelli, Vladimir Oltean,
David S. Miller, Eric Dumazet, Jakub Kicinski, Paolo Abeni,
Rob Herring, Krzysztof Kozlowski, Geert Uytterhoeven
Cc: Clément Léger, linux-renesas-soc, netdev, devicetree,
linux-kernel, Thomas Petazzoni, Herve Codina, Miquèl Raynal,
Milan Stevanovic, Jimmy Lalande, Pascal Eberhard
Describe the switch interrupts (dlr, switch, prp, hub, pattern) which
are connected to the GIC.
Signed-off-by: Clément Léger <clement.leger@bootlin.com>
---
.../bindings/net/dsa/renesas,rzn1-a5psw.yaml | 23 +++++++++++++++++++
1 file changed, 23 insertions(+)
diff --git a/Documentation/devicetree/bindings/net/dsa/renesas,rzn1-a5psw.yaml b/Documentation/devicetree/bindings/net/dsa/renesas,rzn1-a5psw.yaml
index 103b1ef5af1b..51f274c16ed1 100644
--- a/Documentation/devicetree/bindings/net/dsa/renesas,rzn1-a5psw.yaml
+++ b/Documentation/devicetree/bindings/net/dsa/renesas,rzn1-a5psw.yaml
@@ -26,6 +26,22 @@ properties:
reg:
maxItems: 1
+ interrupts:
+ items:
+ - description: DLR interrupt
+ - description: Switch interrupt
+ - description: PRP interrupt
+ - description: Integrated HUB module interrupt
+ - description: RX Pattern interrupt
+
+ interrupts-names:
+ items:
+ - const: dlr
+ - const: switch
+ - const: prp
+ - const: hub
+ - const: ptrn
+
power-domains:
maxItems: 1
@@ -76,6 +92,7 @@ examples:
- |
#include <dt-bindings/gpio/gpio.h>
#include <dt-bindings/clock/r9a06g032-sysctrl.h>
+ #include <dt-bindings/interrupt-controller/arm-gic.h>
switch@44050000 {
compatible = "renesas,r9a06g032-a5psw", "renesas,rzn1-a5psw";
@@ -83,6 +100,12 @@ examples:
clocks = <&sysctrl R9A06G032_HCLK_SWITCH>, <&sysctrl R9A06G032_CLK_SWITCH>;
clock-names = "hclk", "clk";
power-domains = <&sysctrl>;
+ interrupts = <GIC_SPI 40 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 42 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 43 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 44 IRQ_TYPE_LEVEL_HIGH>,
+ <GIC_SPI 45 IRQ_TYPE_LEVEL_HIGH>;
+ interrupts-name = "dlr", "switch", "prp", "hub", "ptrn";
dsa,member = <0 0>;
--
2.36.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH net-next] dt-bindings: net: dsa: renesas,rzn1-a5psw: add interrupts description
2022-06-29 9:13 [PATCH net-next] dt-bindings: net: dsa: renesas,rzn1-a5psw: add interrupts description Clément Léger
@ 2022-06-30 16:17 ` Rob Herring
2022-06-30 16:20 ` Clément Léger
1 sibling, 0 replies; 3+ messages in thread
From: Rob Herring @ 2022-06-30 16:17 UTC (permalink / raw)
To: Clément Léger
Cc: linux-kernel, netdev, Andrew Lunn, Thomas Petazzoni,
Florian Fainelli, David S. Miller, Miquèl Raynal,
Vladimir Oltean, Jakub Kicinski, Rob Herring, Paolo Abeni,
linux-renesas-soc, devicetree, Eric Dumazet, Pascal Eberhard,
Geert Uytterhoeven, Krzysztof Kozlowski, Herve Codina,
Vivien Didelot, Milan Stevanovic, Jimmy Lalande
On Wed, 29 Jun 2022 11:13:04 +0200, Clément Léger wrote:
> Describe the switch interrupts (dlr, switch, prp, hub, pattern) which
> are connected to the GIC.
>
> Signed-off-by: Clément Léger <clement.leger@bootlin.com>
> ---
> .../bindings/net/dsa/renesas,rzn1-a5psw.yaml | 23 +++++++++++++++++++
> 1 file changed, 23 insertions(+)
>
Reviewed-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH net-next] dt-bindings: net: dsa: renesas,rzn1-a5psw: add interrupts description
2022-06-29 9:13 [PATCH net-next] dt-bindings: net: dsa: renesas,rzn1-a5psw: add interrupts description Clément Léger
2022-06-30 16:17 ` Rob Herring
@ 2022-06-30 16:20 ` Clément Léger
1 sibling, 0 replies; 3+ messages in thread
From: Clément Léger @ 2022-06-30 16:20 UTC (permalink / raw)
To: Andrew Lunn, Vivien Didelot, Florian Fainelli, Vladimir Oltean,
David S. Miller, Eric Dumazet, Jakub Kicinski, Paolo Abeni,
Rob Herring, Krzysztof Kozlowski, Geert Uytterhoeven
Cc: linux-renesas-soc, netdev, devicetree, linux-kernel,
Thomas Petazzoni, Herve Codina, Miquèl Raynal,
Milan Stevanovic, Jimmy Lalande, Pascal Eberhard
Le Wed, 29 Jun 2022 11:13:04 +0200,
Clément Léger <clement.leger@bootlin.com> a écrit :
> Describe the switch interrupts (dlr, switch, prp, hub, pattern) which
> are connected to the GIC.
>
> Signed-off-by: Clément Léger <clement.leger@bootlin.com>
> ---
> .../bindings/net/dsa/renesas,rzn1-a5psw.yaml | 23 +++++++++++++++++++
> 1 file changed, 23 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/net/dsa/renesas,rzn1-a5psw.yaml b/Documentation/devicetree/bindings/net/dsa/renesas,rzn1-a5psw.yaml
> index 103b1ef5af1b..51f274c16ed1 100644
> --- a/Documentation/devicetree/bindings/net/dsa/renesas,rzn1-a5psw.yaml
> +++ b/Documentation/devicetree/bindings/net/dsa/renesas,rzn1-a5psw.yaml
> @@ -26,6 +26,22 @@ properties:
> reg:
> maxItems: 1
>
> + interrupts:
> + items:
> + - description: DLR interrupt
> + - description: Switch interrupt
> + - description: PRP interrupt
> + - description: Integrated HUB module interrupt
> + - description: RX Pattern interrupt
> +
> + interrupts-names:
> + items:
> + - const: dlr
> + - const: switch
> + - const: prp
> + - const: hub
> + - const: ptrn
> +
> power-domains:
> maxItems: 1
>
> @@ -76,6 +92,7 @@ examples:
> - |
> #include <dt-bindings/gpio/gpio.h>
> #include <dt-bindings/clock/r9a06g032-sysctrl.h>
> + #include <dt-bindings/interrupt-controller/arm-gic.h>
>
> switch@44050000 {
> compatible = "renesas,r9a06g032-a5psw", "renesas,rzn1-a5psw";
> @@ -83,6 +100,12 @@ examples:
> clocks = <&sysctrl R9A06G032_HCLK_SWITCH>, <&sysctrl R9A06G032_CLK_SWITCH>;
> clock-names = "hclk", "clk";
> power-domains = <&sysctrl>;
> + interrupts = <GIC_SPI 40 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 42 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 43 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 44 IRQ_TYPE_LEVEL_HIGH>,
> + <GIC_SPI 45 IRQ_TYPE_LEVEL_HIGH>;
> + interrupts-name = "dlr", "switch", "prp", "hub", "ptrn";
Wait, there is actually a typo both here and in the property
description, should be "interrupt-names". Was not catched by
dt_binding_check though but probably due to the fact additionnal
properties are allowed.
>
> dsa,member = <0 0>;
>
--
Clément Léger,
Embedded Linux and Kernel engineer at Bootlin
https://bootlin.com
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-06-30 16:21 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-29 9:13 [PATCH net-next] dt-bindings: net: dsa: renesas,rzn1-a5psw: add interrupts description Clément Léger
2022-06-30 16:17 ` Rob Herring
2022-06-30 16:20 ` Clément Léger
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).