Skip to content

: Correct i2c support on am33xx

Heiko Schocher u-boot 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest5 patches 0 repliesNew

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/5Revert "I2C: OMAP: detect more devices when probing an i2c bus"+0 -00 replies
2/5am335x: Correct i2c sysc offset+0 -00 replies
3/5am33xx: Fill in more cm_wkuppll / cm_perpll+0 -00 replies
4/5am33xx: Fix i2c sampling rate typo+0 -00 replies
5/5omap24xx_i2c: Add AM33XX support+0 -00 replies

Reviews trailers per patch

All 5 patches have a Reviewed-by or Acked-by.

PatchHeiko Schocher
1/5A-b
2/5A-b
3/5A-b
4/5A-b
5/5A-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [U-Boot] [PATCH 0/5]: Correct i2c support on am33xx

Hello Tom,

Quoted 7 lines · Tom Rini

The following series corrects i2c support for the am33xx family of devices (including beaglebone) and has been tested on a beagleboard xM and pandaboard. I would like to take these changes in via the u-boot-ti tree as it's mostly changes to arch and board files but I've cc'd Heiko on the i2c related parts. The biggest change here is to revert a previous change to the omap24xx_i2c.c driver for the i2c probe function. In short, the change made before violated the TRM constraints but did no harm on the IP

Quoted 8 lines

block found in omap3/related boards. Moving forward to the IP block found on omap4 and am33xx (and others) it still voilates the TRM and now leaves the bus in an unboundedly defined state like the TRM states. We must revert this change for both correctness and functionality. The rest of the series is minor corrections to structs/defines and adding CONFIG_AM33XX to the block of !CONFIG_OMAP2430. I've thought of, but think it should be separate to change omap24xx_i2c.c to test for CONFIG_OMAP2430 rather than every other present and future user in a few areas.

Ok, just saw, that you delegate this patches in patchwork to you. I am fine with that, but want to ack this patches, so please wait until I sent my Ack ... thanks!

bye,
Heiko

Signature · 3 lines
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