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/southbridge/intel/i82801dx | |
| 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/southbridge/intel/i82801dx')
| -rw-r--r-- | src/southbridge/intel/i82801dx/lpc.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/southbridge/intel/i82801dx/lpc.c b/src/southbridge/intel/i82801dx/lpc.c index 098af838a3..f6c04a5d9f 100644 --- a/src/southbridge/intel/i82801dx/lpc.c +++ b/src/southbridge/intel/i82801dx/lpc.c @@ -94,7 +94,7 @@ static void i82801dx_power_options(struct device *dev) * * If the option is not existent (Laptops), use MAINBOARD_POWER_ON. */ - const int pwr_on = get_int_option("power_on_after_fail", MAINBOARD_POWER_ON); + const unsigned int pwr_on = get_uint_option("power_on_after_fail", MAINBOARD_POWER_ON); reg8 = pci_read_config8(dev, GEN_PMCON_3); reg8 &= 0xfe; @@ -129,7 +129,7 @@ static void i82801dx_power_options(struct device *dev) outb(reg8, 0x61); reg8 = inb(0x70); - const int nmi_option = get_int_option("nmi", NMI_OFF); + const unsigned int nmi_option = get_uint_option("nmi", NMI_OFF); if (nmi_option) { printk(BIOS_INFO, "NMI sources enabled.\n"); reg8 &= ~(1 << 7); /* Set NMI. */ |