* [PATCH] omap3: alsa-soc: Remove restrictive check for cpu type
@ 2011-03-14 13:31 ` Sanjeev Premi
0 siblings, 0 replies; 10+ messages in thread
From: Sanjeev Premi @ 2011-03-14 13:31 UTC (permalink / raw)
To: linux-omap, alsa-devel, linux-arm-kernel; +Cc: Sanjeev Premi
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@ti.com>
---
sound/soc/omap/omap-mcbsp.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/sound/soc/omap/omap-mcbsp.c b/sound/soc/omap/omap-mcbsp.c
index 2175f09..84d434a 100644
--- a/sound/soc/omap/omap-mcbsp.c
+++ b/sound/soc/omap/omap-mcbsp.c
@@ -146,7 +146,7 @@ static int omap_mcbsp_dai_startup(struct snd_pcm_substream *substream,
* 2 channels (stereo): size is 128 / 2 = 64 frames (2 * 64 words)
* 4 channels: size is 128 / 4 = 32 frames (4 * 32 words)
*/
- if (cpu_is_omap343x() || cpu_is_omap44xx()) {
+ if (cpu_is_omap34xx() || cpu_is_omap44xx()) {
/*
* Rule for the buffer size. We should not allow
* smaller buffer than the FIFO size to avoid underruns
--
1.7.2.2
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH] omap3: alsa-soc: Remove restrictive check for cpu type
@ 2011-03-14 13:31 ` Sanjeev Premi
0 siblings, 0 replies; 10+ messages in thread
From: Sanjeev Premi @ 2011-03-14 13:31 UTC (permalink / raw)
To: linux-arm-kernel
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@ti.com>
---
sound/soc/omap/omap-mcbsp.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/sound/soc/omap/omap-mcbsp.c b/sound/soc/omap/omap-mcbsp.c
index 2175f09..84d434a 100644
--- a/sound/soc/omap/omap-mcbsp.c
+++ b/sound/soc/omap/omap-mcbsp.c
@@ -146,7 +146,7 @@ static int omap_mcbsp_dai_startup(struct snd_pcm_substream *substream,
* 2 channels (stereo): size is 128 / 2 = 64 frames (2 * 64 words)
* 4 channels: size is 128 / 4 = 32 frames (4 * 32 words)
*/
- if (cpu_is_omap343x() || cpu_is_omap44xx()) {
+ if (cpu_is_omap34xx() || cpu_is_omap44xx()) {
/*
* Rule for the buffer size. We should not allow
* smaller buffer than the FIFO size to avoid underruns
--
1.7.2.2
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [PATCH] omap3: alsa-soc: Remove restrictive check for cpu type
2011-03-14 13:31 ` Sanjeev Premi
@ 2011-03-14 13:55 ` Jarkko Nikula
-1 siblings, 0 replies; 10+ messages in thread
From: Jarkko Nikula @ 2011-03-14 13:55 UTC (permalink / raw)
To: Sanjeev Premi; +Cc: linux-omap, alsa-devel, linux-arm-kernel
On Mon, 14 Mar 2011 19:01:53 +0530
Sanjeev Premi <premi@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@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?
--
Jarkko
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH] omap3: alsa-soc: Remove restrictive check for cpu type
@ 2011-03-14 13:55 ` Jarkko Nikula
0 siblings, 0 replies; 10+ messages in thread
From: Jarkko Nikula @ 2011-03-14 13:55 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, 14 Mar 2011 19:01:53 +0530
Sanjeev Premi <premi@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@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?
--
Jarkko
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] omap3: alsa-soc: Remove restrictive check for cpu type
2011-03-14 13:55 ` Jarkko Nikula
@ 2011-03-14 14:00 ` Jarkko Nikula
-1 siblings, 0 replies; 10+ messages in thread
From: Jarkko Nikula @ 2011-03-14 14:00 UTC (permalink / raw)
To: Sanjeev Premi; +Cc: linux-omap, alsa-devel, linux-arm-kernel
On Mon, 14 Mar 2011 15:55:42 +0200
Jarkko Nikula <jhnikula@gmail.com> wrote:
> On Mon, 14 Mar 2011 19:01:53 +0530
> Sanjeev Premi <premi@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@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.
--
Jarkko
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH] omap3: alsa-soc: Remove restrictive check for cpu type
@ 2011-03-14 14:00 ` Jarkko Nikula
0 siblings, 0 replies; 10+ messages in thread
From: Jarkko Nikula @ 2011-03-14 14:00 UTC (permalink / raw)
To: linux-arm-kernel
On Mon, 14 Mar 2011 15:55:42 +0200
Jarkko Nikula <jhnikula@gmail.com> wrote:
> On Mon, 14 Mar 2011 19:01:53 +0530
> Sanjeev Premi <premi@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@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.
--
Jarkko
^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH] omap3: alsa-soc: Remove restrictive check for cpu type
2011-03-14 14:00 ` Jarkko Nikula
@ 2011-03-14 14:17 ` Premi, Sanjeev
-1 siblings, 0 replies; 10+ messages in thread
From: Premi, Sanjeev @ 2011-03-14 14:17 UTC (permalink / raw)
To: Jarkko Nikula
Cc: linux-omap@vger.kernel.org, alsa-devel@alsa-project.org,
linux-arm-kernel@lists.infradead.org
> -----Original Message-----
> From: Jarkko Nikula [mailto:jhnikula@gmail.com]
> Sent: Monday, March 14, 2011 7:31 PM
> To: Premi, Sanjeev
> Cc: linux-omap@vger.kernel.org; alsa-devel@alsa-project.org;
> linux-arm-kernel@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@gmail.com> wrote:
>
> > On Mon, 14 Mar 2011 19:01:53 +0530
> > Sanjeev Premi <premi@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@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...
~sanjeev
>
> --
> Jarkko
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH] omap3: alsa-soc: Remove restrictive check for cpu type
@ 2011-03-14 14:17 ` Premi, Sanjeev
0 siblings, 0 replies; 10+ messages in thread
From: Premi, Sanjeev @ 2011-03-14 14:17 UTC (permalink / raw)
To: linux-arm-kernel
> -----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@gmail.com> wrote:
>
> > On Mon, 14 Mar 2011 19:01:53 +0530
> > Sanjeev Premi <premi@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@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...
~sanjeev
>
> --
> Jarkko
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* RE: [PATCH] omap3: alsa-soc: Remove restrictive check for cpu type
2011-03-14 14:17 ` Premi, Sanjeev
@ 2011-03-14 14:25 ` Premi, Sanjeev
-1 siblings, 0 replies; 10+ messages in thread
From: Premi, Sanjeev @ 2011-03-14 14:25 UTC (permalink / raw)
To: Jarkko Nikula
Cc: linux-omap@vger.kernel.org, alsa-devel@alsa-project.org,
linux-arm-kernel@lists.infradead.org
> -----Original Message-----
> From: linux-omap-owner@vger.kernel.org
> [mailto:linux-omap-owner@vger.kernel.org] On Behalf Of Premi, Sanjeev
> Sent: Monday, March 14, 2011 7:47 PM
> To: Jarkko Nikula
> Cc: linux-omap@vger.kernel.org; alsa-devel@alsa-project.org;
> linux-arm-kernel@lists.infradead.org
> Subject: RE: [PATCH] omap3: alsa-soc: Remove restrictive
> check for cpu type
>
> > -----Original Message-----
> > From: Jarkko Nikula [mailto:jhnikula@gmail.com]
> > Sent: Monday, March 14, 2011 7:31 PM
> > To: Premi, Sanjeev
> > Cc: linux-omap@vger.kernel.org; alsa-devel@alsa-project.org;
> > linux-arm-kernel@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@gmail.com> wrote:
> >
> > > On Mon, 14 Mar 2011 19:01:53 +0530
> > > Sanjeev Premi <premi@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@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@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH] omap3: alsa-soc: Remove restrictive check for cpu type
@ 2011-03-14 14:25 ` Premi, Sanjeev
0 siblings, 0 replies; 10+ messages in thread
From: Premi, Sanjeev @ 2011-03-14 14:25 UTC (permalink / raw)
To: linux-arm-kernel
> -----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@gmail.com> wrote:
> >
> > > On Mon, 14 Mar 2011 19:01:53 +0530
> > > Sanjeev Premi <premi@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@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
>
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2011-03-14 14:25 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-03-14 13:31 [PATCH] omap3: alsa-soc: Remove restrictive check for cpu type Sanjeev Premi
2011-03-14 13:31 ` Sanjeev Premi
2011-03-14 13:55 ` Jarkko Nikula
2011-03-14 13:55 ` Jarkko Nikula
2011-03-14 14:00 ` Jarkko Nikula
2011-03-14 14:00 ` Jarkko Nikula
2011-03-14 14:17 ` Premi, Sanjeev
2011-03-14 14:17 ` Premi, Sanjeev
2011-03-14 14:25 ` Premi, Sanjeev
2011-03-14 14:25 ` Premi, Sanjeev
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.