[PATCH] powerpc: fix compile fail in hugetlb cmdline parsing
Benjamin Herrenschmidt
benh at kernel.crashing.org
Wed May 9 13:04:18 EST 2012
On Tue, 2012-05-08 at 13:53 +0930, Rusty Russell wrote:
> On Mon, 7 May 2012 10:32:22 -0400, Paul Gortmaker <paul.gortmaker at windriver.com> 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 at rustcorp.com.au>
> > Cc: Jim Cromie <jim.cromie at gmail.com>
> > Cc: Jason Baron <jbaron at redhat.com>
> > Cc: Greg Kroah-Hartman <gregkh at linuxfoundation.org>
> > Cc: Becky Bruce <beckyb at kernel.crashing.org>
> > Cc: Benjamin Herrenschmidt <benh at kernel.crashing.org>
> > Signed-off-by: Paul Gortmaker <paul.gortmaker at windriver.com>
>
> Acked-by: Rusty Russell <rusty at rustcorp.com.au>
This is for -next right ? IE. I don't seem to have the faulty commit
in my tree so I can't carry the fix for now.... or am I missing
something ?
Cheers,
Ben.
More information about the Linuxppc-dev
mailing list