[coreboot-gerrit] Change in coreboot[master]: mainboard: Use `read_option_X` helpers for HWM settings

[Angel Pons (Code Review)]

Angel Pons has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/47135 )

Change subject: mainboard: Use `read_option_X` helpers for HWM settings
......................................................................

mainboard: Use `read_option_X` helpers for HWM settings

Change-Id: I97fbbf2af76a6d4c44221000da7b36378e066ff3
Signed-off-by: Angel Pons 
---
M src/mainboard/ibase/mb899/superio_hwm.c
M src/mainboard/kontron/986lcd-m/mainboard.c
2 files changed, 10 insertions(+), 20 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/35/47135/1

diff --git a/src/mainboard/ibase/mb899/superio_hwm.c b/src/mainboard/ibase/mb899/superio_hwm.c
index 6b2d77a..c6c8f2c 100644
--- a/src/mainboard/ibase/mb899/superio_hwm.c
+++ b/src/mainboard/ibase/mb899/superio_hwm.c
@@ -52,19 +52,13 @@
    int cpufan_speed = 0, sysfan_speed = 0;
    int cpufan_temperature = 0, sysfan_temperature = 0;

-	cpufan_control = FAN_CRUISE_CONTROL_DISABLED;
-	get_option(&cpufan_control, "cpufan_cruise_control");
-	cpufan_speed = FAN_SPEED_5625;
-	get_option(&cpufan_speed, "cpufan_speed");
-	//cpufan_temperature = FAN_TEMPERATURE_30DEGC;
-	//get_option(&cpufan_temperature, "cpufan_temperature");
+	cpufan_control = read_option_int("cpufan_cruise_control", FAN_CRUISE_CONTROL_DISABLED);
+	cpufan_speed = read_option_int("cpufan_speed", FAN_SPEED_5625);
+	//cpufan_temperature = read_option_int("cpufan_temperature", FAN_TEMPERATURE_30DEGC);

-	sysfan_control = FAN_CRUISE_CONTROL_DISABLED;
-	get_option(&sysfan_control, "sysfan_cruise_control");
-	sysfan_speed = FAN_SPEED_5625;
-	get_option(&sysfan_speed, "sysfan_speed");
-	//sysfan_temperature = FAN_TEMPERATURE_30DEGC;
-	//get_option(&sysfan_temperature, "sysfan_temperature");
+	sysfan_control = read_option_int("sysfan_cruise_control", FAN_CRUISE_CONTROL_DISABLED);
+	sysfan_speed = read_option_int("sysfan_speed", FAN_SPEED_5625);
+	//sysfan_temperature = read_option_int("sysfan_temperature", FAN_TEMPERATURE_30DEGC);

// pnp_write_hwm5_index(HWM_BASE, 0x31, 0x20); // AVCC high limit
    // pnp_write_hwm5_index(HWM_BASE, 0x34, 0x06); // VIN2 low limit
diff --git a/src/mainboard/kontron/986lcd-m/mainboard.c b/src/mainboard/kontron/986lcd-m/mainboard.c
index 26a800a..15cf499 100644
--- a/src/mainboard/kontron/986lcd-m/mainboard.c
+++ b/src/mainboard/kontron/986lcd-m/mainboard.c
@@ -50,15 +50,11 @@
    int cpufan_speed = 0, sysfan_speed = 0;
    int cpufan_temperature = 0, sysfan_temperature = 0;

-	cpufan_control = FAN_CRUISE_CONTROL_DISABLED;
-	get_option(&cpufan_control, "cpufan_cruise_control");
-	cpufan_speed = FAN_SPEED_5625;
-	get_option(&cpufan_speed, "cpufan_speed");
+	cpufan_control = read_option_int("cpufan_cruise_control", FAN_CRUISE_CONTROL_DISABLED);
+	cpufan_speed = read_option_int("cpufan_speed", FAN_SPEED_5625);

-	sysfan_control = FAN_CRUISE_CONTROL_DISABLED;
-	get_option(&sysfan_control, "sysfan_cruise_control");
-	sysfan_speed = FAN_SPEED_5625;
-	get_option(&sysfan_speed, "sysfan_speed");
+	sysfan_control = read_option_int("sysfan_cruise_control", FAN_CRUISE_CONTROL_DISABLED);
+	sysfan_speed = read_option_int("sysfan_speed", FAN_SPEED_5625);

nuvoton_hwm_select_bank(hwm_base, 0);
    pnp_write_hwm5_index(hwm_base, 0x59, 0x20); /* Diode Selection */