]> git.baikalelectronics.ru Git - kernel.git/commit
potential compiler error, irqfunc caller sites update
authorYoann Padioleau <padator@wanadoo.fr>
Fri, 6 Jul 2007 09:39:56 +0000 (02:39 -0700)
committerLinus Torvalds <torvalds@woody.linux-foundation.org>
Fri, 6 Jul 2007 17:23:43 +0000 (10:23 -0700)
commit1bdf174d0f9510e3e2ebb1fe84a588a03113aa7c
tree339a17b19a0a78d007f32182374476974a018805
parent9e4e27d427310c5287f86155e0ed4f0ca4b00e64
potential compiler error, irqfunc caller sites update

In 1ab7fb42f22da0ac82bfc241c5b05cbe4caf46fe David Howells performed
this evolution:
 "IRQ: Maintain regs pointer globally rather than passing to IRQ handlers"

He correctly updated many of the function definitions that were using this
extra regs pointer parameter but forgot to update some caller sites of
those functions.  The reason the modifications was not properly done on all
drivers is that some drivers were rarely compiled because they are for
AMIGA, or that some code sites were inside #ifdefs where the option is not
set or inside #if 0.

Here is the semantic patch that found the occurences
and fixed the problem.

@ rule1 @
identifier fn;
identifier irq, dev_id;
typedef irqreturn_t;
@@

static irqreturn_t fn(int irq, void *dev_id)
{
   ...
}

@@
identifier rule1.fn;
expression E1, E2, E3;
@@

 fn(E1, E2
-   ,E3
   )

Signed-off-by: Yoann Padioleau <padator@wanadoo.fr>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Jeff Garzik <jeff@garzik.org>
Cc: Greg KH <greg@kroah.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/atm/firestream.c
drivers/net/arm/am79c961a.c
drivers/net/ixp2000/ixpdev.c
drivers/net/sb1250-mac.c
drivers/usb/misc/uss720.c