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. julia > > Search and conversion was done with coccinelle. > > Reported-by: Russell King <[email protected]> > Signed-off-by: Thomas Gleixner <[email protected]> > Cc: Julia Lawall <[email protected]> > Cc: Ralf Baechle <[email protected]> > Cc: [email protected] > --- > arch/mips/pci/pci-rt3883.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > Index: tip/arch/mips/pci/pci-rt3883.c > =================================================================== > --- tip.orig/arch/mips/pci/pci-rt3883.c > +++ tip/arch/mips/pci/pci-rt3883.c > @@ -225,8 +225,7 @@ static int rt3883_pci_irq_init(struct de > return -ENODEV; > } > > - irq_set_handler_data(irq, rpc); > - irq_set_chained_handler(irq, rt3883_pci_irq_handler); > + irq_set_chained_handler_and_data(irq, rt3883_pci_irq_handler, rpc); > > return 0; > } > > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
