On Thu, 2020-04-23 at 07:12 +0900, Kuninori Morimoto wrote:
Hi
Hi Ranjani
These patches are tring to not to use snd_soc_rtdcom_lookup() function on each drivers as much as possible, because we might have same name component under multi component situation. It can't find correct component in such case.
I tried to add new feature on each drivers to not to use it, but I can't test. Thus, these patches should get Acked-by or Tested-by from each drivers user/maintenor. Please test these.
After these patches, Intel / SOF drivers are still using snd_soc_rtdcom_lookup(). Because it is very complex, I couldn't try not to use it. If possible, each drivers should try to not use it, and it should be removed from ASoC.
Morimoti-san,
For my education, I understand the concept of multi-cpu/codec components, but when or who would need multiple platform components? This would help me able to remove the snd_soc_rtdcom_lookup() call in SOF.
I don't know concrete system. But it is "possible" today. And, we don't know the future system, having flexibility is good idea, I think.
I'm thinking removing lookup function is nice idea, but don't feel pressure to it. "Now you know it" is very enough for me.
I am having a hard time visualizing a scenario where we would have more than one platform component. And even if we did, I'd think that the driver registering these components would make sure to not duplicate the driver names. Of course, we dont really check if thats really the case. Do you think it makes sense to add that check when registering a component? If we do that, then keeping snd_soc_rtdcom_lookup() might not be such a bad idea.
Thanks, Ranjani