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
