| Message ID | 1513619528-12060-4-git-send-email-fabrizio.castro@bp.renesas.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Simon Horman |
| Headers | show |
On Mon, Dec 18, 2017 at 6:52 PM, Fabrizio Castro <fabrizio.castro@bp.renesas.com> wrote: > Add the definitions for pwm[0123456] to the SoC .dtsi. > > Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com> > Reviewed-by: Biju Das <biju.das@bp.renesas.com> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> > --- a/arch/arm/boot/dts/r8a7743.dtsi > +++ b/arch/arm/boot/dts/r8a7743.dtsi > @@ -1082,6 +1082,83 @@ > status = "disabled"; > }; > > + pwm0: pwm@e6e30000 { > + compatible = "renesas,pwm-r8a7743", > + "renesas,pwm-rcar"; I think the above can fit in one line. 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
Hello Geert, > Subject: Re: [PATCH v2 3/6] ARM: dts: r8a7743: Add PWM SoC support > > On Mon, Dec 18, 2017 at 6:52 PM, Fabrizio Castro > <fabrizio.castro@bp.renesas.com> wrote: > > Add the definitions for pwm[0123456] to the SoC .dtsi. > > > > Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com> > > Reviewed-by: Biju Das <biju.das@bp.renesas.com> > > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> > > > --- a/arch/arm/boot/dts/r8a7743.dtsi > > +++ b/arch/arm/boot/dts/r8a7743.dtsi > > @@ -1082,6 +1082,83 @@ > > status = "disabled"; > > }; > > > > + pwm0: pwm@e6e30000 { > > + compatible = "renesas,pwm-r8a7743", > > + "renesas,pwm-rcar"; > > I think the above can fit in one line. do you think this deserves a v3? Thanks, Fab > > 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 [https://www2.renesas.eu/media/email/unicef_2017.jpg] This Christmas, instead of sending out cards, Renesas Electronics Europe have decided to support Unicef with a donation. For further details click here<https://www.unicef.org/> to find out about the valuable work they do, helping children all over the world. We would like to take this opportunity to wish you a Merry Christmas and a prosperous New Year. Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
Hi Fabrizio, On Tue, Dec 19, 2017 at 12:37 PM, Fabrizio Castro <fabrizio.castro@bp.renesas.com> wrote: >> Subject: Re: [PATCH v2 3/6] ARM: dts: r8a7743: Add PWM SoC support >> >> On Mon, Dec 18, 2017 at 6:52 PM, Fabrizio Castro >> <fabrizio.castro@bp.renesas.com> wrote: >> > Add the definitions for pwm[0123456] to the SoC .dtsi. >> > >> > Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com> >> > Reviewed-by: Biju Das <biju.das@bp.renesas.com> >> >> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> >> >> > --- a/arch/arm/boot/dts/r8a7743.dtsi >> > +++ b/arch/arm/boot/dts/r8a7743.dtsi >> > @@ -1082,6 +1082,83 @@ >> > status = "disabled"; >> > }; >> > >> > + pwm0: pwm@e6e30000 { >> > + compatible = "renesas,pwm-r8a7743", >> > + "renesas,pwm-rcar"; >> >> I think the above can fit in one line. > > do you think this deserves a v3? That depends on the fuzziness-acceptance level of Simon... 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
Hello Geert, > Subject: Re: [PATCH v2 3/6] ARM: dts: r8a7743: Add PWM SoC support > > Hi Fabrizio, > > On Tue, Dec 19, 2017 at 12:37 PM, Fabrizio Castro > <fabrizio.castro@bp.renesas.com> wrote: > >> Subject: Re: [PATCH v2 3/6] ARM: dts: r8a7743: Add PWM SoC support > >> > >> On Mon, Dec 18, 2017 at 6:52 PM, Fabrizio Castro > >> <fabrizio.castro@bp.renesas.com> wrote: > >> > Add the definitions for pwm[0123456] to the SoC .dtsi. > >> > > >> > Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com> > >> > Reviewed-by: Biju Das <biju.das@bp.renesas.com> > >> > >> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> > >> > >> > --- a/arch/arm/boot/dts/r8a7743.dtsi > >> > +++ b/arch/arm/boot/dts/r8a7743.dtsi > >> > @@ -1082,6 +1082,83 @@ > >> > status = "disabled"; > >> > }; > >> > > >> > + pwm0: pwm@e6e30000 { > >> > + compatible = "renesas,pwm-r8a7743", > >> > + "renesas,pwm-rcar"; > >> > >> I think the above can fit in one line. > > > > do you think this deserves a v3? > > That depends on the fuzziness-acceptance level of Simon... I'll send a v3 then. Thanks, Fab > > 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 [https://www2.renesas.eu/media/email/unicef_2017.jpg] This Christmas, instead of sending out cards, Renesas Electronics Europe have decided to support Unicef with a donation. For further details click here<https://www.unicef.org/> to find out about the valuable work they do, helping children all over the world. We would like to take this opportunity to wish you a Merry Christmas and a prosperous New Year. Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
diff --git a/arch/arm/boot/dts/r8a7743.dtsi b/arch/arm/boot/dts/r8a7743.dtsi index c65d8f4..d73617a 100644 --- a/arch/arm/boot/dts/r8a7743.dtsi +++ b/arch/arm/boot/dts/r8a7743.dtsi @@ -1082,6 +1082,83 @@ status = "disabled"; }; + pwm0: pwm@e6e30000 { + compatible = "renesas,pwm-r8a7743", + "renesas,pwm-rcar"; + reg = <0 0xe6e30000 0 0x8>; + clocks = <&cpg CPG_MOD 523>; + power-domains = <&sysc R8A7743_PD_ALWAYS_ON>; + resets = <&cpg 523>; + #pwm-cells = <2>; + status = "disabled"; + }; + + pwm1: pwm@e6e31000 { + compatible = "renesas,pwm-r8a7743", + "renesas,pwm-rcar"; + reg = <0 0xe6e31000 0 0x8>; + clocks = <&cpg CPG_MOD 523>; + power-domains = <&sysc R8A7743_PD_ALWAYS_ON>; + resets = <&cpg 523>; + #pwm-cells = <2>; + status = "disabled"; + }; + + pwm2: pwm@e6e32000 { + compatible = "renesas,pwm-r8a7743", + "renesas,pwm-rcar"; + reg = <0 0xe6e32000 0 0x8>; + clocks = <&cpg CPG_MOD 523>; + power-domains = <&sysc R8A7743_PD_ALWAYS_ON>; + resets = <&cpg 523>; + #pwm-cells = <2>; + status = "disabled"; + }; + + pwm3: pwm@e6e33000 { + compatible = "renesas,pwm-r8a7743", + "renesas,pwm-rcar"; + reg = <0 0xe6e33000 0 0x8>; + clocks = <&cpg CPG_MOD 523>; + power-domains = <&sysc R8A7743_PD_ALWAYS_ON>; + resets = <&cpg 523>; + #pwm-cells = <2>; + status = "disabled"; + }; + + pwm4: pwm@e6e34000 { + compatible = "renesas,pwm-r8a7743", + "renesas,pwm-rcar"; + reg = <0 0xe6e34000 0 0x8>; + clocks = <&cpg CPG_MOD 523>; + power-domains = <&sysc R8A7743_PD_ALWAYS_ON>; + resets = <&cpg 523>; + #pwm-cells = <2>; + status = "disabled"; + }; + + pwm5: pwm@e6e35000 { + compatible = "renesas,pwm-r8a7743", + "renesas,pwm-rcar"; + reg = <0 0xe6e35000 0 0x8>; + clocks = <&cpg CPG_MOD 523>; + power-domains = <&sysc R8A7743_PD_ALWAYS_ON>; + resets = <&cpg 523>; + #pwm-cells = <2>; + status = "disabled"; + }; + + pwm6: pwm@e6e36000 { + compatible = "renesas,pwm-r8a7743", + "renesas,pwm-rcar"; + reg = <0 0xe6e36000 0 0x8>; + clocks = <&cpg CPG_MOD 523>; + power-domains = <&sysc R8A7743_PD_ALWAYS_ON>; + resets = <&cpg 523>; + #pwm-cells = <2>; + status = "disabled"; + }; + sdhi0: sd@ee100000 { compatible = "renesas,sdhi-r8a7743", "renesas,rcar-gen2-sdhi";