Attention is currently required from: Filip Lewiński, Michał Kopeć, Sergii Dmytruk.
Michał Żygowski has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/79080?usp=email )
Change subject: util: add smmstoretool for editing SMMSTORE
......................................................................
Patch Set 7:
(1 comment)
Patchset:
PS7:
Works on Ubuntu now too. Thanks. LGTM. It just needs a rebase
--
To view, visit https://review.coreboot.org/c/coreboot/+/79080?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I6c1c06f1d0c39c13b5be76a3070f09b715aca6e0
Gerrit-Change-Number: 79080
Gerrit-PatchSet: 7
Gerrit-Owner: Sergii Dmytruk <sergii.dmytruk(a)3mdeb.com>
Gerrit-Reviewer: Filip Lewiński
Gerrit-Reviewer: Michał Kopeć <michal.kopec(a)3mdeb.com>
Gerrit-Reviewer: Michał Żygowski <michal.zygowski(a)3mdeb.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Maciej Pijanowski <maciej.pijanowski(a)3mdeb.com>
Gerrit-Attention: Filip Lewiński
Gerrit-Attention: Michał Kopeć <michal.kopec(a)3mdeb.com>
Gerrit-Attention: Sergii Dmytruk <sergii.dmytruk(a)3mdeb.com>
Gerrit-Comment-Date: Thu, 29 Feb 2024 11:32:01 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Cliff Huang, Lance Zhao, Tim Wawrzynczak, minyu li.
Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80781?usp=email )
Change subject: src/acpi: Modify ACPI revision
......................................................................
Patch Set 1:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/80781/comment/2d9aeeb4_9ae5cb99 :
PS1, Line 9: Modify ACPI revision from 6.0 to 6.4
Mention a reason. Why is this change needed?
--
To view, visit https://review.coreboot.org/c/coreboot/+/80781?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I77b978f07b296e55e67d4de5e981bd48fd28eb03
Gerrit-Change-Number: 80781
Gerrit-PatchSet: 1
Gerrit-Owner: minyu li <li.minyu(a)h3c.com>
Gerrit-Reviewer: Cliff Huang <cliff.huang(a)intel.com>
Gerrit-Reviewer: Lance Zhao <lance.zhao(a)gmail.com>
Gerrit-Reviewer: Tim Wawrzynczak <inforichland(a)gmail.com>
Gerrit-CC: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Attention: Lance Zhao <lance.zhao(a)gmail.com>
Gerrit-Attention: Cliff Huang <cliff.huang(a)intel.com>
Gerrit-Attention: Tim Wawrzynczak <inforichland(a)gmail.com>
Gerrit-Attention: minyu li <li.minyu(a)h3c.com>
Gerrit-Comment-Date: Thu, 29 Feb 2024 11:29:51 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Michał Żygowski.
Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80790?usp=email )
Change subject: lib/rtc: Fix off-by-one error in February day count in leap year
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/80790?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: If451e3e3471fef0d429e255cf297050a525ca1a2
Gerrit-Change-Number: 80790
Gerrit-PatchSet: 1
Gerrit-Owner: Michał Żygowski <michal.zygowski(a)3mdeb.com>
Gerrit-Reviewer: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-Reviewer: Michał Kopeć <michal.kopec(a)3mdeb.com>
Gerrit-Reviewer: Werner Zeh <werner.zeh(a)siemens.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Michał Żygowski <michal.zygowski(a)3mdeb.com>
Gerrit-Comment-Date: Thu, 29 Feb 2024 11:27:23 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Angel Pons, Arthur Heymans, Jérémy Compostella, Nick Vaccaro, Nico Huber, Shelley Chen, Tarun Tuli, Tim Wawrzynczak.
Kapil Porwal has posted comments on this change. ( https://review.coreboot.org/c/libgfxinit/+/67490?usp=email )
Change subject: gma: Update DP training with new fields
......................................................................
Patch Set 23:
(1 comment)
File common/ironlake/hw-gfx-gma-pch-dp.ads:
https://review.coreboot.org/c/libgfxinit/+/67490/comment/e9d2ed3c_d4ca4451 :
PS19, Line 26: Reason => "Needed for a common interface"
> > > @arthur@aheymans.xyz could you please share your thoughts on the above comment? […]
Could you please also help with "Verified+1" vote?
--
To view, visit https://review.coreboot.org/c/libgfxinit/+/67490?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: libgfxinit
Gerrit-Branch: main
Gerrit-Change-Id: I3f7f8a93408a74cfa29d1493a48c88ce73dc690c
Gerrit-Change-Number: 67490
Gerrit-PatchSet: 23
Gerrit-Owner: Tim Wawrzynczak <inforichland(a)gmail.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Jérémy Compostella <jeremy.compostella(a)intel.com>
Gerrit-Reviewer: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Tarun Tuli <taruntuli(a)google.com>
Gerrit-CC: Lean Sheng Tan <sheng.tan(a)9elements.com>
Gerrit-CC: Shelley Chen <shchen(a)google.com>
Gerrit-CC: Stefan Reinauer <stefan.reinauer(a)coreboot.org>
Gerrit-Attention: Shelley Chen <shchen(a)google.com>
Gerrit-Attention: Tarun Tuli <taruntuli(a)google.com>
Gerrit-Attention: Nico Huber <nico.h(a)gmx.de>
Gerrit-Attention: Jérémy Compostella <jeremy.compostella(a)intel.com>
Gerrit-Attention: Tim Wawrzynczak <inforichland(a)gmail.com>
Gerrit-Attention: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Attention: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Attention: Nick Vaccaro <nvaccaro(a)google.com>
Gerrit-Comment-Date: Thu, 29 Feb 2024 11:19:09 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Shelley Chen <shchen(a)google.com>
Comment-In-Reply-To: Kapil Porwal <kapilporwal(a)google.com>
Comment-In-Reply-To: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-MessageType: comment
Attention is currently required from: Michał Żygowski.
Werner Zeh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80790?usp=email )
Change subject: lib/rtc: Fix off-by-one error in February day count in leap year
......................................................................
Patch Set 1: Code-Review+2
(1 comment)
Patchset:
PS1:
Thanks for fixing and sorry for the inconvenience.
--
To view, visit https://review.coreboot.org/c/coreboot/+/80790?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: If451e3e3471fef0d429e255cf297050a525ca1a2
Gerrit-Change-Number: 80790
Gerrit-PatchSet: 1
Gerrit-Owner: Michał Żygowski <michal.zygowski(a)3mdeb.com>
Gerrit-Reviewer: Michał Kopeć <michal.kopec(a)3mdeb.com>
Gerrit-Reviewer: Werner Zeh <werner.zeh(a)siemens.com>
Gerrit-Attention: Michał Żygowski <michal.zygowski(a)3mdeb.com>
Gerrit-Comment-Date: Thu, 29 Feb 2024 11:16:52 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Michał Żygowski, Werner Zeh.
Michał Kopeć has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80790?usp=email )
Change subject: lib/rtc: Fix off-by-one error in February day count in leap year
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/80790?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: If451e3e3471fef0d429e255cf297050a525ca1a2
Gerrit-Change-Number: 80790
Gerrit-PatchSet: 1
Gerrit-Owner: Michał Żygowski <michal.zygowski(a)3mdeb.com>
Gerrit-Reviewer: Michał Kopeć <michal.kopec(a)3mdeb.com>
Gerrit-Reviewer: Werner Zeh <werner.zeh(a)siemens.com>
Gerrit-Attention: Michał Żygowski <michal.zygowski(a)3mdeb.com>
Gerrit-Attention: Werner Zeh <werner.zeh(a)siemens.com>
Gerrit-Comment-Date: Thu, 29 Feb 2024 11:11:17 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
minyu li has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/80781?usp=email )
Change subject: src/acpi: Modify ACPI revision
......................................................................
src/acpi: Modify ACPI revision
Modify ACPI revision from 6.0 to 6.4
Change-Id: I77b978f07b296e55e67d4de5e981bd48fd28eb03
Signed-off-by: liminyu <li.minyu(a)h3c.com>
---
M src/acpi/acpi.c
M src/include/acpi/acpi.h
2 files changed, 2 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/81/80781/1
diff --git a/src/acpi/acpi.c b/src/acpi/acpi.c
index 98c8ecd..f100108 100644
--- a/src/acpi/acpi.c
+++ b/src/acpi/acpi.c
@@ -1749,7 +1749,7 @@
u8 get_acpi_fadt_minor_version(void)
{
- return ACPI_FADT_MINOR_VERSION_0;
+ return ACPI_FADT_MINOR_VERSION_4;
}
int get_acpi_table_revision(enum acpi_tables table)
diff --git a/src/include/acpi/acpi.h b/src/include/acpi/acpi.h
index d6c30c1..5b4784d2 100644
--- a/src/include/acpi/acpi.h
+++ b/src/include/acpi/acpi.h
@@ -988,7 +988,7 @@
* value of 2 would be compatible with Errata B, and so on
* Version 6.3 Errata A would be: (1 << 4) | 3
*/
-#define ACPI_FADT_MINOR_VERSION_0 0 /* coreboot currently use this version */
+#define ACPI_FADT_MINOR_VERSION_4 4 /* coreboot currently use this version */
/* Flags for p_lvl2_lat and p_lvl3_lat */
#define ACPI_FADT_C2_NOT_SUPPORTED 101
--
To view, visit https://review.coreboot.org/c/coreboot/+/80781?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I77b978f07b296e55e67d4de5e981bd48fd28eb03
Gerrit-Change-Number: 80781
Gerrit-PatchSet: 1
Gerrit-Owner: minyu li <li.minyu(a)h3c.com>
Gerrit-MessageType: newchange
Attention is currently required from: Werner Zeh.
Michał Żygowski has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/54913?usp=email )
Change subject: lib/rtc: Add sanity check for time and date
......................................................................
Patch Set 3:
(1 comment)
File src/lib/rtc.c:
https://review.coreboot.org/c/coreboot/+/54913/comment/3f8496ea_2fc3ea71 :
PS3, Line 129: month == 2
First leap year and we already hit this off-by-one error 😊
Month is passed as 0-based, not 1-based.
This is quite critical as it may break certificates, file systems, etc.
CB:80790
--
To view, visit https://review.coreboot.org/c/coreboot/+/54913?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I0a381d445c894eee4f82b50fe86dad22cc587605
Gerrit-Change-Number: 54913
Gerrit-PatchSet: 3
Gerrit-Owner: Werner Zeh <werner.zeh(a)siemens.com>
Gerrit-Reviewer: Arthur Heymans <arthur.heymans(a)9elements.com>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Patrick Georgi <patrick(a)coreboot.org>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: 9elements QA <hardwaretestrobot(a)gmail.com>
Gerrit-CC: Jan Dabros <jsd(a)semihalf.com>
Gerrit-CC: Michał Żygowski <michal.zygowski(a)3mdeb.com>
Gerrit-Attention: Werner Zeh <werner.zeh(a)siemens.com>
Gerrit-Comment-Date: Thu, 29 Feb 2024 10:53:38 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Alexander Couzens, Maciej Pijanowski, Michał Kopeć, Paul Menzel.
Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80609?usp=email )
Change subject: mb/lenovo: Add ThinkCentre M920q (Cannon Lake)
......................................................................
Patch Set 4:
(1 comment)
Patchset:
PS4:
Just a ping. A few comments are open and I think it's ready to be merged then.
--
To view, visit https://review.coreboot.org/c/coreboot/+/80609?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Iea1dc5745c0ecf687fa18b793f0aab4b0855d6d4
Gerrit-Change-Number: 80609
Gerrit-PatchSet: 4
Gerrit-Owner: Maciej Pijanowski <maciej.pijanowski(a)3mdeb.com>
Gerrit-Reviewer: Alexander Couzens <lynxis(a)fe80.eu>
Gerrit-Reviewer: Michał Kopeć <michal.kopec(a)3mdeb.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Felix Singer <service+coreboot-gerrit(a)felixsinger.de>
Gerrit-CC: Nicholas Sudsgaard <devel+coreboot(a)nsudsgaard.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Maciej Pijanowski <maciej.pijanowski(a)3mdeb.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Michał Kopeć <michal.kopec(a)3mdeb.com>
Gerrit-Attention: Alexander Couzens <lynxis(a)fe80.eu>
Gerrit-Comment-Date: Thu, 29 Feb 2024 10:53:31 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment