Re: [PATCH v1 1/1] pinctrl: eswin: Fix Handling of PIN_CONFIG_PERSIST_STATE

Linus Walleij <[email protected]> Mon, 3 Aug 2026 09:43:27 +0200

Newsgroups	org.kernel.vger.linux-gpio,org.kernel.vger.linux-kernel
Message-ID	<CAD++jLn_jN025L_=UYJi-g3mx1n83A3g50-CLbLqSuNX+d332g@mail.gmail.com>

On Tue, Jul 28, 2026 at 9:30=E2=80=AFAM Yulin Lu <[email protected]=
m> 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 <[email protected]>
> 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