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 <rmk+kernel@xxxxxxxxxxxxxxxx> > Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx> > Cc: Julia Lawall <Julia.Lawall@xxxxxxx> > Cc: Ralf Baechle <ralf@xxxxxxxxxxxxxx> > Cc: linux-mips@xxxxxxxxxxxxxx > --- > 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; > } > > >
