[PATCH v2 1/4] pinctrl: armada-37xx: Add missing GPIO-only pins
Pali Rohár
pali at kernel.org
Tue Aug 9 13:23:08 PDT 2022
On Tuesday 09 August 2022 22:19:33 Andrew Lunn wrote:
> On Fri, Aug 05, 2022 at 02:21:59PM +0200, Pali Rohár wrote:
> > gpio1_5 and gpio2_2 are GPIO-only pins. Add them into MPP groups table
> > so they are properly exported as valid pin numbers.
> >
> > Fixes: 87466ccd9401 ("pinctrl: armada-37xx: Add pin controller support for Armada 37xx")
> > Signed-off-by: Pali Rohár <pali at kernel.org>
> > ---
> > drivers/pinctrl/mvebu/pinctrl-armada-37xx.c | 12 ++++++++++++
> > 1 file changed, 12 insertions(+)
> >
> > diff --git a/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c b/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c
> > index a140b6bfbfaa..2b44c634ccb5 100644
> > --- a/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c
> > +++ b/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c
> > @@ -122,6 +122,16 @@ struct armada_37xx_pinctrl {
> > .funcs = {_func1, _func2} \
> > }
> >
> > +#define PIN_GRP_GPIO_0(_name, _start, _nr) \
>
> The naming of these macros are a bit odd, but this does not make it
> any worse.
Yea... but I'm continuing to use existing naming convention. Suffix _N
means how many non-gpio functions have particular group.
> Reviewed-by: Andrew Lunn <andrew at lunn.ch>
>
> Andrew
More information about the linux-arm-kernel
mailing list