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 - 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/