* [PATCH 0/3] Add support for linking gpt with poeg @ 2022-11-04 14:59 Biju Das 2022-11-04 14:59 ` [PATCH 1/3] dt-bindings: pwm: Add RZ/G2L GPT binding Biju Das 2022-11-04 14:59 ` [PATCH 2/3] dt-bindings: pwm: rzg2l-gpt: Document renesas,poegs property Biju Das 0 siblings, 2 replies; 10+ messages in thread From: Biju Das @ 2022-11-04 14:59 UTC (permalink / raw) To: Thierry Reding, Rob Herring, Krzysztof Kozlowski Cc: Biju Das, Uwe Kleine-König, linux-pwm, devicetree, Geert Uytterhoeven, Chris Paterson, Prabhakar Mahadev Lad, linux-renesas-soc Renesas RZ/G2L gpt supports output pin disable function by dead time error and detecting short-circuits between output pins. There are 4 poeg groups shared between gpt channels. This patch adds support for linking poeg group with gpt channel, so that gpt channel can control the output disable function through poeg module(port output enable for gpt) patch#1 is just added to avoid bot errors. Patch#3 depend upon [1] [1] https://lore.kernel.org/linux-renesas-soc/20221028104231.220426-3-biju.das.jz@bp.renesas.com/T/#u Biju Das (3): dt-bindings: pwm: Add RZ/G2L GPT binding dt-bindings: pwm: rzg2l-gpt: Document renesas,poegs property pwm: rzg2l-gpt: Add support for linking with poeg .../bindings/pwm/renesas,rzg2l-gpt.yaml | 397 ++++++++++++++++++ drivers/pwm/pwm-rzg2l-gpt.c | 69 +++ 2 files changed, 466 insertions(+) create mode 100644 Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml base-commit: 0cdb3579f1ee4c1e55acf8dfb0697b660067b1f8 prerequisite-patch-id: 7fc6823735fb2b0b321ece8ee749e9a987ca0da6 -- 2.25.1 ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 1/3] dt-bindings: pwm: Add RZ/G2L GPT binding 2022-11-04 14:59 [PATCH 0/3] Add support for linking gpt with poeg Biju Das @ 2022-11-04 14:59 ` Biju Das 2022-11-04 15:02 ` Krzysztof Kozlowski 2022-11-04 14:59 ` [PATCH 2/3] dt-bindings: pwm: rzg2l-gpt: Document renesas,poegs property Biju Das 1 sibling, 1 reply; 10+ messages in thread From: Biju Das @ 2022-11-04 14:59 UTC (permalink / raw) To: Thierry Reding, Rob Herring, Krzysztof Kozlowski Cc: Biju Das, Uwe Kleine-König, linux-pwm, devicetree, Geert Uytterhoeven, Chris Paterson, Prabhakar Mahadev Lad, linux-renesas-soc, Rob Herring Add device tree bindings for the General PWM Timer (GPT). Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> Reviewed-by: Rob Herring <robh@kernel.org> --- sending to avoid bot error for the next patch. --- .../bindings/pwm/renesas,rzg2l-gpt.yaml | 378 ++++++++++++++++++ 1 file changed, 378 insertions(+) create mode 100644 Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml diff --git a/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml b/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml new file mode 100644 index 000000000000..620d5ae4ae30 --- /dev/null +++ b/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml @@ -0,0 +1,378 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/pwm/renesas,rzg2l-gpt.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Renesas RZ/G2L General PWM Timer (GPT) + +maintainers: + - Biju Das <biju.das.jz@bp.renesas.com> + +description: | + RZ/G2L General PWM Timer (GPT) composed of 8 channels with 32-bit timer + (GPT32E). It supports the following functions + * 32 bits × 8 channels. + * Up-counting or down-counting (saw waves) or up/down-counting + (triangle waves) for each counter. + * Clock sources independently selectable for each channel. + * Two I/O pins per channel. + * Two output compare/input capture registers per channel. + * For the two output compare/input capture registers of each channel, + four registers are provided as buffer registers and are capable of + operating as comparison registers when buffering is not in use. + * In output compare operation, buffer switching can be at crests or + troughs, enabling the generation of laterally asymmetric PWM waveforms. + * Registers for setting up frame cycles in each channel (with capability + for generating interrupts at overflow or underflow) + * Generation of dead times in PWM operation. + * Synchronous starting, stopping and clearing counters for arbitrary + channels. + * Starting, stopping, clearing and up/down counters in response to input + level comparison. + * Starting, clearing, stopping and up/down counters in response to a + maximum of four external triggers. + * Output pin disable function by dead time error and detected + short-circuits between output pins. + * A/D converter start triggers can be generated (GPT32E0 to GPT32E3) + * Enables the noise filter for input capture and external trigger + operation. + + The below pwm channels are supported. + pwm0 - GPT32E0.GTIOC0A channel + pwm1 - GPT32E0.GTIOC0B channel + pwm2 - GPT32E1.GTIOC1A channel + pwm3 - GPT32E1.GTIOC1B channel + pwm4 - GPT32E2.GTIOC2A channel + pwm5 - GPT32E2.GTIOC2B channel + pwm6 - GPT32E3.GTIOC3A channel + pwm7 - GPT32E3.GTIOC3B channel + pwm8 - GPT32E4.GTIOC4A channel + pwm9 - GPT32E4.GTIOC4B channel + pwm10 - GPT32E5.GTIOC5A channel + pwm11 - GPT32E5.GTIOC5B channel + pwm12 - GPT32E6.GTIOC6A channel + pwm13 - GPT32E6.GTIOC6B channel + pwm14 - GPT32E7.GTIOC7A channel + pwm15 - GPT32E7.GTIOC7B channel + +properties: + compatible: + items: + - enum: + - renesas,r9a07g044-gpt # RZ/G2{L,LC} + - renesas,r9a07g054-gpt # RZ/V2L + - const: renesas,rzg2l-gpt + + reg: + maxItems: 1 + + '#pwm-cells': + const: 2 + + interrupts: + items: + - description: GPT32E0.GTCCRA input capture/compare match + - description: GPT32E0.GTCCRB input capture/compare + - description: GPT32E0.GTCCRC compare match + - description: GPT32E0.GTCCRD compare match + - description: GPT32E0.GTCCRE compare match + - description: GPT32E0.GTCCRF compare match + - description: GPT32E0.GTADTRA compare match + - description: GPT32E0.GTADTRB compare match + - description: GPT32E0.GTCNT overflow/GTPR compare match + - description: GPT32E0.GTCNT underflow + - description: GPT32E1.GTCCRA input capture/compare match + - description: GPT32E1.GTCCRB input capture/compare + - description: GPT32E1.GTCCRC compare match + - description: GPT32E1.GTCCRD compare match + - description: GPT32E1.GTCCRE compare match + - description: GPT32E1.GTCCRF compare match + - description: GPT32E1.GTADTRA compare match + - description: GPT32E1.GTADTRB compare match + - description: GPT32E1.GTCNT overflow/GTPR compare match + - description: GPT32E1.GTCNT underflow + - description: GPT32E2.GTCCRA input capture/compare match + - description: GPT32E2.GTCCRB input capture/compare + - description: GPT32E2.GTCCRC compare match + - description: GPT32E2.GTCCRD compare match + - description: GPT32E2.GTCCRE compare match + - description: GPT32E2.GTCCRF compare match + - description: GPT32E2.GTADTRA compare match + - description: GPT32E2.GTADTRB compare match + - description: GPT32E2.GTCNT overflow/GTPR compare match + - description: GPT32E2.GTCNT underflow + - description: GPT32E3.GTCCRA input capture/compare match + - description: GPT32E3.GTCCRB input capture/compare + - description: GPT32E3.GTCCRC compare match + - description: GPT32E3.GTCCRD compare match + - description: GPT32E3.GTCCRE compare match + - description: GPT32E3.GTCCRF compare match + - description: GPT32E3.GTADTRA compare match + - description: GPT32E3.GTADTRB compare match + - description: GPT32E3.GTCNT overflow/GTPR compare match + - description: GPT32E3.GTCNT underflow + - description: GPT32E4.GTCCRA input capture/compare match + - description: GPT32E4.GTCCRB input capture/compare + - description: GPT32E4.GTCCRC compare match + - description: GPT32E4.GTCCRD compare match + - description: GPT32E4.GTCCRE compare match + - description: GPT32E4.GTCCRF compare match + - description: GPT32E4.GTADTRA compare match + - description: GPT32E4.GTADTRB compare match + - description: GPT32E4.GTCNT overflow/GTPR compare match + - description: GPT32E4.GTCNT underflow + - description: GPT32E5.GTCCRA input capture/compare match + - description: GPT32E5.GTCCRB input capture/compare + - description: GPT32E5.GTCCRC compare match + - description: GPT32E5.GTCCRD compare match + - description: GPT32E5.GTCCRE compare match + - description: GPT32E5.GTCCRF compare match + - description: GPT32E5.GTADTRA compare match + - description: GPT32E5.GTADTRB compare match + - description: GPT32E5.GTCNT overflow/GTPR compare match + - description: GPT32E5.GTCNT underflow + - description: GPT32E6.GTCCRA input capture/compare match + - description: GPT32E6.GTCCRB input capture/compare + - description: GPT32E6.GTCCRC compare match + - description: GPT32E6.GTCCRD compare match + - description: GPT32E6.GTCCRE compare match + - description: GPT32E6.GTCCRF compare match + - description: GPT32E6.GTADTRA compare match + - description: GPT32E6.GTADTRB compare match + - description: GPT32E6.GTCNT overflow/GTPR compare match + - description: GPT32E6.GTCNT underflow + - description: GPT32E7.GTCCRA input capture/compare match + - description: GPT32E7.GTCCRB input capture/compare + - description: GPT32E7.GTCCRC compare match + - description: GPT32E7.GTCCRD compare match + - description: GPT32E7.GTCCRE compare match + - description: GPT32E7.GTCCRF compare match + - description: GPT32E7.GTADTRA compare match + - description: GPT32E7.GTADTRB compare match + - description: GPT32E7.GTCNT overflow/GTPR compare match + - description: GPT32E7.GTCNT underflow + + interrupt-names: + items: + - const: ccmpa0 + - const: ccmpb0 + - const: cmpc0 + - const: cmpd0 + - const: cmpe0 + - const: cmpf0 + - const: adtrga0 + - const: adtrgb0 + - const: ovf0 + - const: unf0 + - const: ccmpa1 + - const: ccmpb1 + - const: cmpc1 + - const: cmpd1 + - const: cmpe1 + - const: cmpf1 + - const: adtrga1 + - const: adtrgb1 + - const: ovf1 + - const: unf1 + - const: ccmpa2 + - const: ccmpb2 + - const: cmpc2 + - const: cmpd2 + - const: cmpe2 + - const: cmpf2 + - const: adtrga2 + - const: adtrgb2 + - const: ovf2 + - const: unf2 + - const: ccmpa3 + - const: ccmpb3 + - const: cmpc3 + - const: cmpd3 + - const: cmpe3 + - const: cmpf3 + - const: adtrga3 + - const: adtrgb3 + - const: ovf3 + - const: unf3 + - const: ccmpa4 + - const: ccmpb4 + - const: cmpc4 + - const: cmpd4 + - const: cmpe4 + - const: cmpf4 + - const: adtrga4 + - const: adtrgb4 + - const: ovf4 + - const: unf4 + - const: ccmpa5 + - const: ccmpb5 + - const: cmpc5 + - const: cmpd5 + - const: cmpe5 + - const: cmpf5 + - const: adtrga5 + - const: adtrgb5 + - const: ovf5 + - const: unf5 + - const: ccmpa6 + - const: ccmpb6 + - const: cmpc6 + - const: cmpd6 + - const: cmpe6 + - const: cmpf6 + - const: adtrga6 + - const: adtrgb6 + - const: ovf6 + - const: unf6 + - const: ccmpa7 + - const: ccmpb7 + - const: cmpc7 + - const: cmpd7 + - const: cmpe7 + - const: cmpf7 + - const: adtrga7 + - const: adtrgb7 + - const: ovf7 + - const: unf7 + + clocks: + maxItems: 1 + + power-domains: + maxItems: 1 + + resets: + maxItems: 1 + +required: + - compatible + - reg + - interrupts + - interrupt-names + - clocks + - power-domains + - resets + +allOf: + - $ref: pwm.yaml# + +additionalProperties: false + +examples: + - | + #include <dt-bindings/clock/r9a07g044-cpg.h> + #include <dt-bindings/interrupt-controller/arm-gic.h> + + gpt: pwm@10048000 { + compatible = "renesas,r9a07g044-gpt", "renesas,rzg2l-gpt"; + reg = <0x10048000 0x800>; + interrupts = <GIC_SPI 218 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 219 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 220 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 221 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 222 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 223 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 224 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 225 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 226 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 227 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 231 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 232 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 233 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 234 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 235 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 236 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 237 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 238 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 239 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 240 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 244 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 245 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 246 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 247 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 248 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 249 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 250 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 251 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 252 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 253 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 257 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 258 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 259 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 260 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 261 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 262 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 263 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 264 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 265 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 266 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 270 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 271 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 272 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 273 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 274 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 275 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 276 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 277 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 278 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 279 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 283 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 284 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 285 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 286 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 287 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 288 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 289 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 290 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 291 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 292 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 296 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 297 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 298 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 299 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 300 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 301 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 302 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 303 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 304 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 305 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 309 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 310 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 311 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 312 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 313 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 314 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 315 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 316 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 317 IRQ_TYPE_EDGE_RISING>, + <GIC_SPI 318 IRQ_TYPE_EDGE_RISING>; + interrupt-names = "ccmpa0", "ccmpb0", "cmpc0", "cmpd0", + "cmpe0", "cmpf0", "adtrga0", "adtrgb0", + "ovf0", "unf0", + "ccmpa1", "ccmpb1", "cmpc1", "cmpd1", + "cmpe1", "cmpf1", "adtrga1", "adtrgb1", + "ovf1", "unf1", + "ccmpa2", "ccmpb2", "cmpc2", "cmpd2", + "cmpe2", "cmpf2", "adtrga2", "adtrgb2", + "ovf2", "unf2", + "ccmpa3", "ccmpb3", "cmpc3", "cmpd3", + "cmpe3", "cmpf3", "adtrga3", "adtrgb3", + "ovf3", "unf3", + "ccmpa4", "ccmpb4", "cmpc4", "cmpd4", + "cmpe4", "cmpf4", "adtrga4", "adtrgb4", + "ovf4", "unf4", + "ccmpa5", "ccmpb5", "cmpc5", "cmpd5", + "cmpe5", "cmpf5", "adtrga5", "adtrgb5", + "ovf5", "unf5", + "ccmpa6", "ccmpb6", "cmpc6", "cmpd6", + "cmpe6", "cmpf6", "adtrga6", "adtrgb6", + "ovf6", "unf6", + "ccmpa7", "ccmpb7", "cmpc7", "cmpd7", + "cmpe7", "cmpf7", "adtrga7", "adtrgb7", + "ovf7", "unf7"; + clocks = <&cpg CPG_MOD R9A07G044_GPT_PCLK>; + power-domains = <&cpg>; + resets = <&cpg R9A07G044_GPT_RST_C>; + #pwm-cells = <2>; + }; -- 2.25.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 1/3] dt-bindings: pwm: Add RZ/G2L GPT binding 2022-11-04 14:59 ` [PATCH 1/3] dt-bindings: pwm: Add RZ/G2L GPT binding Biju Das @ 2022-11-04 15:02 ` Krzysztof Kozlowski 2022-11-04 15:58 ` Biju Das 0 siblings, 1 reply; 10+ messages in thread From: Krzysztof Kozlowski @ 2022-11-04 15:02 UTC (permalink / raw) To: Biju Das, Thierry Reding, Rob Herring, Krzysztof Kozlowski Cc: Uwe Kleine-König, linux-pwm, devicetree, Geert Uytterhoeven, Chris Paterson, Prabhakar Mahadev Lad, linux-renesas-soc, Rob Herring On 04/11/2022 10:59, Biju Das wrote: > Add device tree bindings for the General PWM Timer (GPT). > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> > Reviewed-by: Rob Herring <robh@kernel.org> > --- > sending to avoid bot error for the next patch. Then subject maybe: [DO NOT APPLY PATCH 1/3] ? Best regards, Krzysztof ^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH 1/3] dt-bindings: pwm: Add RZ/G2L GPT binding 2022-11-04 15:02 ` Krzysztof Kozlowski @ 2022-11-04 15:58 ` Biju Das 0 siblings, 0 replies; 10+ messages in thread From: Biju Das @ 2022-11-04 15:58 UTC (permalink / raw) To: Krzysztof Kozlowski, Thierry Reding, Rob Herring, Krzysztof Kozlowski Cc: Uwe Kleine-König, linux-pwm@vger.kernel.org, devicetree@vger.kernel.org, Geert Uytterhoeven, Chris Paterson, Prabhakar Mahadev Lad, linux-renesas-soc@vger.kernel.org, Rob Herring Hi Krzysztof Kozlowski, Thanks for the feedback. > Subject: Re: [PATCH 1/3] dt-bindings: pwm: Add RZ/G2L GPT binding > > On 04/11/2022 10:59, Biju Das wrote: > > Add device tree bindings for the General PWM Timer (GPT). > > > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> > > Reviewed-by: Rob Herring <robh@kernel.org> > > --- > > sending to avoid bot error for the next patch. > > Then subject maybe: > [DO NOT APPLY PATCH 1/3] > ? OK. Will take care next time. Cheers, Biju ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 2/3] dt-bindings: pwm: rzg2l-gpt: Document renesas,poegs property 2022-11-04 14:59 [PATCH 0/3] Add support for linking gpt with poeg Biju Das 2022-11-04 14:59 ` [PATCH 1/3] dt-bindings: pwm: Add RZ/G2L GPT binding Biju Das @ 2022-11-04 14:59 ` Biju Das 2022-11-04 15:04 ` Krzysztof Kozlowski 1 sibling, 1 reply; 10+ messages in thread From: Biju Das @ 2022-11-04 14:59 UTC (permalink / raw) To: Thierry Reding, Rob Herring, Krzysztof Kozlowski Cc: Biju Das, Uwe Kleine-König, linux-pwm, devicetree, Geert Uytterhoeven, Chris Paterson, Prabhakar Mahadev Lad, linux-renesas-soc RZ/G2L GPT IP supports output pin disable function by dead time error and detecting short-circuits between output pins. Add documentation for the optional property renesas,poegs to link a pair of GPT IOs with POEG. Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> --- .../bindings/pwm/renesas,rzg2l-gpt.yaml | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) diff --git a/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml b/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml index 620d5ae4ae30..32f9deae89e5 100644 --- a/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml +++ b/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml @@ -245,6 +245,24 @@ properties: resets: maxItems: 1 + renesas,poegs: + $ref: "/schemas/types.yaml#/definitions/phandle-array" + items: + items: + - description: phandle to POEG instance that serves the output disable + - description: An index identifying pair of GPT channels. + <0> - GPT channels 0 and 1 + <1> - GPT channels 2 and 3 + <2> - GPT channels 4 and 5 + <3> - GPT channels 6 and 7 + <4> - GPT channels 8 and 9 + <5> - GPT channels 10 and 11 + <6> - GPT channels 12 and 13 + <7> - GPT channels 14 and 15 + description: + A list of phandle and channel index pair tuples to the POEGs that handle the + output disable for the GPT channels. + required: - compatible - reg @@ -375,4 +393,5 @@ examples: power-domains = <&cpg>; resets = <&cpg R9A07G044_GPT_RST_C>; #pwm-cells = <2>; + renesas,poegs = <&poeggd 4>; }; -- 2.25.1 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 2/3] dt-bindings: pwm: rzg2l-gpt: Document renesas,poegs property 2022-11-04 14:59 ` [PATCH 2/3] dt-bindings: pwm: rzg2l-gpt: Document renesas,poegs property Biju Das @ 2022-11-04 15:04 ` Krzysztof Kozlowski 2022-11-04 15:32 ` Biju Das 2022-11-04 16:52 ` Biju Das 0 siblings, 2 replies; 10+ messages in thread From: Krzysztof Kozlowski @ 2022-11-04 15:04 UTC (permalink / raw) To: Biju Das, Thierry Reding, Rob Herring, Krzysztof Kozlowski Cc: Uwe Kleine-König, linux-pwm, devicetree, Geert Uytterhoeven, Chris Paterson, Prabhakar Mahadev Lad, linux-renesas-soc On 04/11/2022 10:59, Biju Das wrote: > RZ/G2L GPT IP supports output pin disable function by dead time > error and detecting short-circuits between output pins. > > Add documentation for the optional property renesas,poegs to > link a pair of GPT IOs with POEG. > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> > --- > .../bindings/pwm/renesas,rzg2l-gpt.yaml | 19 +++++++++++++++++++ > 1 file changed, 19 insertions(+) > > diff --git a/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml b/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml > index 620d5ae4ae30..32f9deae89e5 100644 > --- a/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml > +++ b/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml > @@ -245,6 +245,24 @@ properties: > resets: > maxItems: 1 > > + renesas,poegs: > + $ref: "/schemas/types.yaml#/definitions/phandle-array" No need for quotes. > + items: You miss here maxItems... but if you have just one item, then below "items" should be " - items" > + items: > + - description: phandle to POEG instance that serves the output disable > + - description: An index identifying pair of GPT channels. > + <0> - GPT channels 0 and 1 > + <1> - GPT channels 2 and 3 > + <2> - GPT channels 4 and 5 > + <3> - GPT channels 6 and 7 > + <4> - GPT channels 8 and 9 > + <5> - GPT channels 10 and 11 > + <6> - GPT channels 12 and 13 > + <7> - GPT channels 14 and 15 then this could bave enum or minimum/maximum. Can you try if these work? > + description: > + A list of phandle and channel index pair tuples to the POEGs that handle the > + output disable for the GPT channels. > + > required: > - compatible > - reg > @@ -375,4 +393,5 @@ examples: > power-domains = <&cpg>; > resets = <&cpg R9A07G044_GPT_RST_C>; > #pwm-cells = <2>; > + renesas,poegs = <&poeggd 4>; > }; Best regards, Krzysztof ^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH 2/3] dt-bindings: pwm: rzg2l-gpt: Document renesas,poegs property 2022-11-04 15:04 ` Krzysztof Kozlowski @ 2022-11-04 15:32 ` Biju Das 2022-11-04 16:52 ` Biju Das 1 sibling, 0 replies; 10+ messages in thread From: Biju Das @ 2022-11-04 15:32 UTC (permalink / raw) To: Krzysztof Kozlowski, Thierry Reding, Rob Herring, Krzysztof Kozlowski Cc: Uwe Kleine-König, linux-pwm@vger.kernel.org, devicetree@vger.kernel.org, Geert Uytterhoeven, Chris Paterson, Prabhakar Mahadev Lad, linux-renesas-soc@vger.kernel.org Hi Krzysztof Kozlowski, Thanks for the feedback. > Subject: Re: [PATCH 2/3] dt-bindings: pwm: rzg2l-gpt: Document > renesas,poegs property > > On 04/11/2022 10:59, Biju Das wrote: > > RZ/G2L GPT IP supports output pin disable function by dead time > error > > and detecting short-circuits between output pins. > > > > Add documentation for the optional property renesas,poegs to link a > > pair of GPT IOs with POEG. > > > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> > > --- > > .../bindings/pwm/renesas,rzg2l-gpt.yaml | 19 > +++++++++++++++++++ > > 1 file changed, 19 insertions(+) > > > > diff --git > > a/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml > > b/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml > > index 620d5ae4ae30..32f9deae89e5 100644 > > --- a/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml > > +++ b/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml > > @@ -245,6 +245,24 @@ properties: > > resets: > > maxItems: 1 > > > > + renesas,poegs: > > + $ref: "/schemas/types.yaml#/definitions/phandle-array" > > No need for quotes. OK, I just referred renesas,vsps in [1] as example [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/display/renesas,du.yaml?h=v6.1-rc3#n85 > > > + items: > > You miss here maxItems... but if you have just one item, then below > "items" should be " - items" Max Items is 8, as we have 8 GPT hw channels. Based on HW board design, we should have flexibility to define something like below, renesas,poegs = <&poegga 0>, <&poeggb 1> <&poegga 2> <&poeggd 4>; > > > + items: > > + - description: phandle to POEG instance that serves the > output disable > > + - description: An index identifying pair of GPT channels. > > + <0> - GPT channels 0 and 1 > > + <1> - GPT channels 2 and 3 > > + <2> - GPT channels 4 and 5 > > + <3> - GPT channels 6 and 7 > > + <4> - GPT channels 8 and 9 > > + <5> - GPT channels 10 and 11 > > + <6> - GPT channels 12 and 13 > > + <7> - GPT channels 14 and 15 > > then this could bave enum or minimum/maximum. Can you try if these > work? OK, I will try second items as enum as it is predefined values and Will let you know the results. Cheers, Biju ^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH 2/3] dt-bindings: pwm: rzg2l-gpt: Document renesas,poegs property 2022-11-04 15:04 ` Krzysztof Kozlowski 2022-11-04 15:32 ` Biju Das @ 2022-11-04 16:52 ` Biju Das 2022-11-04 17:10 ` Krzysztof Kozlowski 1 sibling, 1 reply; 10+ messages in thread From: Biju Das @ 2022-11-04 16:52 UTC (permalink / raw) To: Krzysztof Kozlowski, Thierry Reding, Rob Herring, Krzysztof Kozlowski Cc: Uwe Kleine-König, linux-pwm@vger.kernel.org, devicetree@vger.kernel.org, Geert Uytterhoeven, Chris Paterson, Prabhakar Mahadev Lad, linux-renesas-soc@vger.kernel.org Hi Krzysztof Kozlowski, > -----Original Message----- > From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > Subject: Re: [PATCH 2/3] dt-bindings: pwm: rzg2l-gpt: Document > renesas,poegs property > > On 04/11/2022 10:59, Biju Das wrote: > > RZ/G2L GPT IP supports output pin disable function by dead time > error > > and detecting short-circuits between output pins. > > > > Add documentation for the optional property renesas,poegs to link a > > pair of GPT IOs with POEG. > > > > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> > > --- > > .../bindings/pwm/renesas,rzg2l-gpt.yaml | 19 > +++++++++++++++++++ > > 1 file changed, 19 insertions(+) > > > > diff --git > > a/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml > > b/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml > > index 620d5ae4ae30..32f9deae89e5 100644 > > --- a/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml > > +++ b/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml > > @@ -245,6 +245,24 @@ properties: > > resets: > > maxItems: 1 > > > > + renesas,poegs: > > + $ref: "/schemas/types.yaml#/definitions/phandle-array" > > No need for quotes. > > > + items: > > You miss here maxItems... but if you have just one item, then below > "items" should be " - items" > > > + items: > > + - description: phandle to POEG instance that serves the > output disable > > + - description: An index identifying pair of GPT channels. > > + <0> - GPT channels 0 and 1 > > + <1> - GPT channels 2 and 3 > > + <2> - GPT channels 4 and 5 > > + <3> - GPT channels 6 and 7 > > + <4> - GPT channels 8 and 9 > > + <5> - GPT channels 10 and 11 > > + <6> - GPT channels 12 and 13 > > + <7> - GPT channels 14 and 15 > > then this could bave enum or minimum/maximum. Can you try if these > work? Yes, checks are passing with below changes. Will send V2 later once I get feedback for driver changes/ from other reviewers. renesas,poegs: - $ref: "/schemas/types.yaml#/definitions/phandle-array" + $ref: /schemas/types.yaml#/definitions/phandle-array items: + maxItems: 8 items: - description: phandle to POEG instance that serves the output disable - - description: An index identifying pair of GPT channels. - <0> - GPT channels 0 and 1 - <1> - GPT channels 2 and 3 - <2> - GPT channels 4 and 5 - <3> - GPT channels 6 and 7 - <4> - GPT channels 8 and 9 - <5> - GPT channels 10 and 11 - <6> - GPT channels 12 and 13 - <7> - GPT channels 14 and 15 + - enum: [ 0, 1, 2, 3, 4, 5, 6, 7 ] + description: | + An index identifying pair of GPT channels. + <0> : GPT channels 0 and 1 + <1> : GPT channels 2 and 3 + <2> : GPT channels 4 and 5 + <3> : GPT channels 6 and 7 + <4> : GPT channels 8 and 9 + <5> : GPT channels 10 and 11 + <6> : GPT channels 12 and 13 + <7> : GPT channels 14 and 15 Cheers, Biju ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 2/3] dt-bindings: pwm: rzg2l-gpt: Document renesas,poegs property 2022-11-04 16:52 ` Biju Das @ 2022-11-04 17:10 ` Krzysztof Kozlowski 2022-11-11 6:50 ` Biju Das 0 siblings, 1 reply; 10+ messages in thread From: Krzysztof Kozlowski @ 2022-11-04 17:10 UTC (permalink / raw) To: Biju Das, Thierry Reding, Rob Herring, Krzysztof Kozlowski Cc: Uwe Kleine-König, linux-pwm@vger.kernel.org, devicetree@vger.kernel.org, Geert Uytterhoeven, Chris Paterson, Prabhakar Mahadev Lad, linux-renesas-soc@vger.kernel.org On 04/11/2022 12:52, Biju Das wrote: > Hi Krzysztof Kozlowski, > >> -----Original Message----- >> From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >> Subject: Re: [PATCH 2/3] dt-bindings: pwm: rzg2l-gpt: Document >> renesas,poegs property >> >> On 04/11/2022 10:59, Biju Das wrote: >>> RZ/G2L GPT IP supports output pin disable function by dead time >> error >>> and detecting short-circuits between output pins. >>> >>> Add documentation for the optional property renesas,poegs to link a >>> pair of GPT IOs with POEG. >>> >>> Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> >>> --- >>> .../bindings/pwm/renesas,rzg2l-gpt.yaml | 19 >> +++++++++++++++++++ >>> 1 file changed, 19 insertions(+) >>> >>> diff --git >>> a/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml >>> b/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml >>> index 620d5ae4ae30..32f9deae89e5 100644 >>> --- a/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml >>> +++ b/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml >>> @@ -245,6 +245,24 @@ properties: >>> resets: >>> maxItems: 1 >>> >>> + renesas,poegs: >>> + $ref: "/schemas/types.yaml#/definitions/phandle-array" >> >> No need for quotes. >> >>> + items: >> >> You miss here maxItems... but if you have just one item, then below >> "items" should be " - items" >> >>> + items: >>> + - description: phandle to POEG instance that serves the >> output disable >>> + - description: An index identifying pair of GPT channels. >>> + <0> - GPT channels 0 and 1 >>> + <1> - GPT channels 2 and 3 >>> + <2> - GPT channels 4 and 5 >>> + <3> - GPT channels 6 and 7 >>> + <4> - GPT channels 8 and 9 >>> + <5> - GPT channels 10 and 11 >>> + <6> - GPT channels 12 and 13 >>> + <7> - GPT channels 14 and 15 >> >> then this could bave enum or minimum/maximum. Can you try if these >> work? > > Yes, checks are passing with below changes. > Will send V2 later once I get feedback for driver changes/ from other reviewers. > > renesas,poegs: > - $ref: "/schemas/types.yaml#/definitions/phandle-array" > + $ref: /schemas/types.yaml#/definitions/phandle-array > items: > + maxItems: 8 and it might also require minItems: 1 Thank you for changes. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH 2/3] dt-bindings: pwm: rzg2l-gpt: Document renesas,poegs property 2022-11-04 17:10 ` Krzysztof Kozlowski @ 2022-11-11 6:50 ` Biju Das 0 siblings, 0 replies; 10+ messages in thread From: Biju Das @ 2022-11-11 6:50 UTC (permalink / raw) To: Krzysztof Kozlowski, Thierry Reding, Rob Herring, Krzysztof Kozlowski Cc: Uwe Kleine-König, linux-pwm@vger.kernel.org, devicetree@vger.kernel.org, Geert Uytterhoeven, Chris Paterson, Prabhakar Mahadev Lad, linux-renesas-soc@vger.kernel.org Hi Krzysztof Kozlowski, > -----Original Message----- > From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > Sent: 04 November 2022 17:11 > To: Biju Das <biju.das.jz@bp.renesas.com>; Thierry Reding > <thierry.reding@gmail.com>; Rob Herring <robh+dt@kernel.org>; Krzysztof > Kozlowski <krzysztof.kozlowski+dt@linaro.org> > Cc: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>; linux- > pwm@vger.kernel.org; devicetree@vger.kernel.org; Geert Uytterhoeven > <geert+renesas@glider.be>; Chris Paterson <Chris.Paterson2@renesas.com>; > Prabhakar Mahadev Lad <prabhakar.mahadev-lad.rj@bp.renesas.com>; linux- > renesas-soc@vger.kernel.org > Subject: Re: [PATCH 2/3] dt-bindings: pwm: rzg2l-gpt: Document renesas,poegs > property > > On 04/11/2022 12:52, Biju Das wrote: > > Hi Krzysztof Kozlowski, > > > >> -----Original Message----- > >> From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > >> Subject: Re: [PATCH 2/3] dt-bindings: pwm: rzg2l-gpt: Document > >> renesas,poegs property > >> > >> On 04/11/2022 10:59, Biju Das wrote: > >>> RZ/G2L GPT IP supports output pin disable function by dead time > >> error > >>> and detecting short-circuits between output pins. > >>> > >>> Add documentation for the optional property renesas,poegs to link a > >>> pair of GPT IOs with POEG. > >>> > >>> Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com> > >>> --- > >>> .../bindings/pwm/renesas,rzg2l-gpt.yaml | 19 > >> +++++++++++++++++++ > >>> 1 file changed, 19 insertions(+) > >>> > >>> diff --git > >>> a/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml > >>> b/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml > >>> index 620d5ae4ae30..32f9deae89e5 100644 > >>> --- a/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml > >>> +++ b/Documentation/devicetree/bindings/pwm/renesas,rzg2l-gpt.yaml > >>> @@ -245,6 +245,24 @@ properties: > >>> resets: > >>> maxItems: 1 > >>> > >>> + renesas,poegs: > >>> + $ref: "/schemas/types.yaml#/definitions/phandle-array" > >> > >> No need for quotes. > >> > >>> + items: > >> > >> You miss here maxItems... but if you have just one item, then below > >> "items" should be " - items" > >> > >>> + items: > >>> + - description: phandle to POEG instance that serves the > >> output disable > >>> + - description: An index identifying pair of GPT channels. > >>> + <0> - GPT channels 0 and 1 > >>> + <1> - GPT channels 2 and 3 > >>> + <2> - GPT channels 4 and 5 > >>> + <3> - GPT channels 6 and 7 > >>> + <4> - GPT channels 8 and 9 > >>> + <5> - GPT channels 10 and 11 > >>> + <6> - GPT channels 12 and 13 > >>> + <7> - GPT channels 14 and 15 > >> > >> then this could bave enum or minimum/maximum. Can you try if these > >> work? > > > > Yes, checks are passing with below changes. > > Will send V2 later once I get feedback for driver changes/ from other > reviewers. > > > > renesas,poegs: > > - $ref: "/schemas/types.yaml#/definitions/phandle-array" > > + $ref: /schemas/types.yaml#/definitions/phandle-array > > items: > > + maxItems: 8 > > and it might also require minItems: 1 OK will send V2 with these changes. Cheers, Biju ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2022-11-11 6:50 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2022-11-04 14:59 [PATCH 0/3] Add support for linking gpt with poeg Biju Das 2022-11-04 14:59 ` [PATCH 1/3] dt-bindings: pwm: Add RZ/G2L GPT binding Biju Das 2022-11-04 15:02 ` Krzysztof Kozlowski 2022-11-04 15:58 ` Biju Das 2022-11-04 14:59 ` [PATCH 2/3] dt-bindings: pwm: rzg2l-gpt: Document renesas,poegs property Biju Das 2022-11-04 15:04 ` Krzysztof Kozlowski 2022-11-04 15:32 ` Biju Das 2022-11-04 16:52 ` Biju Das 2022-11-04 17:10 ` Krzysztof Kozlowski 2022-11-11 6:50 ` Biju Das
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).