[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