Re: [PATCH v3 08/12] pinctrl: rp1: Implement RaspberryPi RP1 gpio support

From: Andrea della Porta
Date: Thu Oct 31 2024 - 10:44:37 EST


Hi Linus,

On 23:18 Mon 28 Oct , Linus Walleij wrote:
> On Mon, Oct 28, 2024 at 3:07 PM Andrea della Porta
> <andrea.porta@xxxxxxxx> wrote:
>
> > +config PINCTRL_RP1
> > + bool "Pinctrl driver for RP1"
> > + select PINMUX
> > + select PINCONF
> > + select GENERIC_PINCONF
> > + select GPIOLIB
> > + select GPIOLIB_IRQCHIP
>
> Just a quick thing:
>
> You don't happen to want:
> depends on MISC_RP1
> default MISC_RP1
>
> So it will always come in tandem with
> MISC_RP1?

You're right! Added.
However I will postpone the "default MISC_RP1" line after checking why
the pinctrl driver does not work when compiled as a module (right now
it's bool, not tristate).

Many thanks,
Andrea

>
> Yours,
> Linus Walleij