On Sun, Feb 20, 2005 at 07:40:24AM -0800, Mickey Stein wrote:
> Greg KH wrote:
>
> >On Sat, Feb 19, 2005 at 08:58:48AM -0800, Mickey Stein wrote:
> >
> >
> >>From: Mickey Stein
> >>Versions: linux-2.6.11-rc4-bk7, gcc4 (GCC) 4.0.0 20050217 (latest fc
> >>rawhide from 19Feb DL)
> >>
> >>gcc4 cvs seems to dislike "include/linux/i2c.h file":
> >>
> >>Error msg: include/linux/i2c.h:{55,194} error: array type has
> >>incomplete element type
> >>
> >>A. Daplas has recently done a workaround for this on another header
> >>file. A thread discussing this
> >>can be found by following the link below:
> >>
> >>http://gcc.gnu.org/ml/gcc/2005-02/msg00053.html
> >>
> >>The patch changes the array declaration from "struct x y[]" format to
> >>"struct x *y".
> >>I realize its only a workaround, but the gcc guys seem to be aware of
> >>this.
> >>** Note: I'm a noob at this, so feel free to make chopped liver out of
> >>this if its incorrect.
> >>patch below is also attached since I'm not sure formatting survives
> >>the cut&paste.
> >>
> >>
> >
> >The patch looks sane, but before I apply it, care to also fix up all of
> >the function pointers that are affected by this patch? Also the
> >i2c_transfer() function itself should be changed (not just the header
> >file.)
> >
> >thanks,
> >
> >greg k-h
> >
> >
> >
> Greg,
>
> I took a look for other references similar to those in my first take at
> this and found a couple more files.
> Attached is another patch that hopefully addresses the all the similar
> cases. I tried it on today's (-bk8) kernel,
> with all i2c switches enabled.
What about the i2c drivers that implement the master_xfer function? You
will now have a bunch of compiler warnings if you build them with this
patch, right?
thanks,
greg k-h
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
