* [PATCH] riscv: dts: Add DT support for SiFive FU540 PWM driver @ 2019-08-21 9:23 Yash Shah 2019-09-10 9:52 ` Yash Shah 0 siblings, 1 reply; 5+ messages in thread From: Yash Shah @ 2019-08-21 9:23 UTC (permalink / raw) To: robh+dt, mark.rutland, paul.walmsley Cc: devicetree, aou, palmer, linux-kernel, sachin.ghadi, Yash Shah, sagar.kadam, linux-riscv, bmeng.cn Add the PWM DT node in SiFive FU540 soc-specific DT file. Enable the PWM nodes in HiFive Unleashed board-specific DT file. Signed-off-by: Yash Shah <yash.shah@sifive.com> --- arch/riscv/boot/dts/sifive/fu540-c000.dtsi | 19 +++++++++++++++++++ arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts | 8 ++++++++ 2 files changed, 27 insertions(+) diff --git a/arch/riscv/boot/dts/sifive/fu540-c000.dtsi b/arch/riscv/boot/dts/sifive/fu540-c000.dtsi index 42b5ec2..bb422db 100644 --- a/arch/riscv/boot/dts/sifive/fu540-c000.dtsi +++ b/arch/riscv/boot/dts/sifive/fu540-c000.dtsi @@ -230,6 +230,25 @@ #size-cells = <0>; status = "disabled"; }; + pwm0: pwm@10020000 { + compatible = "sifive,pwm0"; + reg = <0x0 0x10020000 0x0 0x1000>; + interrupt-parent = <&plic0>; + interrupts = <42 43 44 45>; + clocks = <&prci PRCI_CLK_TLCLK>; + #pwm-cells = <3>; + status = "disabled"; + }; + pwm1: pwm@10021000 { + compatible = "sifive,pwm0"; + reg = <0x0 0x10021000 0x0 0x1000>; + interrupt-parent = <&plic0>; + interrupts = <46 47 48 49>; + reg-names = "control"; + clocks = <&prci PRCI_CLK_TLCLK>; + #pwm-cells = <3>; + status = "disabled"; + }; }; }; diff --git a/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts b/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts index 93d68cb..104d334 100644 --- a/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts +++ b/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts @@ -85,3 +85,11 @@ reg = <0>; }; }; + +&pwm0 { + status = "okay"; +}; + +&pwm1 { + status = "okay"; +}; -- 1.9.1 _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] riscv: dts: Add DT support for SiFive FU540 PWM driver 2019-08-21 9:23 [PATCH] riscv: dts: Add DT support for SiFive FU540 PWM driver Yash Shah @ 2019-09-10 9:52 ` Yash Shah 2019-09-13 21:20 ` Palmer Dabbelt 0 siblings, 1 reply; 5+ messages in thread From: Yash Shah @ 2019-09-10 9:52 UTC (permalink / raw) To: Rob Herring, Mark Rutland, Paul Walmsley Cc: devicetree, Albert Ou, Palmer Dabbelt, linux-kernel@vger.kernel.org List, Sachin Ghadi, Sagar Kadam, linux-riscv, Bin Meng Hi, Any comments on this patch? - Yash On Wed, Aug 21, 2019 at 2:53 PM Yash Shah <yash.shah@sifive.com> wrote: > > Add the PWM DT node in SiFive FU540 soc-specific DT file. > Enable the PWM nodes in HiFive Unleashed board-specific DT file. > > Signed-off-by: Yash Shah <yash.shah@sifive.com> > --- > arch/riscv/boot/dts/sifive/fu540-c000.dtsi | 19 +++++++++++++++++++ > arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts | 8 ++++++++ > 2 files changed, 27 insertions(+) > > diff --git a/arch/riscv/boot/dts/sifive/fu540-c000.dtsi b/arch/riscv/boot/dts/sifive/fu540-c000.dtsi > index 42b5ec2..bb422db 100644 > --- a/arch/riscv/boot/dts/sifive/fu540-c000.dtsi > +++ b/arch/riscv/boot/dts/sifive/fu540-c000.dtsi > @@ -230,6 +230,25 @@ > #size-cells = <0>; > status = "disabled"; > }; > + pwm0: pwm@10020000 { > + compatible = "sifive,pwm0"; > + reg = <0x0 0x10020000 0x0 0x1000>; > + interrupt-parent = <&plic0>; > + interrupts = <42 43 44 45>; > + clocks = <&prci PRCI_CLK_TLCLK>; > + #pwm-cells = <3>; > + status = "disabled"; > + }; > + pwm1: pwm@10021000 { > + compatible = "sifive,pwm0"; > + reg = <0x0 0x10021000 0x0 0x1000>; > + interrupt-parent = <&plic0>; > + interrupts = <46 47 48 49>; > + reg-names = "control"; > + clocks = <&prci PRCI_CLK_TLCLK>; > + #pwm-cells = <3>; > + status = "disabled"; > + }; > > }; > }; > diff --git a/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts b/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts > index 93d68cb..104d334 100644 > --- a/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts > +++ b/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts > @@ -85,3 +85,11 @@ > reg = <0>; > }; > }; > + > +&pwm0 { > + status = "okay"; > +}; > + > +&pwm1 { > + status = "okay"; > +}; > -- > 1.9.1 > _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] riscv: dts: Add DT support for SiFive FU540 PWM driver 2019-09-10 9:52 ` Yash Shah @ 2019-09-13 21:20 ` Palmer Dabbelt 2019-09-16 11:18 ` Yash Shah 0 siblings, 1 reply; 5+ messages in thread From: Palmer Dabbelt @ 2019-09-13 21:20 UTC (permalink / raw) To: yash.shah Cc: mark.rutland, devicetree, aou, linux-kernel, Sachin Ghadi, robh+dt, sagar.kadam, Paul Walmsley, linux-riscv, bmeng.cn On Tue, 10 Sep 2019 02:52:07 PDT (-0700), yash.shah@sifive.com wrote: > Hi, > > Any comments on this patch? I don't see "sifive,pwm0" in the DT bindings documentation, and it doesn't match our standard way of doing these things (which would have at least "sifive,fu540-c000-pwm"). > > - Yash > > On Wed, Aug 21, 2019 at 2:53 PM Yash Shah <yash.shah@sifive.com> wrote: >> >> Add the PWM DT node in SiFive FU540 soc-specific DT file. >> Enable the PWM nodes in HiFive Unleashed board-specific DT file. >> >> Signed-off-by: Yash Shah <yash.shah@sifive.com> >> --- >> arch/riscv/boot/dts/sifive/fu540-c000.dtsi | 19 +++++++++++++++++++ >> arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts | 8 ++++++++ >> 2 files changed, 27 insertions(+) >> >> diff --git a/arch/riscv/boot/dts/sifive/fu540-c000.dtsi b/arch/riscv/boot/dts/sifive/fu540-c000.dtsi >> index 42b5ec2..bb422db 100644 >> --- a/arch/riscv/boot/dts/sifive/fu540-c000.dtsi >> +++ b/arch/riscv/boot/dts/sifive/fu540-c000.dtsi >> @@ -230,6 +230,25 @@ >> #size-cells = <0>; >> status = "disabled"; >> }; >> + pwm0: pwm@10020000 { >> + compatible = "sifive,pwm0"; >> + reg = <0x0 0x10020000 0x0 0x1000>; >> + interrupt-parent = <&plic0>; >> + interrupts = <42 43 44 45>; >> + clocks = <&prci PRCI_CLK_TLCLK>; >> + #pwm-cells = <3>; >> + status = "disabled"; >> + }; >> + pwm1: pwm@10021000 { >> + compatible = "sifive,pwm0"; >> + reg = <0x0 0x10021000 0x0 0x1000>; >> + interrupt-parent = <&plic0>; >> + interrupts = <46 47 48 49>; >> + reg-names = "control"; >> + clocks = <&prci PRCI_CLK_TLCLK>; >> + #pwm-cells = <3>; >> + status = "disabled"; >> + }; >> >> }; >> }; >> diff --git a/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts b/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts >> index 93d68cb..104d334 100644 >> --- a/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts >> +++ b/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts >> @@ -85,3 +85,11 @@ >> reg = <0>; >> }; >> }; >> + >> +&pwm0 { >> + status = "okay"; >> +}; >> + >> +&pwm1 { >> + status = "okay"; >> +}; >> -- >> 1.9.1 >> _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] riscv: dts: Add DT support for SiFive FU540 PWM driver 2019-09-13 21:20 ` Palmer Dabbelt @ 2019-09-16 11:18 ` Yash Shah 2019-09-18 13:41 ` Paul Walmsley 0 siblings, 1 reply; 5+ messages in thread From: Yash Shah @ 2019-09-16 11:18 UTC (permalink / raw) To: Palmer Dabbelt Cc: Mark Rutland, devicetree, Albert Ou, linux-kernel@vger.kernel.org List, Sachin Ghadi, Rob Herring, Sagar Kadam, Paul Walmsley, linux-riscv, Bin Meng On Sat, Sep 14, 2019 at 2:50 AM Palmer Dabbelt <palmer@sifive.com> wrote: > > On Tue, 10 Sep 2019 02:52:07 PDT (-0700), yash.shah@sifive.com wrote: > > Hi, > > > > Any comments on this patch? > > I don't see "sifive,pwm0" in the DT bindings documentation, and it doesn't > match our standard way of doing these things (which would have at least > "sifive,fu540-c000-pwm"). "sifive,pwm0" is present in the DT bindings documentation at Documentation/devicetree/bindings/pwm/pwm-sifive.txt Yes, I agree that this patch is missing "sifive,fu540-c000-pwm". I will add it along with "sifive,pwm0" and repost as version 2. Thanks for your comment. - Yash > > > > > - Yash > > > > On Wed, Aug 21, 2019 at 2:53 PM Yash Shah <yash.shah@sifive.com> wrote: > >> > >> Add the PWM DT node in SiFive FU540 soc-specific DT file. > >> Enable the PWM nodes in HiFive Unleashed board-specific DT file. > >> > >> Signed-off-by: Yash Shah <yash.shah@sifive.com> > >> --- > >> arch/riscv/boot/dts/sifive/fu540-c000.dtsi | 19 +++++++++++++++++++ > >> arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts | 8 ++++++++ > >> 2 files changed, 27 insertions(+) > >> > >> diff --git a/arch/riscv/boot/dts/sifive/fu540-c000.dtsi b/arch/riscv/boot/dts/sifive/fu540-c000.dtsi > >> index 42b5ec2..bb422db 100644 > >> --- a/arch/riscv/boot/dts/sifive/fu540-c000.dtsi > >> +++ b/arch/riscv/boot/dts/sifive/fu540-c000.dtsi > >> @@ -230,6 +230,25 @@ > >> #size-cells = <0>; > >> status = "disabled"; > >> }; > >> + pwm0: pwm@10020000 { > >> + compatible = "sifive,pwm0"; > >> + reg = <0x0 0x10020000 0x0 0x1000>; > >> + interrupt-parent = <&plic0>; > >> + interrupts = <42 43 44 45>; > >> + clocks = <&prci PRCI_CLK_TLCLK>; > >> + #pwm-cells = <3>; > >> + status = "disabled"; > >> + }; > >> + pwm1: pwm@10021000 { > >> + compatible = "sifive,pwm0"; > >> + reg = <0x0 0x10021000 0x0 0x1000>; > >> + interrupt-parent = <&plic0>; > >> + interrupts = <46 47 48 49>; > >> + reg-names = "control"; > >> + clocks = <&prci PRCI_CLK_TLCLK>; > >> + #pwm-cells = <3>; > >> + status = "disabled"; > >> + }; > >> > >> }; > >> }; > >> diff --git a/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts b/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts > >> index 93d68cb..104d334 100644 > >> --- a/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts > >> +++ b/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts > >> @@ -85,3 +85,11 @@ > >> reg = <0>; > >> }; > >> }; > >> + > >> +&pwm0 { > >> + status = "okay"; > >> +}; > >> + > >> +&pwm1 { > >> + status = "okay"; > >> +}; > >> -- > >> 1.9.1 > >> _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] riscv: dts: Add DT support for SiFive FU540 PWM driver 2019-09-16 11:18 ` Yash Shah @ 2019-09-18 13:41 ` Paul Walmsley 0 siblings, 0 replies; 5+ messages in thread From: Paul Walmsley @ 2019-09-18 13:41 UTC (permalink / raw) To: Yash Shah Cc: Mark Rutland, devicetree, Albert Ou, Palmer Dabbelt, linux-kernel@vger.kernel.org List, Sachin Ghadi, Rob Herring, Sagar Kadam, linux-riscv, Bin Meng On Mon, 16 Sep 2019, Yash Shah wrote: > On Sat, Sep 14, 2019 at 2:50 AM Palmer Dabbelt <palmer@sifive.com> wrote: > > > > On Tue, 10 Sep 2019 02:52:07 PDT (-0700), yash.shah@sifive.com wrote: > > > Hi, > > > > > > Any comments on this patch? > > > > I don't see "sifive,pwm0" in the DT bindings documentation, and it doesn't > > match our standard way of doing these things (which would have at least > > "sifive,fu540-c000-pwm"). > > "sifive,pwm0" is present in the DT bindings documentation at > Documentation/devicetree/bindings/pwm/pwm-sifive.txt > Yes, I agree that this patch is missing "sifive,fu540-c000-pwm". I > will add it along with "sifive,pwm0" and repost as version 2. Fixed the compat string here and also dropped the superfluous reg-names property from pwm1. Queued for v5.4-rc, thanks. - Paul From: Yash Shah <yash.shah@sifive.com> Date: Wed, 21 Aug 2019 14:53:40 +0530 Subject: [PATCH] riscv: dts: Add DT support for SiFive FU540 PWM driver Add the PWM DT node in SiFive FU540 soc-specific DT file. Enable the PWM nodes in HiFive Unleashed board-specific DT file. Signed-off-by: Yash Shah <yash.shah@sifive.com> Cc: Palmer Dabbelt <palmer@sifive.com> [paul.walmsley@sifive.com: added chip-specific compatible string; dropped reg-names string from pwm1] Signed-off-by: Paul Walmsley <paul.walmsley@sifive.com> --- arch/riscv/boot/dts/sifive/fu540-c000.dtsi | 18 ++++++++++++++++++ .../boot/dts/sifive/hifive-unleashed-a00.dts | 8 ++++++++ 2 files changed, 26 insertions(+) diff --git a/arch/riscv/boot/dts/sifive/fu540-c000.dtsi b/arch/riscv/boot/dts/sifive/fu540-c000.dtsi index 42b5ec223100..5a29211d396e 100644 --- a/arch/riscv/boot/dts/sifive/fu540-c000.dtsi +++ b/arch/riscv/boot/dts/sifive/fu540-c000.dtsi @@ -230,6 +230,24 @@ #size-cells = <0>; status = "disabled"; }; + pwm0: pwm@10020000 { + compatible = "sifive,fu540-c000-pwm", "sifive,pwm0"; + reg = <0x0 0x10020000 0x0 0x1000>; + interrupt-parent = <&plic0>; + interrupts = <42 43 44 45>; + clocks = <&prci PRCI_CLK_TLCLK>; + #pwm-cells = <3>; + status = "disabled"; + }; + pwm1: pwm@10021000 { + compatible = "sifive,fu540-c000-pwm", "sifive,pwm0"; + reg = <0x0 0x10021000 0x0 0x1000>; + interrupt-parent = <&plic0>; + interrupts = <46 47 48 49>; + clocks = <&prci PRCI_CLK_TLCLK>; + #pwm-cells = <3>; + status = "disabled"; + }; }; }; diff --git a/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts b/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts index 93d68cbd64fe..104d334511cd 100644 --- a/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts +++ b/arch/riscv/boot/dts/sifive/hifive-unleashed-a00.dts @@ -85,3 +85,11 @@ reg = <0>; }; }; + +&pwm0 { + status = "okay"; +}; + +&pwm1 { + status = "okay"; +}; -- 2.23.0 _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv ^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-09-18 13:41 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-08-21 9:23 [PATCH] riscv: dts: Add DT support for SiFive FU540 PWM driver Yash Shah 2019-09-10 9:52 ` Yash Shah 2019-09-13 21:20 ` Palmer Dabbelt 2019-09-16 11:18 ` Yash Shah 2019-09-18 13:41 ` Paul Walmsley
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).