On Tue, Jan 15, 2019 at 03:16:57PM -0600, Pierre-Louis Bossart wrote:
Maybe the defer card probe logic needs to be extended to also check if dai_link_name had already been registered (either cpu or cpu_dai_name needs to be set), not 100% sure which problem the defer card probe patch was trying to solve.
We were getting cards probing without the platforms being registered (which in turn meant we were just skipping their init) and had patches proposed to implement the deferral in the cards. The deferral stuff is supposed to making sure that everything is registered when we instantiate.
same here, I don't get why the deferred probe stuff only deals with one of the two options.
I think it's just an oversight - I think the change you were proposing to check the cpu_dai_name is a good idea anyway as it makes things more consistent and work more obviously by intention. And more generally if we can simplify the code by removing legacy options that'd be good but that's a bigger bit of work...