diff options
| author | Angel Pons <th3fanbus@gmail.com> | 2021-04-26 17:10:28 +0200 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2021-05-06 14:48:15 +0000 |
| commit | 88dcb3179b4b78a2376609577ae4dd4327fb59c7 (patch) | |
| tree | 9c1ae01959fb4d084ea30893a196687c7b611fff /src/superio | |
| parent | a2cf34129fb3b2a9302bb7cf06e4ee758b9bb85a (diff) | |
src: Retype option API to use unsigned integers
The CMOS option system does not support negative integers. Thus, retype
and rename the option API functions to reflect this.
Change-Id: Id3480e5cfc0ec90674def7ef0919e0b7ac5b19b3
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/52672
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>
Reviewed-by: Felix Singer <felixsinger@posteo.net>
Diffstat (limited to 'src/superio')
| -rw-r--r-- | src/superio/ite/it8720f/superio.c | 2 | ||||
| -rw-r--r-- | src/superio/nuvoton/nct5572d/superio.c | 2 | ||||
| -rw-r--r-- | src/superio/nuvoton/npcd378/superio.c | 2 | ||||
| -rw-r--r-- | src/superio/winbond/w83627ehg/superio.c | 2 | ||||
| -rw-r--r-- | src/superio/winbond/w83627hf/superio.c | 2 | ||||
| -rw-r--r-- | src/superio/winbond/w83667hg-a/superio.c | 2 |
6 files changed, 6 insertions, 6 deletions
diff --git a/src/superio/ite/it8720f/superio.c b/src/superio/ite/it8720f/superio.c index 0ecbddc6a2..c63fa9ab4f 100644 --- a/src/superio/ite/it8720f/superio.c +++ b/src/superio/ite/it8720f/superio.c @@ -17,7 +17,7 @@ static void power_control_init(struct device *dev) { - int power_on = get_int_option("power_on_after_fail", MAINBOARD_POWER_OFF); + unsigned int power_on = get_uint_option("power_on_after_fail", MAINBOARD_POWER_OFF); u8 value; pnp_enter_conf_mode(dev); diff --git a/src/superio/nuvoton/nct5572d/superio.c b/src/superio/nuvoton/nct5572d/superio.c index b26954df9d..b0e00fbf08 100644 --- a/src/superio/nuvoton/nct5572d/superio.c +++ b/src/superio/nuvoton/nct5572d/superio.c @@ -47,7 +47,7 @@ static void nct5572d_init(struct device *dev) break; case NCT5572D_ACPI: /* Set power state after power fail */ - power_status = get_int_option("power_on_after_fail", + power_status = get_uint_option("power_on_after_fail", CONFIG_MAINBOARD_POWER_FAILURE_STATE); pnp_enter_conf_mode(dev); pnp_set_logical_device(dev); diff --git a/src/superio/nuvoton/npcd378/superio.c b/src/superio/nuvoton/npcd378/superio.c index 36458329a8..9889980650 100644 --- a/src/superio/nuvoton/npcd378/superio.c +++ b/src/superio/nuvoton/npcd378/superio.c @@ -68,7 +68,7 @@ static void npcd378_init(struct device *dev) npcd378_hwm_write_start(res->base); - unsigned int fan_lvl = get_int_option("psu_fan_lvl", 3); + unsigned int fan_lvl = get_uint_option("psu_fan_lvl", 3); if (fan_lvl > 7) fan_lvl = 3; diff --git a/src/superio/winbond/w83627ehg/superio.c b/src/superio/winbond/w83627ehg/superio.c index 2f0a3a4869..1e36048146 100644 --- a/src/superio/winbond/w83627ehg/superio.c +++ b/src/superio/winbond/w83627ehg/superio.c @@ -23,7 +23,7 @@ static void enable_hwm_smbus(struct device *dev) static void init_acpi(struct device *dev) { u8 value; - int power_on = get_int_option("power_on_after_fail", 1); + unsigned int power_on = get_uint_option("power_on_after_fail", 1); pnp_enter_conf_mode(dev); pnp_set_logical_device(dev); diff --git a/src/superio/winbond/w83627hf/superio.c b/src/superio/winbond/w83627hf/superio.c index 5af2d37c45..de2a0c6491 100644 --- a/src/superio/winbond/w83627hf/superio.c +++ b/src/superio/winbond/w83627hf/superio.c @@ -23,7 +23,7 @@ static void enable_hwm_smbus(struct device *dev) static void init_acpi(struct device *dev) { u8 value; - int power_on = get_int_option("power_on_after_fail", 1); + unsigned int power_on = get_uint_option("power_on_after_fail", 1); pnp_enter_conf_mode(dev); pnp_set_logical_device(dev); diff --git a/src/superio/winbond/w83667hg-a/superio.c b/src/superio/winbond/w83667hg-a/superio.c index 5f9683299b..0a068983a8 100644 --- a/src/superio/winbond/w83667hg-a/superio.c +++ b/src/superio/winbond/w83667hg-a/superio.c @@ -47,7 +47,7 @@ static void w83667hg_a_init(struct device *dev) break; case W83667HG_A_ACPI: /* Set power state after power fail */ - power_status = get_int_option("power_on_after_fail", + power_status = get_uint_option("power_on_after_fail", CONFIG_MAINBOARD_POWER_FAILURE_STATE); pnp_enter_conf_mode(dev); pnp_set_logical_device(dev); |