Re: [PATCH v1 11/15] ASoC: fsl_ssi: Setup AC97 in dai_probe()
On Thu, Jan 04, 2018 at 09:38:52PM +0100, Maciej S. Szmigiero wrote: > > Hmm...What's the dependency here? Why is it required like this? > And a AC'97 CODEC probe needs AC'97 communication to be working, > since it has to detect the CODEC model, configure it, etc. Okay. If the CODEC configurations depend on (or are done via) the AC link, it has to be in this way then. > > I am okay to put everything to a separate fsl_ssi_hw_init() and > > move it back to the platform probe() though. > > This could be a solution - I assume that by "everything" in the above > sentence you mean (at least) enabling the AC'97 communication at the > SSI. All register configurations -- I was trying to move them to dai probe() so a deferring probe or the platform remove() does not need to revert these register configurations.
Re: [PATCH v1 11/15] ASoC: fsl_ssi: Setup AC97 in dai_probe()
On 04.01.2018 20:07, Nicolin Chen wrote: > On Mon, Jan 01, 2018 at 04:17:20PM +0100, Maciej S. Szmigiero wrote: >>> AC97 configures some registers earlier to start a communication >>> with CODECs, so this patch moves those register settings to the >>> dai_probe() as well, along with other register configurations. > >> This patch breaks AC'97 CODEC probing. >> >> Namely, the fsl_ssi DAI probe callback is only called after the AC'97 >> CODEC probe callback, so when you move SSI AC'97 startup to its DAI >> probe callback it won't be done yet when the CODEC is probed (and this >> requires a working AC'97 interface to successfully complete). > > Hmm...What's the dependency here? Why is it required like this? This patch moves enabling AC'97 communication (done by fsl_ssi_setup_ac97() ) from SSI _platform device_ probe path to SSI _DAI_ probe path. However, it turns out that a SSI _DAI_ probe happens after a AC'97 CODEC probe (that is, ac97_soc_probe() in sound/soc/codecs/ac97.c). And a AC'97 CODEC probe needs AC'97 communication to be working, since it has to detect the CODEC model, configure it, etc. > I am okay to put everything to a separate fsl_ssi_hw_init() and > move it back to the platform probe() though. > This could be a solution - I assume that by "everything" in the above sentence you mean (at least) enabling the AC'97 communication at the SSI. Maciej
Re: [PATCH v1 11/15] ASoC: fsl_ssi: Setup AC97 in dai_probe()
On Mon, Jan 01, 2018 at 04:17:20PM +0100, Maciej S. Szmigiero wrote: > > AC97 configures some registers earlier to start a communication > > with CODECs, so this patch moves those register settings to the > > dai_probe() as well, along with other register configurations. > This patch breaks AC'97 CODEC probing. > > Namely, the fsl_ssi DAI probe callback is only called after the AC'97 > CODEC probe callback, so when you move SSI AC'97 startup to its DAI > probe callback it won't be done yet when the CODEC is probed (and this > requires a working AC'97 interface to successfully complete). Hmm...What's the dependency here? Why is it required like this? I am okay to put everything to a separate fsl_ssi_hw_init() and move it back to the platform probe() though.
Re: [PATCH v1 11/15] ASoC: fsl_ssi: Setup AC97 in dai_probe()
On 19.12.2017 18:00, Nicolin Chen wrote: > AC97 configures some registers earlier to start a communication > with CODECs, so this patch moves those register settings to the > dai_probe() as well, along with other register configurations. > > It also applies _fsl_ssi_set_dai_fmt() to AC97 only since other > formats would be configured via fsl_ssi_set_dai_fmt() directly. > > Meanwhile, this patch adds fsl_ssi_dai_ac97_remove() to cleanup > some control bits for AC97. > > Signed-off-by: Nicolin Chen This patch breaks AC'97 CODEC probing. Namely, the fsl_ssi DAI probe callback is only called after the AC'97 CODEC probe callback, so when you move SSI AC'97 startup to its DAI probe callback it won't be done yet when the CODEC is probed (and this requires a working AC'97 interface to successfully complete). Maciej
