Attention is currently required from: Patrick Rudolph, Julius Werner, Yu-Ping Wu.
Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/70295 )
Change subject: mb/hp/z220_series: Drop VBOOT support
......................................................................
Patch Set 2: Code-Review-1
(1 comment)
Patchset:
PS2:
What's the benefit here? I don't understand the problem. Vboot is not enabled by default and must be enabled by the user. The lines you are removing just configure some defaults in case of when Vboot gets enabled.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/70295
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I9dcabda5e8ee3a0084736d3674dca31d1f5a641f
Gerrit-Change-Number: 70295
Gerrit-PatchSet: 2
Gerrit-Owner: Yu-Ping Wu
yupingso@google.com
Gerrit-Reviewer: Angel Pons
th3fanbus@gmail.com
Gerrit-Reviewer: Felix Singer
felixsinger@posteo.net
Gerrit-Reviewer: Julius Werner
jwerner@chromium.org
Gerrit-Reviewer: Patrick Rudolph
patrick.rudolph@9elements.com
Gerrit-Reviewer: Patrick Rudolph
siro@das-labor.org
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Attention: Patrick Rudolph
patrick.rudolph@9elements.com
Gerrit-Attention: Julius Werner
jwerner@chromium.org
Gerrit-Attention: Yu-Ping Wu
yupingso@google.com
Gerrit-Attention: Patrick Rudolph
siro@das-labor.org
Gerrit-Comment-Date: Fri, 06 Jan 2023 03:10:14 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment