On Sun, 11 May 2008, Jean Delvare wrote: > > > + if (!i2c_check_functionality(adapter, I2C_FUNC_SMBUS_BYTE_DATA | > > > + I2C_FUNC_SMBUS_WORD_DATA)) > > > > Indentation (obviously it is the driver you are referring to that has got > > it wrong in the first place). > > There's nothing wrong with indentation here. Alignment on opening > parenthesis is a common strategy, and checkpatch.pl accepts it.
Well, it suggests I2C_FUNC_SMBUS_WORD_DATA is an argument on its own. The call is short here though, so it may not matter in practice. Maciej _______________________________________________ i2c mailing list [email protected] http://lists.lm-sensors.org/mailman/listinfo/i2c
