summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAngel Pons <th3fanbus@gmail.com>2021-04-26 16:49:40 +0200
committerFelix Held <felix-coreboot@felixheld.de>2021-04-28 16:12:11 +0000
commita35f1810a238f92cdac50524f89a006ec3cddb13 (patch)
tree9f560b276d12f3495be480ececeb777277589cd1
parent8580047dfab84c3ecc1b3fed07c0f8b20f49c014 (diff)
superio/nuvoton/npcd378: Fall back to non-negative value
This change is needed to update the option API to use unsigned integers. The CMOS option system does not support negative numbers. So, adjust the call to get_int_option() to use 3 as fallback instead of -1. Change-Id: I46c5f5c6f47f99379cbafc0d60258b99dc512e9d Signed-off-by: Angel Pons <th3fanbus@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/52671 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Patrick Rudolph <siro@das-labor.org> Reviewed-by: Nico Huber <nico.h@gmx.de>
-rw-r--r--src/superio/nuvoton/npcd378/superio.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/superio/nuvoton/npcd378/superio.c b/src/superio/nuvoton/npcd378/superio.c
index 6a56ac8b89..36458329a8 100644
--- a/src/superio/nuvoton/npcd378/superio.c
+++ b/src/superio/nuvoton/npcd378/superio.c
@@ -68,8 +68,8 @@ static void npcd378_init(struct device *dev)
npcd378_hwm_write_start(res->base);
- int fan_lvl = get_int_option("psu_fan_lvl", -1);
- if (fan_lvl < 0 || fan_lvl > 7)
+ unsigned int fan_lvl = get_int_option("psu_fan_lvl", 3);
+ if (fan_lvl > 7)
fan_lvl = 3;
uint8_t pwm = NPCD378_HWM_PSU_FAN_MIN +