On 12/03/2015 10:52 PM, Wolfram Sang wrote:
On Wed, Dec 02, 2015 at 01:02:34PM +0200, Jarkko Nikula wrote:
I2C adapter drivers that are using 1 second timeout can leave the timeout
unset and let the i2c-core.c: i2c_register_adapter() to set it instead.
Signed-off-by: Jarkko Nikula <jarkko.nikula@xxxxxxxxxxxxxxx>
My take on this is that I prefer the explicit (and thus visible)
assignment. What the core does was intended as a fallback to prevent
strange things happening with a 0 value.
Fair enough. It's hard to say if some of these really needs to be 1
second instead of being stetson guessed or copy-pasted. Now if we ever
change the fall back value from HZ to HZ/10 or similar we might get a
regression.
--
Jarkko
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
