Re: [PATCH] ALSA: firewire-lib: fix wrong value as length of header for CIP_NO_HEADER case

[Takashi Sakamoto <o-takashi@sakamocchi.jp>]

Hi,

On Thu, Jul 25, 2024 at 07:52:32PM +0200, Takashi Iwai wrote:
> On Thu, 25 Jul 2024 18:25:37 +0200,
> Takashi Sakamoto wrote:
> > 
> > Hi,
> > 
> > On Thu, Jul 25, 2024 at 10:16:36AM -0600, Gustavo A. R. Silva wrote:
> > > Yes, but why have two separate patches when the root cause can be addressed by
> > > a single one, which will prevent other potential issues from occurring?
> > > 
> > > The main issue in this case is the __counted_by() annotation. The DEFINE_FLEX()
> > > bug was a consequence.
> > 
> > Just now I sent a patch to revert the issued commit[1].
> > 
> > I guess that we need the association between the two fixes. For example,
> > we can append more 'Fixes' tag to the patch in sound subsystem into the
> > patch in firewire subsystem (or vice versa).
> 
> OK, then I drop your patch for the sound stuff, and you can take it
> through firewire tree.
> 
> Feel free to take my ack:
> Reviewed-by: Takashi Iwai <tiwai@suse.de>
 
Okay. I'll send them to mainline as fixes for v6.11-rc1.
 
> thanks,
> 
> Takashi


Thanks

Takashi Sakamoto