diff options
| author | Francis Rowe <info@gluglug.org.uk> | 2015-06-15 15:15:36 (EDT) |
|---|---|---|
| committer | Francis Rowe <info@gluglug.org.uk> | 2015-06-15 23:36:26 (EDT) |
| commit | bd95009839337576c1d7ac6d022228c4ec4248a5 (patch) | |
| tree | 29622510346a315c5cb0fd766ac883147f3b4b15 /resources/libreboot/patch/0015-ec-lenovo-h8-re-factor-handling-of-power_management_.patch | |
| parent | 9f8eced929a99b2ad7b10d1b8d237779afdd98d5 (diff) | |
| download | libreboot-bd95009839337576c1d7ac6d022228c4ec4248a5.zip libreboot-bd95009839337576c1d7ac6d022228c4ec4248a5.tar.gz libreboot-bd95009839337576c1d7ac6d022228c4ec4248a5.tar.bz2 | |
Update coreboot-libre
Rebase all patches. Remove the ones that are no longer needed.
More CPU microcode updates were moved to coreboot's 3rdparty
repository, so there are less blobs for libreboot to delete
now (because the 3rdparty repository is not checked out in
libreboot).
Correct HDA verbs used for T400 (also R400, T500) (patch is in
coreboot, merged).
Diffstat (limited to 'resources/libreboot/patch/0015-ec-lenovo-h8-re-factor-handling-of-power_management_.patch')
| -rw-r--r-- | resources/libreboot/patch/0015-ec-lenovo-h8-re-factor-handling-of-power_management_.patch | 53 |
1 files changed, 53 insertions, 0 deletions
diff --git a/resources/libreboot/patch/0015-ec-lenovo-h8-re-factor-handling-of-power_management_.patch b/resources/libreboot/patch/0015-ec-lenovo-h8-re-factor-handling-of-power_management_.patch new file mode 100644 index 0000000..e5d6deb --- /dev/null +++ b/resources/libreboot/patch/0015-ec-lenovo-h8-re-factor-handling-of-power_management_.patch @@ -0,0 +1,53 @@ +From 5fdab56649a84249c8eecd30caa34318ebde6b1d Mon Sep 17 00:00:00 2001 +From: Francis Rowe <info@gluglug.org.uk> +Date: Fri, 12 Jun 2015 23:10:52 +0100 +Subject: [PATCH 15/17] ec/lenovo/h8: re-factor handling of + power_management_beeps + +The current code duplicates the same check unnecessarily, +and has no handling of when the option power_management_beeps +is not set. + +Change-Id: I189c5ce382e1a270d24b9b6e897358268b9a141d +Signed-off-by: Francis Rowe <info@gluglug.org.uk> +--- + src/ec/lenovo/h8/h8.c | 15 ++++++++------- + 1 file changed, 8 insertions(+), 7 deletions(-) + +diff --git a/src/ec/lenovo/h8/h8.c b/src/ec/lenovo/h8/h8.c +index b6bc406..fde408c 100644 +--- a/src/ec/lenovo/h8/h8.c ++++ b/src/ec/lenovo/h8/h8.c +@@ -2,6 +2,7 @@ + * This file is part of the coreboot project. + * + * Copyright (C) 2011 Sven Schnelle <svens@stackframe.org> ++ * Copyright (C) 2015 Francis Rowe <info@gluglug.org.uk> + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by +@@ -212,14 +213,14 @@ static void h8_enable(struct device *dev) + beepmask0 = conf->beepmask0; + beepmask1 = conf->beepmask1; + +- if (conf->has_power_management_beeps +- && get_option(&val, "power_management_beeps") == CB_SUCCESS +- && val == 0) { +- beepmask0 = 0x00; +- beepmask1 = 0x00; +- } +- + if (conf->has_power_management_beeps) { ++ if (get_option(&val, "power_management_beeps") != CB_SUCCESS) ++ val = 1; ++ if (!val) { ++ beepmask0 = 0x00; ++ beepmask1 = 0x00; ++ } ++ + if (get_option(&val, "low_battery_beep") != CB_SUCCESS) + val = 1; + if (val) +-- +1.9.1 + |