Re: [PATCH 0/4] pinctrl: free maps on DT mapping failures

[Thierry Reding <thierry.reding@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 1130 bytes --]

On Thu, Oct 01, 2026 at 09:49:09AM +0200, Linus Walleij wrote:
> On Thu, Sep 24, 2026 at 1:19 PM Jiale Yao <yaojiale02@163.com> wrote:
> 
> > Several pinctrl drivers reserve map storage before appending mux and
> > configuration entries while translating device tree nodes.  If a later
> > operation fails, their error paths free only temporary configuration data
> > and leave the partially built map allocated.
> >
> > Commit 17007cd700601777d9ee203a13d97e64ece3a10f fixed the same cleanup
> > problem in the generic mapping helper.  These drivers use their own mapping
> > callbacks and need equivalent error handling.
> >
> > Free the accumulated map and clear the output parameters on every failure
> > path in the Spreadtrum, Renesas RZ/N1, Tegra XUSB, and Samsung drivers.
> > Each patch changes one driver and remains independently buildable.
> 
> Patches applied as nonurgent fixes.

Jiale said they were going to update this to drop the useless reset of
the output parameters. Those changes should probably be integrated into
the patches you apply. Could of course be a follow-up, too.

Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]