* [PATCH] OMAP: McBSP: Do not use extensive spin locks for dma_op_mode
@ 2009-10-27 11:00 Peter Ujfalusi
2009-10-27 11:07 ` Eero Nurkkala
2009-11-12 22:17 ` [APPLIED] " Tony Lindgren
0 siblings, 2 replies; 15+ messages in thread
From: Peter Ujfalusi @ 2009-10-27 11:00 UTC (permalink / raw)
To: alsa-devel, linux-omap; +Cc: tony, broonie, eduardo.valentin
The use of the spin lock, which supposed to protect the the
dma_op_mode causing "INFO: inconsistent lock state" on
playback start.
Remove the spin locks around the dma_op_mode, when it's
purpuse is to protect the dma_op_mode.
Signed-off-by: Peter Ujfalusi <peter.ujfalusi@nokia.com>
---
arch/arm/plat-omap/mcbsp.c | 6 ------
1 files changed, 0 insertions(+), 6 deletions(-)
diff --git a/arch/arm/plat-omap/mcbsp.c b/arch/arm/plat-omap/mcbsp.c
index 88ac976..9b33671 100644
--- a/arch/arm/plat-omap/mcbsp.c
+++ b/arch/arm/plat-omap/mcbsp.c
@@ -298,9 +298,7 @@ int omap_mcbsp_get_dma_op_mode(unsigned int id)
}
mcbsp = id_to_mcbsp_ptr(id);
- spin_lock_irq(&mcbsp->lock);
dma_op_mode = mcbsp->dma_op_mode;
- spin_unlock_irq(&mcbsp->lock);
return dma_op_mode;
}
@@ -318,7 +316,6 @@ static inline void omap34xx_mcbsp_request(struct omap_mcbsp *mcbsp)
syscon = OMAP_MCBSP_READ(mcbsp->io_base, SYSCON);
syscon &= ~(ENAWAKEUP | SIDLEMODE(0x03) | CLOCKACTIVITY(0x03));
- spin_lock_irq(&mcbsp->lock);
if (mcbsp->dma_op_mode == MCBSP_DMA_MODE_THRESHOLD) {
syscon |= (ENAWAKEUP | SIDLEMODE(0x02) |
CLOCKACTIVITY(0x02));
@@ -327,7 +324,6 @@ static inline void omap34xx_mcbsp_request(struct omap_mcbsp *mcbsp)
} else {
syscon |= SIDLEMODE(0x01);
}
- spin_unlock_irq(&mcbsp->lock);
OMAP_MCBSP_WRITE(mcbsp->io_base, SYSCON, syscon);
}
@@ -1145,9 +1141,7 @@ static ssize_t dma_op_mode_show(struct device *dev,
ssize_t len = 0;
const char * const *s;
- spin_lock_irq(&mcbsp->lock);
dma_op_mode = mcbsp->dma_op_mode;
- spin_unlock_irq(&mcbsp->lock);
for (s = &dma_op_modes[i]; i < ARRAY_SIZE(dma_op_modes); s++, i++) {
if (dma_op_mode == i)
--
1.6.5.1
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH] OMAP: McBSP: Do not use extensive spin locks for dma_op_mode
2009-10-27 11:00 [PATCH] OMAP: McBSP: Do not use extensive spin locks for dma_op_mode Peter Ujfalusi
@ 2009-10-27 11:07 ` Eero Nurkkala
2009-10-27 11:17 ` Peter Ujfalusi
2009-10-27 14:00 ` Jarkko Nikula
2009-11-12 22:17 ` [APPLIED] " Tony Lindgren
1 sibling, 2 replies; 15+ messages in thread
From: Eero Nurkkala @ 2009-10-27 11:07 UTC (permalink / raw)
To: Ujfalusi Peter (Nokia-D/Tampere)
Cc: tony, alsa-devel, linux-omap, Valentin Eduardo (Nokia-D/Helsinki),
broonie
On Tue, 2009-10-27 at 12:00 +0100, Ujfalusi Peter (Nokia-D/Tampere)
wrote:
> The use of the spin lock, which supposed to protect the the
> dma_op_mode causing "INFO: inconsistent lock state" on
> playback start.
> Remove the spin locks around the dma_op_mode, when it's
> purpuse is to protect the dma_op_mode.
>
> Signed-off-by: Peter Ujfalusi <peter.ujfalusi@nokia.com>
> ---
You may wish to double read the message above, as it's quite
confusing =)
Otherwise, that spinlocking is highly unnecessary and things are
far better without than with it. The only case it could be useful
is in SMPs, but OMAPs are not such quite yet - and when they
are, things will need to be re-though anyway.
- Eero
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH] OMAP: McBSP: Do not use extensive spin locks for dma_op_mode
2009-10-27 11:07 ` Eero Nurkkala
@ 2009-10-27 11:17 ` Peter Ujfalusi
2009-10-27 12:00 ` Mark Brown
2009-10-27 14:00 ` Jarkko Nikula
1 sibling, 1 reply; 15+ messages in thread
From: Peter Ujfalusi @ 2009-10-27 11:17 UTC (permalink / raw)
To: Nurkkala Eero.An (EXT-Offcode/Oulu)
Cc: tony, alsa-devel, linux-omap, Valentin Eduardo (Nokia-D/Helsinki),
broonie
On Tuesday 27 October 2009 13:07:23 Nurkkala Eero.An (EXT-Offcode/Oulu) wrote:
> On Tue, 2009-10-27 at 12:00 +0100, Ujfalusi Peter (Nokia-D/Tampere)
>
> wrote:
> > The use of the spin lock, which supposed to protect the the
> > dma_op_mode causing "INFO: inconsistent lock state" on
> > playback start.
> > Remove the spin locks around the dma_op_mode, when it's
> > purpuse is to protect the dma_op_mode.
> >
> > Signed-off-by: Peter Ujfalusi <peter.ujfalusi@nokia.com>
> > ---
>
> You may wish to double read the message above, as it's quite
> confusing =)
Yes it is ;)
I have left one spinlock around the dma_op_mode, when it also protects the
mcbsp->active, so that is why that last sentence.
--
Péter
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH] OMAP: McBSP: Do not use extensive spin locks for dma_op_mode
2009-10-27 11:17 ` Peter Ujfalusi
@ 2009-10-27 12:00 ` Mark Brown
2009-10-27 12:04 ` Eero Nurkkala
2009-10-27 12:04 ` [alsa-devel] " Peter Ujfalusi
0 siblings, 2 replies; 15+ messages in thread
From: Mark Brown @ 2009-10-27 12:00 UTC (permalink / raw)
To: Peter Ujfalusi
Cc: tony, alsa-devel, linux-omap, Valentin Eduardo (Nokia-D/Helsinki),
Nurkkala Eero.An (EXT-Offcode/Oulu)
On Tue, Oct 27, 2009 at 01:17:52PM +0200, Peter Ujfalusi wrote:
> On Tuesday 27 October 2009 13:07:23 Nurkkala Eero.An (EXT-Offcode/Oulu) wrote:
> > You may wish to double read the message above, as it's quite
> > confusing =)
> Yes it is ;)
> I have left one spinlock around the dma_op_mode, when it also protects the
> mcbsp->active, so that is why that last sentence.
I have to confess that I'm still not entirely clear what the lock is
supposed to be doing or why it's OK to drop it. I gather that it's just
that dmap_on_mode() doesn't need a lock at all?
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH] OMAP: McBSP: Do not use extensive spin locks for dma_op_mode
2009-10-27 12:00 ` Mark Brown
@ 2009-10-27 12:04 ` Eero Nurkkala
2009-10-27 12:04 ` [alsa-devel] " Peter Ujfalusi
1 sibling, 0 replies; 15+ messages in thread
From: Eero Nurkkala @ 2009-10-27 12:04 UTC (permalink / raw)
To: ext Mark Brown
Cc: tony, Valentin Eduardo (Nokia-D/Helsinki),
alsa-devel, linux-omap, Ujfalusi Peter (Nokia-D/Tampere)
On Tue, 2009-10-27 at 13:00 +0100, ext Mark Brown wrote:
> On Tue, Oct 27, 2009 at 01:17:52PM +0200, Peter Ujfalusi wrote:
> > On Tuesday 27 October 2009 13:07:23 Nurkkala Eero.An (EXT-Offcode/Oulu) wrote:
>
> > > You may wish to double read the message above, as it's quite
> > > confusing =)
>
> > Yes it is ;)
> > I have left one spinlock around the dma_op_mode, when it also protects the
> > mcbsp->active, so that is why that last sentence.
>
> I have to confess that I'm still not entirely clear what the lock is
> supposed to be doing or why it's OK to drop it. I gather that it's just
> that dmap_on_mode() doesn't need a lock at all?
Mostly thinko's with this locking:
- spin_lock_irq(&mcbsp->lock);
dma_op_mode = mcbsp->dma_op_mode;
- spin_unlock_irq(&mcbsp->lock);
return dma_op_mode;
--> same as
return mcbsp->dma_op_mode;
and worst, it's called from pcm_trigger() ->
so irqs are enabled -> lockdep isn't happy ->
so currently things are pretty much "broken".
- Eero
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [alsa-devel] [PATCH] OMAP: McBSP: Do not use extensive spin locks for dma_op_mode
2009-10-27 12:00 ` Mark Brown
2009-10-27 12:04 ` Eero Nurkkala
@ 2009-10-27 12:04 ` Peter Ujfalusi
1 sibling, 0 replies; 15+ messages in thread
From: Peter Ujfalusi @ 2009-10-27 12:04 UTC (permalink / raw)
To: ext Mark Brown
Cc: Nurkkala Eero.An (EXT-Offcode/Oulu),
alsa-devel, linux-omap, tony, Valentin Eduardo (Nokia-D/Helsinki)
On Tuesday 27 October 2009 14:00:09 ext Mark Brown wrote:
>
> I have to confess that I'm still not entirely clear what the lock is
> supposed to be doing or why it's OK to drop it. I gather that it's just
> that dmap_on_mode() doesn't need a lock at all?
In my opinion it does not need a lock.
If the user space is so broken that it is doing different things in parallel, I
would rather crash the kernel if it is possible to force someone to fix the mess
upstairs.
But the lock is not needed, I think it was a result of over-engineering.
--
Péter
--
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] 15+ messages in thread
* Re: [alsa-devel] [PATCH] OMAP: McBSP: Do not use extensive spin locks for dma_op_mode
2009-10-27 11:07 ` Eero Nurkkala
2009-10-27 11:17 ` Peter Ujfalusi
@ 2009-10-27 14:00 ` Jarkko Nikula
2009-10-28 5:52 ` Peter Ujfalusi
1 sibling, 1 reply; 15+ messages in thread
From: Jarkko Nikula @ 2009-10-27 14:00 UTC (permalink / raw)
To: ext-eero.nurkkala
Cc: Ujfalusi Peter (Nokia-D/Tampere),
tony, alsa-devel, linux-omap, Valentin Eduardo (Nokia-D/Helsinki),
broonie
On Tue, 27 Oct 2009 13:07:23 +0200
Eero Nurkkala <ext-eero.nurkkala@nokia.com> wrote:
> Otherwise, that spinlocking is highly unnecessary and things are
> far better without than with it. The only case it could be useful
> is in SMPs, but OMAPs are not such quite yet - and when they
> are, things will need to be re-though anyway.
>
Following commit is suggesting that mcbsp code *must* be SMP safe
already now:
commit a5b92cc348299c20be215b9f4b50853ecfbf3864
Author: Syed Rafiuddin <rafiuddin.syed@ti.com>
Date: Tue Jul 28 18:57:10 2009 +0530
ARM: OMAP4: Add McBSP support
--
Jarkko
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH] OMAP: McBSP: Do not use extensive spin locks for dma_op_mode
2009-10-27 14:00 ` Jarkko Nikula
@ 2009-10-28 5:52 ` Peter Ujfalusi
2009-10-28 6:53 ` Eero Nurkkala
0 siblings, 1 reply; 15+ messages in thread
From: Peter Ujfalusi @ 2009-10-28 5:52 UTC (permalink / raw)
To: ext Jarkko Nikula
Cc: alsa-devel, tony, broonie, Valentin Eduardo (Nokia-D/Helsinki),
Nurkkala Eero.An (EXT-Offcode/Oulu),
linux-omap
On Tuesday 27 October 2009 16:00:00 ext Jarkko Nikula wrote:
> On Tue, 27 Oct 2009 13:07:23 +0200
>
> Eero Nurkkala <ext-eero.nurkkala@nokia.com> wrote:
> > Otherwise, that spinlocking is highly unnecessary and things are
> > far better without than with it. The only case it could be useful
> > is in SMPs, but OMAPs are not such quite yet - and when they
> > are, things will need to be re-though anyway.
>
> Following commit is suggesting that mcbsp code *must* be SMP safe
> already now:
>
> commit a5b92cc348299c20be215b9f4b50853ecfbf3864
> Author: Syed Rafiuddin <rafiuddin.syed@ti.com>
> Date: Tue Jul 28 18:57:10 2009 +0530
>
> ARM: OMAP4: Add McBSP support
Yeah, but I think this locking issue has nothing to do with SMP safe or not.
On playback start in omap_mcbsp_request the mcbsp->free is cleared.
Further modification to the dma_op_mode in dma_op_mode_store is not allowed if
the mcbsp port is in use, thus the dma_op_mode is protected against change while
the port is in use (ensuring that the mode is same in omap34xx_mcbsp_request and
omap_mcbsp_get_dma_op_mode functions). This alone makes the use of spinlock
around the dma_op_mode unnecessary.
--
Péter
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH] OMAP: McBSP: Do not use extensive spin locks for dma_op_mode
2009-10-28 5:52 ` Peter Ujfalusi
@ 2009-10-28 6:53 ` Eero Nurkkala
2009-10-29 6:35 ` Peter Ujfalusi
2009-10-29 7:13 ` [alsa-devel] " Jarkko Nikula
0 siblings, 2 replies; 15+ messages in thread
From: Eero Nurkkala @ 2009-10-28 6:53 UTC (permalink / raw)
To: Ujfalusi Peter (Nokia-D/Tampere)
Cc: alsa-devel, tony, broonie, Valentin Eduardo (Nokia-D/Helsinki),
linux-omap
On Wed, 2009-10-28 at 06:52 +0100, Ujfalusi Peter (Nokia-D/Tampere)
wrote:
> On Tuesday 27 October 2009 16:00:00 ext Jarkko Nikula wrote:
> > On Tue, 27 Oct 2009 13:07:23 +0200
> >
> > Eero Nurkkala <ext-eero.nurkkala@nokia.com> wrote:
> > > Otherwise, that spinlocking is highly unnecessary and things are
> > > far better without than with it. The only case it could be useful
> > > is in SMPs, but OMAPs are not such quite yet - and when they
> > > are, things will need to be re-though anyway.
> >
> > Following commit is suggesting that mcbsp code *must* be SMP safe
> > already now:
> >
> > commit a5b92cc348299c20be215b9f4b50853ecfbf3864
> > Author: Syed Rafiuddin <rafiuddin.syed@ti.com>
> > Date: Tue Jul 28 18:57:10 2009 +0530
> >
> > ARM: OMAP4: Add McBSP support
>
> Yeah, but I think this locking issue has nothing to do with SMP safe or not.
> On playback start in omap_mcbsp_request the mcbsp->free is cleared.
> Further modification to the dma_op_mode in dma_op_mode_store is not allowed if
> the mcbsp port is in use, thus the dma_op_mode is protected against change while
> the port is in use (ensuring that the mode is same in omap34xx_mcbsp_request and
> omap_mcbsp_get_dma_op_mode functions). This alone makes the use of spinlock
> around the dma_op_mode unnecessary.
>
Yeah, maybe I took the SMP safeness into play without looking any code,
my bad =) I was remembering a different version of this McBSP thing, now
that I looked into it, it looked different.
Right, I reviewed the code, and it was first looking really bad at
sound/soc/omap/omap-mcbsp.c, where it calls omap_mcbsp_get_dma_op_mode()
from different places. However, it's not an issue because in:
arch/arm/plat-omap/mcbsp.c : dma_op_mode_store(),
the dma_op_mode is written only if the mcbsp is unoccupied. So it is SMP
safe.
..and a single read is always atomic, so this is buggy code:
301 spin_lock_irq(&mcbsp->lock);
302 dma_op_mode = mcbsp->dma_op_mode;
303 spin_unlock_irq(&mcbsp->lock);
304
305 return dma_op_mode;
The spinlocks are unnecessary. In the above example, you get the same
with just "return mcbsp->dma_op_mode;"
-> Peter's patch is a good cleanup.
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH] OMAP: McBSP: Do not use extensive spin locks for dma_op_mode
2009-10-28 6:53 ` Eero Nurkkala
@ 2009-10-29 6:35 ` Peter Ujfalusi
2009-10-29 7:15 ` [alsa-devel] " Jarkko Nikula
2009-10-29 7:13 ` [alsa-devel] " Jarkko Nikula
1 sibling, 1 reply; 15+ messages in thread
From: Peter Ujfalusi @ 2009-10-29 6:35 UTC (permalink / raw)
To: Nurkkala Eero.An (EXT-Offcode/Oulu)
Cc: alsa-devel, tony, broonie, Valentin Eduardo (Nokia-D/Helsinki),
linux-omap
On Wednesday 28 October 2009 08:53:34 Nurkkala Eero.An (EXT-Offcode/Oulu) wrote:
> Yeah, maybe I took the SMP safeness into play without looking any code,
> my bad =) I was remembering a different version of this McBSP thing, now
> that I looked into it, it looked different.
>
> Right, I reviewed the code, and it was first looking really bad at
> sound/soc/omap/omap-mcbsp.c, where it calls omap_mcbsp_get_dma_op_mode()
> from different places. However, it's not an issue because in:
> arch/arm/plat-omap/mcbsp.c : dma_op_mode_store(),
> the dma_op_mode is written only if the mcbsp is unoccupied. So it is SMP
> safe.
>
> ..and a single read is always atomic, so this is buggy code:
>
> 301 spin_lock_irq(&mcbsp->lock);
> 302 dma_op_mode = mcbsp->dma_op_mode;
> 303 spin_unlock_irq(&mcbsp->lock);
> 304
> 305 return dma_op_mode;
>
> The spinlocks are unnecessary. In the above example, you get the same
> with just "return mcbsp->dma_op_mode;"
>
> -> Peter's patch is a good cleanup.
Jarkko: are we going to take this?
--
Péter
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [alsa-devel] [PATCH] OMAP: McBSP: Do not use extensive spin locks for dma_op_mode
2009-10-28 6:53 ` Eero Nurkkala
2009-10-29 6:35 ` Peter Ujfalusi
@ 2009-10-29 7:13 ` Jarkko Nikula
1 sibling, 0 replies; 15+ messages in thread
From: Jarkko Nikula @ 2009-10-29 7:13 UTC (permalink / raw)
To: ext-eero.nurkkala
Cc: Ujfalusi Peter (Nokia-D/Tampere),
tony, alsa-devel, linux-omap, Valentin Eduardo (Nokia-D/Helsinki),
broonie
On Wed, 28 Oct 2009 08:53:34 +0200
Eero Nurkkala <ext-eero.nurkkala@nokia.com> wrote:
> On Wed, 2009-10-28 at 06:52 +0100, Ujfalusi Peter (Nokia-D/Tampere)
> > Yeah, but I think this locking issue has nothing to do with SMP safe or not.
> > On playback start in omap_mcbsp_request the mcbsp->free is cleared.
> > Further modification to the dma_op_mode in dma_op_mode_store is not allowed if
> > the mcbsp port is in use, thus the dma_op_mode is protected against change while
> > the port is in use (ensuring that the mode is same in omap34xx_mcbsp_request and
> > omap_mcbsp_get_dma_op_mode functions). This alone makes the use of spinlock
> > around the dma_op_mode unnecessary.
> >
>
...
> Right, I reviewed the code, and it was first looking really bad at
> sound/soc/omap/omap-mcbsp.c, where it calls omap_mcbsp_get_dma_op_mode()
> from different places. However, it's not an issue because in:
> arch/arm/plat-omap/mcbsp.c : dma_op_mode_store(),
> the dma_op_mode is written only if the mcbsp is unoccupied. So it is SMP
> safe.
>
...
> The spinlocks are unnecessary. In the above example, you get the same
> with just "return mcbsp->dma_op_mode;"
>
> -> Peter's patch is a good cleanup.
>
Yeah, agree, only now I looked the patch itself :-)
Sorry that I didn't express it clearly that my comment was a side note
for general awareness that SMP's are becoming reality for ARMs as well
than nak for Peter's patch.
According to defconfigs the OMAP4 will be the first commercial ARM SMP
having mainline support (I count realview as development device):
grep 'CONFIG_SMP=y' -r arch/arm/configs/
arch/arm/configs/realview-smp_defconfig:CONFIG_SMP=y
arch/arm/configs/omap_4430sdp_defconfig:CONFIG_SMP=y
--
Jarkko
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [alsa-devel] [PATCH] OMAP: McBSP: Do not use extensive spin locks for dma_op_mode
2009-10-29 6:35 ` Peter Ujfalusi
@ 2009-10-29 7:15 ` Jarkko Nikula
2009-11-09 7:49 ` Peter Ujfalusi
0 siblings, 1 reply; 15+ messages in thread
From: Jarkko Nikula @ 2009-10-29 7:15 UTC (permalink / raw)
To: Peter Ujfalusi
Cc: Nurkkala Eero.An (EXT-Offcode/Oulu),
tony, alsa-devel, linux-omap, Valentin Eduardo (Nokia-D/Helsinki),
broonie
On Thu, 29 Oct 2009 08:35:51 +0200
Peter Ujfalusi <peter.ujfalusi@nokia.com> wrote:
> > The spinlocks are unnecessary. In the above example, you get the same
> > with just "return mcbsp->dma_op_mode;"
> >
> > -> Peter's patch is a good cleanup.
>
> Jarkko: are we going to take this?
>
Yep, you can have my
Acked-by: Jarkko Nikula <jhnikula@gmail.com>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [alsa-devel] [PATCH] OMAP: McBSP: Do not use extensive spin locks for dma_op_mode
2009-10-29 7:15 ` [alsa-devel] " Jarkko Nikula
@ 2009-11-09 7:49 ` Peter Ujfalusi
2009-11-09 13:18 ` Mark Brown
0 siblings, 1 reply; 15+ messages in thread
From: Peter Ujfalusi @ 2009-11-09 7:49 UTC (permalink / raw)
To: ext Jarkko Nikula
Cc: Nurkkala Eero.An (EXT-Offcode/Oulu),
tony, alsa-devel, linux-omap, Valentin Eduardo (Nokia-D/Helsinki),
broonie
On Thursday 29 October 2009 09:15:24 ext Jarkko Nikula wrote:
> On Thu, 29 Oct 2009 08:35:51 +0200
>
> Peter Ujfalusi <peter.ujfalusi@nokia.com> wrote:
> > > The spinlocks are unnecessary. In the above example, you get the same
> > > with just "return mcbsp->dma_op_mode;"
> > >
> > > -> Peter's patch is a good cleanup.
> >
> > Jarkko: are we going to take this?
>
> Yep, you can have my
>
> Acked-by: Jarkko Nikula <jhnikula@gmail.com>
Just want to bring up this thread.
Should I resend the patch?
--
Péter
--
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] 15+ messages in thread
* Re: [PATCH] OMAP: McBSP: Do not use extensive spin locks for dma_op_mode
2009-11-09 7:49 ` Peter Ujfalusi
@ 2009-11-09 13:18 ` Mark Brown
0 siblings, 0 replies; 15+ messages in thread
From: Mark Brown @ 2009-11-09 13:18 UTC (permalink / raw)
To: Peter Ujfalusi
Cc: alsa-devel, tony, Valentin Eduardo (Nokia-D/Helsinki),
Nurkkala Eero.An (EXT-Offcode/Oulu),
linux-omap
On Mon, Nov 09, 2009 at 09:49:11AM +0200, Peter Ujfalusi wrote:
> Just want to bring up this thread.
> Should I resend the patch?
This is an arch/arm thing so I'd expect it to go via the OMAP tree.
^ permalink raw reply [flat|nested] 15+ messages in thread
* [APPLIED] [PATCH] OMAP: McBSP: Do not use extensive spin locks for dma_op_mode
2009-10-27 11:00 [PATCH] OMAP: McBSP: Do not use extensive spin locks for dma_op_mode Peter Ujfalusi
2009-10-27 11:07 ` Eero Nurkkala
@ 2009-11-12 22:17 ` Tony Lindgren
1 sibling, 0 replies; 15+ messages in thread
From: Tony Lindgren @ 2009-11-12 22:17 UTC (permalink / raw)
To: linux-omap
This patch has been applied to the linux-omap
by youw fwiendly patch wobot.
Branch in linux-omap: for-next
Initial commit ID (Likely to change): e1d8daf0013a8c6dd9f53dbde4c80f4723dee85f
PatchWorks
http://patchwork.kernel.org/patch/56075/
Git (Likely to change, and takes a while to get mirrored)
http://git.kernel.org/?p=linux/kernel/git/tmlind/linux-omap-2.6.git;a=commit;h=e1d8daf0013a8c6dd9f53dbde4c80f4723dee85f
^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2009-11-12 22:17 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-10-27 11:00 [PATCH] OMAP: McBSP: Do not use extensive spin locks for dma_op_mode Peter Ujfalusi
2009-10-27 11:07 ` Eero Nurkkala
2009-10-27 11:17 ` Peter Ujfalusi
2009-10-27 12:00 ` Mark Brown
2009-10-27 12:04 ` Eero Nurkkala
2009-10-27 12:04 ` [alsa-devel] " Peter Ujfalusi
2009-10-27 14:00 ` Jarkko Nikula
2009-10-28 5:52 ` Peter Ujfalusi
2009-10-28 6:53 ` Eero Nurkkala
2009-10-29 6:35 ` Peter Ujfalusi
2009-10-29 7:15 ` [alsa-devel] " Jarkko Nikula
2009-11-09 7:49 ` Peter Ujfalusi
2009-11-09 13:18 ` Mark Brown
2009-10-29 7:13 ` [alsa-devel] " Jarkko Nikula
2009-11-12 22:17 ` [APPLIED] " Tony Lindgren
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.