diff options
| author | Matt DeVillier <matt.devillier@gmail.com> | 2018-02-11 16:07:12 -0600 |
|---|---|---|
| committer | Duncan Laurie <dlaurie@chromium.org> | 2018-02-12 15:56:55 +0000 |
| commit | 047b23fc31e069c0cb372748e980da194ad59b95 (patch) | |
| tree | 780aa537f364d3f72edc4cd4aa580dcb440a8924 /src/ec | |
| parent | 3be35976d6a640d0bc4fd40e232b6499b170431f (diff) | |
| download | coreboot-047b23fc31e069c0cb372748e980da194ad59b95.tar.xz | |
ec/chromeec: Fix battery ACPI mutex level
Commit 07fe618 [chromeec: Add support for reading second battery info]
added a mutex as part of the ACPI code to determine battery statuses.
Windows is extremely picky about ACPI code, and attempting to acquire
a level 1 mutex without first having acquired a level 0 mutex causes
Windows to hang on boot. Since there's no reason to use a level 1
mutex here, change it to level 0.
Test: Boot Windows on device with ChromeEC without hanging
Change-Id: Icfb0817cfe0c49eb4527a12b507362939a6d32c6
Signed-off-by: Matt DeVillier <matt.devillier@gmail.com>
Reviewed-on: https://review.coreboot.org/23697
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Duncan Laurie <dlaurie@chromium.org>
Reviewed-by: Furquan Shaikh <furquan@google.com>
Reviewed-by: Nicolas Boichat <drinkcat@chromium.org>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Diffstat (limited to 'src/ec')
| -rw-r--r-- | src/ec/google/chromeec/acpi/battery.asl | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/ec/google/chromeec/acpi/battery.asl b/src/ec/google/chromeec/acpi/battery.asl index d8056b03b2..0a0caec279 100644 --- a/src/ec/google/chromeec/acpi/battery.asl +++ b/src/ec/google/chromeec/acpi/battery.asl @@ -17,7 +17,7 @@ // Scope (EC0) /* Mutex for EC battery index interface */ -Mutex (BATM, 1) +Mutex (BATM, 0) // Wait for desired battery index to be presented in shared memory // Arg0 = battery index |