Re: [PATCH 14/32] dmaengine: ste_dma40: Remove 'always true' checking

2013-04-25 Thread Lee Jones
On Thu, 25 Apr 2013, Linus Walleij wrote: > On Thu, Apr 18, 2013 at 12:11 PM, Lee Jones wrote: > > > Before allocating memory for logical channels, the current driver > > checks to see if there are any. Well as the dma40_memcpy_channels[] > > array is always populated, the check is always true.

Re: [PATCH 14/32] dmaengine: ste_dma40: Remove 'always true' checking

2013-04-25 Thread Linus Walleij
On Thu, Apr 18, 2013 at 12:11 PM, Lee Jones wrote: > Before allocating memory for logical channels, the current driver > checks to see if there are any. Well as the dma40_memcpy_channels[] > array is always populated, the check is always true. Best to > remove the check. > > Cc: Vinod Koul >

Re: [PATCH 14/32] dmaengine: ste_dma40: Remove 'always true' checking

2013-04-25 Thread Linus Walleij
On Thu, Apr 18, 2013 at 12:11 PM, Lee Jones lee.jo...@linaro.org wrote: Before allocating memory for logical channels, the current driver checks to see if there are any. Well as the dma40_memcpy_channels[] array is always populated, the check is always true. Best to remove the check. Cc:

Re: [PATCH 14/32] dmaengine: ste_dma40: Remove 'always true' checking

2013-04-25 Thread Lee Jones
On Thu, 25 Apr 2013, Linus Walleij wrote: On Thu, Apr 18, 2013 at 12:11 PM, Lee Jones lee.jo...@linaro.org wrote: Before allocating memory for logical channels, the current driver checks to see if there are any. Well as the dma40_memcpy_channels[] array is always populated, the check is

Re: [PATCH 14/32] dmaengine: ste_dma40: Remove 'always true' checking

2013-04-22 Thread Vinod Koul
On Thu, Apr 18, 2013 at 11:11:56AM +0100, Lee Jones wrote: > Before allocating memory for logical channels, the current driver > checks to see if there are any. Well as the dma40_memcpy_channels[] > array is always populated, the check is always true. Best to > remove the check. > > Cc: Vinod

Re: [PATCH 14/32] dmaengine: ste_dma40: Remove 'always true' checking

2013-04-22 Thread Vinod Koul
On Thu, Apr 18, 2013 at 11:11:56AM +0100, Lee Jones wrote: Before allocating memory for logical channels, the current driver checks to see if there are any. Well as the dma40_memcpy_channels[] array is always populated, the check is always true. Best to remove the check. Cc: Vinod Koul

Re: [PATCH 14/32] dmaengine: ste_dma40: Remove 'always true' checking

2013-04-18 Thread Arnd Bergmann
On Thursday 18 April 2013, Lee Jones wrote: > Before allocating memory for logical channels, the current driver > checks to see if there are any. Well as the dma40_memcpy_channels[] > array is always populated, the check is always true. Best to > remove the check. > > Cc: Vinod Koul > Cc: Dan

[PATCH 14/32] dmaengine: ste_dma40: Remove 'always true' checking

2013-04-18 Thread Lee Jones
Before allocating memory for logical channels, the current driver checks to see if there are any. Well as the dma40_memcpy_channels[] array is always populated, the check is always true. Best to remove the check. Cc: Vinod Koul Cc: Dan Williams Cc: Per Forlin Cc: Rabin Vincent Signed-off-by:

[PATCH 14/32] dmaengine: ste_dma40: Remove 'always true' checking

2013-04-18 Thread Lee Jones
Before allocating memory for logical channels, the current driver checks to see if there are any. Well as the dma40_memcpy_channels[] array is always populated, the check is always true. Best to remove the check. Cc: Vinod Koul vinod.k...@intel.com Cc: Dan Williams d...@fb.com Cc: Per Forlin

Re: [PATCH 14/32] dmaengine: ste_dma40: Remove 'always true' checking

2013-04-18 Thread Arnd Bergmann
On Thursday 18 April 2013, Lee Jones wrote: Before allocating memory for logical channels, the current driver checks to see if there are any. Well as the dma40_memcpy_channels[] array is always populated, the check is always true. Best to remove the check. Cc: Vinod Koul