[alsa-devel] [PATCH] omap3: alsa-soc: Remove restrictive check for cpu type

[Premi, Sanjeev]

> -----Original Message-----
> From: linux-omap-owner at vger.kernel.org 
> [mailto:linux-omap-owner at vger.kernel.org] On Behalf Of Premi, Sanjeev
> Sent: Monday, March 14, 2011 7:47 PM
> To: Jarkko Nikula
> Cc: linux-omap at vger.kernel.org; alsa-devel at alsa-project.org; 
> linux-arm-kernel at lists.infradead.org
> Subject: RE: [PATCH] omap3: alsa-soc: Remove restrictive 
> check for cpu type
> 
> > -----Original Message-----
> > From: Jarkko Nikula [mailto:jhnikula at gmail.com] 
> > Sent: Monday, March 14, 2011 7:31 PM
> > To: Premi, Sanjeev
> > Cc: linux-omap at vger.kernel.org; alsa-devel at alsa-project.org; 
> > linux-arm-kernel at lists.infradead.org
> > Subject: Re: [PATCH] omap3: alsa-soc: Remove restrictive 
> > check for cpu type
> > 
> > On Mon, 14 Mar 2011 15:55:42 +0200
> > Jarkko Nikula <jhnikula at gmail.com> wrote:
> > 
> > > On Mon, 14 Mar 2011 19:01:53 +0530
> > > Sanjeev Premi <premi at ti.com> wrote:
> > > 
> > > > Current check for cpu type is too restrictive leading to
> > > > failures for other silicons in same family.
> > > > 
> > > > The problem was found while testing audio playback on
> > > > AM37x and AM35x processors. But should exist on OMAP36xx
> > > > as well.
> > > > 
> > > > Signed-off-by: Sanjeev Premi <premi at ti.com>
> > > > ---
> > > >  sound/soc/omap/omap-mcbsp.c |    2 +-
> > > >  1 files changed, 1 insertions(+), 1 deletions(-)
> > > > 
> > > There are two other cpu_is_omap343x tests as well. Care to 
> > check them
> > > too?
> > > 
> > Sorry, only one additional test. I forgot the commit 2686e07 "ASoC:
> > McBSP: get hw params from McBSP driver" that is coming via 
> linux-omap
> > and which removes one test.
> 
> Checking now...

Other check shouldn't be there are well... Had asked someone (off-this
-list) to help me test the change - but seems to be in todo.

Will test it myself and post updated patch today...

> 
> ~sanjeev
> 
> > 
> > -- 
> > Jarkko
> > --
> To unsubscribe from this list: send the line "unsubscribe 
> linux-omap" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>