summaryrefslogtreecommitdiffstats
path: root/src/ec/lenovo
diff options
context:
space:
mode:
authorAngel Pons <th3fanbus@gmail.com>2021-04-26 17:10:28 +0200
committerFelix Held <felix-coreboot@felixheld.de>2021-05-06 14:48:15 +0000
commit88dcb3179b4b78a2376609577ae4dd4327fb59c7 (patch)
tree9c1ae01959fb4d084ea30893a196687c7b611fff /src/ec/lenovo
parenta2cf34129fb3b2a9302bb7cf06e4ee758b9bb85a (diff)
downloadcoreboot-88dcb3179b4b78a2376609577ae4dd4327fb59c7.tar.gz
coreboot-88dcb3179b4b78a2376609577ae4dd4327fb59c7.tar.bz2
coreboot-88dcb3179b4b78a2376609577ae4dd4327fb59c7.zip
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/ec/lenovo')
-rw-r--r--src/ec/lenovo/h8/bluetooth.c2
-rw-r--r--src/ec/lenovo/h8/h8.c22
-rw-r--r--src/ec/lenovo/h8/wwan.c2
-rw-r--r--src/ec/lenovo/pmh7/pmh7.c4
4 files changed, 15 insertions, 15 deletions
diff --git a/src/ec/lenovo/h8/bluetooth.c b/src/ec/lenovo/h8/bluetooth.c
index 463e0e3332dc..16fc8dce39a9 100644
--- a/src/ec/lenovo/h8/bluetooth.c
+++ b/src/ec/lenovo/h8/bluetooth.c
@@ -48,5 +48,5 @@ bool h8_has_bdc(const struct device *dev)
*/
bool h8_bluetooth_nv_enable(void)
{
- return get_int_option("bluetooth", true);
+ return get_uint_option("bluetooth", true);
}
diff --git a/src/ec/lenovo/h8/h8.c b/src/ec/lenovo/h8/h8.c
index e17f0fb2d413..b081ec8f226a 100644
--- a/src/ec/lenovo/h8/h8.c
+++ b/src/ec/lenovo/h8/h8.c
@@ -243,7 +243,7 @@ static void h8_enable(struct device *dev)
reg8 = conf->config1;
if (conf->has_keyboard_backlight) {
/* Default to both backlights */
- reg8 = (reg8 & 0xf3) | ((get_int_option("backlight", 0) & 0x3) << 2);
+ reg8 = (reg8 & 0xf3) | ((get_uint_option("backlight", 0) & 0x3) << 2);
}
ec_write(H8_CONFIG1, reg8);
ec_write(H8_CONFIG2, conf->config2);
@@ -253,14 +253,14 @@ static void h8_enable(struct device *dev)
beepmask1 = conf->beepmask1;
if (conf->has_power_management_beeps) {
- if (get_int_option("power_management_beeps", 1) == 0) {
+ if (get_uint_option("power_management_beeps", 1) == 0) {
beepmask0 = 0x00;
beepmask1 = 0x00;
}
}
if (conf->has_power_management_beeps) {
- if (get_int_option("low_battery_beep", 1))
+ if (get_uint_option("low_battery_beep", 1))
beepmask0 |= 2;
else
beepmask0 &= ~2;
@@ -292,14 +292,14 @@ static void h8_enable(struct device *dev)
ec_write(H8_FAN_CONTROL, H8_FAN_CONTROL_AUTO);
- h8_usb_always_on_enable(get_int_option("usb_always_on", 0));
+ h8_usb_always_on_enable(get_uint_option("usb_always_on", 0));
- h8_wlan_enable(get_int_option("wlan", 1));
+ h8_wlan_enable(get_uint_option("wlan", 1));
h8_trackpoint_enable(1);
h8_usb_power_enable(1);
- unsigned int volume = get_int_option("volume", ~0);
+ unsigned int volume = get_uint_option("volume", ~0);
if (volume <= 0xff && !acpi_is_wakeup_s3())
ec_write(H8_VOLUME_CONTROL, volume);
@@ -311,16 +311,16 @@ static void h8_enable(struct device *dev)
h8_wwan_enable(val);
if (conf->has_uwb)
- h8_uwb_enable(get_int_option("uwb", 1));
+ h8_uwb_enable(get_uint_option("uwb", 1));
- h8_fn_ctrl_swap(get_int_option("fn_ctrl_swap", 0));
+ h8_fn_ctrl_swap(get_uint_option("fn_ctrl_swap", 0));
- h8_sticky_fn(get_int_option("sticky_fn", 0));
+ h8_sticky_fn(get_uint_option("sticky_fn", 0));
if (CONFIG(H8_HAS_PRIMARY_FN_KEYS))
- f1_to_f12_as_primary(get_int_option("f1_to_f12_as_primary", 1));
+ f1_to_f12_as_primary(get_uint_option("f1_to_f12_as_primary", 1));
- h8_charge_priority(get_int_option("first_battery", PRIMARY_BATTERY));
+ h8_charge_priority(get_uint_option("first_battery", PRIMARY_BATTERY));
h8_set_audio_mute(0);
h8_mb_init();
diff --git a/src/ec/lenovo/h8/wwan.c b/src/ec/lenovo/h8/wwan.c
index 90f6b47bbb8d..685886fcced4 100644
--- a/src/ec/lenovo/h8/wwan.c
+++ b/src/ec/lenovo/h8/wwan.c
@@ -46,5 +46,5 @@ bool h8_has_wwan(const struct device *dev)
*/
bool h8_wwan_nv_enable(void)
{
- return get_int_option("wwan", true);
+ return get_uint_option("wwan", true);
}
diff --git a/src/ec/lenovo/pmh7/pmh7.c b/src/ec/lenovo/pmh7/pmh7.c
index f32c9f18d912..ebcbcde08909 100644
--- a/src/ec/lenovo/pmh7/pmh7.c
+++ b/src/ec/lenovo/pmh7/pmh7.c
@@ -117,9 +117,9 @@ static void enable_dev(struct device *dev)
pmh7_backlight_enable(conf->backlight_enable);
pmh7_dock_event_enable(conf->dock_event_enable);
- pmh7_touchpad_enable(get_int_option("touchpad", 1));
+ pmh7_touchpad_enable(get_uint_option("touchpad", 1));
- pmh7_trackpoint_enable(get_int_option("trackpoint", 1));
+ pmh7_trackpoint_enable(get_uint_option("trackpoint", 1));
printk(BIOS_INFO, "PMH7: ID %02x Revision %02x\n",
pmh7_register_read(EC_LENOVO_PMH7_REG_ID),