Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/50190 )
Change subject: device/i2c_bus.c: Correct code style ......................................................................
device/i2c_bus.c: Correct code style
device: Correct code style
Revise the following aspects to follow coreboot's coding style: - Drop braces for single-statement condition. - Remove unnecessary newlines.
BUG = N/A TEST = Build Compulab Intense-PC with secure oprom enabled
Change-Id: I78ce97b0ce1587119a71893c867c2dd062552a31 Signed-off-by: Frans Hendriks fhendriks@eltan.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/50190 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Wim Vervoorn wvervoorn@eltan.com --- M src/device/i2c_bus.c 1 file changed, 4 insertions(+), 8 deletions(-)
Approvals: build bot (Jenkins): Verified Wim Vervoorn: Looks good to me, approved
diff --git a/src/device/i2c_bus.c b/src/device/i2c_bus.c index 93ec854..597a530 100644 --- a/src/device/i2c_bus.c +++ b/src/device/i2c_bus.c @@ -25,10 +25,8 @@ link = NULL; }
- if (!link) { - printk(BIOS_ALERT, "%s Cannot find I2C or SMBus bus operations", - dev_path(dev)); - } + if (!link) + printk(BIOS_ALERT, "%s Cannot find I2C or SMBus bus operations", dev_path(dev));
return link; } @@ -79,8 +77,7 @@ return busdev->ops->ops_smbus_bus->send_byte(dev, val); }
- printk(BIOS_ERR, "%s Missing ops_smbus_bus->send_byte", - dev_path(busdev)); + printk(BIOS_ERR, "%s Missing ops_smbus_bus->send_byte", dev_path(busdev)); return -1; }
@@ -140,8 +137,7 @@ return busdev->ops->ops_smbus_bus->write_byte(dev, off, val); }
- printk(BIOS_ERR, "%s Missing ops_smbus_bus->write_byte", - dev_path(busdev)); + printk(BIOS_ERR, "%s Missing ops_smbus_bus->write_byte", dev_path(busdev)); return -1; }