Attention is currently required from: Martin Roth, Marshall Dawson, Richard Spiegel, Zheng Bao, Matt Papageorge, Felix Held.
Bao Zheng has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58555 )
Change subject: amdfwtool: Set soc name for Stoneyridge
......................................................................
Patch Set 7:
(1 comment)
File src/soc/amd/stoneyridge/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/58555/comment/0306a295_9301ea93
PS5, Line 150: --soc-name "Stoneyridge" \
> not 100% sure, but this might need to depend on FIRMWARE_TYPE; see my comment on CB:55454
Done
--
To view, visit https://review.coreboot.org/c/coreboot/+/58555
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I0e82188ce64733420a578446e22a077ef789be92
Gerrit-Change-Number: 58555
Gerrit-PatchSet: 7
Gerrit-Owner: Bao Zheng <fishbaozi(a)gmail.com>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Matt Papageorge <matthewpapa07(a)gmail.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Richard Spiegel <richard.spiegel(a)silverbackltd.com>
Gerrit-Reviewer: Zheng Bao
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: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Attention: Richard Spiegel <richard.spiegel(a)silverbackltd.com>
Gerrit-Attention: Zheng Bao
Gerrit-Attention: Matt Papageorge <matthewpapa07(a)gmail.com>
Gerrit-Attention: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Comment-Date: Thu, 04 Nov 2021 15:25:10 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-MessageType: comment
Attention is currently required from: Sean Rhodes.
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/58854
to look at the new patch set (#16).
Change subject: [WIP] util: Add coreboot Configurator
......................................................................
[WIP] util: Add coreboot Configurator
A simply GUI to change settings in coreboot's CBFS, via the nvramtool utility.
Signed-off-by: Sean Rhodes <sean(a)starlabs.systems>
Change-Id: Ie654f6aca57a792be0c295afc081a95ed4d875d1
---
A util/coreboot-configurator/README.md
A util/coreboot-configurator/contrib/PKGBUILD
A util/coreboot-configurator/contrib/README.md
A util/coreboot-configurator/contrib/debian/changelog
A util/coreboot-configurator/contrib/debian/compat
A util/coreboot-configurator/contrib/debian/control
A util/coreboot-configurator/contrib/debian/copyright
A util/coreboot-configurator/contrib/debian/files
A util/coreboot-configurator/contrib/debian/rules
A util/coreboot-configurator/contrib/debian/source/format
A util/coreboot-configurator/contrib/flatpak/coreboot-configurator.json
A util/coreboot-configurator/contrib/flatpak/org.coreboot.coreboot-configurator.json
A util/coreboot-configurator/meson.build
A util/coreboot-configurator/meson_options.txt
A util/coreboot-configurator/screenshots/coreboot-configurator.gif
A util/coreboot-configurator/screenshots/coreboot-configurator.png
A util/coreboot-configurator/src/Documentation.txt
A util/coreboot-configurator/src/application/aboutwindow.cpp
A util/coreboot-configurator/src/application/aboutwindow.h
A util/coreboot-configurator/src/application/aboutwindow.ui
A util/coreboot-configurator/src/application/categories.yaml
A util/coreboot-configurator/src/application/configuration.cpp
A util/coreboot-configurator/src/application/configuration.h
A util/coreboot-configurator/src/application/coreboot_configurator.cpp
A util/coreboot-configurator/src/application/coreboot_configurator.h
A util/coreboot-configurator/src/application/coreboot_configurator.ui
A util/coreboot-configurator/src/application/images.qrc
A util/coreboot-configurator/src/application/main.cpp
A util/coreboot-configurator/src/application/meson.build
A util/coreboot-configurator/src/application/qtyaml.h
A util/coreboot-configurator/src/application/toggle-off.svg
A util/coreboot-configurator/src/application/toggle-on.svg
A util/coreboot-configurator/src/meson.build
A util/coreboot-configurator/src/static/aboutIcon.png
A util/coreboot-configurator/src/static/coreboot-configurator.desktop
A util/coreboot-configurator/src/static/coreboot_configurator.svg
A util/coreboot-configurator/src/static/meson.build
A util/coreboot-configurator/src/static/org.coreboot.nvramtool.policy
A util/coreboot-configurator/src/static/org.coreboot.reboot.policy
39 files changed, 2,941 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/54/58854/16
--
To view, visit https://review.coreboot.org/c/coreboot/+/58854
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ie654f6aca57a792be0c295afc081a95ed4d875d1
Gerrit-Change-Number: 58854
Gerrit-PatchSet: 16
Gerrit-Owner: Sean Rhodes <admin(a)starlabs.systems>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Sean Rhodes <admin(a)starlabs.systems>
Gerrit-MessageType: newpatchset
Attention is currently required from: Sean Rhodes.
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/58854
to look at the new patch set (#15).
Change subject: [WIP] util: Add coreboot Configurator
......................................................................
[WIP] util: Add coreboot Configurator
A simply GUI to change settings in coreboot's CBFS, via the nvramtool utility.
Signed-off-by: Sean Rhodes <sean(a)starlabs.systems>
Change-Id: Ie654f6aca57a792be0c295afc081a95ed4d875d1
---
A util/coreboot-configurator/README.md
A util/coreboot-configurator/contrib/PKGBUILD
A util/coreboot-configurator/contrib/README.md
A util/coreboot-configurator/contrib/debian/.debhelper/generated/coreboot-configurator/installed-by-dh_install
A util/coreboot-configurator/contrib/debian/.debhelper/generated/coreboot-configurator/installed-by-dh_installdocs
A util/coreboot-configurator/contrib/debian/changelog
A util/coreboot-configurator/contrib/debian/compat
A util/coreboot-configurator/contrib/debian/control
A util/coreboot-configurator/contrib/debian/copyright
A util/coreboot-configurator/contrib/debian/coreboot-configurator.debhelper.log
A util/coreboot-configurator/contrib/debian/coreboot-configurator.substvars
A util/coreboot-configurator/contrib/debian/debhelper-build-stamp
A util/coreboot-configurator/contrib/debian/files
A util/coreboot-configurator/contrib/debian/rules
A util/coreboot-configurator/contrib/debian/source/format
A util/coreboot-configurator/contrib/flatpak/coreboot-configurator.json
A util/coreboot-configurator/contrib/flatpak/org.coreboot.coreboot-configurator.json
A util/coreboot-configurator/meson.build
A util/coreboot-configurator/meson_options.txt
A util/coreboot-configurator/screenshots/coreboot-configurator.gif
A util/coreboot-configurator/screenshots/coreboot-configurator.png
A util/coreboot-configurator/src/Documentation.txt
A util/coreboot-configurator/src/application/aboutwindow.cpp
A util/coreboot-configurator/src/application/aboutwindow.h
A util/coreboot-configurator/src/application/aboutwindow.ui
A util/coreboot-configurator/src/application/categories.yaml
A util/coreboot-configurator/src/application/configuration.cpp
A util/coreboot-configurator/src/application/configuration.h
A util/coreboot-configurator/src/application/coreboot_configurator.cpp
A util/coreboot-configurator/src/application/coreboot_configurator.h
A util/coreboot-configurator/src/application/coreboot_configurator.ui
A util/coreboot-configurator/src/application/images.qrc
A util/coreboot-configurator/src/application/main.cpp
A util/coreboot-configurator/src/application/meson.build
A util/coreboot-configurator/src/application/qtyaml.h
A util/coreboot-configurator/src/application/toggle-off.svg
A util/coreboot-configurator/src/application/toggle-on.svg
A util/coreboot-configurator/src/meson.build
A util/coreboot-configurator/src/static/aboutIcon.png
A util/coreboot-configurator/src/static/coreboot-configurator.desktop
A util/coreboot-configurator/src/static/coreboot_configurator.svg
A util/coreboot-configurator/src/static/meson.build
A util/coreboot-configurator/src/static/org.coreboot.nvramtool.policy
A util/coreboot-configurator/src/static/org.coreboot.reboot.policy
44 files changed, 2,945 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/54/58854/15
--
To view, visit https://review.coreboot.org/c/coreboot/+/58854
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ie654f6aca57a792be0c295afc081a95ed4d875d1
Gerrit-Change-Number: 58854
Gerrit-PatchSet: 15
Gerrit-Owner: Sean Rhodes <admin(a)starlabs.systems>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Sean Rhodes <admin(a)starlabs.systems>
Gerrit-MessageType: newpatchset
Attention is currently required from: Martin Roth, Marshall Dawson, Richard Spiegel, Zheng Bao, Matt Papageorge.
Bao Zheng has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58555 )
Change subject: amdfwtool: Set soc name for Stoneyridge
......................................................................
Patch Set 7:
(1 comment)
Patchset:
PS7:
Jenkins got confused.
--
To view, visit https://review.coreboot.org/c/coreboot/+/58555
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I0e82188ce64733420a578446e22a077ef789be92
Gerrit-Change-Number: 58555
Gerrit-PatchSet: 7
Gerrit-Owner: Bao Zheng <fishbaozi(a)gmail.com>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Matt Papageorge <matthewpapa07(a)gmail.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Richard Spiegel <richard.spiegel(a)silverbackltd.com>
Gerrit-Reviewer: Zheng Bao
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: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Attention: Richard Spiegel <richard.spiegel(a)silverbackltd.com>
Gerrit-Attention: Zheng Bao
Gerrit-Attention: Matt Papageorge <matthewpapa07(a)gmail.com>
Gerrit-Comment-Date: Thu, 04 Nov 2021 15:22:22 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Jason Glenesk, Raul Rangel, Martin Roth, Marshall Dawson, Zheng Bao, Felix Held.
Bao Zheng has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56773 )
Change subject: amdfwtool: Add support for AMD's BIOS A/B recovery feature
......................................................................
Patch Set 28:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/56773/comment/04d93984_41e13d5a
PS28, Line 12:
> Commit need to be added. […]
Done
--
To view, visit https://review.coreboot.org/c/coreboot/+/56773
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I27f5d3476f648fcecafb8d258ccb6cfad4f50036
Gerrit-Change-Number: 56773
Gerrit-PatchSet: 28
Gerrit-Owner: Bao Zheng <fishbaozi(a)gmail.com>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Jason Glenesk <jason.glenesk(a)gmail.com>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Reviewer: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Reviewer: Zheng Bao
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Bao Zheng <zheng.bao(a)amd.corp-partner.google.com>
Gerrit-CC: Martin Roth <martinroth(a)google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Jason Glenesk <jason.glenesk(a)gmail.com>
Gerrit-Attention: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Attention: Martin Roth <martinroth(a)google.com>
Gerrit-Attention: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Attention: Zheng Bao
Gerrit-Attention: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Comment-Date: Thu, 04 Nov 2021 15:19:53 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Bao Zheng <fishbaozi(a)gmail.com>
Gerrit-MessageType: comment
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/58854
to look at the new patch set (#13).
Change subject: [WIP] util: Add coreboot Configurator
......................................................................
[WIP] util: Add coreboot Configurator
A simply GUI to change settings in coreboot's CBFS, via the nvramtool utility.
Signed-off-by: Sean Rhodes <sean(a)starlabs.systems>
Change-Id: Ie654f6aca57a792be0c295afc081a95ed4d875d1
---
M 3rdparty/blobs
A util/coreboot-configurator/LICENSE
A util/coreboot-configurator/README.md
A util/coreboot-configurator/contrib/PKGBUILD
A util/coreboot-configurator/contrib/README.md
A util/coreboot-configurator/contrib/debian/.debhelper/generated/coreboot-configurator/installed-by-dh_install
A util/coreboot-configurator/contrib/debian/.debhelper/generated/coreboot-configurator/installed-by-dh_installdocs
A util/coreboot-configurator/contrib/debian/changelog
A util/coreboot-configurator/contrib/debian/compat
A util/coreboot-configurator/contrib/debian/control
A util/coreboot-configurator/contrib/debian/copyright
A util/coreboot-configurator/contrib/debian/coreboot-configurator.debhelper.log
A util/coreboot-configurator/contrib/debian/coreboot-configurator.substvars
A util/coreboot-configurator/contrib/debian/coreboot-configurator/DEBIAN/control
A util/coreboot-configurator/contrib/debian/coreboot-configurator/DEBIAN/md5sums
A util/coreboot-configurator/contrib/debian/coreboot-configurator/usr/sbin/coreboot-configurator
A util/coreboot-configurator/contrib/debian/coreboot-configurator/usr/share/applications/coreboot-configurator.desktop
A util/coreboot-configurator/contrib/debian/coreboot-configurator/usr/share/coreboot-configurator/aboutIcon.png
A util/coreboot-configurator/contrib/debian/coreboot-configurator/usr/share/doc/coreboot-configurator/changelog.gz
A util/coreboot-configurator/contrib/debian/coreboot-configurator/usr/share/doc/coreboot-configurator/copyright
A util/coreboot-configurator/contrib/debian/coreboot-configurator/usr/share/icons/hicolor/128x128/apps/coreboot-configurator.png
A util/coreboot-configurator/contrib/debian/coreboot-configurator/usr/share/icons/hicolor/16x16/apps/coreboot-configurator.png
A util/coreboot-configurator/contrib/debian/coreboot-configurator/usr/share/icons/hicolor/24x24/apps/coreboot-configurator.png
A util/coreboot-configurator/contrib/debian/coreboot-configurator/usr/share/icons/hicolor/256x256/apps/coreboot-configurator.png
A util/coreboot-configurator/contrib/debian/coreboot-configurator/usr/share/icons/hicolor/32x32/apps/coreboot-configurator.png
A util/coreboot-configurator/contrib/debian/coreboot-configurator/usr/share/icons/hicolor/48x48/apps/coreboot-configurator.png
A util/coreboot-configurator/contrib/debian/coreboot-configurator/usr/share/icons/hicolor/512x512/apps/coreboot-configurator.png
A util/coreboot-configurator/contrib/debian/coreboot-configurator/usr/share/icons/hicolor/64x64/apps/coreboot-configurator.png
A util/coreboot-configurator/contrib/debian/coreboot-configurator/usr/share/icons/hicolor/96x96/apps/coreboot-configurator.png
A util/coreboot-configurator/contrib/debian/coreboot-configurator/usr/share/polkit-1/actions/org.coreboot.nvramtool.policy
A util/coreboot-configurator/contrib/debian/coreboot-configurator/usr/share/polkit-1/actions/org.coreboot.reboot.policy
A util/coreboot-configurator/contrib/debian/debhelper-build-stamp
A util/coreboot-configurator/contrib/debian/files
A util/coreboot-configurator/contrib/debian/rules
A util/coreboot-configurator/contrib/debian/source/format
A util/coreboot-configurator/contrib/flatpak/coreboot-configurator.json
A util/coreboot-configurator/contrib/flatpak/org.coreboot.coreboot-configurator.json
A util/coreboot-configurator/meson.build
A util/coreboot-configurator/meson_options.txt
A util/coreboot-configurator/screenshots/coreboot-configurator.gif
A util/coreboot-configurator/screenshots/coreboot-configurator.png
A util/coreboot-configurator/src/Documentation.txt
A util/coreboot-configurator/src/application/aboutwindow.cpp
A util/coreboot-configurator/src/application/aboutwindow.h
A util/coreboot-configurator/src/application/aboutwindow.ui
A util/coreboot-configurator/src/application/categories.yaml
A util/coreboot-configurator/src/application/configuration.cpp
A util/coreboot-configurator/src/application/configuration.h
A util/coreboot-configurator/src/application/coreboot_configurator.cpp
A util/coreboot-configurator/src/application/coreboot_configurator.h
A util/coreboot-configurator/src/application/coreboot_configurator.ui
A util/coreboot-configurator/src/application/images.qrc
A util/coreboot-configurator/src/application/main.cpp
A util/coreboot-configurator/src/application/meson.build
A util/coreboot-configurator/src/application/qtyaml.h
A util/coreboot-configurator/src/application/toggle-off.svg
A util/coreboot-configurator/src/application/toggle-on.svg
A util/coreboot-configurator/src/meson.build
A util/coreboot-configurator/src/static/aboutIcon.png
A util/coreboot-configurator/src/static/coreboot-configurator.desktop
A util/coreboot-configurator/src/static/coreboot_configurator.svg
A util/coreboot-configurator/src/static/meson.build
A util/coreboot-configurator/src/static/org.coreboot.nvramtool.policy
A util/coreboot-configurator/src/static/org.coreboot.reboot.policy
64 files changed, 3,367 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/54/58854/13
--
To view, visit https://review.coreboot.org/c/coreboot/+/58854
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ie654f6aca57a792be0c295afc081a95ed4d875d1
Gerrit-Change-Number: 58854
Gerrit-PatchSet: 13
Gerrit-Owner: Sean Rhodes <admin(a)starlabs.systems>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-MessageType: newpatchset
Attention is currently required from: Jason Glenesk, Raul Rangel, Martin Roth, Marshall Dawson, Zheng Bao, Felix Held.
Bao Zheng has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/56773 )
Change subject: amdfwtool: Add support for AMD's BIOS A/B recovery feature
......................................................................
Patch Set 28:
(1 comment)
Patchset:
PS26:
> no. […]
Done.
checked grunt, trembyle, majolica.
--
To view, visit https://review.coreboot.org/c/coreboot/+/56773
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I27f5d3476f648fcecafb8d258ccb6cfad4f50036
Gerrit-Change-Number: 56773
Gerrit-PatchSet: 28
Gerrit-Owner: Bao Zheng <fishbaozi(a)gmail.com>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Jason Glenesk <jason.glenesk(a)gmail.com>
Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Reviewer: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Reviewer: Zheng Bao
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Bao Zheng <zheng.bao(a)amd.corp-partner.google.com>
Gerrit-CC: Martin Roth <martinroth(a)google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Jason Glenesk <jason.glenesk(a)gmail.com>
Gerrit-Attention: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Attention: Martin Roth <martinroth(a)google.com>
Gerrit-Attention: Marshall Dawson <marshalldawson3rd(a)gmail.com>
Gerrit-Attention: Zheng Bao
Gerrit-Attention: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Comment-Date: Thu, 04 Nov 2021 15:17:06 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Bao Zheng <fishbaozi(a)gmail.com>
Comment-In-Reply-To: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-MessageType: comment