mbox series

[v2,0/2] ARM: dts: sun8i: v3s: add pwm controller to v3s dts

Message ID 20210513203527.2072090-1-t.schramm@manjaro.org
Headers show
Series ARM: dts: sun8i: v3s: add pwm controller to v3s dts | expand

Message

Tobias Schramm May 13, 2021, 8:35 p.m. UTC
This patchset adds the pwm controller to the Allwinner V3s dts. The PWM
controller is compatible with the one found in the Allwinner A20.

This is v2 of this patchset.

Cheers,
Tobias

Changelog:
 - v2: add compatible for pwm controller of the V3s SoC

Tobias Schramm (2):
  dt-bindings: pwm: allwinner: add v3s pwm compatible
  ARM: dts: sun8i: v3s: add pwm controller to v3s dts

 .../devicetree/bindings/pwm/allwinner,sun4i-a10-pwm.yaml | 3 +++
 arch/arm/boot/dts/sun8i-v3s.dtsi                         | 9 +++++++++
 2 files changed, 12 insertions(+)

Comments

Andre Przywara May 14, 2021, 7:03 a.m. UTC | #1
On Thu, 13 May 2021 22:35:27 +0200
Tobias Schramm <t.schramm@manjaro.org> wrote:

> The Allwinner V3s and V3 SoCs feature a pwm controller identical to the
> one used in the Allwinner A20.
> This commit adds it to the V3s dtsi.
> 
> Signed-off-by: Tobias Schramm <t.schramm@manjaro.org>

Address and size match the manual, and the register description look
identical between A20 and V3s.

Reviewed-by: Andre Przywara <andre.przywara@arm.com>

Cheers,
Andre

> ---
>  arch/arm/boot/dts/sun8i-v3s.dtsi | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/sun8i-v3s.dtsi b/arch/arm/boot/dts/sun8i-v3s.dtsi
> index eb4cb63fef13..456dee9de87f 100644
> --- a/arch/arm/boot/dts/sun8i-v3s.dtsi
> +++ b/arch/arm/boot/dts/sun8i-v3s.dtsi
> @@ -422,6 +422,15 @@ wdt0: watchdog@1c20ca0 {
>  			clocks = <&osc24M>;
>  		};
>  
> +		pwm: pwm@1c21400 {
> +			compatible = "allwinner,sun8i-v3s-pwm",
> +				     "allwinner,sun7i-a20-pwm";
> +			reg = <0x01c21400 0xc>;
> +			clocks = <&osc24M>;
> +			#pwm-cells = <3>;
> +			status = "disabled";
> +		};
> +
>  		lradc: lradc@1c22800 {
>  			compatible = "allwinner,sun4i-a10-lradc-keys";
>  			reg = <0x01c22800 0x400>;
Maxime Ripard May 18, 2021, 2:41 p.m. UTC | #2
On Thu, May 13, 2021 at 10:35:25PM +0200, Tobias Schramm wrote:
> This patchset adds the pwm controller to the Allwinner V3s dts. The PWM
> controller is compatible with the one found in the Allwinner A20.
> 
> This is v2 of this patchset.

Applied both, thanks
Maxime