On Thu, Oct 29, 2009 at 04:34:54PM +0000, Martyn Welch wrote: > -int ca91cx42_request_irq(int level, int statid, > - void (*callback)(int level, int vector, void *priv_data), > - void *priv_data) > +int ca91cx42_irq_set(int level, int state, int sync) This needs to return a void, otherwise you have a compiler warning telling you you messed up :( I fixed it up in the patch. Please be careful about adding new warnings. I know there are 2 in this file already, which I'll go resolve now as well. thanks, greg k-h _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel