Re: [alsa-devel] [PATCH 1/2] ASoC: max98090: Add master clock handling

On Fri, May 23, 2014 at 11:05:17AM +0530, Tushar Behera wrote:

I tested with moving clk_enable/clk_disable calls to set_bias_level(): SND_SOC_BIAS_PREPARE. That works well for me. Does it look okay?

•           if (!IS_ERR(max98090->mclk)) {
  

•                   if (codec->dapm.bias_level == SND_SOC_BIAS_ON)
  

•                           clk_disable(max98090->mclk);
  

•                   else
  

•                           clk_enable(max98090->mclk);
  

•           }
            break;
  

Should be clk_prepare_enable() and similarly for the disable and you should check the error codes but yes, that looks good.