On Fri, Aug 01, 2014 at 11:04:52PM +0200, Guennadi Liakhovetski wrote: > 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... 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 dmaengine" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
