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