Re: [PATCH v2] pinctrl: sunxi: Implement gpiochip::get_direction()

[Michal Piekos]

On Tue, Feb 24, 2026 at 10:52:23AM +0100, Linus Walleij wrote:
> On Tue, Feb 24, 2026 at 10:24 AM Chen-Yu Tsai <wens@xxxxxxxxxx> wrote:
> 
> > After commit 471e998c0e31 ("gpiolib: remove redundant callback check"),
> > a warning will be printed if the gpio driver does not implement this
> > callback. The warning was added in commit e623c4303ed1 ("gpiolib:
> > sanitize the return value of gpio_chip::get_direction()"), but was
> > masked by the "redundant" check.
> >
> > The warning can be triggered by any action that calls the callback,
> > such as dumping the GPIO state from /sys/kernel/debug/gpio.
> >
> > Implement it for the sunxi driver. This is simply a matter of reading
> > out the mux value from the registers, then checking if it is one of
> > the GPIO functions and which direction it is.
> >
> > Signed-off-by: Chen-Yu Tsai <wens@xxxxxxxxxx>
> > Reviewed-by: Jernej Skrabec <jernej.skrabec@xxxxxxxxx>
> > Reviewed-by: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxxxxxxxx>
> 
> Pulled out v1 of my tree and put in this one instead!
>

Please do not merge the correction yet. As I was pointed out by Jernej
Skrabec the solution is faulty and just cover the issue. I will provide
proper solution as soon as I find it.

Michal Piekos

> Yours,
> Linus Walleij
>