diff options
| author | Martin Roth <martinroth@google.com> | 2015-11-23 11:21:25 -0700 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2015-11-24 22:38:24 +0100 |
| commit | d26c9d603e25b8ae10ed7f7ce36f2661241cde06 (patch) | |
| tree | 8f959e5edb906fc5e26e61bf21cc76f757813c42 /src/mainboard/iwave/iWRainbowG6/acpi | |
| parent | 6149233cce1b220f3b78a301cf26660925eb3c48 (diff) | |
| download | coreboot-d26c9d603e25b8ae10ed7f7ce36f2661241cde06.tar.xz | |
iwave/IWRainBowG6: Fix IASL warning and remark
- Add an empty Operating Region for the empty _REG method
- Serialize _CRS Method
- Remove Kconfig default disabling IASL warnings as errors
Fixes IASL Warning:
dsdt.aml 1362: Method (_REG, 2)
Warning 3079 - ^ _REG has no corresponding Operation Region
Fixes IASL remark:
dsdt.aml 1353: Method (_CRS, 0)
Remark 2120 - ^ Control Method should be made Serialized
(due to creation of named objects within)
Change-Id: Iff01613a6e3238469c1fcb8d74f5e98d18420aaf
Signed-off-by: Martin Roth <martinroth@google.com>
Reviewed-on: https://review.coreboot.org/12515
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Diffstat (limited to 'src/mainboard/iwave/iWRainbowG6/acpi')
| -rw-r--r-- | src/mainboard/iwave/iWRainbowG6/acpi/ec.asl | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/src/mainboard/iwave/iWRainbowG6/acpi/ec.asl b/src/mainboard/iwave/iWRainbowG6/acpi/ec.asl index 1de2ef402c..63abc00ccc 100644 --- a/src/mainboard/iwave/iWRainbowG6/acpi/ec.asl +++ b/src/mainboard/iwave/iWRainbowG6/acpi/ec.asl @@ -19,7 +19,13 @@ Device(EC0) Name (_HID, EISAID("PNP0C09")) Name (_UID, 1) - Method (_CRS, 0) + // _REG method requires that an operation region be defined. + OperationRegion (ERAM, EmbeddedControl, 0x00, 0xff) + Field (ERAM, ByteAcc, Lock, Preserve) + { + } + + Method (_CRS, 0, Serialized) { Name (ECMD, ResourceTemplate() { |