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/winbond | |
| 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/winbond')
| -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 |
3 files changed, 3 insertions, 3 deletions
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); |