Re: [PATCH 13/31] ASoC: rt1320-sdw: Use auto-cleanup for firmware loading

Takashi Iwai <[email protected]> Thu, 06 Aug 2026 15:48:42 +0200
Newsgroups org.kernel.vger.linux-sound
Message-ID <[email protected]>
On Thu, 06 Aug 2026 13:52:48 +0200,
Mark Brown wrote:
> 
> On Thu, Aug 06, 2026 at 09:37:14AM +0200, Takashi Iwai wrote:
> > Mark Brown wrote:
> 
> > > There's a goto further up that jumps over this which clang warns about,
> > > this will break the build.  Landmines like that are why cleanup.h warns
> > > about mixing it with goto.
> 
> > That's an issue because I moved the declaration of the variable fw;
> > if it's left at the beginning of the function as it was, it should
> > work fine, like the patch below.  The same applied to the error for
> > wm2000.
> 
> Yes, it's fine when the declaration is at the beginning IIRC (though not
> ideal stylistically) - the warning is generated when a goto jumps over a
> declaration.

OK, I'm going to submit v2 series.


thanks,

Takashi