coreboot-gerrit October 2022

coreboot-gerrit@coreboot.org

1 participants
5881 discussions

[S] Change in coreboot[master]: Update intel-sec-tools submodule to upstream master
by build bot (Jenkins) (Code Review) 21 Oct '22

21 Oct '22

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68654 ) Change subject: Update intel-sec-tools submodule to upstream master ...................................................................... Patch Set 1: (2 comments) Commit Message: Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-160993): https://review.coreboot.org/c/coreboot/+/68654/comment/d3508e96_d5e983a0 PS1, Line 9: Updating from commit id 0031ac7: Possible unwrapped commit description (prefer a maximum 72 chars per line) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-160993): https://review.coreboot.org/c/coreboot/+/68654/comment/27785d4b_990ea06e PS1, Line 12: to commit id a4552d6: Possible unwrapped commit description (prefer a maximum 72 chars per line) -- To view, visit https://review.coreboot.org/c/coreboot/+/68654 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I0769b3b90c58cece514d9e0ec4c08653ec57ddaf Gerrit-Change-Number: 68654 Gerrit-PatchSet: 1 Gerrit-Owner: Elyes Haouas <ehaouas(a)noos.fr> Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 21 Oct 2022 12:14:02 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

[M] Change in coreboot[master]: Update fsp submodule to upstream master
by build bot (Jenkins) (Code Review) 21 Oct '22

21 Oct '22

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68653 ) Change subject: Update fsp submodule to upstream master ...................................................................... Patch Set 1: (3 comments) Commit Message: Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-160992): https://review.coreboot.org/c/coreboot/+/68653/comment/7cffb72e_4a85dd17 PS1, Line 20: e14d687 Delete ElkhartLake_FSP_Integration_Guide.pdf 'Readme' may be misspelled - perhaps 'Readme'? Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-160992): https://review.coreboot.org/c/coreboot/+/68653/comment/02ab84cc_f3168d05 PS1, Line 21: 7c1f6a2 Readme.md 'README' may be misspelled - perhaps 'README'? Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-160992): https://review.coreboot.org/c/coreboot/+/68653/comment/035fbc88_0418611f PS1, Line 44: 13a144f Added IoT ADL-PS PV (3354_00) FSP 'README' may be misspelled - perhaps 'README'? -- To view, visit https://review.coreboot.org/c/coreboot/+/68653 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I5d8984ed79e6b1ce4978a83e180b569493f47a18 Gerrit-Change-Number: 68653 Gerrit-PatchSet: 1 Gerrit-Owner: Elyes Haouas <ehaouas(a)noos.fr> Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 21 Oct 2022 12:13:40 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

[S] Change in coreboot[master]: Update cmocka submodule to upstream master
by build bot (Jenkins) (Code Review) 21 Oct '22

21 Oct '22

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68652 ) Change subject: Update cmocka submodule to upstream master ...................................................................... Patch Set 1: (2 comments) Commit Message: Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-160991): https://review.coreboot.org/c/coreboot/+/68652/comment/7bba851e_67ac85a2 PS1, Line 9: Updating from commit id 8931845: Possible unwrapped commit description (prefer a maximum 72 chars per line) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-160991): https://review.coreboot.org/c/coreboot/+/68652/comment/65b1a920_9b1073f0 PS1, Line 12: to commit id 59dc001: Possible unwrapped commit description (prefer a maximum 72 chars per line) -- To view, visit https://review.coreboot.org/c/coreboot/+/68652 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I424db2eff6fcdcadd117296292c6dc1fc947ad4c Gerrit-Change-Number: 68652 Gerrit-PatchSet: 1 Gerrit-Owner: Elyes Haouas <ehaouas(a)noos.fr> Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 21 Oct 2022 12:13:14 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

[S] Change in coreboot[master]: Update arm-trusted-firmware submodule to upstream master
by build bot (Jenkins) (Code Review) 21 Oct '22

21 Oct '22

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68651 ) Change subject: Update arm-trusted-firmware submodule to upstream master ...................................................................... Patch Set 1: (6 comments) Commit Message: Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-160990): https://review.coreboot.org/c/coreboot/+/68651/comment/46ef8780_0e08e55f PS1, Line 9: Updating from commit id c45d2febb: Possible unwrapped commit description (prefer a maximum 72 chars per line) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-160990): https://review.coreboot.org/c/coreboot/+/68651/comment/3926f227_b5424bab PS1, Line 12: to commit id 61fe7826d: Possible unwrapped commit description (prefer a maximum 72 chars per line) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-160990): https://review.coreboot.org/c/coreboot/+/68651/comment/c521310b_bd4764a1 PS1, Line 15: This brings in 10 new commits: Possible unwrapped commit description (prefer a maximum 72 chars per line) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-160990): https://review.coreboot.org/c/coreboot/+/68651/comment/65db28ab_9c771b03 PS1, Line 17: cf17f7c45 Merge "chore(rpi3): remove redundant code" into integration Possible unwrapped commit description (prefer a maximum 72 chars per line) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-160990): https://review.coreboot.org/c/coreboot/+/68651/comment/93a8ca9d_093f3b2d PS1, Line 19: 60c439435 docs(maintainers): add NPU driver owners Possible unwrapped commit description (prefer a maximum 72 chars per line) Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-160990): https://review.coreboot.org/c/coreboot/+/68651/comment/5f1a14b5_ffd3a555 PS1, Line 22: 0271eddb0 feat(fvp): build delegated attestation in BL31 Possible unwrapped commit description (prefer a maximum 72 chars per line) -- To view, visit https://review.coreboot.org/c/coreboot/+/68651 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ic32c0889961f529e1762b208ef118a94369c34e6 Gerrit-Change-Number: 68651 Gerrit-PatchSet: 1 Gerrit-Owner: Elyes Haouas <ehaouas(a)noos.fr> Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 21 Oct 2022 12:13:04 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

[S] Change in coreboot[master]: lib/Kconfig: Disable `DECOMPRESS_OFAST` for x86_64
by Arthur Heymans (Code Review) 21 Oct '22

21 Oct '22

Attention is currently required from: Nico Huber, Martin L Roth, Angel Pons, Felix Held. Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68648 ) Change subject: lib/Kconfig: Disable `DECOMPRESS_OFAST` for x86_64 ...................................................................... Patch Set 1: (1 comment) File src/lib/Kconfig: https://review.coreboot.org/c/coreboot/+/68648/comment/19fdb56f_99eb2098 PS1, Line 121: USE_EXP_X86_64_SUPPORT Can you use 'ARCH_RAMSTAGE_X86_64' (that's a good enough proxy for needing lzma decompression code + 64bit)? USE_EXP_X86_64_SUPPORT is more like a toggle option and may not be present on some platforms where only 64bit is used, which seems likely to happen at some point. -- To view, visit https://review.coreboot.org/c/coreboot/+/68648 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I4e1475b71754cdda309b6c4ed72086c084dbcb1f Gerrit-Change-Number: 68648 Gerrit-PatchSet: 1 Gerrit-Owner: Angel Pons <th3fanbus(a)gmail.com> Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de> Gerrit-Reviewer: Felix Singer <felixsinger(a)posteo.net> Gerrit-Reviewer: Martin L Roth <gaumless(a)gmail.com> Gerrit-Reviewer: Marvin Drees <marvin.drees(a)9elements.com> Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-CC: Bao Zheng <zheng.bao(a)amd.corp-partner.google.com> Gerrit-CC: Julius Werner <jwerner(a)chromium.org> Gerrit-Attention: Nico Huber <nico.h(a)gmx.de> Gerrit-Attention: Martin L Roth <gaumless(a)gmail.com> Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com> Gerrit-Attention: Felix Held <felix-coreboot(a)felixheld.de> Gerrit-Comment-Date: Fri, 21 Oct 2022 12:12:55 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

[S] Change in coreboot[master]: Update opensbi submodule to upstream master
by Elyes Haouas (Code Review) 21 Oct '22

21 Oct '22

Elyes Haouas has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/68655 ) Change subject: Update opensbi submodule to upstream master ...................................................................... Update opensbi submodule to upstream master Updating from commit id 5019fd1: 2022-09-01 16:53:28 +0530 - (include: sbi: Reduce includes in sbi_pmu.h) to commit id b6e520b: 2022-10-13 09:52:09 +0530 - (platform: generic: allwinner: add support for c9xx pmu) This brings in 14 new commits: b6e520b platform: generic: allwinner: add support for c9xx pmu 2f63f24 platform: generic: add extensions_init handler and platform-override 4f2acb5 lib: sbi_platform: expose hart_features to extension_init callback c316fa3 lib: sbi_hart: move hart_features struct to a public location e54cb32 lib: sbi_pmu: move pmu irq information into pmu itself 3f3d401 docs: Fix some typos 7105c18 docs/firmware: Update FW_JUMP documentation 7f09fba lib: utils/serial: add semihosting support 49372f2 lib: sbi: Fix sbi_strnlen wrong count decrement 8ce486a lib: utils/fdt: Fix DT parsing in fdt_pmu_setup() 46e744a lib: sbi_misaligned_ldst: Set GVA if not emulating 37a0d83 lib: sbi_trap: Add helper to get GVA in sbi_trap_regs 19664f6 docs: pmu: extend bindings example for Unmatched d32b0a9 docs: pmu: fix Unmatched example typo Change-Id: I1e90f9f89effa4d1e641723859885d5874f4c7c0 Signed-off-by: Elyes Haouas <ehaouas(a)noos.fr> --- M 3rdparty/opensbi 1 file changed, 33 insertions(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/55/68655/1 diff --git a/3rdparty/opensbi b/3rdparty/opensbi index 5019fd1..b6e520b 160000 --- a/3rdparty/opensbi +++ b/3rdparty/opensbi @@ -1 +1 @@ -Subproject commit 5019fd124b4c46e1581129c5154fc2cdd3b777ed +Subproject commit b6e520b2a836cd7cc8dc99c25a21a470e8589888 -- To view, visit https://review.coreboot.org/c/coreboot/+/68655 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I1e90f9f89effa4d1e641723859885d5874f4c7c0 Gerrit-Change-Number: 68655 Gerrit-PatchSet: 1 Gerrit-Owner: Elyes Haouas <ehaouas(a)noos.fr> Gerrit-MessageType: newchange

1 0

[S] Change in coreboot[master]: Update intel-sec-tools submodule to upstream master
by Elyes Haouas (Code Review) 21 Oct '22

21 Oct '22

Elyes Haouas has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/68654 ) Change subject: Update intel-sec-tools submodule to upstream master ...................................................................... Update intel-sec-tools submodule to upstream master Updating from commit id 0031ac7: 2021-06-22 14:22:03 +0200 - (Merge pull request #245 from 9elements/meparser) to commit id a4552d6: 2022-10-04 11:17:05 +0100 - (Merge pull request #335 from 9elements/feature/uefi_node_add_offset) This brings in 172 new commits. Change-Id: I0769b3b90c58cece514d9e0ec4c08653ec57ddaf Signed-off-by: Elyes Haouas <ehaouas(a)noos.fr> --- M 3rdparty/intel-sec-tools 1 file changed, 19 insertions(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/54/68654/1 diff --git a/3rdparty/intel-sec-tools b/3rdparty/intel-sec-tools index 0031ac7..a4552d6 160000 --- a/3rdparty/intel-sec-tools +++ b/3rdparty/intel-sec-tools @@ -1 +1 @@ -Subproject commit 0031ac73447baeb197fb2d80e5fba2470716e76d +Subproject commit a4552d635bf33bc1dda0cbfbd92a125a7188dbcb -- To view, visit https://review.coreboot.org/c/coreboot/+/68654 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I0769b3b90c58cece514d9e0ec4c08653ec57ddaf Gerrit-Change-Number: 68654 Gerrit-PatchSet: 1 Gerrit-Owner: Elyes Haouas <ehaouas(a)noos.fr> Gerrit-MessageType: newchange

1 0

[M] Change in coreboot[master]: Update fsp submodule to upstream master
by Elyes Haouas (Code Review) 21 Oct '22

21 Oct '22

Elyes Haouas has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/68653 ) Change subject: Update fsp submodule to upstream master ...................................................................... Update fsp submodule to upstream master Updating from commit id 12160fe: 2022-08-02 17:21:49 +0800 - (IoT ADL-P MR1 (3267_00) FSP) to commit id 17cd61f: 2022-10-19 15:08:23 +0800 - (Elkhart Lake MR5 FSP) This brings in 36 new commits: 17cd61f Elkhart Lake MR5 FSP e5a1931 Elkhart Lake MR5 FSP 4f7b483 Delete DoxygenFspIntegrationGuide.txt a1691c3 Elkhart Lake MR5 FSP e14d687 Delete ElkhartLake_FSP_Integration_Guide.pdf 7c1f6a2 Readme.md 83c1f1a Update README.md 3905c55 Elkhart Lake MR5 FSP 6941a97 Delete Vbt.bsf fec7167 Elkhart Lake MR5 FSP 117abb6 Delete Vbt.bin fcb9eb4 Elkhart Lake MR5 FSP 1072f3c Delete MemInfoHob.h 4c6560e Elkhart Lake MR5 FSP e3265df Delete FsptUpd.h 11a58d1 Elkhart Lake MR5 FSP 98562a6 Delete FspsUpd.h 14fde1a Elkhart Lake MR5 FSP 1299631 Delete FspmUpd.h c00b955 Elkhart Lake MR5 FSP 4b7a7a0 Delete FspUpd.h 68f6535 Elkhart Lake MR5 FSP 53bc5ed Delete FirmwareVersionInfoHob.h 29aafdb Elkhart Lake MR5 FSP 6e9b4b3 Delete FSPRel.bin fb79ac3 Elkhart Lake MR5 FSP d8fcdf1 Delete FSP.bsf 3221d3f Update FSP pkg 13a144f Added IoT ADL-PS PV (3354_00) FSP 308bd6a Update README.md af464fc Initial base tree check in for Idaville Icelake-D LCC/HCC FSP 5d78fcc IoT ADL-S MR2 (3267_00) FSP 432e2fe IoT ADL-S MR2 (3267_00) FSP da42c69 IoT ADL-S MR2 (3267_00) FSP e55a291 IoT ADL-S MR2 (3267_00) FSP 5dad172 IoT ADL-S MR2 (3267_00) FSP Change-Id: I5d8984ed79e6b1ce4978a83e180b569493f47a18 Signed-off-by: Elyes Haouas <ehaouas(a)noos.fr> --- M 3rdparty/fsp 1 file changed, 55 insertions(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/53/68653/1 diff --git a/3rdparty/fsp b/3rdparty/fsp index 12160fe..17cd61f 160000 --- a/3rdparty/fsp +++ b/3rdparty/fsp @@ -1 +1 @@ -Subproject commit 12160fe64bc1caaba3c2d6be44da0cd8787a2561 +Subproject commit 17cd61faa68cab02ddea6a3e24317fa2a6fae992 -- To view, visit https://review.coreboot.org/c/coreboot/+/68653 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I5d8984ed79e6b1ce4978a83e180b569493f47a18 Gerrit-Change-Number: 68653 Gerrit-PatchSet: 1 Gerrit-Owner: Elyes Haouas <ehaouas(a)noos.fr> Gerrit-MessageType: newchange

1 0

[S] Change in coreboot[master]: Update cmocka submodule to upstream master
by Elyes Haouas (Code Review) 21 Oct '22

21 Oct '22

Elyes Haouas has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/68652 ) Change subject: Update cmocka submodule to upstream master ...................................................................... Update cmocka submodule to upstream master Updating from commit id 8931845: 2020-10-12 08:24:45 +0200 - (cmocka: Fix cm_print_error() in check_for_leftover_values(_list)) to commit id 59dc001: 2022-08-10 07:03:17 +0200 - (include: Fix macro names to avoid conflicts) This brings in 94 new commits. Change-Id: I424db2eff6fcdcadd117296292c6dc1fc947ad4c Signed-off-by: Elyes Haouas <ehaouas(a)noos.fr> --- M 3rdparty/cmocka 1 file changed, 19 insertions(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/52/68652/1 diff --git a/3rdparty/cmocka b/3rdparty/cmocka index 8931845..59dc001 160000 --- a/3rdparty/cmocka +++ b/3rdparty/cmocka @@ -1 +1 @@ -Subproject commit 8931845c35e78b5123d73430b071affd537d5935 +Subproject commit 59dc0013f9f29fcf212fe4911c78e734263ce24c -- To view, visit https://review.coreboot.org/c/coreboot/+/68652 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I424db2eff6fcdcadd117296292c6dc1fc947ad4c Gerrit-Change-Number: 68652 Gerrit-PatchSet: 1 Gerrit-Owner: Elyes Haouas <ehaouas(a)noos.fr> Gerrit-MessageType: newchange

1 0

[S] Change in coreboot[master]: Update arm-trusted-firmware submodule to upstream master
by Elyes Haouas (Code Review) 21 Oct '22

21 Oct '22

Elyes Haouas has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/68651 ) Change subject: Update arm-trusted-firmware submodule to upstream master ...................................................................... Update arm-trusted-firmware submodule to upstream master Updating from commit id c45d2febb: 2022-10-12 15:56:24 +0200 - (Merge "fix(ufs): retry commands on unit attention" into integration) to commit id 61fe7826d: 2022-10-18 16:20:05 +0200 - (Merge "feat(fvp): build delegated attestation in BL31" into integration) This brings in 10 new commits: 61fe7826d Merge "feat(fvp): build delegated attestation in BL31" into integration cf17f7c45 Merge "chore(rpi3): remove redundant code" into integration 70360382b Merge "docs(maintainers): add NPU driver owners" into integration 60c439435 docs(maintainers): add NPU driver owners e504ce5fa Merge "fix(versal_net): Enable a78 errata workarounds" into integration bcc6e4a02 fix(versal_net): Enable a78 errata workarounds 0271eddb0 feat(fvp): build delegated attestation in BL31 6047ab122 Merge "fix(versal): enable a72 erratum 859971 and 1319367" into integration 769446a68 fix(versal): enable a72 erratum 859971 and 1319367 2594759d2 chore(rpi3): remove redundant code Change-Id: Ic32c0889961f529e1762b208ef118a94369c34e6 Signed-off-by: Elyes Haouas <ehaouas(a)noos.fr> --- M 3rdparty/arm-trusted-firmware 1 file changed, 29 insertions(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/51/68651/1 diff --git a/3rdparty/arm-trusted-firmware b/3rdparty/arm-trusted-firmware index c45d2fe..61fe782 160000 --- a/3rdparty/arm-trusted-firmware +++ b/3rdparty/arm-trusted-firmware @@ -1 +1 @@ -Subproject commit c45d2febb98058efa77d812b094bd1456a916c29 +Subproject commit 61fe7826d8ae41c906b2189136a1ae482f91f84a -- To view, visit https://review.coreboot.org/c/coreboot/+/68651 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ic32c0889961f529e1762b208ef118a94369c34e6 Gerrit-Change-Number: 68651 Gerrit-PatchSet: 1 Gerrit-Owner: Elyes Haouas <ehaouas(a)noos.fr> Gerrit-MessageType: newchange

1 0

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

coreboot-gerrit October 2022