Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/63208 )
Change subject: device/i2c_bus: Constify i2c_busdev and i2c_link ......................................................................
device/i2c_bus: Constify i2c_busdev and i2c_link
Change-Id: If795087ecdaea24ad7834dcc6d5bf6a72f2aea8f Signed-off-by: Matt DeVillier matt.devillier@gmail.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/63208 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Tim Wawrzynczak twawrzynczak@chromium.org Reviewed-by: Raul Rangel rrangel@chromium.org --- M src/device/i2c_bus.c M src/include/device/i2c_bus.h 2 files changed, 3 insertions(+), 3 deletions(-)
Approvals: build bot (Jenkins): Verified Raul Rangel: Looks good to me, approved Tim Wawrzynczak: Looks good to me, approved
diff --git a/src/device/i2c_bus.c b/src/device/i2c_bus.c index 597a530..a63086c 100644 --- a/src/device/i2c_bus.c +++ b/src/device/i2c_bus.c @@ -6,7 +6,7 @@ #include <device/i2c_bus.h> #include <commonlib/endian.h>
-struct bus *i2c_link(struct device *const dev) +struct bus *i2c_link(const struct device *const dev) { if (!dev || !dev->bus) return NULL; diff --git a/src/include/device/i2c_bus.h b/src/include/device/i2c_bus.h index b5e7710..42f461d 100644 --- a/src/include/device/i2c_bus.h +++ b/src/include/device/i2c_bus.h @@ -21,14 +21,14 @@ * `->dev->ops->ops_i2c_bus` or `->dev->ops->ops_smbus_bus` are * not NULL. */ -struct bus *i2c_link(struct device *); +struct bus *i2c_link(const struct device *dev);
/* * Shorthand for `i2c_link(dev)->dev`. * * Returns NULL if i2c_link(dev) returns NULL. */ -static inline DEVTREE_CONST struct device *i2c_busdev(struct device *dev) +static inline DEVTREE_CONST struct device *i2c_busdev(const struct device *dev) { struct bus *const link = i2c_link(dev); return link ? link->dev : NULL;