Re: [PATCH v2 4/8] ASoC: topology: ABI - Add name element to snd_soc_tplg_stream

[Lars-Peter Clausen <lars@metafoo.de>]

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

On 10/06/2015 06:25 PM, Mark Brown wrote:
> On Tue, Oct 06, 2015 at 03:16:30PM +0000, Koul, Vinod wrote:
>> On Tue, 2015-10-06 at 11:34 +0100, Mark Brown wrote:
> 
>>> No, I mean that if someone changes a name it'll invalidate topology
>>> files that reference that name.
> 
>> Shouldn't this sort be checked in the alsa-lib while building the topology. It should
>> validate the names, graph and complete only when stuff is right.
> 
> The problem isn't detecting the error, the point is that if someone
> upgrades their kernel and someone changed the DAI link name then their
> existing userspace will break.  We don't want that error to be there to
> be detected in the first place.

Another thing to consider is that the topology firmware might be developed
against a non-upstream driver. And in order for the driver to go upstream it
needs changes that breaks the ABI interface to the firmware which is already
shipped and the vendor might not provide support for updating the firmware
file. Having these kind of tightly coupled interdependencies between driver
and firmware is quite risky business.



[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

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