[U-Boot] [PATCH 3/3] mpc5200: make i2c faster
Wolfgang Denk
wd at denx.de
Mon Mar 23 09:24:36 CET 2009
Dear Jon,
In message <9e4733910903221959n452a067kd46b313b9b453ec3 at mail.gmail.com> you wrote:
>
> > You did not answer my question in which way this actually makes I2C
> > faster? Where do we save time, and how much?
>
> I increased the retry loop to 10,000. This definitely makes my system
> faster. On my bus the actual i2c delay is 40-55us. The original code
> always delayed 1,000us so for me it a gain of 940us on each i2c
> operation. This is visible during things like probe.
Hm... you don't really convince me. Being "visible" is nice, but is
it a reason to change the code? I mean, if you said something like:
"without that patch operation FOO takes 20 seconds, and with the
patch it takes less than 3 seconds", then everybody can understand
why this is a good thing to do.
And: the I2C probe operation itself is just a debug thing, nothing
you do in normal operation, so it doesn't look that time critical to
me.
I agree that the change itself looks harmless, and I'm willing to
accept it, but at least make sure not to reduce the total timeout
values - that would require re-testing on some boards.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"Deliver yesterday, code today, think tomorrow."
More information about the U-Boot
mailing list