Re: [PATCH v4 0/3] pwm: meson: add pwm support for A1

[George Stark <gnstark@xxxxxxxxxxxxxxxxx>]

Hello

Excuse me, should I fix/improve anything on this series?

On 7/11/24 02:41, George Stark wrote:
> Add support for Amlogic meson A1 SoC family PWM
>
> Changes in v2:
>    add patch with optional power-domains to pwm bindings;
>    fix syntax in a1 bindigns patch:
>    - use enum over const for amlogic,meson-a1-pwm beacuse adding more devices here
>      are expected
>    - leave only base compatible amlogic,meson-s4-pwm in check section
>    dt_binding_check and dtbs_check run ok now;
>    previous version: [1]
>
> Changes in v3:
>    squash power-domains patch into main bindigns patch
>    add conditional to bindings that power-domains property is required only for a1
>    previous version: [2]
>
> Changes in v4:
>    split bindings patch into power-domains patch and new compatible patch
>    previous version: [3]
>
> [1] https://lore.kernel.org/lkml/20240701130113.433169-3-gnstark@xxxxxxxxxxxxxxxxx/T/
> [2] https://lore.kernel.org/lkml/20240701172016.523402-1-gnstark@xxxxxxxxxxxxxxxxx/T/
> [3] https://lore.kernel.org/lkml/20240702123425.584610-1-gnstark@xxxxxxxxxxxxxxxxx/T/
>
> George Stark (3):
>    dt-bindings: pwm: amlogic: Add optional power-domains
>    dt-bindings: pwm: amlogic: Add new bindings for meson A1 PWM
>    arm64: dts: meson: a1: add definitions for meson PWM
>
>   .../devicetree/bindings/pwm/pwm-amlogic.yaml  |  17 ++
>   arch/arm64/boot/dts/amlogic/meson-a1.dtsi     | 215 ++++++++++++++++++
>   2 files changed, 232 insertions(+)
>
> --
> 2.25.1

--
Best regards
George