Re: [PATCH v1 1/1] pinctrl: eswin: Fix Handling of PIN_CONFIG_PERSIST_STATE
From: Linus Walleij
Date: Mon Aug 03 2026 - 03:44:59 EST
On Tue, Jul 28, 2026 at 9:30 AM Yulin Lu <luyulin@xxxxxxxxxxxxxxxxxx> wrote:
> The EIC7700 pinctrl driver does not handle PIN_CONFIG_PERSIST_STATE
> specifically, and returns -EOPNOTSUPP from the default case.
>
> Since all pins on the EIC7700 SoC are persistent over suspend, the
> correct behaviour is to accept this parameter and return success.
>
> Add an explicit case for PIN_CONFIG_PERSIST_STATE that returns 0 to
> prevent errors when this parameter is set.
>
> Signed-off-by: Yulin Lu <luyulin@xxxxxxxxxxxxxxxxxx>
> Fixes: 5b797bcc00ef ("pinctrl: eswin: Add EIC7700 pinctrl driver")
Patch applied, I don't see any indication that this is a regression or
urgent so applied as nonurgent fix.
Yours,
Linus Walleij