Re: [PATCH v2 02/10] ARM: dts: suniv: f1c100s: add PWM node

Thierry Reding
2022-11-15T10:45:35Z

[-- Attachment #1.1: Type: text/plain, Size: 947 bytes --]

On Tue, Nov 15, 2022 at 11:19:26AM +0100, Uwe Kleine-König wrote:
> On Mon, Nov 07, 2022 at 12:54:25AM +0000, Andre Przywara wrote:
> > The Allwinner F1C100s family of SoCs contain a PWM controller compatible
> > to the one used in the A20 chip.
> > Add the DT node so that any users can simply enable it in their board
> > DT.
> >
> > Signed-off-by: Andre Przywara <andre.przywara@arm.com>
>
> I checked "by hand" that this matches the modified binding in patch
> 01/10.
>
> Reviewed-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
>
> I assume the whole series will go in via a tree different from the PWM
> tree? While this is Thierry's area of maintenance, I'd be surprised if
> he had concerns about that.

Yeah, it's probably best for the Allwinner maintainers to pick this up
into their tree so that the bindings changes go along with the DT
changes.

Acked-by: Thierry Reding <thierry.reding@gmail.com>

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

[-- Attachment #2: Type: text/plain, Size: 176 bytes --]