Vinod Koul <vinod.koul@xxxxxxxxx> writes: > On Tue, Jul 05, 2016 at 08:24:06PM +0530, Vinod Koul wrote: >> This series attempts to do few cleanup for drivers. My aim was to fix irq >> cleanup but on the way found few more things to be fixed as well. >> o explicitly freeup irq: Many drivers nowadays use devm variant for >> requesting irq, which is fine from cleanup POV, but on remove you leave irq >> enabled and able to fire. This can lead to nasty races, so better idea is >> to explicitly freeup. >> o same situation is with tasklets. They need to be killed before we exit, >> so that has been fixed too >> o while at it noticed few drivers still set .owner, removed them as well. >> o also fixed few sparse complaints > > Merging this now with the ACKs and changes as discussed Hi Vinod, I thought that we agreed that commit "dmaengine: pxa_dma: implement device_synchronize" superseeded "dmaengine: pxa_dm: explicitly freeup irq" which wasn't necessary anymore. Yet I see it in your next tree. Can you tell me what is happening ? Cheers. -- Robert -- 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
