* [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG binding
2022-05-10 15:11 [RFC 0/8] Add RZ/G2L POEG support Biju Das
@ 2022-05-10 15:11 ` Biju Das
2022-05-17 21:04 ` Rob Herring
2022-05-10 15:11 ` [RFC 2/8] drivers: soc: renesas: Add POEG driver support Biju Das
` (7 subsequent siblings)
8 siblings, 1 reply; 22+ messages in thread
From: Biju Das @ 2022-05-10 15:11 UTC (permalink / raw)
To: Rob Herring, Thierry Reding, Lee Jones, Krzysztof Kozlowski
Cc: Biju Das, linux-pwm, devicetree, Geert Uytterhoeven,
Chris Paterson, Biju Das, Prabhakar Mahadev Lad,
linux-renesas-soc
Add device tree bindings for the RZ/G2L Port Output Enable for GPT (POEG).
Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
---
.../soc/renesas/renesas,rzg2l-poeg.yaml | 65 +++++++++++++++++++
1 file changed, 65 insertions(+)
create mode 100644 Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-poeg.yaml
diff --git a/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-poeg.yaml b/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-poeg.yaml
new file mode 100644
index 000000000000..5737dbf3fa45
--- /dev/null
+++ b/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-poeg.yaml
@@ -0,0 +1,65 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: "http://devicetree.org/schemas/soc/renesas/renesas,rzg2l-poeg.yaml#"
+$schema: "http://devicetree.org/meta-schemas/core.yaml#"
+
+title: Renesas RZ/G2L Port Output Enable for GPT (POEG)
+
+maintainers:
+ - Biju Das <biju.das.jz@bp.renesas.com>
+
+description:
+ The output pins of the general PWM timer (GPT) can be disabled by using
+ the port output enabling function for the GPT (POEG). Specifically,
+ either of the following ways can be used.
+ * Input level detection of the GTETRGA to GTETRGD pins.
+ * Output-disable request from the GPT.
+ * Register settings.
+
+properties:
+ compatible:
+ items:
+ - enum:
+ - renesas,r9a07g044-poeg # RZ/G2{L,LC}
+ - renesas,r9a07g054-poeg # RZ/V2L
+ - const: renesas,rzg2l-poeg
+
+ reg:
+ maxItems: 1
+
+ interrupts:
+ maxItems: 1
+
+ clocks:
+ maxItems: 1
+
+ power-domains:
+ maxItems: 1
+
+ resets:
+ maxItems: 1
+
+required:
+ - compatible
+ - reg
+ - interrupts
+ - clocks
+ - power-domains
+ - resets
+
+additionalProperties: false
+
+examples:
+ - |
+ #include <dt-bindings/clock/r9a07g044-cpg.h>
+ #include <dt-bindings/interrupt-controller/arm-gic.h>
+
+ poeggd: poeg@10049400 {
+ compatible = "renesas,r9a07g044-poeg", "renesas,rzg2l-poeg";
+ reg = <0x10049400 0x4>;
+ interrupts = <GIC_SPI 325 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&cpg CPG_MOD R9A07G044_POEG_D_CLKP>;
+ power-domains = <&cpg>;
+ resets = <&cpg R9A07G044_POEG_D_RST>;
+ };
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread* Re: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG binding
2022-05-10 15:11 ` [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG binding Biju Das
@ 2022-05-17 21:04 ` Rob Herring
2022-05-18 5:58 ` Biju Das
0 siblings, 1 reply; 22+ messages in thread
From: Rob Herring @ 2022-05-17 21:04 UTC (permalink / raw)
To: Biju Das
Cc: Thierry Reding, Lee Jones, Krzysztof Kozlowski, linux-pwm,
devicetree, Geert Uytterhoeven, Chris Paterson, Biju Das,
Prabhakar Mahadev Lad, linux-renesas-soc
On Tue, May 10, 2022 at 04:11:05PM +0100, Biju Das wrote:
> Add device tree bindings for the RZ/G2L Port Output Enable for GPT (POEG).
>
> Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> ---
> .../soc/renesas/renesas,rzg2l-poeg.yaml | 65 +++++++++++++++++++
> 1 file changed, 65 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-poeg.yaml
>
> diff --git a/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-poeg.yaml b/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-poeg.yaml
> new file mode 100644
> index 000000000000..5737dbf3fa45
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-poeg.yaml
> @@ -0,0 +1,65 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: "http://devicetree.org/schemas/soc/renesas/renesas,rzg2l-poeg.yaml#"
> +$schema: "http://devicetree.org/meta-schemas/core.yaml#"
> +
> +title: Renesas RZ/G2L Port Output Enable for GPT (POEG)
> +
> +maintainers:
> + - Biju Das <biju.das.jz@bp.renesas.com>
> +
> +description:
'|' needed.
> + The output pins of the general PWM timer (GPT) can be disabled by using
> + the port output enabling function for the GPT (POEG). Specifically,
> + either of the following ways can be used.
> + * Input level detection of the GTETRGA to GTETRGD pins.
> + * Output-disable request from the GPT.
> + * Register settings.
> +
> +properties:
> + compatible:
> + items:
> + - enum:
> + - renesas,r9a07g044-poeg # RZ/G2{L,LC}
> + - renesas,r9a07g054-poeg # RZ/V2L
> + - const: renesas,rzg2l-poeg
> +
> + reg:
> + maxItems: 1
> +
> + interrupts:
> + maxItems: 1
> +
> + clocks:
> + maxItems: 1
> +
> + power-domains:
> + maxItems: 1
> +
> + resets:
> + maxItems: 1
> +
> +required:
> + - compatible
> + - reg
> + - interrupts
> + - clocks
> + - power-domains
> + - resets
> +
> +additionalProperties: false
> +
> +examples:
> + - |
> + #include <dt-bindings/clock/r9a07g044-cpg.h>
> + #include <dt-bindings/interrupt-controller/arm-gic.h>
> +
> + poeggd: poeg@10049400 {
> + compatible = "renesas,r9a07g044-poeg", "renesas,rzg2l-poeg";
> + reg = <0x10049400 0x4>;
This looks like it is part of some larger block?
> + interrupts = <GIC_SPI 325 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&cpg CPG_MOD R9A07G044_POEG_D_CLKP>;
> + power-domains = <&cpg>;
> + resets = <&cpg R9A07G044_POEG_D_RST>;
> + };
> --
> 2.25.1
>
>
^ permalink raw reply [flat|nested] 22+ messages in thread* RE: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG binding
2022-05-17 21:04 ` Rob Herring
@ 2022-05-18 5:58 ` Biju Das
2022-05-18 18:17 ` Rob Herring
0 siblings, 1 reply; 22+ messages in thread
From: Biju Das @ 2022-05-18 5:58 UTC (permalink / raw)
To: Rob Herring
Cc: Thierry Reding, Lee Jones, Krzysztof Kozlowski, linux-pwm,
devicetree, Geert Uytterhoeven, Chris Paterson, Biju Das,
Prabhakar Mahadev Lad, linux-renesas-soc, Uwe Kleine-König
Hi Rob,
Thanks for the feedback.
> Subject: Re: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG binding
>
> On Tue, May 10, 2022 at 04:11:05PM +0100, Biju Das wrote:
> > Add device tree bindings for the RZ/G2L Port Output Enable for GPT
> (POEG).
> >
> > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> > ---
> > .../soc/renesas/renesas,rzg2l-poeg.yaml | 65 +++++++++++++++++++
> > 1 file changed, 65 insertions(+)
> > create mode 100644
> > Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-poeg.yaml
> >
> > diff --git
> > a/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-poeg.yam
> > l
> > b/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-poeg.yam
> > l
> > new file mode 100644
> > index 000000000000..5737dbf3fa45
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-poeg
> > +++ .yaml
> > @@ -0,0 +1,65 @@
> > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) %YAML 1.2
> > +---
> > +$id:
"
> > +
> > +title: Renesas RZ/G2L Port Output Enable for GPT (POEG)
> > +
> > +maintainers:
> > + - Biju Das <biju.das.jz@bp.renesas.com>
> > +
> > +description:
>
> '|' needed.
OK.
>
> > + The output pins of the general PWM timer (GPT) can be disabled by
> > + using the port output enabling function for the GPT (POEG).
> > + Specifically, either of the following ways can be used.
> > + * Input level detection of the GTETRGA to GTETRGD pins.
> > + * Output-disable request from the GPT.
> > + * Register settings.
> > +
> > +properties:
> > + compatible:
> > + items:
> > + - enum:
> > + - renesas,r9a07g044-poeg # RZ/G2{L,LC}
> > + - renesas,r9a07g054-poeg # RZ/V2L
> > + - const: renesas,rzg2l-poeg
> > +
> > + reg:
> > + maxItems: 1
> > +
> > + interrupts:
> > + maxItems: 1
> > +
> > + clocks:
> > + maxItems: 1
> > +
> > + power-domains:
> > + maxItems: 1
> > +
> > + resets:
> > + maxItems: 1
> > +
> > +required:
> > + - compatible
> > + - reg
> > + - interrupts
> > + - clocks
> > + - power-domains
> > + - resets
> > +
> > +additionalProperties: false
> > +
> > +examples:
> > + - |
> > + #include <dt-bindings/clock/r9a07g044-cpg.h>
> > + #include <dt-bindings/interrupt-controller/arm-gic.h>
> > +
> > + poeggd: poeg@10049400 {
> > + compatible = "renesas,r9a07g044-poeg", "renesas,rzg2l-poeg";
> > + reg = <0x10049400 0x4>;
>
> This looks like it is part of some larger block?
There are 2 IP blocks GPT(PWM) and POEG with its own resources like (register map, clk, reset and interrupts)
Larger block is GPT, which has lot of functionalities. The output from GPT block can be disabled
by this IP either by external trigger, request from GPT(Deadtime error, both output low/high)
or explicit software control). This IP has only a single register. Currently I am not sure which framework
to be used for this IP?? Or should it be merged with larger block GPT by combining the resources?
Cheers,
Biju
>
> > + interrupts = <GIC_SPI 325 IRQ_TYPE_LEVEL_HIGH>;
> > + clocks = <&cpg CPG_MOD R9A07G044_POEG_D_CLKP>;
> > + power-domains = <&cpg>;
> > + resets = <&cpg R9A07G044_POEG_D_RST>;
> > + };
> > --
> > 2.25.1
> >
> >
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG binding
2022-05-18 5:58 ` Biju Das
@ 2022-05-18 18:17 ` Rob Herring
2022-05-18 18:34 ` Biju Das
0 siblings, 1 reply; 22+ messages in thread
From: Rob Herring @ 2022-05-18 18:17 UTC (permalink / raw)
To: Biju Das
Cc: Thierry Reding, Lee Jones, Krzysztof Kozlowski, linux-pwm,
devicetree, Geert Uytterhoeven, Chris Paterson, Biju Das,
Prabhakar Mahadev Lad, linux-renesas-soc, Uwe Kleine-König
On Wed, May 18, 2022 at 05:58:00AM +0000, Biju Das wrote:
> Hi Rob,
>
> Thanks for the feedback.
>
> > Subject: Re: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG binding
> >
> > On Tue, May 10, 2022 at 04:11:05PM +0100, Biju Das wrote:
> > > Add device tree bindings for the RZ/G2L Port Output Enable for GPT
> > (POEG).
> > >
> > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> > > ---
> > > .../soc/renesas/renesas,rzg2l-poeg.yaml | 65 +++++++++++++++++++
> > > 1 file changed, 65 insertions(+)
> > > create mode 100644
> > > Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-poeg.yaml
> > >
> > > diff --git
> > > a/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-poeg.yam
> > > l
> > > b/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-poeg.yam
> > > l
> > > new file mode 100644
> > > index 000000000000..5737dbf3fa45
> > > --- /dev/null
> > > +++ b/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-poeg
> > > +++ .yaml
> > > @@ -0,0 +1,65 @@
> > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) %YAML 1.2
> > > +---
> > > +$id:
> "
> > > +
> > > +title: Renesas RZ/G2L Port Output Enable for GPT (POEG)
> > > +
> > > +maintainers:
> > > + - Biju Das <biju.das.jz@bp.renesas.com>
> > > +
> > > +description:
> >
> > '|' needed.
>
> OK.
>
> >
> > > + The output pins of the general PWM timer (GPT) can be disabled by
> > > + using the port output enabling function for the GPT (POEG).
> > > + Specifically, either of the following ways can be used.
> > > + * Input level detection of the GTETRGA to GTETRGD pins.
> > > + * Output-disable request from the GPT.
> > > + * Register settings.
> > > +
> > > +properties:
> > > + compatible:
> > > + items:
> > > + - enum:
> > > + - renesas,r9a07g044-poeg # RZ/G2{L,LC}
> > > + - renesas,r9a07g054-poeg # RZ/V2L
> > > + - const: renesas,rzg2l-poeg
> > > +
> > > + reg:
> > > + maxItems: 1
> > > +
> > > + interrupts:
> > > + maxItems: 1
> > > +
> > > + clocks:
> > > + maxItems: 1
> > > +
> > > + power-domains:
> > > + maxItems: 1
> > > +
> > > + resets:
> > > + maxItems: 1
> > > +
> > > +required:
> > > + - compatible
> > > + - reg
> > > + - interrupts
> > > + - clocks
> > > + - power-domains
> > > + - resets
> > > +
> > > +additionalProperties: false
> > > +
> > > +examples:
> > > + - |
> > > + #include <dt-bindings/clock/r9a07g044-cpg.h>
> > > + #include <dt-bindings/interrupt-controller/arm-gic.h>
> > > +
> > > + poeggd: poeg@10049400 {
> > > + compatible = "renesas,r9a07g044-poeg", "renesas,rzg2l-poeg";
> > > + reg = <0x10049400 0x4>;
> >
> > This looks like it is part of some larger block?
>
> There are 2 IP blocks GPT(PWM) and POEG with its own resources like (register map, clk, reset and interrupts)
>
> Larger block is GPT, which has lot of functionalities. The output from GPT block can be disabled
> by this IP either by external trigger, request from GPT(Deadtime error, both output low/high)
> or explicit software control). This IP has only a single register. Currently I am not sure which framework
> to be used for this IP?? Or should it be merged with larger block GPT by combining the resources?
Usually, IP blocks would have some minimum address alignment (typ 4K or
64K to be page aligned), but if there's no other IP in this address
range as-is is fine. The question is what's before or after the above
address?
Rob
^ permalink raw reply [flat|nested] 22+ messages in thread* RE: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG binding
2022-05-18 18:17 ` Rob Herring
@ 2022-05-18 18:34 ` Biju Das
2022-05-19 9:06 ` Geert Uytterhoeven
0 siblings, 1 reply; 22+ messages in thread
From: Biju Das @ 2022-05-18 18:34 UTC (permalink / raw)
To: Rob Herring
Cc: Thierry Reding, Lee Jones, Krzysztof Kozlowski, linux-pwm,
devicetree, Geert Uytterhoeven, Chris Paterson, Biju Das,
Prabhakar Mahadev Lad, linux-renesas-soc, Uwe Kleine-König
Hi Rob,
Thanks for the feedback.
> Subject: Re: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG binding
>
> On Wed, May 18, 2022 at 05:58:00AM +0000, Biju Das wrote:
> > Hi Rob,
> >
> > Thanks for the feedback.
> >
> > > Subject: Re: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG
> > > binding
> > >
> > > On Tue, May 10, 2022 at 04:11:05PM +0100, Biju Das wrote:
> > > > Add device tree bindings for the RZ/G2L Port Output Enable for GPT
> > > (POEG).
> > > >
> > > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> > > > ---
> > > > .../soc/renesas/renesas,rzg2l-poeg.yaml | 65
> +++++++++++++++++++
> > > > 1 file changed, 65 insertions(+)
> > > > create mode 100644
> > > > Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-poeg.y
> > > > aml
> > > >
> > > > diff --git
> > > > a/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-poeg
> > > > .yam
> > > > l
> > > > b/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-poeg
> > > > .yam
> > > > l
> > > > new file mode 100644
> > > > index 000000000000..5737dbf3fa45
> > > > --- /dev/null
> > > > +++ b/Documentation/devicetree/bindings/soc/renesas/renesas,rzg2l-
> > > > +++ poeg
> > > > +++ .yaml
> > > > @@ -0,0 +1,65 @@
> > > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) %YAML
> > > > +1.2
> > > > +---
> > > > +$id:
> > "
> > > > +
> > > > +title: Renesas RZ/G2L Port Output Enable for GPT (POEG)
> > > > +
> > > > +maintainers:
> > > > + - Biju Das <biju.das.jz@bp.renesas.com>
> > > > +
> > > > +description:
> > >
> > > '|' needed.
> >
> > OK.
> >
> > >
> > > > + The output pins of the general PWM timer (GPT) can be disabled
> > > > + by using the port output enabling function for the GPT (POEG).
> > > > + Specifically, either of the following ways can be used.
> > > > + * Input level detection of the GTETRGA to GTETRGD pins.
> > > > + * Output-disable request from the GPT.
> > > > + * Register settings.
> > > > +
> > > > +properties:
> > > > + compatible:
> > > > + items:
> > > > + - enum:
> > > > + - renesas,r9a07g044-poeg # RZ/G2{L,LC}
> > > > + - renesas,r9a07g054-poeg # RZ/V2L
> > > > + - const: renesas,rzg2l-poeg
> > > > +
> > > > + reg:
> > > > + maxItems: 1
> > > > +
> > > > + interrupts:
> > > > + maxItems: 1
> > > > +
> > > > + clocks:
> > > > + maxItems: 1
> > > > +
> > > > + power-domains:
> > > > + maxItems: 1
> > > > +
> > > > + resets:
> > > > + maxItems: 1
> > > > +
> > > > +required:
> > > > + - compatible
> > > > + - reg
> > > > + - interrupts
> > > > + - clocks
> > > > + - power-domains
> > > > + - resets
> > > > +
> > > > +additionalProperties: false
> > > > +
> > > > +examples:
> > > > + - |
> > > > + #include <dt-bindings/clock/r9a07g044-cpg.h>
> > > > + #include <dt-bindings/interrupt-controller/arm-gic.h>
> > > > +
> > > > + poeggd: poeg@10049400 {
> > > > + compatible = "renesas,r9a07g044-poeg", "renesas,rzg2l-poeg";
> > > > + reg = <0x10049400 0x4>;
> > >
> > > This looks like it is part of some larger block?
> >
> > There are 2 IP blocks GPT(PWM) and POEG with its own resources like
> > (register map, clk, reset and interrupts)
> >
> > Larger block is GPT, which has lot of functionalities. The output from
> > GPT block can be disabled by this IP either by external trigger,
> > request from GPT(Deadtime error, both output low/high) or explicit
> > software control). This IP has only a single register. Currently I am not
> sure which framework to be used for this IP?? Or should it be merged with
> larger block GPT by combining the resources?
>
> Usually, IP blocks would have some minimum address alignment (typ 4K or 64K
> to be page aligned), but if there's no other IP in this address range as-is
> is fine. The question is what's before or after the above address?
As per the HW manual, before GPT IP block and after POE3 block(Port Output Enable 3 (POE3) for MTU).
Before
H'0_1004_8000 H'0_1004_87FF 2 Kbytes GPT
After
H'0_1004_9800 H'0_1004_9BFF 1 Kbyte POE3
Please find the address map for the IP blocks near to it.
H'0_1004_A000 H'0_1004_A3FF 1 Kbyte SSIF ch1
H'0_1004_9C00 H'0_1004_9FFF 1 Kbyte SSIF ch0
H'0_1004_9800 H'0_1004_9BFF 1 Kbyte POE3
H'0_1004_9400 H'0_1004_97FF 1 Kbyte POEGD
H'0_1004_9000 H'0_1004_93FF 1 Kbyte POEGC
H'0_1004_8C00 H'0_1004_8FFF 1 Kbyte POEGB
H'0_1004_8800 H'0_1004_8BFF 1 Kbyte POEGA
H'0_1004_8000 H'0_1004_87FF 2 Kbytes GPT
H'0_1004_7000 H'0_1004_7FFF 4 Kbytes SRC (Reg)
H'0_1004_0000 H'0_1004_6FFF 28 Kbytes SRC (Memory)
Cheers,
Biju
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG binding
2022-05-18 18:34 ` Biju Das
@ 2022-05-19 9:06 ` Geert Uytterhoeven
2022-05-19 9:30 ` Biju Das
0 siblings, 1 reply; 22+ messages in thread
From: Geert Uytterhoeven @ 2022-05-19 9:06 UTC (permalink / raw)
To: Biju Das
Cc: Rob Herring, Thierry Reding, Lee Jones, Krzysztof Kozlowski,
linux-pwm, devicetree, Geert Uytterhoeven, Chris Paterson,
Biju Das, Prabhakar Mahadev Lad, linux-renesas-soc,
Uwe Kleine-König
Hi Biju,
On Wed, May 18, 2022 at 8:34 PM Biju Das <biju.das.jz@bp.renesas.com> wrote:
> > Subject: Re: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG binding
> > On Wed, May 18, 2022 at 05:58:00AM +0000, Biju Das wrote:
> > > > Subject: Re: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG
> > > > binding
> > > >
> > > > On Tue, May 10, 2022 at 04:11:05PM +0100, Biju Das wrote:
> > > > > Add device tree bindings for the RZ/G2L Port Output Enable for GPT
> > > > (POEG).
> > > > >
> > > > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> > > > > +examples:
> > > > > + - |
> > > > > + #include <dt-bindings/clock/r9a07g044-cpg.h>
> > > > > + #include <dt-bindings/interrupt-controller/arm-gic.h>
> > > > > +
> > > > > + poeggd: poeg@10049400 {
> > > > > + compatible = "renesas,r9a07g044-poeg", "renesas,rzg2l-poeg";
> > > > > + reg = <0x10049400 0x4>;
> > > >
> > > > This looks like it is part of some larger block?
> > >
> > > There are 2 IP blocks GPT(PWM) and POEG with its own resources like
> > > (register map, clk, reset and interrupts)
> > >
> > > Larger block is GPT, which has lot of functionalities. The output from
> > > GPT block can be disabled by this IP either by external trigger,
> > > request from GPT(Deadtime error, both output low/high) or explicit
> > > software control). This IP has only a single register. Currently I am not
> > sure which framework to be used for this IP?? Or should it be merged with
Yeah, POEG is a weird beast.
Some of it fits under pin control, but not all of it.
From a quick glance, most of its configuration is intended to be
static, i.e. could be done from DT, like pin control?
I have no idea how to use the POEG interrupts, though.
> > larger block GPT by combining the resources?
> >
> > Usually, IP blocks would have some minimum address alignment (typ 4K or 64K
> > to be page aligned), but if there's no other IP in this address range as-is
> > is fine. The question is what's before or after the above address?
>
> As per the HW manual, before GPT IP block and after POE3 block(Port Output Enable 3 (POE3) for MTU).
>
> Before
> H'0_1004_8000 H'0_1004_87FF 2 Kbytes GPT
>
> After
> H'0_1004_9800 H'0_1004_9BFF 1 Kbyte POE3
>
> Please find the address map for the IP blocks near to it.
>
> H'0_1004_A000 H'0_1004_A3FF 1 Kbyte SSIF ch1
> H'0_1004_9C00 H'0_1004_9FFF 1 Kbyte SSIF ch0
> H'0_1004_9800 H'0_1004_9BFF 1 Kbyte POE3
> H'0_1004_9400 H'0_1004_97FF 1 Kbyte POEGD
> H'0_1004_9000 H'0_1004_93FF 1 Kbyte POEGC
> H'0_1004_8C00 H'0_1004_8FFF 1 Kbyte POEGB
> H'0_1004_8800 H'0_1004_8BFF 1 Kbyte POEGA
> H'0_1004_8000 H'0_1004_87FF 2 Kbytes GPT
This is actually 8 x 256 bytes, for 8 GPT instances.
> H'0_1004_7000 H'0_1004_7FFF 4 Kbytes SRC (Reg)
> H'0_1004_0000 H'0_1004_6FFF 28 Kbytes SRC (Memory)
So you can combine GPT and POEG[A-D] into a single block.
However, doing so will make life harder when reusing the driver on
an SoC with a different layout, or a different number of POEG blocks
and GPT channels.
BTW, POE3 is a similar (in spirit) block on top of the MTU
(Multi-Function Timer Pulse Unit 3, which seems to be an
enhanced version of the already-supported MTU2 on RZ/A1?).
But the POE3 block is not located next to the MTU block, so you cannot
combine them without overlap.
Note that the minimum page size on Cortex-A seems to be 4 kiB, and
several blocks are spaced apart less, so even with a different OS
than Linux you cannot implement page-based access control.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 22+ messages in thread* RE: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG binding
2022-05-19 9:06 ` Geert Uytterhoeven
@ 2022-05-19 9:30 ` Biju Das
2022-05-19 20:04 ` Rob Herring
0 siblings, 1 reply; 22+ messages in thread
From: Biju Das @ 2022-05-19 9:30 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Rob Herring, Thierry Reding, Lee Jones, Krzysztof Kozlowski,
linux-pwm, devicetree, Geert Uytterhoeven, Chris Paterson,
Biju Das, Prabhakar Mahadev Lad, linux-renesas-soc,
Uwe Kleine-König
Hi Geert,
Thanks for the feedback.
> Subject: Re: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG binding
>
> Hi Biju,
>
> On Wed, May 18, 2022 at 8:34 PM Biju Das <biju.das.jz@bp.renesas.com>
> wrote:
> > > Subject: Re: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG
> > > binding On Wed, May 18, 2022 at 05:58:00AM +0000, Biju Das wrote:
> > > > > Subject: Re: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L
> > > > > POEG binding
> > > > >
> > > > > On Tue, May 10, 2022 at 04:11:05PM +0100, Biju Das wrote:
> > > > > > Add device tree bindings for the RZ/G2L Port Output Enable for
> > > > > > GPT
> > > > > (POEG).
> > > > > >
> > > > > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
>
> > > > > > +examples:
> > > > > > + - |
> > > > > > + #include <dt-bindings/clock/r9a07g044-cpg.h>
> > > > > > + #include <dt-bindings/interrupt-controller/arm-gic.h>
> > > > > > +
> > > > > > + poeggd: poeg@10049400 {
> > > > > > + compatible = "renesas,r9a07g044-poeg", "renesas,rzg2l-
> poeg";
> > > > > > + reg = <0x10049400 0x4>;
> > > > >
> > > > > This looks like it is part of some larger block?
> > > >
> > > > There are 2 IP blocks GPT(PWM) and POEG with its own resources
> > > > like (register map, clk, reset and interrupts)
> > > >
> > > > Larger block is GPT, which has lot of functionalities. The output
> > > > from GPT block can be disabled by this IP either by external
> > > > trigger, request from GPT(Deadtime error, both output low/high) or
> > > > explicit software control). This IP has only a single register.
> > > > Currently I am not
> > > sure which framework to be used for this IP?? Or should it be merged
> > > with
>
> Yeah, POEG is a weird beast.
> Some of it fits under pin control, but not all of it.
> From a quick glance, most of its configuration is intended to be static,
> i.e. could be done from DT, like pin control?
> I have no idea how to use the POEG interrupts, though.
If there is a GPT request(Dead time error or Both output low/high condition) output is disabled automatically and we get an
Interrupt. May be to clear it , we need to implement interrupt. Otherwise output will be always disabled,
even if the outputs are out of phase after the fault condition.
I have done a quick test with interrupts previously for output disable using GPT request:-
Use both A and B in phase, output is disabled automatically and you get an interrupt in POEG block.
If you inverse B, it is out of phase and fault condition is no more, but still output is disabled.
In this condition, If we want to enable outputs, we need to clear interrupt status bits.
>
> > > larger block GPT by combining the resources?
> > >
> > > Usually, IP blocks would have some minimum address alignment (typ 4K
> > > or 64K to be page aligned), but if there's no other IP in this
> > > address range as-is is fine. The question is what's before or after
> the above address?
> >
> > As per the HW manual, before GPT IP block and after POE3 block(Port
> Output Enable 3 (POE3) for MTU).
> >
> > Before
> > H'0_1004_8000 H'0_1004_87FF 2 Kbytes GPT
> >
> > After
> > H'0_1004_9800 H'0_1004_9BFF 1 Kbyte POE3
> >
> > Please find the address map for the IP blocks near to it.
> >
> > H'0_1004_A000 H'0_1004_A3FF 1 Kbyte SSIF ch1
> > H'0_1004_9C00 H'0_1004_9FFF 1 Kbyte SSIF ch0
> > H'0_1004_9800 H'0_1004_9BFF 1 Kbyte POE3
> > H'0_1004_9400 H'0_1004_97FF 1 Kbyte POEGD
> > H'0_1004_9000 H'0_1004_93FF 1 Kbyte POEGC
> > H'0_1004_8C00 H'0_1004_8FFF 1 Kbyte POEGB
> > H'0_1004_8800 H'0_1004_8BFF 1 Kbyte POEGA
> > H'0_1004_8000 H'0_1004_87FF 2 Kbytes GPT
>
> This is actually 8 x 256 bytes, for 8 GPT instances.
Yes correct.
>
> > H'0_1004_7000 H'0_1004_7FFF 4 Kbytes SRC (Reg)
> > H'0_1004_0000 H'0_1004_6FFF 28 Kbytes SRC (Memory)
>
> So you can combine GPT and POEG[A-D] into a single block.
> However, doing so will make life harder when reusing the driver on an SoC
> with a different layout, or a different number of POEG blocks and GPT
> channels.
I agree. Modelling as a different driver gives lots of flexibility.
Cheers,
Biju
>
> BTW, POE3 is a similar (in spirit) block on top of the MTU (Multi-Function
> Timer Pulse Unit 3, which seems to be an enhanced version of the already-
> supported MTU2 on RZ/A1?).
> But the POE3 block is not located next to the MTU block, so you cannot
> combine them without overlap.
>
> Note that the minimum page size on Cortex-A seems to be 4 kiB, and several
> blocks are spaced apart less, so even with a different OS than Linux you
> cannot implement page-based access control.
>
^ permalink raw reply [flat|nested] 22+ messages in thread* Re: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG binding
2022-05-19 9:30 ` Biju Das
@ 2022-05-19 20:04 ` Rob Herring
2022-06-08 16:11 ` Biju Das
0 siblings, 1 reply; 22+ messages in thread
From: Rob Herring @ 2022-05-19 20:04 UTC (permalink / raw)
To: Biju Das
Cc: Geert Uytterhoeven, Thierry Reding, Lee Jones,
Krzysztof Kozlowski, linux-pwm, devicetree, Geert Uytterhoeven,
Chris Paterson, Biju Das, Prabhakar Mahadev Lad,
linux-renesas-soc, Uwe Kleine-König
On Thu, May 19, 2022 at 09:30:19AM +0000, Biju Das wrote:
> Hi Geert,
>
> Thanks for the feedback.
>
> > Subject: Re: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG binding
> >
> > Hi Biju,
> >
> > On Wed, May 18, 2022 at 8:34 PM Biju Das <biju.das.jz@bp.renesas.com>
> > wrote:
> > > > Subject: Re: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG
> > > > binding On Wed, May 18, 2022 at 05:58:00AM +0000, Biju Das wrote:
> > > > > > Subject: Re: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L
> > > > > > POEG binding
> > > > > >
> > > > > > On Tue, May 10, 2022 at 04:11:05PM +0100, Biju Das wrote:
> > > > > > > Add device tree bindings for the RZ/G2L Port Output Enable for
> > > > > > > GPT
> > > > > > (POEG).
> > > > > > >
> > > > > > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> >
> > > > > > > +examples:
> > > > > > > + - |
> > > > > > > + #include <dt-bindings/clock/r9a07g044-cpg.h>
> > > > > > > + #include <dt-bindings/interrupt-controller/arm-gic.h>
> > > > > > > +
> > > > > > > + poeggd: poeg@10049400 {
> > > > > > > + compatible = "renesas,r9a07g044-poeg", "renesas,rzg2l-
> > poeg";
> > > > > > > + reg = <0x10049400 0x4>;
> > > > > >
> > > > > > This looks like it is part of some larger block?
> > > > >
> > > > > There are 2 IP blocks GPT(PWM) and POEG with its own resources
> > > > > like (register map, clk, reset and interrupts)
> > > > >
> > > > > Larger block is GPT, which has lot of functionalities. The output
> > > > > from GPT block can be disabled by this IP either by external
> > > > > trigger, request from GPT(Deadtime error, both output low/high) or
> > > > > explicit software control). This IP has only a single register.
> > > > > Currently I am not
> > > > sure which framework to be used for this IP?? Or should it be merged
> > > > with
> >
> > Yeah, POEG is a weird beast.
> > Some of it fits under pin control, but not all of it.
> > From a quick glance, most of its configuration is intended to be static,
> > i.e. could be done from DT, like pin control?
> > I have no idea how to use the POEG interrupts, though.
>
> If there is a GPT request(Dead time error or Both output low/high condition) output is disabled automatically and we get an
> Interrupt. May be to clear it , we need to implement interrupt. Otherwise output will be always disabled,
> even if the outputs are out of phase after the fault condition.
>
> I have done a quick test with interrupts previously for output disable using GPT request:-
> Use both A and B in phase, output is disabled automatically and you get an interrupt in POEG block.
> If you inverse B, it is out of phase and fault condition is no more, but still output is disabled.
> In this condition, If we want to enable outputs, we need to clear interrupt status bits.
>
> >
> > > > larger block GPT by combining the resources?
> > > >
> > > > Usually, IP blocks would have some minimum address alignment (typ 4K
> > > > or 64K to be page aligned), but if there's no other IP in this
> > > > address range as-is is fine. The question is what's before or after
> > the above address?
> > >
> > > As per the HW manual, before GPT IP block and after POE3 block(Port
> > Output Enable 3 (POE3) for MTU).
> > >
> > > Before
> > > H'0_1004_8000 H'0_1004_87FF 2 Kbytes GPT
> > >
> > > After
> > > H'0_1004_9800 H'0_1004_9BFF 1 Kbyte POE3
> > >
> > > Please find the address map for the IP blocks near to it.
> > >
> > > H'0_1004_A000 H'0_1004_A3FF 1 Kbyte SSIF ch1
> > > H'0_1004_9C00 H'0_1004_9FFF 1 Kbyte SSIF ch0
> > > H'0_1004_9800 H'0_1004_9BFF 1 Kbyte POE3
> > > H'0_1004_9400 H'0_1004_97FF 1 Kbyte POEGD
> > > H'0_1004_9000 H'0_1004_93FF 1 Kbyte POEGC
> > > H'0_1004_8C00 H'0_1004_8FFF 1 Kbyte POEGB
> > > H'0_1004_8800 H'0_1004_8BFF 1 Kbyte POEGA
> > > H'0_1004_8000 H'0_1004_87FF 2 Kbytes GPT
> >
> > This is actually 8 x 256 bytes, for 8 GPT instances.
>
> Yes correct.
>
> >
> > > H'0_1004_7000 H'0_1004_7FFF 4 Kbytes SRC (Reg)
> > > H'0_1004_0000 H'0_1004_6FFF 28 Kbytes SRC (Memory)
> >
> > So you can combine GPT and POEG[A-D] into a single block.
> > However, doing so will make life harder when reusing the driver on an SoC
> > with a different layout, or a different number of POEG blocks and GPT
> > channels.
>
> I agree. Modelling as a different driver gives lots of flexibility.
The question is different h/w blocks or 1, not driver(s). It's
convenient when the answer is the same (i.e. h/w node:driver is 1:1),
but h/w is sometimes messy.
In any case, that looks like different blocks to me.
Rob
^ permalink raw reply [flat|nested] 22+ messages in thread* RE: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG binding
2022-05-19 20:04 ` Rob Herring
@ 2022-06-08 16:11 ` Biju Das
0 siblings, 0 replies; 22+ messages in thread
From: Biju Das @ 2022-06-08 16:11 UTC (permalink / raw)
To: Rob Herring
Cc: Geert Uytterhoeven, Thierry Reding, Lee Jones,
Krzysztof Kozlowski, linux-pwm, devicetree, Geert Uytterhoeven,
Chris Paterson, Biju Das, Prabhakar Mahadev Lad,
linux-renesas-soc, Uwe Kleine-König
Hi Rob,
Thanks for the feedback.
> Subject: Re: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG
> binding
>
> On Thu, May 19, 2022 at 09:30:19AM +0000, Biju Das wrote:
> > Hi Geert,
> >
> > Thanks for the feedback.
> >
> > > Subject: Re: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG
> > > binding
> > >
> > > Hi Biju,
> > >
> > > On Wed, May 18, 2022 at 8:34 PM Biju Das
> > > <biju.das.jz@bp.renesas.com>
> > > wrote:
> > > > > Subject: Re: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L
> > > > > POEG binding On Wed, May 18, 2022 at 05:58:00AM +0000, Biju Das
> wrote:
> > > > > > > Subject: Re: [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L
> > > > > > > POEG binding
> > > > > > >
> > > > > > > On Tue, May 10, 2022 at 04:11:05PM +0100, Biju Das wrote:
> > > > > > > > Add device tree bindings for the RZ/G2L Port Output Enable
> > > > > > > > for GPT
> > > > > > > (POEG).
> > > > > > > >
> > > > > > > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> > >
> > > > > > > > +examples:
> > > > > > > > + - |
> > > > > > > > + #include <dt-bindings/clock/r9a07g044-cpg.h>
> > > > > > > > + #include <dt-bindings/interrupt-controller/arm-gic.h>
> > > > > > > > +
> > > > > > > > + poeggd: poeg@10049400 {
> > > > > > > > + compatible = "renesas,r9a07g044-poeg",
> > > > > > > > + "renesas,rzg2l-
> > > poeg";
> > > > > > > > + reg = <0x10049400 0x4>;
> > > > > > >
> > > > > > > This looks like it is part of some larger block?
> > > > > >
> > > > > > There are 2 IP blocks GPT(PWM) and POEG with its own resources
> > > > > > like (register map, clk, reset and interrupts)
> > > > > >
> > > > > > Larger block is GPT, which has lot of functionalities. The
> > > > > > output from GPT block can be disabled by this IP either by
> > > > > > external trigger, request from GPT(Deadtime error, both output
> > > > > > low/high) or explicit software control). This IP has only a
> single register.
> > > > > > Currently I am not
> > > > > sure which framework to be used for this IP?? Or should it be
> > > > > merged with
> > >
> > > Yeah, POEG is a weird beast.
> > > Some of it fits under pin control, but not all of it.
> > > From a quick glance, most of its configuration is intended to be
> > > static, i.e. could be done from DT, like pin control?
> > > I have no idea how to use the POEG interrupts, though.
> >
> > If there is a GPT request(Dead time error or Both output low/high
> > condition) output is disabled automatically and we get an Interrupt.
> > May be to clear it , we need to implement interrupt. Otherwise output
> will be always disabled, even if the outputs are out of phase after the
> fault condition.
> >
> > I have done a quick test with interrupts previously for output disable
> using GPT request:-
> > Use both A and B in phase, output is disabled automatically and
> you get an interrupt in POEG block.
> > If you inverse B, it is out of phase and fault condition is no
> more, but still output is disabled.
> > In this condition, If we want to enable outputs, we need to
> clear interrupt status bits.
> >
> > >
> > > > > larger block GPT by combining the resources?
> > > > >
> > > > > Usually, IP blocks would have some minimum address alignment
> > > > > (typ 4K or 64K to be page aligned), but if there's no other IP
> > > > > in this address range as-is is fine. The question is what's
> > > > > before or after
> > > the above address?
> > > >
> > > > As per the HW manual, before GPT IP block and after POE3
> > > > block(Port
> > > Output Enable 3 (POE3) for MTU).
> > > >
> > > > Before
> > > > H'0_1004_8000 H'0_1004_87FF 2 Kbytes GPT
> > > >
> > > > After
> > > > H'0_1004_9800 H'0_1004_9BFF 1 Kbyte POE3
> > > >
> > > > Please find the address map for the IP blocks near to it.
> > > >
> > > > H'0_1004_A000 H'0_1004_A3FF 1 Kbyte SSIF ch1
> > > > H'0_1004_9C00 H'0_1004_9FFF 1 Kbyte SSIF ch0
> > > > H'0_1004_9800 H'0_1004_9BFF 1 Kbyte POE3
> > > > H'0_1004_9400 H'0_1004_97FF 1 Kbyte POEGD
> > > > H'0_1004_9000 H'0_1004_93FF 1 Kbyte POEGC
> > > > H'0_1004_8C00 H'0_1004_8FFF 1 Kbyte POEGB
> > > > H'0_1004_8800 H'0_1004_8BFF 1 Kbyte POEGA
> > > > H'0_1004_8000 H'0_1004_87FF 2 Kbytes GPT
> > >
> > > This is actually 8 x 256 bytes, for 8 GPT instances.
> >
> > Yes correct.
> >
> > >
> > > > H'0_1004_7000 H'0_1004_7FFF 4 Kbytes SRC (Reg)
> > > > H'0_1004_0000 H'0_1004_6FFF 28 Kbytes SRC (Memory)
> > >
> > > So you can combine GPT and POEG[A-D] into a single block.
> > > However, doing so will make life harder when reusing the driver on
> > > an SoC with a different layout, or a different number of POEG blocks
> > > and GPT channels.
> >
> > I agree. Modelling as a different driver gives lots of flexibility.
>
> The question is different h/w blocks or 1, not driver(s). It's
> convenient when the answer is the same (i.e. h/w node:driver is 1:1),
> but h/w is sometimes messy.
>
> In any case, that looks like different blocks to me.
OK, As Geert suggested will model this block as pinctrl.
Cheers,
Biju
^ permalink raw reply [flat|nested] 22+ messages in thread
* [RFC 2/8] drivers: soc: renesas: Add POEG driver support
2022-05-10 15:11 [RFC 0/8] Add RZ/G2L POEG support Biju Das
2022-05-10 15:11 ` [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG binding Biju Das
@ 2022-05-10 15:11 ` Biju Das
2022-05-10 15:11 ` [RFC 3/8] dt-bindings: pwm: rzg2l-gpt: Document renesas,poeg-group property Biju Das
` (6 subsequent siblings)
8 siblings, 0 replies; 22+ messages in thread
From: Biju Das @ 2022-05-10 15:11 UTC (permalink / raw)
To: Philipp Zabel, Rob Herring, Thierry Reding, Lee Jones,
Krzysztof Kozlowski
Cc: Biju Das, Geert Uytterhoeven, Magnus Damm, linux-pwm,
linux-renesas-soc, Chris Paterson, Biju Das,
Prabhakar Mahadev Lad
The output pins of the general PWM timer (GPT) can be disabled by
using the port output enabling function for the GPT (POEG).
This patch series add basic support using s/w control through
sysfs to enable/disable output from GPT.
Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
---
drivers/soc/renesas/Kconfig | 2 +
drivers/soc/renesas/Makefile | 2 +
drivers/soc/renesas/poeg/Kconfig | 12 +++
drivers/soc/renesas/poeg/Makefile | 2 +
drivers/soc/renesas/poeg/poeg-rzg2l.c | 148 ++++++++++++++++++++++++++
5 files changed, 166 insertions(+)
create mode 100644 drivers/soc/renesas/poeg/Kconfig
create mode 100644 drivers/soc/renesas/poeg/Makefile
create mode 100644 drivers/soc/renesas/poeg/poeg-rzg2l.c
diff --git a/drivers/soc/renesas/Kconfig b/drivers/soc/renesas/Kconfig
index c50a6ce1b99d..363bfe3340fa 100644
--- a/drivers/soc/renesas/Kconfig
+++ b/drivers/soc/renesas/Kconfig
@@ -330,6 +330,8 @@ config ARCH_R9A09G011
help
This enables support for the Renesas RZ/V2M SoC.
+source "drivers/soc/renesas/poeg/Kconfig"
+
endif # ARM64
config RST_RCAR
diff --git a/drivers/soc/renesas/Makefile b/drivers/soc/renesas/Makefile
index 535868c9c7e4..935bf2cbf31e 100644
--- a/drivers/soc/renesas/Makefile
+++ b/drivers/soc/renesas/Makefile
@@ -36,3 +36,5 @@ obj-$(CONFIG_RST_RCAR) += rcar-rst.o
obj-$(CONFIG_SYSC_RCAR) += rcar-sysc.o
obj-$(CONFIG_SYSC_RCAR_GEN4) += rcar-gen4-sysc.o
obj-$(CONFIG_SYSC_RMOBILE) += rmobile-sysc.o
+
+obj-$(CONFIG_POEG_RZG2L) += poeg/
diff --git a/drivers/soc/renesas/poeg/Kconfig b/drivers/soc/renesas/poeg/Kconfig
new file mode 100644
index 000000000000..d0f123adb8da
--- /dev/null
+++ b/drivers/soc/renesas/poeg/Kconfig
@@ -0,0 +1,12 @@
+# SPDX-License-Identifier: GPL-2.0
+config POEG_RZG2L
+ tristate "RZ/G2L poeg support"
+ depends on ARCH_R9A07G044
+ depends on PWM_RZG2L_GPT
+ depends on HAS_IOMEM
+ help
+ This driver exposes the General Port Output Enable for PWM found
+ in RZ/G2L.
+
+ To compile this driver as a module, choose M here: the module
+ will be called poeg-rzg2l.
diff --git a/drivers/soc/renesas/poeg/Makefile b/drivers/soc/renesas/poeg/Makefile
new file mode 100644
index 000000000000..cc1ec08d2021
--- /dev/null
+++ b/drivers/soc/renesas/poeg/Makefile
@@ -0,0 +1,2 @@
+# SPDX-License-Identifier: GPL-2.0
+obj-$(CONFIG_POEG_RZG2L) += poeg-rzg2l.o
diff --git a/drivers/soc/renesas/poeg/poeg-rzg2l.c b/drivers/soc/renesas/poeg/poeg-rzg2l.c
new file mode 100644
index 000000000000..34f638dc45a6
--- /dev/null
+++ b/drivers/soc/renesas/poeg/poeg-rzg2l.c
@@ -0,0 +1,148 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * Renesas RZ/G2L Port Output Enable for GPT (POEG) driver
+ *
+ * Copyright (C) 2022 Renesas Electronics Corporation
+ */
+
+#include <linux/io.h>
+#include <linux/module.h>
+#include <linux/of.h>
+#include <linux/platform_device.h>
+#include <linux/pm_runtime.h>
+#include <linux/reset.h>
+
+#define POEGG 0
+#define POEGG_SSF BIT(3)
+
+struct rzg2l_poeg_chip {
+ struct reset_control *rstc;
+ void __iomem *mmio;
+};
+
+static void rzg2l_poeg_write(struct rzg2l_poeg_chip *chip, u32 reg, u32 data)
+{
+ iowrite32(data, chip->mmio + reg);
+}
+
+static u32 rzg2l_poeg_read(struct rzg2l_poeg_chip *chip, u32 reg)
+{
+ return ioread32(chip->mmio + reg);
+}
+
+static ssize_t output_disable_store(struct device *dev,
+ struct device_attribute *attr,
+ const char *buf, size_t count)
+{
+ struct platform_device *pdev = to_platform_device(dev);
+ struct rzg2l_poeg_chip *chip = platform_get_drvdata(pdev);
+ unsigned int val;
+ u32 reg;
+ int ret;
+
+ ret = kstrtouint(buf, 0, &val);
+ if (ret)
+ return ret;
+
+ reg = rzg2l_poeg_read(chip, POEGG);
+ if (val)
+ reg |= POEGG_SSF;
+ else
+ reg &= ~POEGG_SSF;
+
+ rzg2l_poeg_write(chip, POEGG, reg);
+
+ return ret ? : count;
+}
+
+static ssize_t output_disable_show(struct device *dev,
+ struct device_attribute *attr, char *buf)
+{
+ struct platform_device *pdev = to_platform_device(dev);
+ struct rzg2l_poeg_chip *chip = platform_get_drvdata(pdev);
+ u32 reg;
+
+ reg = rzg2l_poeg_read(chip, POEGG);
+
+ return sprintf(buf, "%u\n", (reg & POEGG_SSF) ? 1 : 0);
+}
+static DEVICE_ATTR_RW(output_disable);
+
+static const struct of_device_id rzg2l_poeg_of_table[] = {
+ { .compatible = "renesas,rzg2l-poeg", },
+ { /* Sentinel */ }
+};
+MODULE_DEVICE_TABLE(of, rzg2l_poeg_of_table);
+
+static int rzg2l_poeg_probe(struct platform_device *pdev)
+{
+ struct rzg2l_poeg_chip *chip;
+ int ret;
+
+ chip = devm_kzalloc(&pdev->dev, sizeof(*chip), GFP_KERNEL);
+ if (!chip)
+ return -ENOMEM;
+
+ chip->mmio = devm_platform_ioremap_resource(pdev, 0);
+ if (IS_ERR(chip->mmio))
+ return PTR_ERR(chip->mmio);
+
+ chip->rstc = devm_reset_control_get_exclusive(&pdev->dev, NULL);
+ if (IS_ERR(chip->rstc))
+ return dev_err_probe(&pdev->dev, PTR_ERR(chip->rstc),
+ "get reset failed\n");
+
+ ret = reset_control_deassert(chip->rstc);
+ if (ret)
+ return ret;
+
+ pm_runtime_enable(&pdev->dev);
+ platform_set_drvdata(pdev, chip);
+
+ ret = pm_runtime_resume_and_get(&pdev->dev);
+ if (ret < 0) {
+ dev_err(&pdev->dev, "pm_runtime_resume_and_get failed: %d\n", ret);
+ goto err;
+ }
+
+ ret = device_create_file(&pdev->dev, &dev_attr_output_disable);
+ if (ret < 0) {
+ dev_err(&pdev->dev, "Failed to create poeg sysfs: %d\n", ret);
+ goto err;
+ }
+
+ return 0;
+
+err:
+ pm_runtime_disable(&pdev->dev);
+ reset_control_assert(chip->rstc);
+
+ return ret;
+}
+
+static int rzg2l_poeg_remove(struct platform_device *pdev)
+{
+ struct rzg2l_poeg_chip *chip = platform_get_drvdata(pdev);
+
+ device_remove_file(&pdev->dev, &dev_attr_output_disable);
+ pm_runtime_put(&pdev->dev);
+ pm_runtime_disable(&pdev->dev);
+ reset_control_assert(chip->rstc);
+
+ return 0;
+}
+
+static struct platform_driver rzg2l_poeg_driver = {
+ .driver = {
+ .name = "poeg-rzg2l",
+ .owner = THIS_MODULE,
+ .of_match_table = of_match_ptr(rzg2l_poeg_of_table),
+ },
+ .probe = rzg2l_poeg_probe,
+ .remove = rzg2l_poeg_remove,
+};
+module_platform_driver(rzg2l_poeg_driver);
+
+MODULE_AUTHOR("Biju Das <biju.das.jz@bp.renesas.com>");
+MODULE_DESCRIPTION("Renesas RZ/G2L POEG Driver");
+MODULE_LICENSE("GPL v2");
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread* [RFC 3/8] dt-bindings: pwm: rzg2l-gpt: Document renesas,poeg-group property
2022-05-10 15:11 [RFC 0/8] Add RZ/G2L POEG support Biju Das
2022-05-10 15:11 ` [RFC 1/8] dt-bindings: soc: renesas: Add RZ/G2L POEG binding Biju Das
2022-05-10 15:11 ` [RFC 2/8] drivers: soc: renesas: Add POEG driver support Biju Das
@ 2022-05-10 15:11 ` Biju Das
2022-05-10 18:29 ` Rob Herring
2022-05-10 15:11 ` [RFC 4/8] pwm: rzg2l-gpt: Add support for linking with POEG Biju Das
` (5 subsequent siblings)
8 siblings, 1 reply; 22+ messages in thread
From: Biju Das @ 2022-05-10 15:11 UTC (permalink / raw)
To: Thierry Reding, Lee Jones, Rob Herring, Krzysztof Kozlowski
Cc: Biju Das, Uwe Kleine-König, linux-pwm, devicetree,
Geert Uytterhoeven, Chris Paterson, Biju Das,
Prabhakar Mahadev Lad, linux-renesas-soc
For output disable operation, POEG group needs to be linked with
GPT. Document renesas,poeg-group property for linking both GPT and
POEG devices.
Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
---
.../devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml b/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml
index b57c1b256a86..94be441d742c 100644
--- a/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml
+++ b/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml
@@ -89,6 +89,14 @@ properties:
resets:
maxItems: 1
+ renesas,poeg-group:
+ $ref: "/schemas/types.yaml#/definitions/phandle-array"
+ items:
+ maxItems: 1
+ description:
+ phandle to the POEGGroup instance present in the SoC, one for each
+ available GPT channel.
+
required:
- compatible
- reg
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread* Re: [RFC 3/8] dt-bindings: pwm: rzg2l-gpt: Document renesas,poeg-group property
2022-05-10 15:11 ` [RFC 3/8] dt-bindings: pwm: rzg2l-gpt: Document renesas,poeg-group property Biju Das
@ 2022-05-10 18:29 ` Rob Herring
0 siblings, 0 replies; 22+ messages in thread
From: Rob Herring @ 2022-05-10 18:29 UTC (permalink / raw)
To: Biju Das
Cc: devicetree, Krzysztof Kozlowski, Uwe Kleine-König,
Lee Jones, Prabhakar Mahadev Lad, Rob Herring, linux-pwm,
Chris Paterson, linux-renesas-soc, Geert Uytterhoeven, Biju Das,
Thierry Reding
On Tue, 10 May 2022 16:11:07 +0100, Biju Das wrote:
> For output disable operation, POEG group needs to be linked with
> GPT. Document renesas,poeg-group property for linking both GPT and
> POEG devices.
>
> Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> ---
> .../devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 22+ messages in thread
* [RFC 4/8] pwm: rzg2l-gpt: Add support for linking with POEG
2022-05-10 15:11 [RFC 0/8] Add RZ/G2L POEG support Biju Das
` (2 preceding siblings ...)
2022-05-10 15:11 ` [RFC 3/8] dt-bindings: pwm: rzg2l-gpt: Document renesas,poeg-group property Biju Das
@ 2022-05-10 15:11 ` Biju Das
2022-05-19 9:15 ` Geert Uytterhoeven
2022-05-10 15:11 ` [RFC 5/8] arm64: dts: renesas: r9a07g044: Add POEG nodes Biju Das
` (4 subsequent siblings)
8 siblings, 1 reply; 22+ messages in thread
From: Biju Das @ 2022-05-10 15:11 UTC (permalink / raw)
To: Thierry Reding, Lee Jones
Cc: Biju Das, Uwe Kleine-König, linux-pwm, Geert Uytterhoeven,
Chris Paterson, Biju Das, Prabhakar Mahadev Lad,
linux-renesas-soc
This patch add support for linking POEG group with pwm, so that
POEG can control the output disable function.
Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
---
drivers/pwm/pwm-rzg2l-gpt.c | 59 +++++++++++++++++++++++++++++++++++++
1 file changed, 59 insertions(+)
diff --git a/drivers/pwm/pwm-rzg2l-gpt.c b/drivers/pwm/pwm-rzg2l-gpt.c
index d5d22b1ff792..8eaf96b2052d 100644
--- a/drivers/pwm/pwm-rzg2l-gpt.c
+++ b/drivers/pwm/pwm-rzg2l-gpt.c
@@ -21,6 +21,7 @@
#define GTCR 0x2c
#define GTUDDTYC 0x30
#define GTIOR 0x34
+#define GTINTAD 0x38
#define GTBER 0x40
#define GTCNT 0x48
#define GTCCRA 0x4c
@@ -37,9 +38,14 @@
#define UP_COUNTING (GTUDDTYC_UP | GTUDDTYC_UDF)
#define GTIOR_GTIOA_MASK GENMASK(4, 0)
+#define GTIOR_OADF_MASK GENMASK(10, 9)
#define GTIOR_GTIOB_MASK GENMASK(20, 16)
+#define GTIOR_OBDF_MASK GENMASK(26, 25)
+
#define GTIOR_OAE BIT(8)
#define GTIOR_OBE BIT(24)
+#define GTIOR_OADF_HIGH_IMP_ON_OUT_DISABLE (1 << 9)
+#define GTIOR_OBDF_HIGH_IMP_ON_OUT_DISABLE (1 << 25)
#define INIT_OUT_LO_OUT_LO_END_TOGGLE (0x07)
#define INIT_OUT_HI_OUT_HI_END_TOGGLE (0x1B)
@@ -48,6 +54,13 @@
#define GTIOR_GTIOB_OUT_HI_END_TOGGLE_CMP_MATCH ((INIT_OUT_HI_OUT_HI_END_TOGGLE << 16) | GTIOR_OBE)
#define GTIOR_GTIOB_OUT_LO_END_TOGGLE_CMP_MATCH ((INIT_OUT_LO_OUT_LO_END_TOGGLE << 16) | GTIOR_OBE)
+#define GTINTAD_GRP_MASK GENMASK(25, 24)
+#define GRP_INVALID (0xFF)
+#define POEG_GRP_A_ADDR (0x10048800)
+#define POEG_GRP_B_ADDR (0x10048c00)
+#define POEG_GRP_C_ADDR (0x10049000)
+#define POEG_GRP_D_ADDR (0x10049400)
+
struct phase {
u32 value;
u32 mask;
@@ -85,6 +98,7 @@ struct rzg2l_gpt_chip {
void __iomem *mmio;
struct reset_control *rstc;
struct clk *clk;
+ u8 poeg_grp;
};
static inline struct rzg2l_gpt_chip *to_rzg2l_gpt_chip(struct pwm_chip *chip)
@@ -220,6 +234,17 @@ static int rzg2l_gpt_config(struct pwm_chip *chip, struct pwm_device *_pwm,
/* Set period */
rzg2l_gpt_write(pc, GTPR, pv);
+ if (pc->poeg_grp != GRP_INVALID) {
+ rzg2l_gpt_modify(pc, GTINTAD, GTINTAD_GRP_MASK, pc->poeg_grp << 24);
+
+ if (pwm->channel)
+ rzg2l_gpt_modify(pc, GTIOR, GTIOR_OBDF_MASK,
+ GTIOR_OBDF_HIGH_IMP_ON_OUT_DISABLE);
+ else
+ rzg2l_gpt_modify(pc, GTIOR, GTIOR_OADF_MASK,
+ GTIOR_OADF_HIGH_IMP_ON_OUT_DISABLE);
+ }
+
/* Enable pin output */
rzg2l_gpt_modify(pc, GTIOR, pwm->ph->mask, pwm->ph->value);
@@ -266,6 +291,36 @@ static int rzg2l_gpt_apply(struct pwm_chip *chip, struct pwm_device *pwm,
return ret;
}
+static int rzg2l_gpt_parse_properties(struct platform_device *pdev,
+ struct rzg2l_gpt_chip *pc)
+{
+ static const u64 poeg_grp_addr[] = {
+ POEG_GRP_A_ADDR, POEG_GRP_B_ADDR, POEG_GRP_C_ADDR, POEG_GRP_D_ADDR
+ };
+ struct device_node *np;
+ unsigned int i;
+ u64 addr;
+
+ pc->poeg_grp = GRP_INVALID;
+ np = of_parse_phandle(pdev->dev.of_node, "renesas,poeg-group", 0);
+ if (!np)
+ return 0;
+
+ if (!of_property_read_u64(np, "reg", &addr)) {
+ for (i = 0; i < ARRAY_SIZE(poeg_grp_addr); i++) {
+ if (addr == poeg_grp_addr[i]) {
+ pc->poeg_grp = i;
+ break;
+ }
+ }
+ }
+
+ if (np)
+ of_node_put(np);
+
+ return 0;
+}
+
static const struct pwm_ops rzg2l_gpt_ops = {
.request = rzg2l_gpt_request,
.free = rzg2l_gpt_free,
@@ -288,6 +343,10 @@ static int rzg2l_gpt_probe(struct platform_device *pdev)
if (!rzg2l_gpt)
return -ENOMEM;
+ ret = rzg2l_gpt_parse_properties(pdev, rzg2l_gpt);
+ if (ret)
+ return ret;
+
rzg2l_gpt->mmio = devm_platform_ioremap_resource(pdev, 0);
if (IS_ERR(rzg2l_gpt->mmio))
return PTR_ERR(rzg2l_gpt->mmio);
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread* Re: [RFC 4/8] pwm: rzg2l-gpt: Add support for linking with POEG
2022-05-10 15:11 ` [RFC 4/8] pwm: rzg2l-gpt: Add support for linking with POEG Biju Das
@ 2022-05-19 9:15 ` Geert Uytterhoeven
2022-06-08 17:01 ` Biju Das
0 siblings, 1 reply; 22+ messages in thread
From: Geert Uytterhoeven @ 2022-05-19 9:15 UTC (permalink / raw)
To: Biju Das
Cc: Thierry Reding, Lee Jones, Uwe Kleine-König, Linux PWM List,
Geert Uytterhoeven, Chris Paterson, Biju Das,
Prabhakar Mahadev Lad, Linux-Renesas
Hi Biju,
On Tue, May 10, 2022 at 5:11 PM Biju Das <biju.das.jz@bp.renesas.com> wrote:
> This patch add support for linking POEG group with pwm, so that
> POEG can control the output disable function.
>
> Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Thanks for your patch!
> --- a/drivers/pwm/pwm-rzg2l-gpt.c
> +++ b/drivers/pwm/pwm-rzg2l-gpt.c
> @@ -266,6 +291,36 @@ static int rzg2l_gpt_apply(struct pwm_chip *chip, struct pwm_device *pwm,
> return ret;
> }
>
> +static int rzg2l_gpt_parse_properties(struct platform_device *pdev,
> + struct rzg2l_gpt_chip *pc)
> +{
> + static const u64 poeg_grp_addr[] = {
> + POEG_GRP_A_ADDR, POEG_GRP_B_ADDR, POEG_GRP_C_ADDR, POEG_GRP_D_ADDR
> + };
> + struct device_node *np;
> + unsigned int i;
> + u64 addr;
> +
> + pc->poeg_grp = GRP_INVALID;
> + np = of_parse_phandle(pdev->dev.of_node, "renesas,poeg-group", 0);
> + if (!np)
> + return 0;
> +
> + if (!of_property_read_u64(np, "reg", &addr)) {
> + for (i = 0; i < ARRAY_SIZE(poeg_grp_addr); i++) {
> + if (addr == poeg_grp_addr[i]) {
Matching on addresses looks fragile to me.
Of course this is code, not DT, so it can be changed later.
Possible alternatives:
1. Use a numeric property instead of a phandle, so you can store
its value directly into pc->poeg_grp.
This loses the linking by phandle, though, which is nice to
have, and might be useful for other purposes later.
2. Add a "renesas,id" property to each POEGx DT node, cfr.
Documentation/devicetree/bindings/media/renesas,vin.yaml.
> + pc->poeg_grp = i;
> + break;
> + }
> + }
> + }
> +
> + if (np)
> + of_node_put(np);
> +
> + return 0;
> +}
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 22+ messages in thread* RE: [RFC 4/8] pwm: rzg2l-gpt: Add support for linking with POEG
2022-05-19 9:15 ` Geert Uytterhoeven
@ 2022-06-08 17:01 ` Biju Das
0 siblings, 0 replies; 22+ messages in thread
From: Biju Das @ 2022-06-08 17:01 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Thierry Reding, Lee Jones, Uwe Kleine-König, Linux PWM List,
Geert Uytterhoeven, Chris Paterson, Biju Das,
Prabhakar Mahadev Lad, Linux-Renesas
Hi Geert,
Thanks for the feedback.
> Subject: Re: [RFC 4/8] pwm: rzg2l-gpt: Add support for linking with POEG
>
> Hi Biju,
>
> On Tue, May 10, 2022 at 5:11 PM Biju Das <biju.das.jz@bp.renesas.com>
> wrote:
> > This patch add support for linking POEG group with pwm, so that POEG
> > can control the output disable function.
> >
> > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
>
> Thanks for your patch!
>
> > --- a/drivers/pwm/pwm-rzg2l-gpt.c
> > +++ b/drivers/pwm/pwm-rzg2l-gpt.c
> > @@ -266,6 +291,36 @@ static int rzg2l_gpt_apply(struct pwm_chip *chip,
> struct pwm_device *pwm,
> > return ret;
> > }
> >
> > +static int rzg2l_gpt_parse_properties(struct platform_device *pdev,
> > + struct rzg2l_gpt_chip *pc) {
> > + static const u64 poeg_grp_addr[] = {
> > + POEG_GRP_A_ADDR, POEG_GRP_B_ADDR, POEG_GRP_C_ADDR,
> POEG_GRP_D_ADDR
> > + };
> > + struct device_node *np;
> > + unsigned int i;
> > + u64 addr;
> > +
> > + pc->poeg_grp = GRP_INVALID;
> > + np = of_parse_phandle(pdev->dev.of_node, "renesas,poeg-group",
> 0);
> > + if (!np)
> > + return 0;
> > +
> > + if (!of_property_read_u64(np, "reg", &addr)) {
> > + for (i = 0; i < ARRAY_SIZE(poeg_grp_addr); i++) {
> > + if (addr == poeg_grp_addr[i]) {
>
> Matching on addresses looks fragile to me.
> Of course this is code, not DT, so it can be changed later.
>
> Possible alternatives:
> 1. Use a numeric property instead of a phandle, so you can store
> its value directly into pc->poeg_grp.
> This loses the linking by phandle, though, which is nice to
> have, and might be useful for other purposes later.
OK, will use a numeric property as default POEGGroup.
Cheers,
Biju
> 2. Add a "renesas,id" property to each POEGx DT node, cfr.
> Documentation/devicetree/bindings/media/renesas,vin.yaml.
>
> > + pc->poeg_grp = i;
> > + break;
> > + }
> > + }
> > + }
> > +
> > + if (np)
> > + of_node_put(np);
> > +
> > + return 0;
> > +}
>
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-
> m68k.org
>
> In personal conversations with technical people, I call myself a hacker.
> But when I'm talking to journalists I just say "programmer" or something
> like that.
> -- Linus Torvalds
^ permalink raw reply [flat|nested] 22+ messages in thread
* [RFC 5/8] arm64: dts: renesas: r9a07g044: Add POEG nodes
2022-05-10 15:11 [RFC 0/8] Add RZ/G2L POEG support Biju Das
` (3 preceding siblings ...)
2022-05-10 15:11 ` [RFC 4/8] pwm: rzg2l-gpt: Add support for linking with POEG Biju Das
@ 2022-05-10 15:11 ` Biju Das
2022-05-10 15:11 ` [RFC 6/8] arm64: dts: renesas: r9a07g054: " Biju Das
` (3 subsequent siblings)
8 siblings, 0 replies; 22+ messages in thread
From: Biju Das @ 2022-05-10 15:11 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski
Cc: Biju Das, Uwe Kleine-König, linux-pwm, devicetree,
Geert Uytterhoeven, Magnus Damm, linux-renesas-soc,
Chris Paterson, Biju Das, Prabhakar Mahadev Lad
Add POEGG{A,B,C,D} nodes to RZ/G2L SoC DTSI.
Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
---
arch/arm64/boot/dts/renesas/r9a07g044.dtsi | 44 ++++++++++++++++++++++
1 file changed, 44 insertions(+)
diff --git a/arch/arm64/boot/dts/renesas/r9a07g044.dtsi b/arch/arm64/boot/dts/renesas/r9a07g044.dtsi
index 28284d537a70..58476519683e 100644
--- a/arch/arm64/boot/dts/renesas/r9a07g044.dtsi
+++ b/arch/arm64/boot/dts/renesas/r9a07g044.dtsi
@@ -360,6 +360,50 @@ gpt7: pwm@10048700 {
status = "disabled";
};
+ poegga: poeg@10048800 {
+ compatible = "renesas,r9a07g044-poeg",
+ "renesas,rzg2l-poeg";
+ reg = <0 0x10048800 0 0x04>;
+ interrupts = <GIC_SPI 322 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&cpg CPG_MOD R9A07G044_POEG_A_CLKP>;
+ power-domains = <&cpg>;
+ resets = <&cpg R9A07G044_POEG_A_RST>;
+ status = "disabled";
+ };
+
+ poeggb: poeg@10048c00 {
+ compatible = "renesas,r9a07g044-poeg",
+ "renesas,rzg2l-poeg";
+ reg = <0 0x10048c00 0 0x04>;
+ interrupts = <GIC_SPI 323 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&cpg CPG_MOD R9A07G044_POEG_B_CLKP>;
+ power-domains = <&cpg>;
+ resets = <&cpg R9A07G044_POEG_B_RST>;
+ status = "disabled";
+ };
+
+ poeggc: poeg@10049000 {
+ compatible = "renesas,r9a07g044-poeg",
+ "renesas,rzg2l-poeg";
+ reg = <0 0x10049000 0 0x04>;
+ interrupts = <GIC_SPI 324 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&cpg CPG_MOD R9A07G044_POEG_C_CLKP>;
+ power-domains = <&cpg>;
+ resets = <&cpg R9A07G044_POEG_C_RST>;
+ status = "disabled";
+ };
+
+ poeggd: poeg@10049400 {
+ compatible = "renesas,r9a07g044-poeg",
+ "renesas,rzg2l-poeg";
+ reg = <0 0x10049400 0 0x04>;
+ interrupts = <GIC_SPI 325 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&cpg CPG_MOD R9A07G044_POEG_D_CLKP>;
+ power-domains = <&cpg>;
+ resets = <&cpg R9A07G044_POEG_D_RST>;
+ status = "disabled";
+ };
+
ssi0: ssi@10049c00 {
compatible = "renesas,r9a07g044-ssi",
"renesas,rz-ssi";
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread* [RFC 6/8] arm64: dts: renesas: r9a07g054: Add POEG nodes
2022-05-10 15:11 [RFC 0/8] Add RZ/G2L POEG support Biju Das
` (4 preceding siblings ...)
2022-05-10 15:11 ` [RFC 5/8] arm64: dts: renesas: r9a07g044: Add POEG nodes Biju Das
@ 2022-05-10 15:11 ` Biju Das
2022-05-10 15:11 ` [RFC 7/8] arm64: dts: renesas: rzg2l-smarc: Enable POEGG{A,B,C,D} on carrier board Biju Das
` (2 subsequent siblings)
8 siblings, 0 replies; 22+ messages in thread
From: Biju Das @ 2022-05-10 15:11 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski
Cc: Biju Das, Uwe Kleine-König, linux-pwm, devicetree,
Geert Uytterhoeven, Magnus Damm, linux-renesas-soc,
Chris Paterson, Biju Das, Prabhakar Mahadev Lad
Add POEGG{A,B,C,D} nodes to RZ/V2L SoC DTSI.
Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
---
arch/arm64/boot/dts/renesas/r9a07g054.dtsi | 44 ++++++++++++++++++++++
1 file changed, 44 insertions(+)
diff --git a/arch/arm64/boot/dts/renesas/r9a07g054.dtsi b/arch/arm64/boot/dts/renesas/r9a07g054.dtsi
index 755c92d92e8b..659f0eb11d2b 100644
--- a/arch/arm64/boot/dts/renesas/r9a07g054.dtsi
+++ b/arch/arm64/boot/dts/renesas/r9a07g054.dtsi
@@ -360,6 +360,50 @@ gpt7: pwm@10048700 {
status = "disabled";
};
+ poegga: poeg@10048800 {
+ compatible = "renesas,r9a07g054-poeg",
+ "renesas,rzg2l-poeg";
+ reg = <0 0x10048800 0 0x04>;
+ interrupts = <GIC_SPI 322 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&cpg CPG_MOD R9A07G054_POEG_A_CLKP>;
+ power-domains = <&cpg>;
+ resets = <&cpg R9A07G054_POEG_A_RST>;
+ status = "disabled";
+ };
+
+ poeggb: poeg@10048C00 {
+ compatible = "renesas,r9a07g054-poeg",
+ "renesas,rzg2l-poeg";
+ reg = <0 0x10048C00 0 0x04>;
+ interrupts = <GIC_SPI 323 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&cpg CPG_MOD R9A07G054_POEG_B_CLKP>;
+ power-domains = <&cpg>;
+ resets = <&cpg R9A07G054_POEG_B_RST>;
+ status = "disabled";
+ };
+
+ poeggc: poeg@10049000 {
+ compatible = "renesas,r9a07g054-poeg",
+ "renesas,rzg2l-poeg";
+ reg = <0 0x10049000 0 0x04>;
+ interrupts = <GIC_SPI 324 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&cpg CPG_MOD R9A07G054_POEG_C_CLKP>;
+ power-domains = <&cpg>;
+ resets = <&cpg R9A07G054_POEG_C_RST>;
+ status = "disabled";
+ };
+
+ poeggd: poeg@10049400 {
+ compatible = "renesas,r9a07g054-poeg",
+ "renesas,rzg2l-poeg";
+ reg = <0 0x10049400 0 0x04>;
+ interrupts = <GIC_SPI 325 IRQ_TYPE_LEVEL_HIGH>;
+ clocks = <&cpg CPG_MOD R9A07G054_POEG_D_CLKP>;
+ power-domains = <&cpg>;
+ resets = <&cpg R9A07G054_POEG_D_RST>;
+ status = "disabled";
+ };
+
ssi0: ssi@10049c00 {
compatible = "renesas,r9a07g054-ssi",
"renesas,rz-ssi";
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread* [RFC 7/8] arm64: dts: renesas: rzg2l-smarc: Enable POEGG{A,B,C,D} on carrier board
2022-05-10 15:11 [RFC 0/8] Add RZ/G2L POEG support Biju Das
` (5 preceding siblings ...)
2022-05-10 15:11 ` [RFC 6/8] arm64: dts: renesas: r9a07g054: " Biju Das
@ 2022-05-10 15:11 ` Biju Das
2022-05-10 15:11 ` [RFC 8/8] arm64: dts: renesas: rzg2l-smarc: Link GPT4 with POEGGD " Biju Das
2022-05-10 16:00 ` [RFC 0/8] Add RZ/G2L POEG support Uwe Kleine-König
8 siblings, 0 replies; 22+ messages in thread
From: Biju Das @ 2022-05-10 15:11 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski
Cc: Biju Das, Uwe Kleine-König, linux-pwm, devicetree,
Geert Uytterhoeven, Magnus Damm, linux-renesas-soc,
Chris Paterson, Biju Das, Prabhakar Mahadev Lad
Enable POEGG{A,B,C,D} on RZ/{G2,V2}L SMARC EVK.
Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
---
arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi | 16 ++++++++++++++++
1 file changed, 16 insertions(+)
diff --git a/arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi b/arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi
index 8fb68e95f1d7..f1fb9cecc49b 100644
--- a/arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi
+++ b/arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi
@@ -42,6 +42,22 @@ wm8978: codec@1a {
};
};
+&poegga {
+ status = "okay";
+};
+
+&poeggb {
+ status = "okay";
+};
+
+&poeggc {
+ status = "okay";
+};
+
+&poeggd {
+ status = "okay";
+};
+
/*
* To enable SCIF2 (SER0) on PMOD1 (CN7)
* SW1 should be at position 2->3 so that SER0_CTS# line is activated
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread* [RFC 8/8] arm64: dts: renesas: rzg2l-smarc: Link GPT4 with POEGGD on carrier board
2022-05-10 15:11 [RFC 0/8] Add RZ/G2L POEG support Biju Das
` (6 preceding siblings ...)
2022-05-10 15:11 ` [RFC 7/8] arm64: dts: renesas: rzg2l-smarc: Enable POEGG{A,B,C,D} on carrier board Biju Das
@ 2022-05-10 15:11 ` Biju Das
2022-05-10 16:00 ` [RFC 0/8] Add RZ/G2L POEG support Uwe Kleine-König
8 siblings, 0 replies; 22+ messages in thread
From: Biju Das @ 2022-05-10 15:11 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski
Cc: Biju Das, Uwe Kleine-König, linux-pwm, devicetree,
Geert Uytterhoeven, Magnus Damm, linux-renesas-soc,
Chris Paterson, Biju Das, Prabhakar Mahadev Lad
This patch links GPT4 with POEGGD on RZ/G2L SMARC EVK.
Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
---
arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi b/arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi
index f1fb9cecc49b..d7cc3897ee61 100644
--- a/arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi
+++ b/arch/arm64/boot/dts/renesas/rzg2l-smarc.dtsi
@@ -25,6 +25,7 @@ &cpu_dai {
&gpt4 {
pinctrl-0 = <&gpt4_pins>;
pinctrl-names = "default";
+ renesas,poeg-group = <&poeggd>;
status = "okay";
};
--
2.25.1
^ permalink raw reply related [flat|nested] 22+ messages in thread* Re: [RFC 0/8] Add RZ/G2L POEG support
2022-05-10 15:11 [RFC 0/8] Add RZ/G2L POEG support Biju Das
` (7 preceding siblings ...)
2022-05-10 15:11 ` [RFC 8/8] arm64: dts: renesas: rzg2l-smarc: Link GPT4 with POEGGD " Biju Das
@ 2022-05-10 16:00 ` Uwe Kleine-König
2022-05-10 16:08 ` Biju Das
8 siblings, 1 reply; 22+ messages in thread
From: Uwe Kleine-König @ 2022-05-10 16:00 UTC (permalink / raw)
To: Biju Das
Cc: Thierry Reding, Lee Jones, Rob Herring, Krzysztof Kozlowski,
linux-pwm, devicetree, Geert Uytterhoeven, Chris Paterson,
Biju Das, Prabhakar Mahadev Lad, linux-renesas-soc
[-- Attachment #1: Type: text/plain, Size: 1287 bytes --]
On Tue, May 10, 2022 at 04:11:04PM +0100, Biju Das wrote:
> The output pins of the general PWM timer (GPT) can be disabled by using
> the port output enabling function for the GPT (POEG). Specifically,
> either of the following ways can be used.
> * Input level detection of the GTETRGA to GTETRGD pins.
> * Output-disable request from the GPT.
> * Register settings.
>
> Added RZ/G2L POEG support under driver/soc/renesas, as currently I am not sure about
> the framework to be used for POEG.
>
> This patch series add support for controlling output disable function using sysfs.
>
> For output disable operation, POEG group needs to be linked with
> GPT. So introduced renesas,poeg-group property in pwm for linking both GPT and
> POEG devices.
>
> Please share your valuable comments.
>
> patch#3 and #4 depend upon [1]
> [1] https://lore.kernel.org/linux-renesas-soc/20220510144259.9908-1-biju.das.jz@bp.renesas.com/T/#t
I suggest to use the --base switch to git-format-patch for the next
submission round. This way the built robots can parse this information,
too.
Best regards
Uwe
--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | https://www.pengutronix.de/ |
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 22+ messages in thread* RE: [RFC 0/8] Add RZ/G2L POEG support
2022-05-10 16:00 ` [RFC 0/8] Add RZ/G2L POEG support Uwe Kleine-König
@ 2022-05-10 16:08 ` Biju Das
0 siblings, 0 replies; 22+ messages in thread
From: Biju Das @ 2022-05-10 16:08 UTC (permalink / raw)
To: Uwe Kleine-König
Cc: Thierry Reding, Lee Jones, Rob Herring, Krzysztof Kozlowski,
linux-pwm, devicetree, Geert Uytterhoeven, Chris Paterson,
Biju Das, Prabhakar Mahadev Lad, linux-renesas-soc
Hi Uwe,
Thanks for the feedback.
> Subject: Re: [RFC 0/8] Add RZ/G2L POEG support
>
> On Tue, May 10, 2022 at 04:11:04PM +0100, Biju Das wrote:
> > The output pins of the general PWM timer (GPT) can be disabled by
> > using the port output enabling function for the GPT (POEG).
> > Specifically, either of the following ways can be used.
> > * Input level detection of the GTETRGA to GTETRGD pins.
> > * Output-disable request from the GPT.
> > * Register settings.
> >
> > Added RZ/G2L POEG support under driver/soc/renesas, as currently I am
> > not sure about the framework to be used for POEG.
> >
> > This patch series add support for controlling output disable function
> using sysfs.
> >
> > For output disable operation, POEG group needs to be linked with GPT.
> > So introduced renesas,poeg-group property in pwm for linking both GPT
> > and POEG devices.
> >
> > Please share your valuable comments.
> >
> > patch#3 and #4 depend upon [1]
> > [1]
> > https://lore.kernel.org/linux-renesas-soc/20220510144259.9908-1-biju.d
> > as.jz@bp.renesas.com/T/#t
>
> I suggest to use the --base switch to git-format-patch for the next
> submission round. This way the built robots can parse this information,
> too.
Agreed. Will take care this in next submission.
Cheers,
Biju
^ permalink raw reply [flat|nested] 22+ messages in thread