[alsa-devel] [PATCH] ASoC: mcbsp: Add McBSP support for OMAP4
Liam Girdwood
lrg at slimlogic.co.uk
Thu Dec 23 15:39:38 CET 2010
On Thu, 2010-12-23 at 07:32 +0000, peter.ujfalusi at nokia.com wrote:
> Hi,
>
> > From: alsa-devel-bounces at alsa-project.org [alsa-devel-bounces at alsa-project.org] on behalf of Ujfalusi Peter (Nokia-MS/Tampere)
> > Sent: Thursday, December 23, 2010 08:42
> > To: lrg at slimlogic.co.uk; jhnikula at gmail.com
> > Cc: magi.olaya at ti.com; alsa-devel at alsa-project.org; broonie at opensource.wolfsonmicro.com
> > Subject: Re: [alsa-devel] [PATCH] ASoC: mcbsp: Add McBSP support for OMAP4
>
> > In the patch:
> > @@ -623,6 +652,9 @@ static int omap_mcbsp_dai_set_dai_sysclk(struct snd_soc_dai *cpu_dai,
> > if (cpu_class_is_omap1()) {
> > err = -EINVAL;
> > break;
> > + } else if (cpu_is_omap44xx()) {
> > + regs->srgr2 &= ~CLKSM;
> >
> > There's no need to clear the CLKSM, since it is by default not set.
> > IMHO you shall handle the OMAP4 in a same way as we handle OMAP3.
> > By doing nothing here.
> >
> > + break;
>
> I think you should not break here, since you might want to select the CLKS/PRCM with the next
> call...
>
> > }
> > err = omap2_mcbsp_set_clks_src(mcbsp_data->bus_id,
> > MCBSP_CLKS_PRCM_SRC);
> >
> >
> > Otherwise:
> > Acked-by: Peter Ujfalusi <peter.ujfalusi at nokia.com>
>
> Just noticed that, this patch is already in, but the issues, I have commented has to
> be fixed in order to have proper support for OMAP4.
Now fixed and added your Ack.
Thanks
Liam
--
Freelance Developer, SlimLogic Ltd
ASoC and Voltage Regulator Maintainer.
http://www.slimlogic.co.uk
More information about the Alsa-devel
mailing list