2022-03-23 22:52:48

by hammer hsieh

[permalink] [raw]
Subject: [PATCH v4 0/2] Add PWM driver for Suplus SP7021 SoC

This is a patch series for PWM driver for Suplus SP7021 SoC.

Sunplus SP7021 is an ARM Cortex A7 (4 cores) based SoC. It integrates
many peripherals (ex: UART. I2C, SPI, SDIO, eMMC, USB, SD card and
etc.) into a single chip. It is designed for industrial control.

Refer to:
https://sunplus-tibbo.atlassian.net/wiki/spaces/doc/overview
https://tibbo.com/store/plus1.html

Refer to (PWM):
https://sunplus.atlassian.net/wiki/spaces/doc/pages/461144198/12.+Pulse+Width+Modulation+PWM

Hammer Hsieh (2):
dt-bindings: pwm: Add bindings doc for Sunplus SoC SP7021 PWM Driver
pwm: sunplus-pwm: Add Sunplus SoC SP7021 PWM Driver

.../bindings/pwm/sunplus,sp7021-pwm.yaml | 42 ++++
MAINTAINERS | 6 +
drivers/pwm/Kconfig | 11 +
drivers/pwm/Makefile | 1 +
drivers/pwm/pwm-sunplus.c | 232 +++++++++++++++++++++
5 files changed, 292 insertions(+)
create mode 100644 Documentation/devicetree/bindings/pwm/sunplus,sp7021-pwm.yaml
create mode 100644 drivers/pwm/pwm-sunplus.c

--
2.7.4


2022-04-22 20:12:42

by Thierry Reding

[permalink] [raw]
Subject: Re: [PATCH v4 0/2] Add PWM driver for Suplus SP7021 SoC

On Tue, Mar 22, 2022 at 06:42:59PM +0800, Hammer Hsieh wrote:
> This is a patch series for PWM driver for Suplus SP7021 SoC.
>
> Sunplus SP7021 is an ARM Cortex A7 (4 cores) based SoC. It integrates
> many peripherals (ex: UART. I2C, SPI, SDIO, eMMC, USB, SD card and
> etc.) into a single chip. It is designed for industrial control.
>
> Refer to:
> https://sunplus-tibbo.atlassian.net/wiki/spaces/doc/overview
> https://tibbo.com/store/plus1.html
>
> Refer to (PWM):
> https://sunplus.atlassian.net/wiki/spaces/doc/pages/461144198/12.+Pulse+Width+Modulation+PWM
>
> Hammer Hsieh (2):
> dt-bindings: pwm: Add bindings doc for Sunplus SoC SP7021 PWM Driver
> pwm: sunplus-pwm: Add Sunplus SoC SP7021 PWM Driver
>
> .../bindings/pwm/sunplus,sp7021-pwm.yaml | 42 ++++
> MAINTAINERS | 6 +
> drivers/pwm/Kconfig | 11 +
> drivers/pwm/Makefile | 1 +
> drivers/pwm/pwm-sunplus.c | 232 +++++++++++++++++++++
> 5 files changed, 292 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/pwm/sunplus,sp7021-pwm.yaml
> create mode 100644 drivers/pwm/pwm-sunplus.c

Applied, thanks.

Thierry


Attachments:
(No filename) (1.25 kB)
signature.asc (849.00 B)
Download all attachments

2022-04-22 22:46:16

by Thierry Reding

[permalink] [raw]
Subject: Re: [PATCH v4 0/2] Add PWM driver for Suplus SP7021 SoC

On Tue, Mar 22, 2022 at 06:42:59PM +0800, Hammer Hsieh wrote:
> This is a patch series for PWM driver for Suplus SP7021 SoC.
>
> Sunplus SP7021 is an ARM Cortex A7 (4 cores) based SoC. It integrates
> many peripherals (ex: UART. I2C, SPI, SDIO, eMMC, USB, SD card and
> etc.) into a single chip. It is designed for industrial control.

I just noticed as I was trying to do test builds that ARCH_SUNPLUS
doesn't exist yet. Are there any plans to add those in the near future?

I can always use COMPILE_TEST to do test build, but I like to also do
proper builds because they sometimes flag things that COMPILE_TEST
misses (like this one).

Thierry


Attachments:
(No filename) (662.00 B)
signature.asc (849.00 B)
Download all attachments

2022-04-25 06:43:03

by hammer hsieh

[permalink] [raw]
Subject: Re: [PATCH v4 0/2] Add PWM driver for Suplus SP7021 SoC

Yes, my colleague is working on it right now.
But he submitted too many items at a time. (clock/reset/interrupt,
defconfig/, arch/arm...)
Current patch v13, still ongoing.
https://lore.kernel.org/all/[email protected]/

Thierry Reding <[email protected]> 於 2022年4月23日 週六 上午1:23寫道:
>
> On Tue, Mar 22, 2022 at 06:42:59PM +0800, Hammer Hsieh wrote:
> > This is a patch series for PWM driver for Suplus SP7021 SoC.
> >
> > Sunplus SP7021 is an ARM Cortex A7 (4 cores) based SoC. It integrates
> > many peripherals (ex: UART. I2C, SPI, SDIO, eMMC, USB, SD card and
> > etc.) into a single chip. It is designed for industrial control.
>
> I just noticed as I was trying to do test builds that ARCH_SUNPLUS
> doesn't exist yet. Are there any plans to add those in the near future?
>
> I can always use COMPILE_TEST to do test build, but I like to also do
> proper builds because they sometimes flag things that COMPILE_TEST
> misses (like this one).
>
> Thierry