From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Tue, 22 May 2012 09:29:11 +0200 Subject: [U-Boot] [PATCH 0/5]: Correct i2c support on am33xx In-Reply-To: <1337618793-13934-1-git-send-email-trini@ti.com> References: <1337618793-13934-1-git-send-email-trini@ti.com> Message-ID: <4FBB4047.3060705@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hello Tom, Tom Rini wrote: > 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 > 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 -- 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