From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mengdong Lin Subject: Re: [PATCH 0/7] topology: Add support for vendor tuples Date: Mon, 28 Mar 2016 15:49:36 +0800 Message-ID: <56F8E210.5000108@linux.intel.com> References: <20160324112051.GW2566@sirena.org.uk> <56F4CF07.6080001@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by alsa0.perex.cz (Postfix) with ESMTP id BB48B260694 for ; Mon, 28 Mar 2016 09:47:27 +0200 (CEST) In-Reply-To: <56F4CF07.6080001@linux.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Mark Brown , tiwai@suse.de Cc: alsa-devel@alsa-project.org, vinod.koul@intel.com, mengdong.lin@intel.com, rakesh.a.ughreja@intel.com, liam.r.girdwood@intel.com, hardik.t.shah@intel.com, subhransu.s.prusty@intel.com List-Id: alsa-devel@alsa-project.org Hi Takashi and Mark, Despite the ABI update, could you help to review if this is a doable solution to define vendor-specific data by tuples? Previously there was a patch for Skylake topology conf file, but this file try to import vendor specific config data as a binary blob from other files. The patch was refused for on license of the binary blob and we were suggested to expose the layout of the data. This patch series is to meet this request, providing a way for vendors to define their structured private data. Here is the link of the refused patch: [alsa-devel] [PATCH] conf: topology: Add topolgy for skylake i2s configuration http://mailman.alsa-project.org/pipermail/alsa-devel/2016-February/104099.html Thanks Mengdong On 03/25/2016 01:39 PM, Mengdong Lin wrote: > http://mailman.alsa-project.org/pipermail/alsa-devel/2016-February/104099.html > On 03/24/2016 07:20 PM, Mark Brown wrote: >> On Thu, Mar 24, 2016 at 11:05:29AM +0800, mengdong.lin@linux.intel.com >> wrote: >>> From: Mengdong Lin >>> >>> 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". >> >> It's now been three kernel releases since the original code was merged >> and we've still not seen any effort to make the ABI actually available. >> Where are we with that? I've mentioned it a few times previously but >> don't really recall any response. >> > > We've been working on this. I'm sorry we're still not fast enough. > > This ABI update is introduced by previous failure to upstream the > topology configuration file for Skylake, which imports vendor-specific > binary data from another file. It was refused because of the license of > the binary data is not obvious. So we turn to use vendor tuples, because > the vendor tuples can show the layout and meaning of binary data, which > can help to check if the data is safe to open source. > This ABI change has no impact on backward compatibility. > > Also, the audio device driver for new platforms are under development > now, which uses topology for DAI and DAI links. We're checking if > current ABI can meet all requirements for the new platforms. This may > still need a few weeks. > > Thanks > Mengdong