Re: [PATCH v3 0/7] topology: Add support for vendor tuples

[Takashi Iwai <tiwai@suse.de>]

On Thu, 07 Apr 2016 16:33:13 +0200,
Lin, Mengdong wrote:
> 
> > -----Original Message-----
> > From: Takashi Iwai [mailto:tiwai@suse.de]
> > Sent: Thursday, April 07, 2016 5:22 PM
> > To: mengdong.lin@linux.intel.com
> > Cc: alsa-devel@alsa-project.org; broonie@kernel.org; perex@perex.cz;
> > Girdwood, Liam R; Lin, Mengdong; Koul, Vinod; Prusty, Subhransu S; Ughreja,
> > Rakesh A; Shah, Hardik T
> > Subject: Re: [PATCH v3 0/7] topology: Add support for vendor tuples
> > 
> > On Thu, 07 Apr 2016 09:27:20 +0200,
> > mengdong.lin@linux.intel.com wrote:
> > >
> > > From: Mengdong Lin <mengdong.lin@linux.intel.com>
> > >
> > > This series addes support for vendor tuples to topology, to avoid
> > > importing binary data blob from other files.
> > >
> > > Backward compatibility of ABI is not impacted. A kernel patch is also
> > > submitted "ASoC: topology: ABI - Define types for vendor tuples".
> > >
> > > The 1st patch is small code cleanup.
> > > The 2nd patch is a preparation, since tuples will need the
> > > type-specific free handler.
> > >
> > > History:
> > > v2: add check on string length, use strtol() to get hex value,
> > >     and fix memory leak.
> > > v3: Fix overflow on uuid string copy. Use strtoul() to get unsigned
> > >     vendor values and check range, tested on 32 & 64 bit systems.
> > 
> > The patches look good to me.  The prerequisite is the change in the kernel
> > side.  Mark, how about that?
> > 
> 
> I saw the kernel patch "ASoC: topology: ABI - Define types for vendor tuples" has been merged to ASoC kernel.
> 
> Here is the link:
> http://git.kernel.org/cgit/linux/kernel/git/broonie/sound.git/commit/?h=for-next&id=e42839b012eb8179ed2e3571bfcd133fba5df154

OK, assuming that the kernel-side change will be in 4.7, I applied
this patch series now.  Thanks.


Takashi