Attention is currently required from: Martin L Roth, Stefan Reinauer, Lean Sheng Tan.
Matt DeVillier has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74926 )
Change subject: payloads/edk2: Hook up PCIe Resizable BARs flag
......................................................................
Patch Set 2: Code-Review+2
(1 comment)
Patchset:
PS2:
tested on any specific board?
--
To view, visit https://review.coreboot.org/c/coreboot/+/74926
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I5cc12d32c5e132b9f99ec650377d7683377c2a9c
Gerrit-Change-Number: 74926
Gerrit-PatchSet: 2
Gerrit-Owner: Lean Sheng Tan <sheng.tan(a)9elements.com>
Gerrit-Reviewer: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-Reviewer: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Reviewer: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-Reviewer: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Attention: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Attention: Lean Sheng Tan <sheng.tan(a)9elements.com>
Gerrit-Comment-Date: Wed, 03 May 2023 15:47:24 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Benjamin Doron, Paul Menzel, Lean Sheng Tan.
Matt DeVillier has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74927 )
Change subject: MAINTAINERS: Add Benjamin, Matt, Sean, Sheng for EDK2 payload
......................................................................
Patch Set 3: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/74927
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I914a7f665e246e7490c127078a5a9795c8334d92
Gerrit-Change-Number: 74927
Gerrit-PatchSet: 3
Gerrit-Owner: Lean Sheng Tan <sheng.tan(a)9elements.com>
Gerrit-Reviewer: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-Reviewer: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Reviewer: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Lean Sheng Tan <sheng.tan(a)9elements.com>
Gerrit-Comment-Date: Wed, 03 May 2023 15:46:26 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Martin L Roth, Matt DeVillier, Stefan Reinauer, Lean Sheng Tan.
Benjamin Doron has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74926 )
Change subject: payloads/edk2: Hook up PCIe Resizable BARs flag
......................................................................
Patch Set 2: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/74926
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I5cc12d32c5e132b9f99ec650377d7683377c2a9c
Gerrit-Change-Number: 74926
Gerrit-PatchSet: 2
Gerrit-Owner: Lean Sheng Tan <sheng.tan(a)9elements.com>
Gerrit-Reviewer: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-Reviewer: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Reviewer: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-Reviewer: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Attention: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-Attention: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Attention: Lean Sheng Tan <sheng.tan(a)9elements.com>
Gerrit-Comment-Date: Wed, 03 May 2023 15:23:47 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Benjamin Doron, Matt DeVillier, Paul Menzel.
Lean Sheng Tan has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74927 )
Change subject: MAINTAINERS: Add Benjamin, Matt, Sean, Sheng for EDK2 payload
......................................................................
Patch Set 3:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/74927/comment/3e6fb51b_5b988d83
PS1, Line 7: MAINTAINERS: Add Maintainers for edk2 payload
> MAINTAINERS: Add Benjamin, Matt, Sean, Lean for EDK2 payload
Done
--
To view, visit https://review.coreboot.org/c/coreboot/+/74927
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I914a7f665e246e7490c127078a5a9795c8334d92
Gerrit-Change-Number: 74927
Gerrit-PatchSet: 3
Gerrit-Owner: Lean Sheng Tan <sheng.tan(a)9elements.com>
Gerrit-Reviewer: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-Reviewer: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Reviewer: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-Attention: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Comment-Date: Wed, 03 May 2023 15:11:57 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-MessageType: comment
Attention is currently required from: Benjamin Doron, Matt DeVillier, Lean Sheng Tan.
Hello build bot (Jenkins), Sean Rhodes, Benjamin Doron, Matt DeVillier, Paul Menzel,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/74927
to look at the new patch set (#3).
Change subject: MAINTAINERS: Add Benjamin, Matt, Sean, Sheng for EDK2 payload
......................................................................
MAINTAINERS: Add Benjamin, Matt, Sean, Sheng for EDK2 payload
Signed-off-by: Lean Sheng Tan <sheng.tan(a)9elements.com>
Change-Id: I914a7f665e246e7490c127078a5a9795c8334d92
---
M MAINTAINERS
1 file changed, 18 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/27/74927/3
--
To view, visit https://review.coreboot.org/c/coreboot/+/74927
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I914a7f665e246e7490c127078a5a9795c8334d92
Gerrit-Change-Number: 74927
Gerrit-PatchSet: 3
Gerrit-Owner: Lean Sheng Tan <sheng.tan(a)9elements.com>
Gerrit-Reviewer: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-Reviewer: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Reviewer: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-Attention: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-Attention: Lean Sheng Tan <sheng.tan(a)9elements.com>
Gerrit-MessageType: newpatchset
Attention is currently required from: Benjamin Doron, Matt DeVillier, Lean Sheng Tan.
Hello build bot (Jenkins), Sean Rhodes, Benjamin Doron, Matt DeVillier, Paul Menzel,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/74927
to look at the new patch set (#2).
Change subject: MAINTAINERS: MAINTAINERS: Add Benjamin, Matt, Sean, Sheng for EDK2 payload
......................................................................
MAINTAINERS: MAINTAINERS: Add Benjamin, Matt, Sean, Sheng for EDK2 payload
Signed-off-by: Lean Sheng Tan <sheng.tan(a)9elements.com>
Change-Id: I914a7f665e246e7490c127078a5a9795c8334d92
---
M MAINTAINERS
1 file changed, 18 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/27/74927/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/74927
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I914a7f665e246e7490c127078a5a9795c8334d92
Gerrit-Change-Number: 74927
Gerrit-PatchSet: 2
Gerrit-Owner: Lean Sheng Tan <sheng.tan(a)9elements.com>
Gerrit-Reviewer: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-Reviewer: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Reviewer: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-Attention: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-Attention: Lean Sheng Tan <sheng.tan(a)9elements.com>
Gerrit-MessageType: newpatchset
Attention is currently required from: Benjamin Doron, Matt DeVillier, Lean Sheng Tan.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74927 )
Change subject: MAINTAINERS: Add Maintainers for edk2 payload
......................................................................
Patch Set 1:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/74927/comment/5fd6e403_86cd91e0
PS1, Line 7: MAINTAINERS: Add Maintainers for edk2 payload
MAINTAINERS: Add Benjamin, Matt, Sean, Lean for EDK2 payload
--
To view, visit https://review.coreboot.org/c/coreboot/+/74927
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I914a7f665e246e7490c127078a5a9795c8334d92
Gerrit-Change-Number: 74927
Gerrit-PatchSet: 1
Gerrit-Owner: Lean Sheng Tan <sheng.tan(a)9elements.com>
Gerrit-Reviewer: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-Reviewer: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Reviewer: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-Attention: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-Attention: Lean Sheng Tan <sheng.tan(a)9elements.com>
Gerrit-Comment-Date: Wed, 03 May 2023 14:59:22 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Kevin3 Yang.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/74889 )
Change subject: mb/google/dedede/var/boxy: Disable EXT_VR for boxy
......................................................................
Patch Set 4:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/74889/comment/f00f4ba0_c8656915
PS4, Line 7: for boxy
Redundant as in prefix.
https://review.coreboot.org/c/coreboot/+/74889/comment/dff0077e_a5ddc80e
PS4, Line 9: Disable EXT_VR for boxy
Please add why it should be disabled.
--
To view, visit https://review.coreboot.org/c/coreboot/+/74889
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ic6667e93de41e84f67363ab7554fe755fe50684a
Gerrit-Change-Number: 74889
Gerrit-PatchSet: 4
Gerrit-Owner: Kevin3 Yang <kevin3.yang(a)lcfc.corp-partner.google.com>
Gerrit-Reviewer: Derek Huang <derekhuang(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Jerry2 Huang <jerry2.huang(a)lcfc.corp-partner.google.com>
Gerrit-CC: Joey Peng <joey.peng(a)lcfc.corp-partner.google.com>
Gerrit-CC: Leo Chou <leo.chou(a)lcfc.corp-partner.google.com>
Gerrit-CC: Melo Chuang <melo.chuang(a)lcfc.corp-partner.google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-CC: Stanley Wu <stanley1.wu(a)lcfc.corp-partner.google.com>
Gerrit-Attention: Kevin3 Yang <kevin3.yang(a)lcfc.corp-partner.google.com>
Gerrit-Comment-Date: Wed, 03 May 2023 14:58:31 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment