Attention is currently required from: David Hendricks.
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/71167?usp=email
to look at the new patch set (#18).
The following approvals got outdated and were removed:
Verified+1 by build bot (Jenkins)
Change subject: util/intelp2m: Rework platforms and fields packages
......................................................................
util/intelp2m: Rework platforms and fields packages
- Add the base platform interface to be able to reuse its methods.
- The macro structure is excluded from the platform structure.
- Instead of a global macro instance, create a new local structure for
each generated macro.
TEST:
1) 'make test' = PASS
2) './intelp2m -p cnl -iiii -file inteltool.log' = gpio.h before and
after the commit is the same.
Change-Id: I2e0aa56efa2430ac6524c6977f8b6fd13113edf9
Signed-off-by: Maxim Polyakov <max.senia.poliak(a)gmail.com>
---
M util/intelp2m/fields/cb/cb.go
M util/intelp2m/fields/cb/cb_test.go
M util/intelp2m/fields/fields.go
M util/intelp2m/fields/fsp/fsp.go
M util/intelp2m/fields/fsp/fsp_test.go
M util/intelp2m/fields/raw/raw.go
M util/intelp2m/fields/raw/raw_test.go
M util/intelp2m/fields/test/suite.go
M util/intelp2m/parser/parser.go
M util/intelp2m/platforms/adl/adl_test.go
M util/intelp2m/platforms/adl/macro.go
M util/intelp2m/platforms/adl/template.go
M util/intelp2m/platforms/apl/apl_test.go
M util/intelp2m/platforms/apl/macro.go
M util/intelp2m/platforms/apl/template.go
M util/intelp2m/platforms/cnl/cnl_test.go
M util/intelp2m/platforms/cnl/macro.go
M util/intelp2m/platforms/cnl/template.go
A util/intelp2m/platforms/common/common.go
M util/intelp2m/platforms/common/macro.go
M util/intelp2m/platforms/common/register/dw0.go
M util/intelp2m/platforms/common/register/dw1.go
A util/intelp2m/platforms/common/register/helper.go
M util/intelp2m/platforms/ebg/ebg_test.go
M util/intelp2m/platforms/ebg/macro.go
M util/intelp2m/platforms/ebg/template.go
D util/intelp2m/platforms/interfaces.go
M util/intelp2m/platforms/jsl/jsl_test.go
M util/intelp2m/platforms/jsl/macro.go
M util/intelp2m/platforms/jsl/template.go
M util/intelp2m/platforms/lbg/lbg_test.go
M util/intelp2m/platforms/lbg/macro.go
M util/intelp2m/platforms/lbg/template.go
M util/intelp2m/platforms/mtl/macro.go
M util/intelp2m/platforms/mtl/mtl_test.go
M util/intelp2m/platforms/mtl/template.go
A util/intelp2m/platforms/platforms.go
M util/intelp2m/platforms/snr/macro.go
M util/intelp2m/platforms/snr/snr_test.go
M util/intelp2m/platforms/snr/template.go
M util/intelp2m/platforms/test/suite.go
M util/intelp2m/platforms/tgl/macro.go
M util/intelp2m/platforms/tgl/template.go
M util/intelp2m/platforms/tgl/tgl_test.go
44 files changed, 1,183 insertions(+), 1,500 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/67/71167/18
--
To view, visit https://review.coreboot.org/c/coreboot/+/71167?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I2e0aa56efa2430ac6524c6977f8b6fd13113edf9
Gerrit-Change-Number: 71167
Gerrit-PatchSet: 18
Gerrit-Owner: Maxim <max.senia.poliak(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Alicja Michalska <ahplka19(a)gmail.com>
Gerrit-CC: David Hendricks <david.hendricks(a)gmail.com>
Gerrit-Attention: David Hendricks <david.hendricks(a)gmail.com>
Attention is currently required from: Angel Pons, Lean Sheng Tan, Patrick Rudolph.
Martin L Roth has posted comments on this change by Angel Pons. ( https://review.coreboot.org/c/coreboot/+/74122?usp=email )
Change subject: mb/prodrive/atlas: Add initial support for options
......................................................................
Patch Set 24:
This change is ready for review.
--
To view, visit https://review.coreboot.org/c/coreboot/+/74122?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I47585a9a6f94ab5005f2ab63a0df267c0caef231
Gerrit-Change-Number: 74122
Gerrit-PatchSet: 24
Gerrit-Owner: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Benjamin Doron <benjamin.doron00(a)gmail.com>
Gerrit-Reviewer: Christian Walter <christian.walter(a)9elements.com>
Gerrit-Reviewer: Lean Sheng Tan <tanleansheng(a)outlook.com>
Gerrit-Reviewer: Maximilian Brune <maximilian.brune(a)9elements.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Kyösti Mälkki <kyosti.malkki(a)gmail.com>
Gerrit-CC: Martin L Roth <gaumless(a)gmail.com>
Gerrit-CC: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-CC: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Attention: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Lean Sheng Tan <tanleansheng(a)outlook.com>
Gerrit-Comment-Date: Sat, 21 Dec 2024 15:31:20 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Attention is currently required from: Bao Zheng, Felix Held, Julius Werner, Paul Menzel, Zheng Bao.
Maximilian Brune has posted comments on this change by Bao Zheng. ( https://review.coreboot.org/c/coreboot/+/84338?usp=email )
Change subject: amdfwtool: Check fletcher of each header
......................................................................
Patch Set 5:
(3 comments)
File util/amdfwtool/amdfwtool.c:
https://review.coreboot.org/c/coreboot/+/84338/comment/57d10763_a99794b0?us… :
PS5, Line 540: psp_directory_table *dir = directory;
nit:
Use same naming scheme as for the others (e.g. pdir instead of dir)
https://review.coreboot.org/c/coreboot/+/84338/comment/2bb5a0af_fd54015d?us… :
PS5, Line 544: uint32_t cookie = ((psp_directory_table *)directory)->header.cookie;
Do you need the extra type conversion? Can't you just use the `dir` from above?
https://review.coreboot.org/c/coreboot/+/84338/comment/178d4ee7_f6db04cb?us… :
PS5, Line 549: if (directory == NULL)
You should probably do this check before dereferencing `directory` above
--
To view, visit https://review.coreboot.org/c/coreboot/+/84338?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ie7540386027c6449317e9b7c0b4fd22d09d4a3a3
Gerrit-Change-Number: 84338
Gerrit-PatchSet: 5
Gerrit-Owner: Bao Zheng <fishbaozi(a)gmail.com>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Zheng Bao
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-CC: Maximilian Brune <maximilian.brune(a)9elements.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Bao Zheng <fishbaozi(a)gmail.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Julius Werner <jwerner(a)chromium.org>
Gerrit-Attention: Zheng Bao
Gerrit-Attention: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Comment-Date: Sat, 21 Dec 2024 15:17:19 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Attention is currently required from: Felix Singer, Martin L Roth.
Elyes Haouas has posted comments on this change by Felix Singer. ( https://review.coreboot.org/c/coreboot/+/85168?usp=email )
Change subject: util/crossgcc: Update CMake from 3.30.2 to 3.31.3
......................................................................
Patch Set 6: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/85168?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I573dace3b752a0d3c4614ece9c0845f8334e2857
Gerrit-Change-Number: 85168
Gerrit-PatchSet: 6
Gerrit-Owner: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Reviewer: Elyes Haouas <ehaouas(a)noos.fr>
Gerrit-Reviewer: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Attention: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Comment-Date: Sat, 21 Dec 2024 15:15:12 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes