From f61fff92e4ff0de9c7e0efc6a8afaa42c5143822 Mon Sep 17 00:00:00 2001 From: Vikram Narayanan Date: Sat, 7 Jan 2012 16:04:46 +0530 Subject: [PATCH] adm1027: add return statement Adds a missing return statment which will stop misleading the users Change-Id: I53741f1136b396e9493ce959b54efc00c9b09764 Signed-off-by: Vikram Narayanan Reviewed-on: http://review.coreboot.org/522 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Georgi Reviewed-by: Ronald G. Minnich Reviewed-by: Stefan Reinauer --- src/drivers/i2c/adm1027/adm1027.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/drivers/i2c/adm1027/adm1027.c b/src/drivers/i2c/adm1027/adm1027.c index 516a8897d..e97ec6956 100644 --- a/src/drivers/i2c/adm1027/adm1027.c +++ b/src/drivers/i2c/adm1027/adm1027.c @@ -35,6 +35,7 @@ static void adm1027_enable_monitoring(device_t dev) result = smbus_read_byte(dev, ADM1027_REG_CONFIG1); if (!(result & CFG1_STRT)) { printk(BIOS_DEBUG, "ADM1027: monitoring would not enable\n"); + return; } printk(BIOS_DEBUG, "ADM1027: monitoring enabled\n"); } -- 2.25.1