Hi Vinod, On Fri, 1 Aug 2014, Vinod Koul wrote: > On Thu, Jul 31, 2014 at 03:42:44PM +0200, Guennadi Liakhovetski wrote: > > 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... Thanks Guennadi -- To unsubscribe from this list: send the line "unsubscribe dmaengine" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html