diff options
author | Vikram Narayanan <vikram186@gmail.com> | 2012-01-07 16:04:46 +0530 |
---|---|---|
committer | Stefan Reinauer <stefan.reinauer@coreboot.org> | 2012-01-08 20:42:08 +0100 |
commit | f61fff92e4ff0de9c7e0efc6a8afaa42c5143822 (patch) | |
tree | 63567be820aac1f97caf2e9fa0bbb28c3a383fd0 /src/drivers/i2c | |
parent | 4b7b320ff80c1047503e26fd387ba3d8acd996d9 (diff) |
adm1027: add return statement
Adds a missing return statment which will stop misleading the users
Change-Id: I53741f1136b396e9493ce959b54efc00c9b09764
Signed-off-by: Vikram Narayanan <vikram186@gmail.com>
Reviewed-on: http://review.coreboot.org/522
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick@georgi-clan.de>
Reviewed-by: Ronald G. Minnich <rminnich@gmail.com>
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Diffstat (limited to 'src/drivers/i2c')
-rw-r--r-- | src/drivers/i2c/adm1027/adm1027.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/drivers/i2c/adm1027/adm1027.c b/src/drivers/i2c/adm1027/adm1027.c index 516a8897d8..e97ec69569 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"); } |