Martin Roth has submitted this change. ( https://review.coreboot.org/c/coreboot/+/59415 )
Change subject: Makefile.inc: Don't ignore IASL's "multiple types" warning ......................................................................
Makefile.inc: Don't ignore IASL's "multiple types" warning
Intel Lynx Point ASL code is fixed. So don't ignore "Multiple types (Device object requires either a _HID or _ADR, but not both)" warning.
Change-Id: Ie9398879a76ad3d36454772a1c23da083af14b59 Signed-off-by: Elyes HAOUAS ehaouas@noos.fr Reviewed-on: https://review.coreboot.org/c/coreboot/+/59415 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Martin Roth martinroth@google.com --- M Makefile.inc 1 file changed, 0 insertions(+), 8 deletions(-)
Approvals: build bot (Jenkins): Verified Martin Roth: Looks good to me, approved
diff --git a/Makefile.inc b/Makefile.inc index 1c76a91..beb7f0e 100644 --- a/Makefile.inc +++ b/Makefile.inc @@ -263,17 +263,9 @@ # Redundant offset remarks are not useful in any way and are masking useful # ones that might indicate an issue so it is better to hide them. REDUNDANT_OFFSET_REMARK = 2158 -# Ignore _HID & _ADR coexisting in Intel Lynxpoint ASL code. -# See cb:38802 -# "Multiple types (Device object requires either a _HID or _ADR, but not both)" -MULTIPLE_TYPES_WARNING = 3073
IASL_WARNINGS_LIST = $(EMPTY_RESOURCE_TEMPLATE_WARNING) $(REDUNDANT_OFFSET_REMARK)
-ifeq ($(CONFIG_SOUTHBRIDGE_INTEL_LYNXPOINT),y) -IASL_WARNINGS_LIST += $(MULTIPLE_TYPES_WARNING) -endif - IGNORED_IASL_WARNINGS = $(addprefix -vw , $(IASL_WARNINGS_LIST))
define asl_template