summaryrefslogtreecommitdiff
path: root/src/device
diff options
context:
space:
mode:
authorAaron Durbin <adurbin@chromium.org>2018-01-22 21:24:35 -0700
committerAaron Durbin <adurbin@chromium.org>2018-01-24 05:02:50 +0000
commit439cee90984ed9b5b614af2707aa0017528890d0 (patch)
treef2891bff4409c6bd0b64756879ec0fb68e71bbb3 /src/device
parent11284d7d4374c15e81e805301d448de1a8576a18 (diff)
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/device')
-rw-r--r--src/device/i2c_bus.c10
1 files changed, 5 insertions, 5 deletions
diff --git a/src/device/i2c_bus.c b/src/device/i2c_bus.c
index 1c543efaba..30bb80ca98 100644
--- a/src/device/i2c_bus.c
+++ b/src/device/i2c_bus.c
@@ -44,7 +44,7 @@ struct bus *i2c_link(struct device *const dev)
return link;
}
-int i2c_readb(struct device *const dev)
+int i2c_dev_readb(struct device *const dev)
{
struct device *const busdev = i2c_busdev(dev);
if (!busdev)
@@ -74,7 +74,7 @@ int i2c_readb(struct device *const dev)
}
}
-int i2c_writeb(struct device *const dev, uint8_t val)
+int i2c_dev_writeb(struct device *const dev, uint8_t val)
{
struct device *const busdev = i2c_busdev(dev);
if (!busdev)
@@ -97,7 +97,7 @@ int i2c_writeb(struct device *const dev, uint8_t val)
}
}
-int i2c_readb_at(struct device *const dev, uint8_t off)
+int i2c_dev_readb_at(struct device *const dev, uint8_t off)
{
struct device *const busdev = i2c_busdev(dev);
if (!busdev)
@@ -135,8 +135,8 @@ int i2c_readb_at(struct device *const dev, uint8_t off)
}
}
-int i2c_writeb_at(struct device *const dev,
- const uint8_t off, const uint8_t val)
+int i2c_dev_writeb_at(struct device *const dev,
+ const uint8_t off, const uint8_t val)
{
struct device *const busdev = i2c_busdev(dev);
if (!busdev)