diff options
author | Marc Jones <marcj303@gmail.com> | 2018-08-15 22:53:34 -0600 |
---|---|---|
committer | Martin Roth <martinroth@google.com> | 2018-08-17 21:09:45 +0000 |
commit | e17ec3e2e84a8e7cd3f19fb6774356684ae6f25f (patch) | |
tree | e4d46db3e95f7cdc271b3bdccf930691fda451f4 /src/include/sdram_mode.h | |
parent | 4ae02818fcf839959a617fae5f7f685b055133e3 (diff) |
ec/google/chromeec: Fix ACPI FWTS error
Fix the following FWTS error:
FAILED [MEDIUM] AMLAsmASL_MSG_RETURN_TYPES: Test 1, Assembler warning in line
3038
Line | AML source
--------------------------------------------------------------------------------
03035| Return (One)
03036| }
03037|
03038| Method (_Q09, 0, NotSerialized) // _Qxx: EC Query
| ^
| Warning 3115: Not all control paths return a value (_Q09)
03039| {
03040| If (Acquire (PATM, 0x03E8))
03041| {
================================================================================
ADVICE: (for Warning #3115, ASL_MSG_RETURN_TYPES): Some of the execution paths
do not return a value. All control paths that return must return a value
otherwise unexpected behaviour may occur. This error occurs because a branch on
an conditional op-code returns a value and another does not, which is
inconsistent behaviour.
_Q09 is a reserved method and can't return a value. Change the logic
so that no return is used and avoid this test error.
BUG=b:112476331
TEST=Run FWTS.
Change-Id: Ibbda1649ec2eb9cdf9966d4ec92bfd203bb78d07
Signed-off-by: Marc Jones <marcj303@gmail.com>
Reviewed-on: https://review.coreboot.org/28123
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Duncan Laurie <dlaurie@chromium.org>
Diffstat (limited to 'src/include/sdram_mode.h')
0 files changed, 0 insertions, 0 deletions