Re: [PATCH v4 2/9] ASoC: Intel: Skylake: Add set_tristate DAI ops to enable SSP MCLK

[Jeeja KP <jeeja.kp@intel.com>]

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.
--