[U-Boot] [PATCH v2 0/4] ARMV7: OMAP: I2C driver: Restructure code to eliminate udelay calls and improve performance
Heiko Schocher
hs at denx.de
Wed Oct 20 15:33:42 CEST 2010
Hello Steve,
Steve Sakoman wrote:
> I've been preparing a patch series for Beagle and Overo that detects expansion
> board configuration information by reading a 128 byte I2C EEPROM on each
> expansion board.
>
> Using the OMAP I2C driver in its current form takes about 5-6 seconds to read
> this small number of bytes! Executing the i2c probe command takes close
> to 10 seconds.
>
> Examining the code I see that there are a large number of fairly long udelay calls
> throughout the driver (10 - 50 milliseconds). I looked through the linux driver
> and did not see equivalent delays in that code. In fact the longest delay in the
> linux code was one millisecond.
>
> In looking at the TRM I2C section for OMAP3 and OMAP4 I don't see any requirement
> for delays in the programming model description.
>
> This patch restructures the i2c driver to eliminate most of the udelay calls
> by monitoring state changes in the status register.
>
> The EEPROM reads and the i2c probe execution are now instantaneous.
>
> This patch series was tested on OMAP3 (Overo) and OMAP4 (Panda). I do not
> have access to OMAP2 hardware for testing.
>
> Steve Sakoman (4):
> ARMV7: OMAP: I2C driver: Use same timeout value as linux kernel
> driver
> ARMV7: OMAP: I2C driver: Restructure i2c_read_byte function
> ARMV7: OMAP: I2C driver: Restructure i2c_write_byte function
> ARMV7: OMAP: I2C driver: Restructure i2c_probe function
>
> drivers/i2c/omap24xx_i2c.c | 204 ++++++++++++++++++++++++-------------------
> 1 files changed, 114 insertions(+), 90 deletions(-)
Applied v2 to u-boot-i2c.git master
Thanks!
bye,
Heiko
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
More information about the U-Boot
mailing list