Re: [PATCH] ARM: dts: am33xx: modify AM33XX_IOPAD for #pinctrl-cells = 2

From: Linus Walleij
Date: Thu Oct 01 2020 - 04:08:55 EST


On Tue, Sep 22, 2020 at 12:57 AM Drew Fustini <drew@xxxxxxxxxxxxxxx> wrote:

> Modify the AM33XX_IOPAD macro so that it works now that #pinctrl-cells =
> <2>. The third parameter is just a zero and the pinctrl-single driver
> will just OR this with the second parameter so it has no actual effect.
>
> There are no longer any dts files using this macro (following my patch
> to am335x-guardian.dts), but this will keep dts files not in mainline
> from breaking.
>
> Fixes: 27c90e5e48d0 ("ARM: dts: am33xx-l4: change #pinctrl-cells from 1 to 2")
> Suggested-by: Tony Lindgren <tony@xxxxxxxxxxx>
> Reported-by: Trent Piepho <tpiepho@xxxxxxxxx>
> Link: https://lore.kernel.org/linux-devicetree/20200921064707.GN7101@xxxxxxxxxxx/
> Signed-off-by: Drew Fustini <drew@xxxxxxxxxxxxxxx>

I didn't get a review tag on this one, Tony is this something I
should be applying?

Yours,
Linus Walleij