All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mengdong Lin <mengdong.lin@linux.intel.com>
To: Mark Brown <broonie@kernel.org>
Cc: alsa-devel@alsa-project.org, tiwai@suse.de,
	hardik.t.shah@intel.com, guneshwor.o.singh@intel.com,
	liam.r.girdwood@linux.intel.com, vinod.koul@intel.com,
	rakesh.a.ughreja@intel.com, mengdong.lin@intel.com
Subject: Re: [PATCH v3 1/8] ASoC: topology: ABI - Add flags to PCM
Date: Sat, 8 Oct 2016 15:39:27 +0800	[thread overview]
Message-ID: <57F8A2AF.3080909@linux.intel.com> (raw)
In-Reply-To: <20161006144419.z6ntryxtrxpwsqad@sirena.org.uk>



On 10/06/2016 10:44 PM, Mark Brown wrote:
> On Thu, Sep 29, 2016 at 02:08:51PM +0800, mengdong.lin@linux.intel.com wrote:
>
>> +	__le32 flag_mask;       /* bitmask of flags to configure */
>> +	__le32 flags;           /* SND_SOC_TPLG_LNK_FLGBIT_* flag value */
>>   } __attribute__((packed));
>
> I just followed up on the userspace patches but to repeat what I was
> saying there I think given that users are actively complaining about
> topology ABI files breaking we need to handle any extensions in a
> backwards compatible fashion.
>

Thanks for your comments!

We'll try to solve this backward compatible issue in kernel. And we'll 
support from topology ABI version 4.

Version 4 is now used by alsa-lib. Before version 4, only Intel use it 
internally and later we moved to version 4.

Thanks
Mengdong

  reply	other threads:[~2016-10-08  7:38 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-29  5:55 [PATCH v3 0/8] ASoC: topology: Remaining kernel patches mengdong.lin
2016-09-29  6:08 ` [PATCH v3 1/8] ASoC: topology: ABI - Add flags to PCM mengdong.lin
2016-10-06 14:44   ` Mark Brown
2016-10-08  7:39     ` Mengdong Lin [this message]
2016-09-29  6:09 ` [PATCH v3 2/8] ASoC: topology: ABI - Add private data " mengdong.lin
2016-09-29  6:09 ` [PATCH v3 3/8] ASoC: topology: ABI - Define DAI physical PCM data formats mengdong.lin
2016-09-29  6:09 ` [PATCH v3 4/8] ASoC: topology: ABI - Add HW configurations to BE DAI links mengdong.lin
2016-09-29  6:09 ` [PATCH v3 5/8] ASoC: topology: ABI - Add flags and private data " mengdong.lin
2016-09-29  6:09 ` [PATCH v3 6/8] ASoC: topology: ABI - Add name and stream name " mengdong.lin
2016-09-29  6:09 ` [PATCH v3 7/8] ASoC: Define API to find a dai link mengdong.lin
2016-11-04 17:23   ` Applied "ASoC: Define API to find a dai link" to the asoc tree Mark Brown
2016-09-29  6:10 ` [PATCH v3 8/8] ASoC: topology: Add support to configure existing BE DAI links mengdong.lin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=57F8A2AF.3080909@linux.intel.com \
    --to=mengdong.lin@linux.intel.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=broonie@kernel.org \
    --cc=guneshwor.o.singh@intel.com \
    --cc=hardik.t.shah@intel.com \
    --cc=liam.r.girdwood@linux.intel.com \
    --cc=mengdong.lin@intel.com \
    --cc=rakesh.a.ughreja@intel.com \
    --cc=tiwai@suse.de \
    --cc=vinod.koul@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.