diff options
| author | Aaron Durbin <adurbin@chromium.org> | 2018-01-22 21:24:35 -0700 |
|---|---|---|
| committer | Aaron Durbin <adurbin@chromium.org> | 2018-01-24 05:02:50 +0000 |
| commit | 439cee90984ed9b5b614af2707aa0017528890d0 (patch) | |
| tree | f2891bff4409c6bd0b64756879ec0fb68e71bbb3 /src/include/device/i2c_bus.h | |
| parent | 11284d7d4374c15e81e805301d448de1a8576a18 (diff) | |
| download | coreboot-439cee90984ed9b5b614af2707aa0017528890d0.tar.gz coreboot-439cee90984ed9b5b614af2707aa0017528890d0.tar.bz2 coreboot-439cee90984ed9b5b614af2707aa0017528890d0.zip | |
device/i2c_bus: allow i2c_bus and i2c_simple to coexist
If one wants to implement both i2c_bus.h and i2c_simple.h APIs
the compilation unit needs to be guarded or coordinated carefully
with different compilation units. Instead, name the i2c_bus
functions with _dev such that it indicates that they operate on
struct device. One other change to allow i2c_bus.h to be built in
non-ramstage environments is to ensure DEVTREE_CONST is used for
the dev field in struct bus.
BUG=b:72121803
Change-Id: I267e27e62c95013e8ff8b0728dbe9e7b523de453
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: https://review.coreboot.org/23370
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Martin Roth <martinroth@google.com>
Reviewed-by: Furquan Shaikh <furquan@google.com>
Diffstat (limited to 'src/include/device/i2c_bus.h')
| -rw-r--r-- | src/include/device/i2c_bus.h | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/include/device/i2c_bus.h b/src/include/device/i2c_bus.h index f1416a7411f6..6aa4f9ba9ddc 100644 --- a/src/include/device/i2c_bus.h +++ b/src/include/device/i2c_bus.h @@ -39,7 +39,7 @@ struct bus *i2c_link(struct device *); * * Returns NULL if i2c_link(dev) returns NULL. */ -static inline struct device *i2c_busdev(struct device *dev) +static inline DEVTREE_CONST struct device *i2c_busdev(struct device *dev) { struct bus *const link = i2c_link(dev); return link ? link->dev : NULL; @@ -63,7 +63,7 @@ static inline struct device *i2c_busdev(struct device *dev) * * Returns the read byte on success, negative `enum cb_err` value on error. */ -int i2c_readb(struct device *); +int i2c_dev_readb(struct device *); /* * Writes the byte `val`. @@ -71,7 +71,7 @@ int i2c_readb(struct device *); * * Returns 0 on success, negative `enum cb_err` value on error. */ -int i2c_writeb(struct device *, uint8_t val); +int i2c_dev_writeb(struct device *, uint8_t val); /* * Sends the register offset `off` and reads one byte. @@ -79,7 +79,7 @@ int i2c_writeb(struct device *, uint8_t val); * * Returns the read byte on success, negative `enum cb_err` value on error. */ -int i2c_readb_at(struct device *, uint8_t off); +int i2c_dev_readb_at(struct device *, uint8_t off); /* * Sends the register offset `off` followed by the byte `val`. @@ -87,6 +87,6 @@ int i2c_readb_at(struct device *, uint8_t off); * * Returns 0 on success, negative `enum cb_err` value on error. */ -int i2c_writeb_at(struct device *, uint8_t off, uint8_t val); +int i2c_dev_writeb_at(struct device *, uint8_t off, uint8_t val); #endif /* _DEVICE_I2C_BUS_H_ */ |