Re: [PATCH v6] staging: Add ST-Ericsson CG2900 driver

[Greg KH <greg@kroah.com>]

On Wed, May 11, 2011 at 10:34:47AM +0200, Par-Gunnar HJALMDAHL wrote:
> > -----Original Message-----
> > From: Greg KH [mailto:greg@kroah.com]
> > Sent: den 10 maj 2011 22:08
> > 
> > {sigh}
> > 
> > This breaks the build again, the error messages are below :(
> > 
> > I'm about to give up here, so the next time you send this patch in, I
> > want some kind of proof that the code will at least build properly.
> > 
> > How you provide this is up to you.
> > 
> > Oh, and if the next one breaks stuff again, you will owe me a lot of
> > beer, although I think you owe me already...
> > 
> > thanks,
> > 
> > greg k-h
> > 
> >   CC [M]  drivers/staging/cg2900/mfd/cg2900_chip.o
> > drivers/staging/cg2900/mfd/cg2900_chip.c:2940:3: error: unknown field
> > ‘platform_data’ specified in initializer
> > drivers/staging/cg2900/mfd/cg2900_chip.c:2940:3: warning:
> > initialization makes integer from pointer without a cast
> > drivers/staging/cg2900/mfd/cg2900_chip.c:2940:3: error: initializer
> > element is not computable at load time
> > drivers/staging/cg2900/mfd/cg2900_chip.c:2940:3: error: (near
> > initialization for ‘cg2900_devs[0].id’)
> 
> Hi Greg,
> 
> What linux-next tag did you base the patch on?

My staging git tree.  I can't apply to linux-next, only my tree,
linux-next then picks it up.

> I used next-20110510 and in that tag the file include/Linux/mfd/core.h
> The struct mfd_cell has a parameter called platform_data, while in your
> build it seems it doesn't.
> This was changed very recently in the core.h file (on April 6).
> So for core.h the last commit ID I have in the tag I built upon is
> 558aa9c0d620b91a896d5e39ce8a570bed65589b.

So do I need that patch on the staging-next tree also?

Or is it an issue of building this on x86-64...

thanks,

greg k-h