Re: [PATCH 6/7] DMA: Freescale: use spin_lock_bh instead of spin_lock_irqsave

2014-04-03 Thread Hongbo Zhang
On 04/03/2014 12:35 AM, Vinod Koul wrote: On Mon, Mar 31, 2014 at 12:08:55PM +0800, Hongbo Zhang wrote: On 03/29/2014 09:45 PM, Vinod Koul wrote: On Fri, Mar 28, 2014 at 02:33:37PM +0800, Hongbo Zhang wrote: On 03/26/2014 03:01 PM, Vinod Koul wrote: On Thu, 2014-01-16 at 13:47 +0800,

Re: [PATCH 6/7] DMA: Freescale: use spin_lock_bh instead of spin_lock_irqsave

2014-04-03 Thread Hongbo Zhang
On 04/03/2014 12:35 AM, Vinod Koul wrote: On Mon, Mar 31, 2014 at 12:08:55PM +0800, Hongbo Zhang wrote: On 03/29/2014 09:45 PM, Vinod Koul wrote: On Fri, Mar 28, 2014 at 02:33:37PM +0800, Hongbo Zhang wrote: On 03/26/2014 03:01 PM, Vinod Koul wrote: On Thu, 2014-01-16 at 13:47 +0800,

Re: [PATCH 6/7] DMA: Freescale: use spin_lock_bh instead of spin_lock_irqsave

2014-04-02 Thread Vinod Koul
On Mon, Mar 31, 2014 at 12:08:55PM +0800, Hongbo Zhang wrote: > > On 03/29/2014 09:45 PM, Vinod Koul wrote: > >On Fri, Mar 28, 2014 at 02:33:37PM +0800, Hongbo Zhang wrote: > >>On 03/26/2014 03:01 PM, Vinod Koul wrote: > >>>On Thu, 2014-01-16 at 13:47 +0800, hongbo.zh...@freescale.com wrote: >

Re: [PATCH 6/7] DMA: Freescale: use spin_lock_bh instead of spin_lock_irqsave

2014-04-02 Thread Vinod Koul
On Mon, Mar 31, 2014 at 12:08:55PM +0800, Hongbo Zhang wrote: On 03/29/2014 09:45 PM, Vinod Koul wrote: On Fri, Mar 28, 2014 at 02:33:37PM +0800, Hongbo Zhang wrote: On 03/26/2014 03:01 PM, Vinod Koul wrote: On Thu, 2014-01-16 at 13:47 +0800, hongbo.zh...@freescale.com wrote: From: Hongbo

Re: [PATCH 6/7] DMA: Freescale: use spin_lock_bh instead of spin_lock_irqsave

2014-03-30 Thread Hongbo Zhang
On 03/29/2014 09:45 PM, Vinod Koul wrote: On Fri, Mar 28, 2014 at 02:33:37PM +0800, Hongbo Zhang wrote: On 03/26/2014 03:01 PM, Vinod Koul wrote: On Thu, 2014-01-16 at 13:47 +0800, hongbo.zh...@freescale.com wrote: From: Hongbo Zhang The usage of spin_lock_irqsave() is a stronger locking

Re: [PATCH 6/7] DMA: Freescale: use spin_lock_bh instead of spin_lock_irqsave

2014-03-30 Thread Hongbo Zhang
On 03/29/2014 09:45 PM, Vinod Koul wrote: On Fri, Mar 28, 2014 at 02:33:37PM +0800, Hongbo Zhang wrote: On 03/26/2014 03:01 PM, Vinod Koul wrote: On Thu, 2014-01-16 at 13:47 +0800, hongbo.zh...@freescale.com wrote: From: Hongbo Zhang hongbo.zh...@freescale.com The usage of

Re: [PATCH 6/7] DMA: Freescale: use spin_lock_bh instead of spin_lock_irqsave

2014-03-29 Thread Vinod Koul
On Fri, Mar 28, 2014 at 02:33:37PM +0800, Hongbo Zhang wrote: > > On 03/26/2014 03:01 PM, Vinod Koul wrote: > >On Thu, 2014-01-16 at 13:47 +0800, hongbo.zh...@freescale.com wrote: > >>From: Hongbo Zhang > >> > >>The usage of spin_lock_irqsave() is a stronger locking mechanism than is >

Re: [PATCH 6/7] DMA: Freescale: use spin_lock_bh instead of spin_lock_irqsave

2014-03-29 Thread Vinod Koul
On Fri, Mar 28, 2014 at 02:33:37PM +0800, Hongbo Zhang wrote: On 03/26/2014 03:01 PM, Vinod Koul wrote: On Thu, 2014-01-16 at 13:47 +0800, hongbo.zh...@freescale.com wrote: From: Hongbo Zhang hongbo.zh...@freescale.com The usage of spin_lock_irqsave() is a stronger locking mechanism than

Re: [PATCH 6/7] DMA: Freescale: use spin_lock_bh instead of spin_lock_irqsave

2014-03-28 Thread Hongbo Zhang
On 03/26/2014 03:01 PM, Vinod Koul wrote: On Thu, 2014-01-16 at 13:47 +0800, hongbo.zh...@freescale.com wrote: From: Hongbo Zhang The usage of spin_lock_irqsave() is a stronger locking mechanism than is required throughout the driver. The minimum locking required should be used instead.

Re: [PATCH 6/7] DMA: Freescale: use spin_lock_bh instead of spin_lock_irqsave

2014-03-28 Thread Hongbo Zhang
On 03/26/2014 03:01 PM, Vinod Koul wrote: On Thu, 2014-01-16 at 13:47 +0800, hongbo.zh...@freescale.com wrote: From: Hongbo Zhang hongbo.zh...@freescale.com The usage of spin_lock_irqsave() is a stronger locking mechanism than is required throughout the driver. The minimum locking required

Re: [PATCH 6/7] DMA: Freescale: use spin_lock_bh instead of spin_lock_irqsave

2014-03-26 Thread Vinod Koul
On Thu, 2014-01-16 at 13:47 +0800, hongbo.zh...@freescale.com wrote: > From: Hongbo Zhang > > The usage of spin_lock_irqsave() is a stronger locking mechanism than is > required throughout the driver. The minimum locking required should be used > instead. Interrupts will be turned off and

Re: [PATCH 6/7] DMA: Freescale: use spin_lock_bh instead of spin_lock_irqsave

2014-03-26 Thread Vinod Koul
On Thu, 2014-01-16 at 13:47 +0800, hongbo.zh...@freescale.com wrote: From: Hongbo Zhang hongbo.zh...@freescale.com The usage of spin_lock_irqsave() is a stronger locking mechanism than is required throughout the driver. The minimum locking required should be used instead. Interrupts will be

[PATCH 6/7] DMA: Freescale: use spin_lock_bh instead of spin_lock_irqsave

2014-01-15 Thread hongbo.zhang
From: Hongbo Zhang The usage of spin_lock_irqsave() is a stronger locking mechanism than is required throughout the driver. The minimum locking required should be used instead. Interrupts will be turned off and context will be saved, it is unnecessary to use irqsave. This patch changes all

[PATCH 6/7] DMA: Freescale: use spin_lock_bh instead of spin_lock_irqsave

2014-01-15 Thread hongbo.zhang
From: Hongbo Zhang hongbo.zh...@freescale.com The usage of spin_lock_irqsave() is a stronger locking mechanism than is required throughout the driver. The minimum locking required should be used instead. Interrupts will be turned off and context will be saved, it is unnecessary to use irqsave.