diff options
| author | Matt DeVillier <matt.devillier@gmail.com> | 2016-12-09 18:14:57 -0600 |
|---|---|---|
| committer | Nico Huber <nico.h@gmx.de> | 2016-12-15 23:39:04 +0100 |
| commit | 678923c2b7f8c50746be4e49f9ad3ccd1ca11c45 (patch) | |
| tree | fc2b021d7c23f3545afcc671cfa31b1ef702e8ce /src/ec | |
| parent | f8bd1dd43cadc01cd93b6f14373d3fd3903d5817 (diff) | |
| download | coreboot-678923c2b7f8c50746be4e49f9ad3ccd1ca11c45.tar.xz | |
ec/ene932: correct ACPI battery data fed into ToString()
ToString() requires the input buffer data to be null-terminated,
but the data returned by the EC is not, leading Windows to fail
to report any battery data at all. Correct this by concatenating
a null terminator (0x00) to the end of the buffer data before
inputting to ToString() where needed
Change-Id: Ic86048d1d6354b9b0dac3c8957df318d0825c905
Signed-off-by: Matt DeVillier <matt.devillier@gmail.com>
Reviewed-on: https://review.coreboot.org/17783
Tested-by: build bot (Jenkins)
Reviewed-by: Nico Huber <nico.h@gmx.de>
Diffstat (limited to 'src/ec')
| -rw-r--r-- | src/ec/compal/ene932/acpi/battery.asl | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/ec/compal/ene932/acpi/battery.asl b/src/ec/compal/ene932/acpi/battery.asl index 12092c7b8a..f86cb0169e 100644 --- a/src/ec/compal/ene932/acpi/battery.asl +++ b/src/ec/compal/ene932/acpi/battery.asl @@ -84,9 +84,9 @@ Device (BATX) Multiply(Local1, 0x03, Local1) Store (Local1, Index (PBIF, 6)) - Store (ToString(BATD), Index (PBIF, 9)) // model + Store (ToString(Concatenate(BATD, 0x00)), Index (PBIF, 9)) // model Store (ToHexString(BSN0), Index (PBIF, 10)) // serial - Store (ToString(BMFN), Index (PBIF, 12)) // venor + Store (ToString(BMFN), Index (PBIF, 12)) // vendor Store(BDN0, BTNM) // Save the battery number |