On Mon, Jul 18, 2011 at 05:10, Jiang, Scott wrote: >>> + int (*setup)(struct i2c_client *client, >>> + unsigned state); >> >> probably want to put a forward i2c_client struct decl at the top of >> this file after the includes to avoid warnings here. > > Why not include <linux/i2c.h> directly?
for platform resources (like this header) that dont need to know about i2c, typically people put a forward decl rather than including the header. the only thing people care about at this point is that it takes a func pointer. -mike _______________________________________________ Linux-kernel-commits mailing list [email protected] https://blackfin.uclinux.org/mailman/listinfo/linux-kernel-commits
