Re: [PATCH] DMA: shdma: fix a bad merge - remove free_irq()

2013-08-28 Thread Guennadi Liakhovetski
Hi Vinod, On Wed, 28 Aug 2013, Vinod Koul wrote: > On Tue, Aug 27, 2013 at 04:46:51PM +0200, Guennadi Liakhovetski wrote: > > Ok, to me it looks like you did fix some patches in the topic/sh branch, > > but: > > > > patch "DMA: shdma: move two macros to a header" is missing > > patch "dma: use

Re: [PATCH] DMA: shdma: fix a bad merge - remove free_irq()

2013-08-27 Thread Vinod Koul
On Tue, Aug 27, 2013 at 04:46:51PM +0200, Guennadi Liakhovetski wrote: > Ok, to me it looks like you did fix some patches in the topic/sh branch, > but: > > patch "DMA: shdma: move two macros to a header" is missing > patch "dma: use dev_get_platdata()" is missing (but ok, it's common for > driv

Re: [PATCH] DMA: shdma: fix a bad merge - remove free_irq()

2013-08-27 Thread Guennadi Liakhovetski
On Tue, 27 Aug 2013, Vinod Koul wrote: > On Tue, Aug 27, 2013 at 11:21:00AM +0200, Guennadi Liakhovetski wrote: > > Hi Vinod, > > > > On Mon, 26 Aug 2013, Vinod Koul wrote: > > > > > On Mon, Aug 26, 2013 at 02:27:51PM +0200, Guennadi Liakhovetski wrote: > > > > A bad merge resulted in a left-ove

Re: [PATCH] DMA: shdma: fix a bad merge - remove free_irq()

2013-08-27 Thread Vinod Koul
On Tue, Aug 27, 2013 at 11:21:00AM +0200, Guennadi Liakhovetski wrote: > Hi Vinod, > > On Mon, 26 Aug 2013, Vinod Koul wrote: > > > On Mon, Aug 26, 2013 at 02:27:51PM +0200, Guennadi Liakhovetski wrote: > > > A bad merge resulted in a left-over free_irq() call. This patch removes > > > it. > > A

Re: [PATCH] DMA: shdma: fix a bad merge - remove free_irq()

2013-08-27 Thread Guennadi Liakhovetski
Hi Vinod, On Mon, 26 Aug 2013, Vinod Koul wrote: > On Mon, Aug 26, 2013 at 02:27:51PM +0200, Guennadi Liakhovetski wrote: > > A bad merge resulted in a left-over free_irq() call. This patch removes it. > Applied, thanks > > Can you also test this branch and see if anything is remaining > > And

Re: [PATCH] DMA: shdma: fix a bad merge - remove free_irq()

2013-08-26 Thread Vinod Koul
On Mon, Aug 26, 2013 at 02:27:51PM +0200, Guennadi Liakhovetski wrote: > A bad merge resulted in a left-over free_irq() call. This patch removes it. Applied, thanks Can you also test this branch and see if anything is remaining And can you take care then ARM patches in last serries are merged thi

[PATCH] DMA: shdma: fix a bad merge - remove free_irq()

2013-08-26 Thread Guennadi Liakhovetski
A bad merge resulted in a left-over free_irq() call. This patch removes it. Signed-off-by: Guennadi Liakhovetski --- Against today's dma-slave next branch. drivers/dma/sh/shdmac.c |5 - 1 files changed, 0 insertions(+), 5 deletions(-) diff --git a/drivers/dma/sh/shdmac.c b/drivers/dma