[PATCH RFC 1/3] pinctrl: mediatek: paris: Expose more configurations to GPIO set_config
Nícolas F. R. A. Prado
nfraprado at collabora.com
Thu Oct 24 10:46:38 PDT 2024
On Thu, Oct 24, 2024 at 05:17:05PM +0200, AngeloGioacchino Del Regno wrote:
> Il 11/09/24 12:10, AngeloGioacchino Del Regno ha scritto:
> > Il 09/09/24 20:37, Nícolas F. R. A. Prado ha scritto:
> > > Currently the set_config callback in the gpio_chip registered by the
> > > pinctrl_paris driver only supports PIN_CONFIG_INPUT_DEBOUNCE, despite
> >
> > [...] only supports operations configuring the input debounce parameter
> > of the EINT controller and denies configuring params on the other AP GPIOs [...]
> >
> > (reword as needed)
> >
> > > many other configurations already being implemented and available
> > > through the pinctrl API for configuration of pins by the Devicetree and
> > > other drivers.
> > >
> > > Expose all configurations currently implemented through the GPIO API so
> > > they can also be set from userspace, which is particularly useful to
> > > allow testing them from userspace.
> > >
> > > Signed-off-by: Nícolas F. R. A. Prado <nfraprado at collabora.com>
> > > ---
> > > drivers/pinctrl/mediatek/pinctrl-paris.c | 20 ++++++++++----------
> >
> > You can do the same for pinctrl-moore too, it's trivial.
> >
> > Other than that, I agree about performing this change, as this may be useful
> > for more than just testing.
> >
>
> Nicolas, please don't forget to respin this patch.
I was hoping to get some feedback on the test itself as well, particularly from
Linus as the pinctrl maintainer, but it's also been a while so I'll send a v2
with the feedback here addressed.
Thanks,
Nícolas
More information about the Linux-mediatek
mailing list