| Message ID | 20220224234133.15708-2-aidanmacdonald.0x0@gmail.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [v2,1/2] dt-bindings: timer: Add PWM compatible for X1000 SoC | expand |
On Thu, Feb 24, 2022 at 11:41:34PM +0000, Aidan MacDonald wrote: > Copied from the jz4740 devicetree and trimmed to 5 timers, which > is what the hardware supports. > > Signed-off-by: Aidan MacDonald <aidanmacdonald.0x0@gmail.com> > --- > arch/mips/boot/dts/ingenic/x1000.dtsi | 13 +++++++++++++ > 1 file changed, 13 insertions(+) However I don't see this upstream yet, neither in Linus' tree nor in linux-next. Paul, do you still have this on your radar? Thierry > > diff --git a/arch/mips/boot/dts/ingenic/x1000.dtsi b/arch/mips/boot/dts/ingenic/x1000.dtsi > index 8bd27edef216..0dcf37527c8e 100644 > --- a/arch/mips/boot/dts/ingenic/x1000.dtsi > +++ b/arch/mips/boot/dts/ingenic/x1000.dtsi > @@ -127,6 +127,19 @@ wdt: watchdog@0 { > clocks = <&tcu TCU_CLK_WDT>; > clock-names = "wdt"; > }; > + > + pwm: pwm@40 { > + compatible = "ingenic,x1000-pwm"; > + reg = <0x40 0x80>; > + > + #pwm-cells = <3>; > + > + clocks = <&tcu TCU_CLK_TIMER0>, <&tcu TCU_CLK_TIMER1>, > + <&tcu TCU_CLK_TIMER2>, <&tcu TCU_CLK_TIMER3>, > + <&tcu TCU_CLK_TIMER4>; > + clock-names = "timer0", "timer1", "timer2", > + "timer3", "timer4"; > + }; > }; > > rtc: rtc@10003000 { > -- > 2.34.1 >
Hi Thierry, Le ven., avril 22 2022 at 18:12:05 +0200, Thierry Reding <thierry.reding@gmail.com> a écrit : > On Thu, Feb 24, 2022 at 11:41:34PM +0000, Aidan MacDonald wrote: >> Copied from the jz4740 devicetree and trimmed to 5 timers, which >> is what the hardware supports. >> >> Signed-off-by: Aidan MacDonald <aidanmacdonald.0x0@gmail.com> >> --- >> arch/mips/boot/dts/ingenic/x1000.dtsi | 13 +++++++++++++ >> 1 file changed, 13 insertions(+) > > However I don't see this upstream yet, neither in Linus' tree nor in > linux-next. Paul, do you still have this on your radar? I don't have a tree so that would be a patch for Thomas (Cc'd) to go through the linux-mips tree. (Note to Thomas: patch 1 is already applied in the pwm tree) The changes look good though, so: Reviewed-by: Paul Cercueil <paul@crapouillou.net> Cheers, -Paul > > Thierry > >> >> diff --git a/arch/mips/boot/dts/ingenic/x1000.dtsi >> b/arch/mips/boot/dts/ingenic/x1000.dtsi >> index 8bd27edef216..0dcf37527c8e 100644 >> --- a/arch/mips/boot/dts/ingenic/x1000.dtsi >> +++ b/arch/mips/boot/dts/ingenic/x1000.dtsi >> @@ -127,6 +127,19 @@ wdt: watchdog@0 { >> clocks = <&tcu TCU_CLK_WDT>; >> clock-names = "wdt"; >> }; >> + >> + pwm: pwm@40 { >> + compatible = "ingenic,x1000-pwm"; >> + reg = <0x40 0x80>; >> + >> + #pwm-cells = <3>; >> + >> + clocks = <&tcu TCU_CLK_TIMER0>, <&tcu TCU_CLK_TIMER1>, >> + <&tcu TCU_CLK_TIMER2>, <&tcu TCU_CLK_TIMER3>, >> + <&tcu TCU_CLK_TIMER4>; >> + clock-names = "timer0", "timer1", "timer2", >> + "timer3", "timer4"; >> + }; >> }; >> >> rtc: rtc@10003000 { >> -- >> 2.34.1 >>
diff --git a/arch/mips/boot/dts/ingenic/x1000.dtsi b/arch/mips/boot/dts/ingenic/x1000.dtsi index 8bd27edef216..0dcf37527c8e 100644 --- a/arch/mips/boot/dts/ingenic/x1000.dtsi +++ b/arch/mips/boot/dts/ingenic/x1000.dtsi @@ -127,6 +127,19 @@ wdt: watchdog@0 { clocks = <&tcu TCU_CLK_WDT>; clock-names = "wdt"; }; + + pwm: pwm@40 { + compatible = "ingenic,x1000-pwm"; + reg = <0x40 0x80>; + + #pwm-cells = <3>; + + clocks = <&tcu TCU_CLK_TIMER0>, <&tcu TCU_CLK_TIMER1>, + <&tcu TCU_CLK_TIMER2>, <&tcu TCU_CLK_TIMER3>, + <&tcu TCU_CLK_TIMER4>; + clock-names = "timer0", "timer1", "timer2", + "timer3", "timer4"; + }; }; rtc: rtc@10003000 {
Copied from the jz4740 devicetree and trimmed to 5 timers, which is what the hardware supports. Signed-off-by: Aidan MacDonald <aidanmacdonald.0x0@gmail.com> --- arch/mips/boot/dts/ingenic/x1000.dtsi | 13 +++++++++++++ 1 file changed, 13 insertions(+)