Attention is currently required from: Sean Rhodes, Martin Roth, Tim Wawrzynczak, Angel Pons.
Hello build bot (Jenkins), Martin Roth, Tim Wawrzynczak, Angel Pons,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/58854
to look at the new patch set (#20).
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/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/obj-x86_64-linux-gnu/meson-info/meson-info.json
A util/coreboot-configurator/obj-x86_64-linux-gnu/meson-logs/meson-log.txt
A util/coreboot-configurator/obj-x86_64-linux-gnu/meson-private/meson.lock
A util/coreboot-configurator/obj-x86_64-linux-gnu/meson-private/sanitycheckcpp.cc
A util/coreboot-configurator/obj-x86_64-linux-gnu/meson-private/sanitycheckcpp.exe
A util/coreboot-configurator/screenshots/coreboot-configurator.gif
A util/coreboot-configurator/screenshots/coreboot-configurator.png
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/application/toggleswitch.cpp
A util/coreboot-configurator/src/application/toggleswitch.h
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, 3,197 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/54/58854/20
--
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: 20
Gerrit-Owner: Sean Rhodes <admin(a)starlabs.systems>
Gerrit-Reviewer: Angel Pons <angel.pons(a)9elements.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Sean Rhodes <admin(a)starlabs.systems>
Gerrit-Attention: Martin Roth <martinroth(a)google.com>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)google.com>
Gerrit-Attention: Angel Pons <angel.pons(a)9elements.com>
Gerrit-MessageType: newpatchset
Attention is currently required from: Sean Rhodes, Martin Roth, Tim Wawrzynczak, Angel Pons.
Hello build bot (Jenkins), Martin Roth, Tim Wawrzynczak, Angel Pons,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/58854
to look at the new patch set (#19).
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,927 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/54/58854/19
--
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: 19
Gerrit-Owner: Sean Rhodes <admin(a)starlabs.systems>
Gerrit-Reviewer: Angel Pons <angel.pons(a)9elements.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Sean Rhodes <admin(a)starlabs.systems>
Gerrit-Attention: Martin Roth <martinroth(a)google.com>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)google.com>
Gerrit-Attention: Angel Pons <angel.pons(a)9elements.com>
Gerrit-MessageType: newpatchset
Attention is currently required from: Sean Rhodes, Martin Roth, Tim Wawrzynczak, Angel Pons.
Hello build bot (Jenkins), Martin Roth, Tim Wawrzynczak, Angel Pons,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/58854
to look at the new patch set (#18).
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,928 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/54/58854/18
--
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: 18
Gerrit-Owner: Sean Rhodes <admin(a)starlabs.systems>
Gerrit-Reviewer: Angel Pons <angel.pons(a)9elements.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Sean Rhodes <admin(a)starlabs.systems>
Gerrit-Attention: Martin Roth <martinroth(a)google.com>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)google.com>
Gerrit-Attention: Angel Pons <angel.pons(a)9elements.com>
Gerrit-MessageType: newpatchset
Attention is currently required from: Martin Roth, Tim Wawrzynczak, Angel Pons.
Hello build bot (Jenkins), Martin Roth, Tim Wawrzynczak, Angel Pons,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/58854
to look at the new patch set (#17).
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,958 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/54/58854/17
--
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: 17
Gerrit-Owner: Sean Rhodes <admin(a)starlabs.systems>
Gerrit-Reviewer: Angel Pons <angel.pons(a)9elements.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Tim Wawrzynczak <twawrzynczak(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Martin Roth <martinroth(a)google.com>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)google.com>
Gerrit-Attention: Angel Pons <angel.pons(a)9elements.com>
Gerrit-MessageType: newpatchset
Attention is currently required from: Paul Menzel, Tim Wawrzynczak.
Reka Norman has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58929 )
Change subject: spd: Add new LP5 parts and generate SPDs
......................................................................
Patch Set 3:
(3 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/58929/comment/f3178d6c_14ff15b1
PS2, Line 12: MT62F512M32D2DR-031 WT:B
: MT62F1G32D4DR-031 WT:B
: H9JCNNNCP3MLYR-N6E
> Please add the vendors.
Done
File spd/lp5/memory_parts.json:
https://review.coreboot.org/c/coreboot/+/58929/comment/a4fe8e4a_8769dd13
PS2, Line 19: 2,
> Perhaps I'm misunderstanding the datasheet, but isn't this also 1 like the 512M32?
I was basing this off Table 2 in the datasheet, which shows 2 ranks per channel.
Also, the block diagram in Figure 3 shows 2 CSs for each channel: CS0_A/B and CS1_A/B.
https://review.coreboot.org/c/coreboot/+/58929/comment/8638de8d_743b3398
PS2, Line 27: 4,
> I only see one ZQ ball per package on this part.
I know for LP4x there was one ZQ ball per die, so this was used as an indicator of the number of dies. But I think it's different for LP5 because multiple dies can share a ZQ ball.
From 16.6 ZQ Wiring in JESD209-5B:
> LPDDR5 devices are designed to allow up to NZQ die within a single package to connect to a common ZQ resource. When multiple die share a ZQ resource, one die is designated as the Initiator die.
The block diagram on this part's datasheet also shows 4 dies sharing a single ZQ pin.
I might be missing something, but based on the LP5 tables in advisory #616599, I think for LP5 the dies per package is literally just the number of physical dies. I.e. since the datasheet says QDP and show 4 dies in the block diagram, diesPerPackage should be 4.
--
To view, visit https://review.coreboot.org/c/coreboot/+/58929
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ifbcadfb78281b2b78a61a9b61180c421748193a0
Gerrit-Change-Number: 58929
Gerrit-PatchSet: 3
Gerrit-Owner: Reka Norman <rekanorman(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Karthik Ramasubramanian <kramasub(a)google.com>
Gerrit-CC: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-CC: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Comment-Date: Thu, 04 Nov 2021 21:29:33 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Paul Menzel <paulepanter(a)mailbox.org>
Comment-In-Reply-To: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-MessageType: comment
Attention is currently required from: Reka Norman.
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/58929
to look at the new patch set (#3).
Change subject: spd: Add new LP5 parts and generate SPDs
......................................................................
spd: Add new LP5 parts and generate SPDs
Add the parts below which will be used by the brya variant Vell. Add
the parts to memory_parts.json and generate the SPDs using spd_gen.
Micron MT62F512M32D2DR-031 WT:B
Micron MT62F1G32D4DR-031 WT:B
Hynix H9JCNNNCP3MLYR-N6E
Generated using:
util/spd_tools/bin/spd_gen spd/lp5/memory_parts.json lp5
BUG=b:204284866
TEST=None
Change-Id: Ifbcadfb78281b2b78a61a9b61180c421748193a0
Signed-off-by: Reka Norman <rekanorman(a)google.com>
---
M spd/lp5/memory_parts.json
M spd/lp5/set-0/parts_spd_manifest.generated.txt
A spd/lp5/set-0/spd-1.hex
A spd/lp5/set-0/spd-2.hex
4 files changed, 99 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/29/58929/3
--
To view, visit https://review.coreboot.org/c/coreboot/+/58929
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ifbcadfb78281b2b78a61a9b61180c421748193a0
Gerrit-Change-Number: 58929
Gerrit-PatchSet: 3
Gerrit-Owner: Reka Norman <rekanorman(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Karthik Ramasubramanian <kramasub(a)google.com>
Gerrit-CC: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-CC: Tim Wawrzynczak <twawrzynczak(a)chromium.org>
Gerrit-Attention: Reka Norman <rekanorman(a)chromium.org>
Gerrit-MessageType: newpatchset