Hello, On Fri, 2 Jun 2017 22:15:02 +0200, Wolfram Sang wrote: > > hardware that doesn't support SDA hold time. But needless spamming have > > negative value so this is worth to fix. (I would do this as a single liner > > by else if ()). > > > > Acked-by: Jarkko Nikula <jarkko.nikula@xxxxxxxxxxxxxxx> > > I agree with Jarkko on everything, including the 'else if' fix. I am not sure to understand what Jarkko suggested to fix, besides using "else if" in one line. Could you be more specific about the changes you expect me to do? Andy suggested to turn the warning into a debug message, to which I objected, and my understanding was that Jarkko agreed with me that a warning message was better, and only made the suggestion of the "else if". Thanks, Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com -- 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
