coreboot-gerrit November 2021

coreboot-gerrit@coreboot.org

1 participants
6394 discussions

Change in blobs[master]: mb/teleplatforms/D4E4S168: Add blobs
by Дмитрий Понаморев (Code Review) 19 Nov '21

19 Nov '21

Attention is currently required from: Дмитрий Понаморев, Paul Menzel, Kyösti Mälkki. Hello Kyösti Mälkki, I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/blobs/+/57196 to look at the new patch set (#5). Change subject: mb/teleplatforms/D4E4S168: Add blobs ...................................................................... mb/teleplatforms/D4E4S168: Add blobs These files are required to build a complete BIOS image for the teleplatforms/D4E4S16P8 motherboard. Change-Id: I71de576aa283fe6738138ad023d70c282045f5cc Signed-off-by: Dmitry Ponamorev <dponamorev(a)gmail.com> --- A mainboard/teleplatforms/D4E4S16P8/DNV_LAN0_SGMII_MRVL_noMNG_2p10_80000BD8.bin A mainboard/teleplatforms/D4E4S16P8/DNV_LAN1_SFI_noMNG_2p10_80000C01.bin A mainboard/teleplatforms/D4E4S16P8/FSP_M.rom A mainboard/teleplatforms/D4E4S16P8/FSP_S.rom A mainboard/teleplatforms/D4E4S16P8/FSP_T.rom A mainboard/teleplatforms/D4E4S16P8/README.md A mainboard/teleplatforms/D4E4S16P8/bootorder A mainboard/teleplatforms/D4E4S16P8/descriptor.bin A mainboard/teleplatforms/D4E4S16P8/me.bin A mainboard/teleplatforms/D4E4S16P8/pci1a03,2000.rom 10 files changed, 17 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/blobs refs/changes/96/57196/5 -- To view, visit https://review.coreboot.org/c/blobs/+/57196 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: blobs Gerrit-Branch: master Gerrit-Change-Id: I71de576aa283fe6738138ad023d70c282045f5cc Gerrit-Change-Number: 57196 Gerrit-PatchSet: 5 Gerrit-Owner: Дмитрий Понаморев <dponamorev(a)gmail.com> Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Gerrit-CC: Angel Pons <th3fanbus(a)gmail.com> Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: Дмитрий Понаморев <dponamorev(a)gmail.com> Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Attention: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Gerrit-MessageType: newpatchset

1 0

Change in coreboot[master]: mainboard/google/brya: Enable dev screen in bios-stage for Brask
by Adam Liu (Code Review) 19 Nov '21

19 Nov '21

Attention is currently required from: Tim Wawrzynczak. Adam Liu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59414 ) Change subject: mainboard/google/brya: Enable dev screen in bios-stage for Brask ...................................................................... Patch Set 7: (1 comment) File src/mainboard/google/brya/Kconfig: https://review.coreboot.org/c/coreboot/+/59414/comment/6d5871fe_c89eac71 PS3, Line 10: INTEL_GMA_HAVE_VBT > For now, the VBTs are in the private repositories (NB I am working to change this but not done yet), […] The config is unnecessary. Removed it. -- To view, visit https://review.coreboot.org/c/coreboot/+/59414 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I5f34be030a6d819a0e93a2d479c4ff41bb14cfe2 Gerrit-Change-Number: 59414 Gerrit-PatchSet: 7 Gerrit-Owner: Adam Liu <adam.liu(a)quanta.corp-partner.google.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-Comment-Date: Fri, 19 Nov 2021 08:06:26 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-MessageType: comment

1 0

Change in coreboot[master]: [TEST] Updating FSP Binary repo path
by Balaji Manigandan (Code Review) 19 Nov '21

19 Nov '21

Attention is currently required from: Balaji Manigandan. Balaji Manigandan has removed Patrick Rudolph from this change. ( https://review.coreboot.org/c/coreboot/+/59470 ) Change subject: [TEST] Updating FSP Binary repo path ...................................................................... Removed reviewer Patrick Rudolph. -- To view, visit https://review.coreboot.org/c/coreboot/+/59470 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I68d57f039ec81bd9a42861984660764258b63ef8 Gerrit-Change-Number: 59470 Gerrit-PatchSet: 3 Gerrit-Owner: Balaji Manigandan <balaji.manigandan(a)intel.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Balaji Manigandan <balaji.manigandan(a)intel.com> Gerrit-MessageType: deleteReviewer

1 0

Change in coreboot[master]: [TEST] Updating FSP Binary repo path
by Balaji Manigandan (Code Review) 19 Nov '21

19 Nov '21

Attention is currently required from: Balaji Manigandan, Patrick Rudolph. Balaji Manigandan has removed Tim Wawrzynczak from this change. ( https://review.coreboot.org/c/coreboot/+/59470 ) Change subject: [TEST] Updating FSP Binary repo path ...................................................................... Removed reviewer Tim Wawrzynczak. -- To view, visit https://review.coreboot.org/c/coreboot/+/59470 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I68d57f039ec81bd9a42861984660764258b63ef8 Gerrit-Change-Number: 59470 Gerrit-PatchSet: 3 Gerrit-Owner: Balaji Manigandan <balaji.manigandan(a)intel.com> Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Balaji Manigandan <balaji.manigandan(a)intel.com> Gerrit-Attention: Patrick Rudolph <siro(a)das-labor.org> Gerrit-MessageType: deleteReviewer

1 0

Change in coreboot[master]: [TEST] Updating FSP Binary repo path
by Balaji Manigandan (Code Review) 19 Nov '21

19 Nov '21

Attention is currently required from: Tim Wawrzynczak, Balaji Manigandan, Patrick Rudolph. Balaji Manigandan has removed Andrey Petrov from this change. ( https://review.coreboot.org/c/coreboot/+/59470 ) Change subject: [TEST] Updating FSP Binary repo path ...................................................................... Removed reviewer Andrey Petrov. -- To view, visit https://review.coreboot.org/c/coreboot/+/59470 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I68d57f039ec81bd9a42861984660764258b63ef8 Gerrit-Change-Number: 59470 Gerrit-PatchSet: 3 Gerrit-Owner: Balaji Manigandan <balaji.manigandan(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-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-Attention: Balaji Manigandan <balaji.manigandan(a)intel.com> Gerrit-Attention: Patrick Rudolph <siro(a)das-labor.org> Gerrit-MessageType: deleteReviewer

1 0

Change in coreboot[master]: [TEST] Updating FSP Binary repo path
by Balaji Manigandan (Code Review) 19 Nov '21

19 Nov '21

Attention is currently required from: Balaji Manigandan. Hello build bot (Jenkins), I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/coreboot/+/59470 to look at the new patch set (#3). Change subject: [TEST] Updating FSP Binary repo path ...................................................................... [TEST] Updating FSP Binary repo path Signed-off-by: Balaji Manigandan B <balaji.manigandan(a)intel.com> Change-Id: I68d57f039ec81bd9a42861984660764258b63ef8 --- M src/drivers/intel/fsp2_0/Makefile.inc M src/soc/intel/alderlake/Kconfig 2 files changed, 4 insertions(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/70/59470/3 -- To view, visit https://review.coreboot.org/c/coreboot/+/59470 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I68d57f039ec81bd9a42861984660764258b63ef8 Gerrit-Change-Number: 59470 Gerrit-PatchSet: 3 Gerrit-Owner: Balaji Manigandan <balaji.manigandan(a)intel.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Balaji Manigandan <balaji.manigandan(a)intel.com> Gerrit-MessageType: newpatchset

1 0

Change in coreboot[master]: src/mb/brya: Enable crashlog on brya
by Kane Chen (Code Review) 19 Nov '21

19 Nov '21

Attention is currently required from: Gaggery Tsai, Francois Toguo Fotso, Kane Chen, Tim Wawrzynczak. Hello Gaggery Tsai, build bot (Jenkins), Francois Toguo Fotso, Tim Wawrzynczak, I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/coreboot/+/59486 to look at the new patch set (#4). Change subject: src/mb/brya: Enable crashlog on brya ...................................................................... src/mb/brya: Enable crashlog on brya Enabling crashlog helpis partners to debug hang issues efficiently. BUG=b:195327879 TEST=Found BERT table is created and the tcss function is ok in depthcharge. Warm/cold/suspend_stress test pass 50 cycles on gimble Change-Id: Ib4bbe5d7cece0c6c5fc170460d55ac820054abb9 Signed-off-by: Kane Chen <kane.chen(a)intel.corp-partner.google.com> --- M src/mainboard/google/brya/Kconfig 1 file changed, 1 insertion(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/86/59486/4 -- To view, visit https://review.coreboot.org/c/coreboot/+/59486 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ib4bbe5d7cece0c6c5fc170460d55ac820054abb9 Gerrit-Change-Number: 59486 Gerrit-PatchSet: 4 Gerrit-Owner: Kane Chen <kane.chen(a)intel.corp-partner.google.com> Gerrit-Reviewer: Francois Toguo Fotso <francois.toguo.fotso(a)intel.com> Gerrit-Reviewer: Gaggery Tsai <gaggery.tsai(a)intel.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Gaggery Tsai <gaggery.tsai(a)intel.com> Gerrit-Attention: Francois Toguo Fotso <francois.toguo.fotso(a)intel.com> Gerrit-Attention: Kane Chen <kane.chen(a)intel.corp-partner.google.com> Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-MessageType: newpatchset

1 0

Change in coreboot[master]: src/mb/brya: Enable crashlog on brya
by Kane Chen (Code Review) 19 Nov '21

19 Nov '21

Attention is currently required from: Kane Chen, Tim Wawrzynczak. Hello build bot (Jenkins), Tim Wawrzynczak, I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/coreboot/+/59486 to look at the new patch set (#3). Change subject: src/mb/brya: Enable crashlog on brya ...................................................................... src/mb/brya: Enable crashlog on brya Enabling crashlog helpis partners to debug hang issues efficiently. BUG=b:195327879 TEST=Found BERT table is created and the tcss function is ok in depthcharge. Warm/cold/suspend_stress test pass 50 cycles on gimble Change-Id: Ib4bbe5d7cece0c6c5fc170460d55ac820054abb9 --- M src/mainboard/google/brya/Kconfig 1 file changed, 1 insertion(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/86/59486/3 -- To view, visit https://review.coreboot.org/c/coreboot/+/59486 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ib4bbe5d7cece0c6c5fc170460d55ac820054abb9 Gerrit-Change-Number: 59486 Gerrit-PatchSet: 3 Gerrit-Owner: Kane Chen <kane.chen(a)intel.corp-partner.google.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Kane Chen <kane.chen(a)intel.corp-partner.google.com> Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-MessageType: newpatchset

1 0

Change in coreboot[master]: src/mb/brya: Enable crashlog on brya
by Kane Chen (Code Review) 19 Nov '21

19 Nov '21

Attention is currently required from: Tim Wawrzynczak. Hello Tim Wawrzynczak, I'd like you to reexamine a change. Please visit https://review.coreboot.org/c/coreboot/+/59486 to look at the new patch set (#2). Change subject: src/mb/brya: Enable crashlog on brya ...................................................................... src/mb/brya: Enable crashlog on brya Enabling crashlog helpis partners to debug hang issues efficiently. BUG=b:195327879 TEST=Found BERT table is created and the tcss function is ok in depthcharge warm/cold/suspend_stress test pass 50 cycles on gimble Change-Id: Ib4bbe5d7cece0c6c5fc170460d55ac820054abb9 --- M src/mainboard/google/brya/Kconfig 1 file changed, 1 insertion(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/86/59486/2 -- To view, visit https://review.coreboot.org/c/coreboot/+/59486 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ib4bbe5d7cece0c6c5fc170460d55ac820054abb9 Gerrit-Change-Number: 59486 Gerrit-PatchSet: 2 Gerrit-Owner: Kane Chen <kane.chen(a)intel.corp-partner.google.com> Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org> Gerrit-MessageType: newpatchset

1 0

Change in coreboot[master]: src/mb/brya: Enable crashlog on brya
by Kane Chen (Code Review) 19 Nov '21

19 Nov '21

Kane Chen has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/59486 ) Change subject: src/mb/brya: Enable crashlog on brya ...................................................................... src/mb/brya: Enable crashlog on brya Enabling crashlog helpis partners to debug hang issues efficiently. BUG=b:195327879 TEST=Found BERT table is created and the tcss function is ok in depthcharge warm/cold/suspend_stress test pass 50 cycles on gimble Change-Id: Ib4bbe5d7cece0c6c5fc170460d55ac820054abb9 --- M src/mainboard/google/brya/Kconfig 1 file changed, 1 insertion(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/86/59486/1 diff --git a/src/mainboard/google/brya/Kconfig b/src/mainboard/google/brya/Kconfig index 136bde6..a572d34 100644 --- a/src/mainboard/google/brya/Kconfig +++ b/src/mainboard/google/brya/Kconfig @@ -48,6 +48,7 @@ select SOC_INTEL_COMMON_BLOCK_PCIE_RTD3 select SOC_INTEL_CSE_LITE_SKU select INTEL_CAR_NEM #TODO - Enable INTEL_CAR_NEM_ENHANCED + select SOC_INTEL_CRASHLOG config BASEBOARD_DIR string -- To view, visit https://review.coreboot.org/c/coreboot/+/59486 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ib4bbe5d7cece0c6c5fc170460d55ac820054abb9 Gerrit-Change-Number: 59486 Gerrit-PatchSet: 1 Gerrit-Owner: Kane Chen <kane.chen(a)intel.corp-partner.google.com> Gerrit-MessageType: newchange

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

coreboot-gerrit November 2021