[PATCH] i2c: designware: do not show SDA hold time warning when not needed
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Sat Jun 3 06:18:21 PDT 2017
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 at linux.intel.com>
>
> 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
More information about the linux-arm-kernel
mailing list