On Mon, 7 May 2012 10:32:22 -0400, Paul Gortmaker <paul.gortmaker@xxxxxxxxxxxxx> wrote: > Commit 9fb48c744ba6a4bf58b666f4e6fdac3008ea1bd4 > > "params: add 3rd arg to option handler callback signature" > > added an extra arg to the function, but didn't catch all the use > cases needing it, causing this compile fail in mpc85xx_defconfig: > > arch/powerpc/mm/hugetlbpage.c:316:4: error: passing argument 7 of > 'parse_args' from incompatible pointer type [-Werror] > > include/linux/moduleparam.h:317:12: note: expected > 'int (*)(char *, char *, const char *)' but argument is of type > 'int (*)(char *, char *)' > > This function has no need to printk out the "doing" value, so > just add the arg as an "unused". > > Cc: Rusty Russell <rusty@xxxxxxxxxxxxxxx> > Cc: Jim Cromie <jim.cromie@xxxxxxxxx> > Cc: Jason Baron <jbaron@xxxxxxxxxx> > Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > Cc: Becky Bruce <beckyb@xxxxxxxxxxxxxxxxxxx> > Cc: Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx> > Signed-off-by: Paul Gortmaker <paul.gortmaker@xxxxxxxxxxxxx> Acked-by: Rusty Russell <rusty@xxxxxxxxxxxxxxx> Thanks, Rusty. -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
