All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Brown <broonie@kernel.org>
To: "Lin, Mengdong" <mengdong.lin@intel.com>
Cc: Takashi Iwai <tiwai@suse.de>,
	"alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>,
	"Girdwood, Liam R" <liam.r.girdwood@intel.com>
Subject: Re: [PATCH v2 09/10] ASoC: topology: Change stream formats to bitwise flag
Date: Sat, 15 Aug 2015 08:14:01 -0700	[thread overview]
Message-ID: <20150815151401.GF10748@sirena.org.uk> (raw)
In-Reply-To: <F46914AEC2663F4A9BB62374E5EEF8F8480D04D0@SHSMSX101.ccr.corp.intel.com>


[-- Attachment #1.1: Type: text/plain, Size: 983 bytes --]

On Sat, Aug 15, 2015 at 01:49:42PM +0000, Lin, Mengdong wrote:

> Is there some topic branch which can accept topology patches for next
> kernel release?  Can we change topology ABI in v4.3?

You're not quite getting the issue here - the problem is that once we
have released the ABI we need to keep supporting applications using it
so we can't just change the ABI, we also need to keep compatibility code
for the original ABI.  This makes it a lot more effort to change things.

> Now only Intel SKL driver is using topology code, so the affect should
> be limited.  And I feel using bitwise flag is simpler in both user
> space and in kernel coding.  We can increase the topology ABI number
> when there is ABI change, and the topology driver always checks this
> version.

Someone else may take this kernel and use it in their product, or some
distro may decide to ship v4.2 as their kernel.  Once it's in a release
we have to assume someone is going to start relying on the ABI.

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]



  parent reply	other threads:[~2015-08-15 15:14 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-10 14:45 [PATCH v2 00/10] ASoC: support adding PCM dynamically from topology mengdong.lin
2015-08-10 14:45 ` [PATCH v2 01/10] ASoC: Change the PCM runtime array to a list mengdong.lin
2015-08-14 20:02   ` Mark Brown
2015-08-17  6:28     ` Lin, Mengdong
2015-08-10 14:46 ` [PATCH v2 02/10] ASoC: Define soc_init_dai_link() to wrap link intialization mengdong.lin
2015-08-10 14:47 ` [PATCH v2 03/10] ASoC: Change 2nd argument of soc_bind_dai_link() to DAI link pointer mengdong.lin
2015-08-10 14:47 ` [PATCH v2 04/10] ASoC: Implement DAI links in a list mengdong.lin
2015-08-10 14:47 ` [PATCH v2 05/10] ASoC: Add support for dummy DAI links and PCM runtimes mengdong.lin
2015-08-14 20:22   ` Mark Brown
2015-08-17 10:01     ` Lin, Mengdong
2015-08-17 18:39       ` Mark Brown
2015-08-10 14:48 ` [PATCH v2 06/10] ASoC: Bind new DAI links after probing components mengdong.lin
2015-08-10 14:48 ` [PATCH v2 07/10] ASoC: Support adding a DAI dynamically mengdong.lin
2015-08-10 14:48 ` [PATCH v2 08/10] ASoC: topology: Change pass number of DAI smaller than graph mengdong.lin
2015-08-10 14:48 ` [PATCH v2 09/10] ASoC: topology: Change stream formats to bitwise flag mengdong.lin
2015-08-14 20:34   ` Mark Brown
2015-08-15  7:37     ` Takashi Iwai
2015-08-15 13:49       ` Lin, Mengdong
2015-08-15 14:45         ` Takashi Iwai
2015-08-15 15:25           ` Lin, Mengdong
2015-08-15 16:50             ` Takashi Iwai
2015-08-15 16:51             ` Mark Brown
2015-08-17 10:05               ` Lin, Mengdong
2015-08-15 15:14         ` Mark Brown [this message]
2015-08-15 14:59       ` Mark Brown
2015-08-15 16:52         ` Takashi Iwai
2015-08-10 14:48 ` [PATCH v2 10/10] ASOC: topology: Add PCM DAIs dynamically when loading them mengdong.lin
2015-08-15  7:56 ` [PATCH v2 00/10] ASoC: support adding PCM dynamically from topology Lars-Peter Clausen
2015-08-15 15:50   ` Mark Brown
2015-08-17  6:13     ` Lin, Mengdong
2015-08-17 10:39       ` Liam Girdwood
2015-08-17 20:05         ` Mark Brown
2015-08-18  5:17           ` Lin, Mengdong
2015-08-18  5:27             ` Mark Brown

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=20150815151401.GF10748@sirena.org.uk \
    --to=broonie@kernel.org \
    --cc=alsa-devel@alsa-project.org \
    --cc=liam.r.girdwood@intel.com \
    --cc=mengdong.lin@intel.com \
    --cc=tiwai@suse.de \
    /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.