On Fri, Jan 06, 2017 at 06:32:33PM +0000, Mark Brown wrote:
On Mon, Jan 02, 2017 at 12:44:29PM +0530, jeeja.kp@intel.com wrote:
From: Jeeja KP jeeja.kp@intel.com
Some machine requires MCLK to be enabled before SSP start up, so add set_tristate() DAI ops to enable/disable SSP MCLK. This ops will be called from machine which requires early MCLK and removes the usage of enabling early MLCK in platform DAI prepare by default which is not machine specific.
This isn't an *obvious* use of the tristate operation...
- /*
* To enable ssp clk explicitly before gateway copier is configured,* need to use dma control IPC.* Disable will be done when DSP gateway copier modules are deleted.*/- if (!tristate)
return skl_dsp_set_dma_control(ctx, mconfig);So we never clear this? And to be frank I'd have no idea looking at this that it's doing anything to do with clocks though that's a bit of a separate thing. It feels like DAPM might be a more natural place to do this, have a clock widget supplying the AIF widget for the interface. That'd be a bit more idiomatic.
The clk will be cleared, when DSP pipeline is deleted, that is when audio device is closed. My bad, yes this is not the correct usage of tristate operation, we need to model it as a clock supply widget, will rework on this patch. --