the following patch was just integrated into master: commit cfb73607be05e57237592c5c94a98589aba04833 Author: David Hendricks dhendrix@chromium.org Date: Fri Apr 5 13:42:39 2013 -0700
replace device/i2c.h with simpler version
The existing header was imported along with the Exynos code and left mostly unchanged. This is the first patch in a series intended to replace the imported u-boot I2C API with a much simpler and cleaner interface:
- We only need to expose i2c_read() and i2c_write() in our public API. Everything else is board/chip-dependent and should remain hidden away.
- i2c_read and i2c_write functions will take bus number as an arg and we'll eliminate i2c_get_bus and i2c_set_bus. Those are prone to error and end up cluttering the code since the user needs to save the old bus number, set the new one, do the read/write, and restore the old value (3 added steps to do a simple transaction).
- Stop setting default values for board-specific things like SPD and RTC bus numbers (as if we always have an SPD or RTC on I2C).
- Death to all the trivial inline wrappers. And in case there was any doubt, we really don't care about the MPC8xx. Though if we did then we would not pollute the public API with its idiosyncrasies.
Change-Id: I4410a3c82ed5a6b2e80e3d8c0163464a9ca7c3b0 Signed-off-by: David Hendricks dhendrix@chromium.org Reviewed-on: http://review.coreboot.org/3043 Tested-by: build bot (Jenkins) Reviewed-by: Ronald G. Minnich rminnich@gmail.com
Build-Tested: build bot (Jenkins) at Tue Apr 9 03:45:09 2013, giving +1 Reviewed-By: Ronald G. Minnich rminnich@gmail.com at Tue Apr 9 23:59:34 2013, giving +2 See http://review.coreboot.org/3043 for details.
-gerrit