Re: [PATCH v2 1/2] pinctrl: renesas: rzg2l: Use -ENOTSUPP instead of -EOPNOTSUPP

[Claudiu Beznea]

Hi, Geert,

On 5/22/26 12:11, Geert Uytterhoeven wrote:
> Hi Claudiu,
>
> On Fri, 15 May 2026 at 14:40, Claudiu Beznea <claudiu.beznea@xxxxxxxxxx> wrote:
> > From: Claudiu Beznea <claudiu.beznea.uj@xxxxxxxxxxxxxx>
> >
> > The pinctrl and GPIO core code make exceptions for the -ENOTSUPP error
> > code. One such example is gpio_set_config_with_argument_optional(), which
> > returns success when gpio_set_config_with_argument() returns -ENOTSUPP, but
> > reports failure for all other error codes.
> >
> > Returning -EOPNOTSUPP from the pinctrl driver on the unsupported pinctrl
> > operation may lead to boot failures when pinctrl drivers implements
> > struct gpio_chip::set_config, the system uses GPIO hogs, and the
> > struct gpio_chip::set_config implementation returns -EOPNOTSUPP for the
> > unsupported operations.
> >
> > Return -ENOTSUPP for the unsupported pinctrl operation.
> >
> > Fixes: 560c633d378a ("pinctrl: renesas: rzg2l: Drop oen_read and oen_write callbacks")
> > Fixes: c4c4637eb57f ("pinctrl: renesas: Add RZ/G2L pin and gpio controller driver")
> > Cc: stable@xxxxxxxxxxxxxxx
> > Signed-off-by: Claudiu Beznea <claudiu.beznea.uj@xxxxxxxxxxxxxx>
>
> Thanks for your patch!
>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> i.e. will queue in renesas-pinctrl for v7.2.
>
> I guess drivers/pinctrl/renesas/pinctrl-rzv2m.c needs a similar patch?

Yes! I'll prepare and send one.

--
Thank you,
Claudiu