On Mon, 13 Jul 2015, Julia Lawall wrote: > On Mon, 13 Jul 2015, Thomas Gleixner wrote: > > > Chained irq handlers usually set up handler data as well. We now have > > a function to set both under irq_desc->lock. Replace the two calls > > with one. > > Are the original calls remaining? If so, should there be a semantic patch > in the kernel to check for this, in case people ut the two calls in teh > future. irq_set_handler_data() can be used in a different context as well. irq_set_chained_handler() has to stay for now, but we probably can replace it with irq_set_chained_handler_and_data(irq, handler, NULL). Have not yet done the analysis. But yes, a semantic check for this would be nice. Thanks, tglx
