[alsa-devel] [PATCH 01/11] dma: imx-sdma: make channel0 operations atomic
shijie8 at gmail.com
Fri Apr 27 17:27:16 CEST 2012
On Fri, Apr 27, 2012 at 11:13 AM, Lothar Waßmann <LW at karo-electronics.de> wrote:
> Richard Zhao writes:
>> On Fri, Apr 27, 2012 at 11:18:31AM +0200, Lothar Waßmann wrote:
>> > Hi,
>> > Richard Zhao writes:
>> > > device_prep_dma_cyclic may be call in audio trigger function which is
>> > > atomic context, so we make it atomic too.
>> > >
>> > > - change channel0 lock to spinlock.
>> > > - Use polling to wait for channel0 finish running.
>> > >
>> > > Signed-off-by: Lothar Waßmann <LW at KARO-electronics.de>
>> > >
>> > Actually I didn't sign off the patch that I posted, because I wanted
>> > to wait for more comments first.
>> I send it out with slight modifications because the series highly
>> depend on it. Will you take it over or let me put it in next version?
>> Both are ok to me.
> I think you should keep it as part of your sound patches and I will
> test your final version on our hardware.
I hope we can get a conclusion that the prep_slave_sg() can be called
in atomic context or not.
My patch "add DMA support to UART" heavily depends on it.
> Lothar Waßmann
> Ka-Ro electronics GmbH | Pascalstraße 22 | D - 52076 Aachen
> Phone: +49 2408 1402-0 | Fax: +49 2408 1402-10
> Geschäftsführer: Matthias Kaussen
> Handelsregistereintrag: Amtsgericht Aachen, HRB 4996
> www.karo-electronics.de | info at karo-electronics.de
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
More information about the Alsa-devel