[coreboot-gerrit] Patch merged into coreboot/master: cfb7360 replace device/i2c.h with simpler version
gerrit at coreboot.org
gerrit at coreboot.org
Tue Apr 9 23:59:36 CEST 2013
the following patch was just integrated into master:
commit cfb73607be05e57237592c5c94a98589aba04833
Author: David Hendricks <dhendrix at 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 at chromium.org>
Reviewed-on: http://review.coreboot.org/3043
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich at gmail.com>
Build-Tested: build bot (Jenkins) at Tue Apr 9 03:45:09 2013, giving +1
Reviewed-By: Ronald G. Minnich <rminnich at gmail.com> at Tue Apr 9 23:59:34 2013, giving +2
See http://review.coreboot.org/3043 for details.
-gerrit
More information about the coreboot-gerrit
mailing list