mbox series

[v3,0/3] Add support for Allwinner PWM on D1/T113s/R329 SoCs

Message ID 20230627082334.1253020-1-privatesub2@gmail.com (mailing list archive)
Headers show
Series Add support for Allwinner PWM on D1/T113s/R329 SoCs | expand

Message

Александр Шубин June 27, 2023, 8:23 a.m. UTC
Hi,

This series adds support for PWM controller on new
Allwinner's SoCs, such as D1, T113s and R329. The implemented driver
provides basic functionality for control PWM channels.

v2:
 - fix dt-bindings
 - fix a remark in the driver

v3:
 - fix dt-bindings
 - fix sunxi-d1s-t113.dtsi

Aleksandr Shubin (3):
  dt-bindings: pwm: Add binding for Allwinner D1/T113-S3/R329 PWM
    controller
  pwm: Add Allwinner's D1/T113-S3/R329 SoCs PWM support
  riscv: dts: allwinner: d1: Add pwm node

 .../bindings/pwm/allwinner,sun20i-pwm.yaml    |  86 +++++
 .../boot/dts/allwinner/sunxi-d1s-t113.dtsi    |  11 +
 drivers/pwm/Kconfig                           |  10 +
 drivers/pwm/Makefile                          |   1 +
 drivers/pwm/pwm-sun20i.c                      | 322 ++++++++++++++++++
 5 files changed, 430 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/pwm/allwinner,sun20i-pwm.yaml
 create mode 100644 drivers/pwm/pwm-sun20i.c

Comments

Palmer Dabbelt July 6, 2023, 5:08 p.m. UTC | #1
On Tue, 27 Jun 2023 01:23:23 PDT (-0700), privatesub2@gmail.com wrote:
> Hi,
>
> This series adds support for PWM controller on new
> Allwinner's SoCs, such as D1, T113s and R329. The implemented driver
> provides basic functionality for control PWM channels.
>
> v2:
>  - fix dt-bindings
>  - fix a remark in the driver
>
> v3:
>  - fix dt-bindings
>  - fix sunxi-d1s-t113.dtsi
>
> Aleksandr Shubin (3):
>   dt-bindings: pwm: Add binding for Allwinner D1/T113-S3/R329 PWM
>     controller
>   pwm: Add Allwinner's D1/T113-S3/R329 SoCs PWM support
>   riscv: dts: allwinner: d1: Add pwm node
>
>  .../bindings/pwm/allwinner,sun20i-pwm.yaml    |  86 +++++
>  .../boot/dts/allwinner/sunxi-d1s-t113.dtsi    |  11 +
>  drivers/pwm/Kconfig                           |  10 +
>  drivers/pwm/Makefile                          |   1 +
>  drivers/pwm/pwm-sun20i.c                      | 322 ++++++++++++++++++
>  5 files changed, 430 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/pwm/allwinner,sun20i-pwm.yaml
>  create mode 100644 drivers/pwm/pwm-sun20i.c

Acked-by: Palmer Dabbelt <palmer@rivosinc.com>

I'm not sure if this ended up somewhere, but I'm assuming it's not aimed 
at my tree.  LMK if you guys want me to pick it up, I'm dropping it from 
patchwork.
Uwe Kleine-König July 6, 2023, 8:20 p.m. UTC | #2
Hello,

On Thu, Jul 06, 2023 at 10:08:47AM -0700, Palmer Dabbelt wrote:
> On Tue, 27 Jun 2023 01:23:23 PDT (-0700), privatesub2@gmail.com wrote:
> > Hi,
> > 
> > This series adds support for PWM controller on new
> > Allwinner's SoCs, such as D1, T113s and R329. The implemented driver
> > provides basic functionality for control PWM channels.
> > 
> > v2:
> >  - fix dt-bindings
> >  - fix a remark in the driver
> > 
> > v3:
> >  - fix dt-bindings
> >  - fix sunxi-d1s-t113.dtsi
> > 
> > Aleksandr Shubin (3):
> >   dt-bindings: pwm: Add binding for Allwinner D1/T113-S3/R329 PWM
> >     controller
> >   pwm: Add Allwinner's D1/T113-S3/R329 SoCs PWM support
> >   riscv: dts: allwinner: d1: Add pwm node
> > 
> >  .../bindings/pwm/allwinner,sun20i-pwm.yaml    |  86 +++++
> >  .../boot/dts/allwinner/sunxi-d1s-t113.dtsi    |  11 +
> >  drivers/pwm/Kconfig                           |  10 +
> >  drivers/pwm/Makefile                          |   1 +
> >  drivers/pwm/pwm-sun20i.c                      | 322 ++++++++++++++++++
> >  5 files changed, 430 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/pwm/allwinner,sun20i-pwm.yaml
> >  create mode 100644 drivers/pwm/pwm-sun20i.c
> 
> Acked-by: Palmer Dabbelt <palmer@rivosinc.com>
> 
> I'm not sure if this ended up somewhere, but I'm assuming it's not aimed at
> my tree.  LMK if you guys want me to pick it up, I'm dropping it from
> patchwork.

At least patch #2 should go via the pwm tree. It's on my list to review.

Best regards
Uwe
Jernej Škrabec July 11, 2023, 6:10 p.m. UTC | #3
Dne četrtek, 06. julij 2023 ob 22:20:57 CEST je Uwe Kleine-König napisal(a):
> Hello,
> 
> On Thu, Jul 06, 2023 at 10:08:47AM -0700, Palmer Dabbelt wrote:
> > On Tue, 27 Jun 2023 01:23:23 PDT (-0700), privatesub2@gmail.com wrote:
> > > Hi,
> > > 
> > > This series adds support for PWM controller on new
> > > Allwinner's SoCs, such as D1, T113s and R329. The implemented driver
> > > provides basic functionality for control PWM channels.
> > > 
> > > v2:
> > >  - fix dt-bindings
> > >  - fix a remark in the driver
> > > 
> > > v3:
> > >  - fix dt-bindings
> > >  - fix sunxi-d1s-t113.dtsi
> > > 
> > > Aleksandr Shubin (3):
> > >   dt-bindings: pwm: Add binding for Allwinner D1/T113-S3/R329 PWM
> > >   
> > >     controller
> > >   
> > >   pwm: Add Allwinner's D1/T113-S3/R329 SoCs PWM support
> > >   riscv: dts: allwinner: d1: Add pwm node
> > >  
> > >  .../bindings/pwm/allwinner,sun20i-pwm.yaml    |  86 +++++
> > >  .../boot/dts/allwinner/sunxi-d1s-t113.dtsi    |  11 +
> > >  drivers/pwm/Kconfig                           |  10 +
> > >  drivers/pwm/Makefile                          |   1 +
> > >  drivers/pwm/pwm-sun20i.c                      | 322 ++++++++++++++++++
> > >  5 files changed, 430 insertions(+)
> > >  create mode 100644
> > >  Documentation/devicetree/bindings/pwm/allwinner,sun20i-pwm.yaml create
> > >  mode 100644 drivers/pwm/pwm-sun20i.c
> > 
> > Acked-by: Palmer Dabbelt <palmer@rivosinc.com>
> > 
> > I'm not sure if this ended up somewhere, but I'm assuming it's not aimed
> > at
> > my tree.  LMK if you guys want me to pick it up, I'm dropping it from
> > patchwork.
> 
> At least patch #2 should go via the pwm tree. It's on my list to review.

Rest will go through sunxi tree, although DT bindings patches are usually 
picked together with driver patches. I'm fine either way.

Best regards,
Jernej