Hello. Chaithrika U S wrote: > Add clock enable and disable calls to resume and suspend respectively. > Also add a member to the audio device data structure which tracks the clock > status. > Tested on DA850/OMAP-L138 EVM. For the purpose of testing, the patches[1] which > add suspend-to-RAM support to DA850/OMAP-L138 SoC were applied. > [1] http://linux.davincidsp.com/pipermail/davinci-linux-open-source/ > 2009-November/016958.html > Signed-off-by: Chaithrika U S <chaithrika@xxxxxx> > --- [...] > diff --git a/sound/soc/davinci/davinci-mcasp.c b/sound/soc/davinci/davinci-mcasp.c > index 0a302e1..0d263f1 100644 > --- a/sound/soc/davinci/davinci-mcasp.c > +++ b/sound/soc/davinci/davinci-mcasp.c > @@ -767,14 +767,27 @@ static int davinci_mcasp_trigger(struct snd_pcm_substream *substream, > int ret = 0; > > switch (cmd) { > - case SNDRV_PCM_TRIGGER_START: > case SNDRV_PCM_TRIGGER_RESUME: > + if (!dev->clk_active) { > + clk_enable(dev->clk); > + dev->clk_active = 1; > + } You should add a comment in the cases where *break* is ommitted deliberately (if it indeed is), like: /* FALL THRU */ > + > + case SNDRV_PCM_TRIGGER_START: > case SNDRV_PCM_TRIGGER_PAUSE_RELEASE: > davinci_mcasp_start(dev, substream->stream); > break; > WBR, Sergei _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
