On Fri, Aug 26, 2011 at 3:18 PM, Dan Carpenter <error27@xxxxxxxxx> wrote: > It looks good to me, but I don't know anything about the iio code. > Jonathan will let you know if there is a problem. :) I'm hoping so. :) Thanks for reviewing. > > This is minor thing, but checkpatch complains that: > +#define ISL29018_MAX_REGS ISL29018_REG_ADD_DATA_MSB+1 > should be written as: > +#define ISL29018_MAX_REGS (ISL29018_REG_ADD_DATA_MSB + 1) ah...I didn't run check_patch. /o\ I'll include that in the version I send to gregkh. cheers, grant _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel