Re: [PATCH v4 2/2] dmaengine: add a driver for AMBA AXI NBPF DMAC IP cores

On Fri, Aug 01, 2014 at 11:04:52PM +0200, Guennadi Liakhovetski wrote:

Hi Vinod,

On Fri, 1 Aug 2014, Vinod Koul wrote:

quoted

On Thu, Jul 31, 2014 at 03:42:44PM +0200, Guennadi Liakhovetski wrote:

quoted

Ok. This doesn't seem to progress. You suggested in your previous mail, 
that I can provide an incremental patch to add devm_free(_threaded)_irq() 
and synchronize_irq() to .release(). I still think, that this doesn't make 
much sense, but if you insist... Alternatively, my proposal would be to 
fix the thoretical race in .free_chan_resources(). Would you accept such a 
patch instead of what you were proposing? I can also convert the driver to 
using a tasklet, if required. Would it suffice, if I do those patches as 
fixes after -rc1? I'm leaving on a holiday tomorrow and I still have to 
finalise some other work, so, preparing those patches today doesn't seem 
realistic to me... Would this be acceptable?

Can you please send both, incrementally is fine.
I will merge them tomorrow

Sorry, do you mean I should send those incremental patches for the 
original one to become acceptable? As I explained above, I'm on a holiday 
now, so, I hoped it would be acceptble to get this patch in and let me do 
the other two later, possibly after -rc1 as fixes. If you definitely would 
like all 3 patches now... Well, I could try, but this won't be easy...

Well I was planning to merge and wait for fixes in rc1 or later during merge
window. Looks like you managed to send, so I will merge all.

Thanks

-- 
~Vinod

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html