On 2022-03-08 8:42 PM, Pierre-Louis Bossart wrote:
On 3/8/22 12:31, Cezary Rojewski wrote:
...
Yes, avs-driver embraces granular sound card approach as opposed to super card approach. There is one topology file per sound card.
That subject is not part of this patchset though.
You would still need to clarify how the same set of modules might be accessed or configured when different cards are created, or if there is a 'clean' split with a 1:1 mapping between module and cards.
The more I think of this the less practical this 'granular' approach looks, e.g. if you want to route the same stream to different interfaces handled by different cards. An example is playing a notification on local speakers controlled by HDaudio and a Bluetooth headset using I2S. This could be really fun to represent even basic volume control to users: controls are card-specific and some parts may be handled in different cards and thus different UCM files.
I really think the best split of a DSP topology is between orthogonal parts. When muxers/demux are used, or multi-input modules such as AEC, the routing complexity outweighs the benefits of a simpler card design.
There is a clean split - for all the typical scenarios, topology tied to given sound card defines the stream patterns completely.
Even when considering "functional" split as you mention in third paragraph, multiple sound cards are still there. So, as I have described earlier, to prevent any unwanted behaviour, cached module information in form of ->mods_info, is being locked with help of a mutex. We do not duplicate such information per sound card as that would be a waste of memory. No matter how many libraries are mentioned by the topology files and loaded throughout the lifetime of a driver, there is always just one base firmware we sent MODULE_INFO to.
In regard to advanced scenarios and more complex routing, it would be good to have that conversation as a part of dedicated patchset which is going to follow this one (sooner or later).
Regards, Czarek