coreboot-gerrit December 2021

coreboot-gerrit@coreboot.org

1 participants
5522 discussions

Change in coreboot[master]: mb/google/brya/var/vell: update gpio override
by Paul Menzel (Code Review) 04 Dec '21

04 Dec '21

Attention is currently required from: Kevin Chiu, Tim Wawrzynczak, Nick Vaccaro, Wisley Chen, Shon Wang. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59305 ) Change subject: mb/google/brya/var/vell: update gpio override ...................................................................... Patch Set 15: (2 comments) Commit Message: https://review.coreboot.org/c/coreboot/+/59305/comment/3afb82fd_fabdf067 PS15, Line 9: Configure GPIOs according to schematics : Update initial gpio configuration for redrix Please format it as a list. https://review.coreboot.org/c/coreboot/+/59305/comment/5e5d8df5_b4517f7b PS15, Line 10: redrix What is retrix? -- To view, visit https://review.coreboot.org/c/coreboot/+/59305 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Icc91866f7555c294af7eed9e5d1550e73d8059d0 Gerrit-Change-Number: 59305 Gerrit-PatchSet: 15 Gerrit-Owner: Kevin Chiu <kevin.chiu.17802(a)gmail.com> Gerrit-Reviewer: Kevin Chiu <Kevin.Chiu(a)quantatw.com> Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)google.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-Reviewer: Wisley Chen <wisley.chen(a)quanta.corp-partner.google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-CC: Shon Wang <shon.wang(a)quanta.corp-partner.google.com> Gerrit-Attention: Kevin Chiu <kevin.chiu.17802(a)gmail.com> Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-Attention: Kevin Chiu <Kevin.Chiu(a)quantatw.com> Gerrit-Attention: Nick Vaccaro <nvaccaro(a)google.com> Gerrit-Attention: Wisley Chen <wisley.chen(a)quanta.corp-partner.google.com> Gerrit-Attention: Shon Wang <shon.wang(a)quanta.corp-partner.google.com> Gerrit-Comment-Date: Sat, 04 Dec 2021 11:59:37 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in coreboot[master]: crossgcc: Upgrade acpica to 20210930 version
by Paul Menzel (Code Review) 04 Dec '21

04 Dec '21

Attention is currently required from: HAOUAS Elyes. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59174 ) Change subject: crossgcc: Upgrade acpica to 20210930 version ...................................................................... Patch Set 4: Code-Review+1 (2 comments) Commit Message: https://review.coreboot.org/c/coreboot/+/59174/comment/1a7449df_2c0bb255 PS4, Line 7: crossgcc: Upgrade acpica to 20210930 version Maybe: > Update acpica from 20210331 to 20210930 Patchset: PS4: Does it produce new warnings, which CB:59880 disables? -- To view, visit https://review.coreboot.org/c/coreboot/+/59174 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I3a03b74e95f910b50aa2f7ce502b1e9ad5b6df37 Gerrit-Change-Number: 59174 Gerrit-PatchSet: 4 Gerrit-Owner: HAOUAS Elyes <ehaouas(a)noos.fr> Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: HAOUAS Elyes <ehaouas(a)noos.fr> Gerrit-Comment-Date: Sat, 04 Dec 2021 11:50:18 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in coreboot[master]: crossgcc: Upgrade llvm to 13.0.0 version
by Paul Menzel (Code Review) 04 Dec '21

04 Dec '21

Attention is currently required from: HAOUAS Elyes. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59400 ) Change subject: crossgcc: Upgrade llvm to 13.0.0 version ...................................................................... Patch Set 3: Code-Review+1 (2 comments) Commit Message: https://review.coreboot.org/c/coreboot/+/59400/comment/8df46fb3_dc403847 PS3, Line 7: crossgcc: Upgrade llvm to 13.0.0 version Maybe: > Upgrade LLVM/clang from 12.0.0 to 13.0.0 https://review.coreboot.org/c/coreboot/+/59400/comment/8b662fde_135a58d4 PS3, Line 8: Did you build/run test one board? -- To view, visit https://review.coreboot.org/c/coreboot/+/59400 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I8c550d3528a5b1c891b318c08ecfba3a9255e69c Gerrit-Change-Number: 59400 Gerrit-PatchSet: 3 Gerrit-Owner: HAOUAS Elyes <ehaouas(a)noos.fr> Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: HAOUAS Elyes <ehaouas(a)noos.fr> Gerrit-Comment-Date: Sat, 04 Dec 2021 11:48:37 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in coreboot[master]: crossgcc: Upgrade CMake to 3.22.0 version
by Paul Menzel (Code Review) 04 Dec '21

04 Dec '21

Attention is currently required from: HAOUAS Elyes. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59399 ) Change subject: crossgcc: Upgrade CMake to 3.22.0 version ...................................................................... Patch Set 3: Code-Review+1 -- To view, visit https://review.coreboot.org/c/coreboot/+/59399 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I4272f72dd6ed686dbad5615a0ab44c8c632b5930 Gerrit-Change-Number: 59399 Gerrit-PatchSet: 3 Gerrit-Owner: HAOUAS Elyes <ehaouas(a)noos.fr> Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: HAOUAS Elyes <ehaouas(a)noos.fr> Gerrit-Comment-Date: Sat, 04 Dec 2021 11:47:28 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in coreboot[master]: Makefile.inc: Ignore IASL's "Missing dependency" warning
by Paul Menzel (Code Review) 04 Dec '21

04 Dec '21

Attention is currently required from: Martin Roth, HAOUAS Elyes. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59880 ) Change subject: Makefile.inc: Ignore IASL's "Missing dependency" warning ...................................................................... Patch Set 1: Code-Review+1 (1 comment) Commit Message: https://review.coreboot.org/c/coreboot/+/59880/comment/5fff273d_47c7103a PS1, Line 16: Do you have a list of devices with this warning? -- To view, visit https://review.coreboot.org/c/coreboot/+/59880 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I28b437194f08232727623009372327fec15215dd Gerrit-Change-Number: 59880 Gerrit-PatchSet: 1 Gerrit-Owner: HAOUAS Elyes <ehaouas(a)noos.fr> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: Martin Roth <martinroth(a)google.com> Gerrit-Attention: HAOUAS Elyes <ehaouas(a)noos.fr> Gerrit-Comment-Date: Sat, 04 Dec 2021 11:47:16 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in coreboot[master]: Makefile.inc: Enable iasl multiple types warning
by Paul Menzel (Code Review) 04 Dec '21

04 Dec '21

Attention is currently required from: Martin Roth, Angel Pons, HAOUAS Elyes. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59415 ) Change subject: Makefile.inc: Enable iasl multiple types warning ...................................................................... Patch Set 3: (1 comment) Commit Message: https://review.coreboot.org/c/coreboot/+/59415/comment/9c4ca32b_2156be52 PS3, Line 9: Lynxpoint Lynx Point -- To view, visit https://review.coreboot.org/c/coreboot/+/59415 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ie9398879a76ad3d36454772a1c23da083af14b59 Gerrit-Change-Number: 59415 Gerrit-PatchSet: 3 Gerrit-Owner: HAOUAS Elyes <ehaouas(a)noos.fr> Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com> Gerrit-Reviewer: Martin Roth <martinroth(a)google.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: Martin Roth <martinroth(a)google.com> Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com> Gerrit-Attention: HAOUAS Elyes <ehaouas(a)noos.fr> Gerrit-Comment-Date: Sat, 04 Dec 2021 11:46:30 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Change in coreboot[master]: crossgcc/buildgcc: Clean-up
by Paul Menzel (Code Review) 04 Dec '21

04 Dec '21

Attention is currently required from: HAOUAS Elyes. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59398 ) Change subject: crossgcc/buildgcc: Clean-up ...................................................................... Patch Set 2: Code-Review+1 (2 comments) Commit Message: https://review.coreboot.org/c/coreboot/+/59398/comment/b32bc734_4d4c1c7a PS2, Line 7: Clean-up Please use your summary below: > Remove unused GCC_AUTOCONF_VERSION https://review.coreboot.org/c/coreboot/+/59398/comment/c4b4c628_a6c6703f PS2, Line 10: Please add: > Fixes: d0f83723 (buildgcc: Don't run autoconf) -- To view, visit https://review.coreboot.org/c/coreboot/+/59398 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I7d293ae2c8663efdc9ad4146ff32671ffd3e176a Gerrit-Change-Number: 59398 Gerrit-PatchSet: 2 Gerrit-Owner: HAOUAS Elyes <ehaouas(a)noos.fr> Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: HAOUAS Elyes <ehaouas(a)noos.fr> Gerrit-Comment-Date: Sat, 04 Dec 2021 11:44:25 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in coreboot[master]: crossgcc/Makefile: Clean-up
by Paul Menzel (Code Review) 04 Dec '21

04 Dec '21

Attention is currently required from: HAOUAS Elyes. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59397 ) Change subject: crossgcc/Makefile: Clean-up ...................................................................... Patch Set 2: Code-Review+1 (3 comments) Commit Message: https://review.coreboot.org/c/coreboot/+/59397/comment/58648720_83d6095b PS2, Line 7: Clean-up Remove obsolete target build_make https://review.coreboot.org/c/coreboot/+/59397/comment/40f298c0_3c39642a PS2, Line 9: do does https://review.coreboot.org/c/coreboot/+/59397/comment/56d54bb5_f659e86d PS2, Line 9: coreboot do not build "Make" anymore. Since what commit? -- To view, visit https://review.coreboot.org/c/coreboot/+/59397 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I0f159fc912d09ebde6ac7ba5be83933aa251f1d5 Gerrit-Change-Number: 59397 Gerrit-PatchSet: 2 Gerrit-Owner: HAOUAS Elyes <ehaouas(a)noos.fr> Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: HAOUAS Elyes <ehaouas(a)noos.fr> Gerrit-Comment-Date: Sat, 04 Dec 2021 11:41:36 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

Change in coreboot[master]: Makefile.inc: Ignore IASL's "Missing dependency" warning
by HAOUAS Elyes (Code Review) 04 Dec '21

04 Dec '21

HAOUAS Elyes has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/59880 ) Change subject: Makefile.inc: Ignore IASL's "Missing dependency" warning ...................................................................... Makefile.inc: Ignore IASL's "Missing dependency" warning IASL compiler check for usage of _CRS, _DIS, _PRS, and _SRS objects: 1) If _PRS is present, must have _CRS and _SRS 2) If _SRS is present, must have _PRS (_PRS requires _CRS and _SRS) 3) If _DIS is present, must have _SRS (_SRS requires _PRS, _PRS requires _CRS and _SRS) 4) If _SRS is present, probably should have a _DIS (Remark only) IASL will issue a warning for each missing dependency. Ignore this warnings for existing ASL code. Change-Id: I28b437194f08232727623009372327fec15215dd Signed-off-by: Elyes HAOUAS <ehaouas(a)noos.fr> --- M Makefile.inc 1 file changed, 219 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/80/59880/1 diff --git a/Makefile.inc b/Makefile.inc index 971a56c..5dcc40d 100644 --- a/Makefile.inc +++ b/Makefile.inc @@ -263,9 +263,228 @@ # 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 +# IASL compiler check for usage of _CRS, _DIS, _PRS, and _SRS objects: +# 1) If _PRS is present, must have _CRS and _SRS +# 2) If _SRS is present, must have _PRS (_PRS requires _CRS and _SRS) +# 3) If _DIS is present, must have _SRS (_SRS requires _PRS, _PRS requires _CRS and _SRS) +# 4) If _SRS is present, probably should have a _DIS (Remark only) +# A warning will be issued for each of these cases. +# For existing ASL code, ignore this warnings +IASL_MISSING_DEPENDENCY = 3141 IASL_WARNINGS_LIST = $(EMPTY_RESOURCE_TEMPLATE_WARNING) $(REDUNDANT_OFFSET_REMARK) +# Please fix your ASL code and remove dedicated lines below. +ifeq ($(CONFIG_BOARD_ACER_G43T_AM3),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_ASROCK_H81M_HDS),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_ASUS_AM1I_A),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_ASUS_P5G41T_M_LX),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_ASUS_P5QPL_AM),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_ASUS_P8H61_M_LX),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_BIOSTAR_AM1ML),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_BOARD_INTEL_ADLRVP_COMMON),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_DELL_OPTIPLEX_9010),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_DELL_OPTIPLEX_9010_SFF),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_FACEBOOK_FBG1701),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_FACEBOOK_MONOLITH),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_FOXCONN_D41S),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_FOXCONN_G41M),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_FOXCONN_G41M_),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_FOXCONN_G41S_K),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GIGABYTE_GA_D510UD),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_BASEBOARD_BELTINO),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_BASEBOARD_KAHLEE),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_BASEBOARD_SLIPPY),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_ZORK),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_BASEBOARD_AURON),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_BASEBOARD_BRYA),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_BUTTERFLY),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_BASEBOARD_CYAN),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BASEBOARD_DEDEDE_LAPTOP),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_BASEBOARD_DEDEDE),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_BASEBOARD_DEDEDE_TPM2),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_BASEBOARD_DELTAUR),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_BASEBOARD_DRALLION),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_EVE),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_BASEBOARD_FIZZ),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_BASEBOARD_GLADOS),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_BASEBOARD_GUYBRUSH),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_BASEBOARD_HATCH),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_BASEBOARD_JECHT),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_LINK),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_BASEBOARD_OCTOPUS),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BASEBOARD_OCTOPUS_LAPTOP),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_BASEBOARD_POPPY),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_BASEBOARD_RAMBI),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_BASEBOARD_REEF),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BASEBOARD_REEF_LAPTOP),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_BASEBOARD_SARIEN),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_STOUT),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_GOOGLE_BASEBOARD_VOLTEER),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_IBASE_MB899),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_INTEL_BASKING_RIDGE),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_INTEL_DCP847SKE),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_INTEL_DG41WV),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_INTEL_DG43GT),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_INTEL_EMERALDLAKE2),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BASEBOARD_GLKRVP_LAPTOP),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_INTEL_BASEBOARD_GLKRVP),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_INTEL_JASPERLAKE_RVP_EXT_EC),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_INTEL_KBLRVP_COMMON),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_INTEL_KUNIMITSU),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_INTEL_SHADOWMOUNTAIN),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_INTEL_STRAGO),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_INTEL_TGLRVP_UP3),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_INTEL_TGLRVP_UP4),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_KONTRON_986LCD_M),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_ODE_E20XX),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_PORTWELL_M107),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_PROTECTLI_FW2B),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_PROTECTLI_FW4B),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_SAMSUNG_LUMPY),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_SAMSUNG_STUMPY),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_SUPERMICRO_X10SLM_PLUS_F),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif +ifeq ($(CONFIG_BOARD_SUPERMICRO_X9SCL),y) +IASL_WARNINGS_LIST += $(IASL_MISSING_DEPENDENCY) +endif IGNORED_IASL_WARNINGS = $(addprefix -vw , $(IASL_WARNINGS_LIST)) define asl_template -- To view, visit https://review.coreboot.org/c/coreboot/+/59880 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I28b437194f08232727623009372327fec15215dd Gerrit-Change-Number: 59880 Gerrit-PatchSet: 1 Gerrit-Owner: HAOUAS Elyes <ehaouas(a)noos.fr> Gerrit-MessageType: newchange

1 0

Change in coreboot[master]: soc/intel/common: Rename compare_cse_version() function name
by 9elements QA (Code Review) 04 Dec '21

04 Dec '21

9elements QA has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59689 ) Change subject: soc/intel/common: Rename compare_cse_version() function name ...................................................................... Patch Set 10: Automatic boot test returned (PASS/FAIL/TOTAL): 5 / 4 / 9 FAIL: x86_32 "ThinkPad T500" , build config LENOVO_T500 and payload SeaBIOS : https://lava.9esec.io/r/83402 FAIL: x86_64 "HP Compaq 8200 Elite SFF PC" , build config HP_COMPAQ_8200_ELITE_SFF_PC.X86_64 and payload SeaBIOS : https://lava.9esec.io/r/83400 FAIL: x86_32 "HP Compaq 8200 Elite SFF PC" , build config HP_COMPAQ_8200_ELITE_SFF_PC and payload SeaBIOS : https://lava.9esec.io/r/83399 FAIL: x86_32 "QEMU x86 q35/ich9" , build config EMULATION_QEMU_X86_Q35_SMM_TSEG and payload SeaBIOS : https://lava.9esec.io/r/83398 PASS: x86_32 "QEMU x86 q35/ich9" , build config EMULATION_QEMU_X86_Q35 and payload SeaBIOS : https://lava.9esec.io/r/83397 PASS: x86_64 "QEMU x86 i440fx/piix4" , build config EMULATION_QEMU_X86_I440FX_X86_64 and payload SeaBIOS : https://lava.9esec.io/r/83396 PASS: x86_32 "QEMU x86 i440fx/piix4" , build config EMULATION_QEMU_X86_I440FX_ASAN and payload SeaBIOS : https://lava.9esec.io/r/83395 PASS: x86_32 "QEMU x86 i440fx/piix4" , build config EMULATION_QEMU_X86_I440FX_ and payload SeaBIOS : https://lava.9esec.io/r/83394 PASS: x86_32 "QEMU x86 i440fx/piix4" , build config EMULATION_QEMU_X86_I440FX and payload SeaBIOS : https://lava.9esec.io/r/83393 Please note: This test is under development and might not be accurate at all! -- To view, visit https://review.coreboot.org/c/coreboot/+/59689 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I88a44a3c0ba2ad8a589602a35ea644dab535b287 Gerrit-Change-Number: 59689 Gerrit-PatchSet: 10 Gerrit-Owner: Sridhar Siricilla <sridhar.siricilla(a)intel.com> Gerrit-Reviewer: Krishna P Bhat D <krishna.p.bhat.d(a)intel.com> Gerrit-Reviewer: Maulik V Vaghela <maulik.v.vaghela(a)intel.com> Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: 9elements QA <hardwaretestrobot(a)gmail.com> Gerrit-CC: Anfernee Chen <anfernee_chen(a)wistron.corp-partner.google.com> Gerrit-CC: Kane Chen <kane.chen(a)intel.com> Gerrit-CC: Mark Hsieh <mark_hsieh(a)wistron.corp-partner.google.com> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-CC: Sridhar Siricilla <sridhar.siricilla(a)intel.corp-partner.google.com> Gerrit-Comment-Date: Sat, 04 Dec 2021 11:00:24 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

coreboot-gerrit December 2021