[U-Boot] [PATCH v3] OMAP: I2C: New read, write and probe functions

Tom Rini trini at ti.com
Thu May 30 16:37:42 CEST 2013


On Thu, May 30, 2013 at 01:24:42AM +0300, Lubomir Popov wrote:

> Tested on OMAP4/5 only, but should work on older OMAPs and
> derivatives as well.
> 
> - Rewritten i2c_read to operate correctly with all types of chips
>   (old function could not read consistent data from some I2C slaves).
> - Optimised i2c_write.
> - New i2c_probe, optionally selectable via CONFIG_I2C_PROBE_WRITE,
>   performs write access vs read. The old probe could hang the system
>   under certain conditions (e.g. unconfigured pads).
> - The read/write/probe functions try to identify unconfigured bus.
> - Status functions now read irqstatus_raw as per TRM guidelines
>   (except for OMAP243X and OMAP34XX).
> - Driver now supports up to I2C5 (OMAP5).
> 
> Signed-off-by: Lubomir Popov <lpopov at mm-sol.com>

With CONFIG_I2C_PROBE_WRITE set:
Tested-by: Tom Rini <trini at ti.com> on Beagleboard / Beagleboard xM

So lets just go with the write probe always being on again.

Now, when I git am'd I saw some whitespace problems, so please make sure
v4 is checkpatch clean.  And note that printf("Long than 80 char wide",
a, b) is OK and expected to NOT break the string (but do align the
args).

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20130530/19c26d6a/attachment.pgp>


More information about the U-Boot mailing list