Angel Pons has uploaded this change for review.

View Change

mainboard: Use `read_option_X` helpers for HWM settings

Change-Id: I97fbbf2af76a6d4c44221000da7b36378e066ff3
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
---
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 */

To view, visit change 47135. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I97fbbf2af76a6d4c44221000da7b36378e066ff3
Gerrit-Change-Number: 47135
Gerrit-PatchSet: 1
Gerrit-Owner: Angel Pons <th3fanbus@gmail.com>
Gerrit-MessageType: newchange