Messages in this thread | | | Date | Wed, 11 May 2011 06:48:59 -0700 | From | Greg KH <> | Subject | Re: [PATCH v6] staging: Add ST-Ericsson CG2900 driver |
| |
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 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |