On Mon, Mar 12, 2018 at 04:10:49PM +0000, Harald Geyer wrote: > This device is compatible with A13, so no new driver is needed. > > Signed-off-by: Harald Geyer <harald@xxxxxxxxx> > --- > I saw that Andre Przywara has been working on A64 pwm too and has > submitted some patches a few days ago. I think his patches are functionally > equivalent to this one here, but clean up things a bit and thus are > preferable. See: > > https://groups.google.com/forum/#!topic/linux-sunxi/hQFeteP591k > > I'm including my patch here mostly to have a consistent series for others > to test. OTOH you might merge the device tree changes here and pick up > the cleanup patches from him. Either way should work fine. > > arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi | 13 +++++++++++++ > 1 file changed, 13 insertions(+) > > diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi > index 05d5e8def68a..95d52f6aa07f 100644 > --- a/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi > +++ b/arch/arm64/boot/dts/allwinner/sun50i-a64.dtsi > @@ -364,6 +364,11 @@ > bias-pull-up; > }; > > + pwm_pin: pwm_pin { > + pins = "PD22"; > + function = "pwm"; > + }; > + > rmii_pins: rmii_pins { > pins = "PD10", "PD11", "PD13", "PD14", "PD17", > "PD18", "PD19", "PD20", "PD22", "PD23"; > @@ -629,6 +634,14 @@ > #interrupt-cells = <3>; > }; > > + pwm: pwm@1c21400 { > + compatible = "allwinner,sun5i-a13-pwm"; Just like for the watchdog, you should have an A64 compatible there. > + reg = <0x01c21400 0x8>; And you should use the full memory range here. Maxime -- Maxime Ripard, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering https://bootlin.com
Attachment:
signature.asc
Description: PGP signature
