Attention is currently required from: Eran Mitrani, Kapil Porwal, Paul Menzel, Tarun, yunhao wang.
Kun Liu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/78290?usp=email )
Change subject: mb/google/rex/var/screebo: Update DTT settings for thermal control
......................................................................
Patch Set 2:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/78290/comment/e7813ad6_2738ae7c :
PS2, Line 9: update DTT settings for thermal control
> Please elaborate, and describe the source and the changes.
please see:https://partnerissuetracker.corp.google.com/issues/291217859#comment14
1.Cancel TCPU trip point and fine tune other protection temperature on the Critical policy table
2.Fine tune EC/Bios protection temperature
--
To view, visit https://review.coreboot.org/c/coreboot/+/78290?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: I0e2ff6eea9fed71ad7680c1fac4921984b87aca5
Gerrit-Change-Number: 78290
Gerrit-PatchSet: 2
Gerrit-Owner: Kun Liu <liukun11(a)huaqin.corp-partner.google.com>
Gerrit-Reviewer: Eran Mitrani <mitrani(a)google.com>
Gerrit-Reviewer: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: Tarun <tstuli(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Reviewer: yunhao wang <wangyunhao(a)huaqin.corp-partner.google.com>
Gerrit-Reviewer: zhongtian wu <wuzhongtian(a)huaqin.corp-partner.google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: yunhao wang <wangyunhao(a)huaqin.corp-partner.google.com>
Gerrit-Attention: Eran Mitrani <mitrani(a)google.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Attention: Tarun <tstuli(a)gmail.com>
Gerrit-Comment-Date: Mon, 09 Oct 2023 11:27:17 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-MessageType: comment
Attention is currently required from: Eran Mitrani, Kapil Porwal, Paul Menzel, Subrata Banik, Tarun.
Kun Liu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/78295?usp=email )
Change subject: Revert "mb/google/rex/var/screebo: Enable GL9750 invert WP function"
......................................................................
Patch Set 3:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/78295/comment/bf37c3d5_47dd6585 :
PS2, Line 11: Added
> Do you mean: In schematic X a sdcard write protection pull-down … was added, so …
yes
https://review.coreboot.org/c/coreboot/+/78295/comment/369916be_6471b170 :
PS2, Line 12: hardware,so
> Please add a space after the comma.
Done
--
To view, visit https://review.coreboot.org/c/coreboot/+/78295?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: I00a8f43094d8b3674a4bbaeed24b96aab64b9b75
Gerrit-Change-Number: 78295
Gerrit-PatchSet: 3
Gerrit-Owner: Kun Liu <liukun11(a)huaqin.corp-partner.google.com>
Gerrit-Reviewer: Eran Mitrani <mitrani(a)google.com>
Gerrit-Reviewer: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Reviewer: Rui Zhou <zhourui(a)huaqin.corp-partner.google.com>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: Tarun <tstuli(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Eran Mitrani <mitrani(a)google.com>
Gerrit-Attention: Subrata Banik <subratabanik(a)google.com>
Gerrit-Attention: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Attention: Tarun <tstuli(a)gmail.com>
Gerrit-Comment-Date: Mon, 09 Oct 2023 11:17:28 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-MessageType: comment
Attention is currently required from: Eran Mitrani, Kapil Porwal, Kun Liu, Subrata Banik, Tarun.
Hello Eran Mitrani, Kapil Porwal, Rui Zhou, Subrata Banik, Tarun, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/78295?usp=email
to look at the new patch set (#3).
Change subject: Revert "mb/google/rex/var/screebo: Enable GL9750 invert WP function"
......................................................................
Revert "mb/google/rex/var/screebo: Enable GL9750 invert WP function"
This reverts commit ee4191852abf9b24f822468250c24edb993497c6.
Reason for revert: Added sdcard write protection pull-down resistor
on hardware, so need to disable GL9750 invert WP function
Change-Id: I00a8f43094d8b3674a4bbaeed24b96aab64b9b75
Signed-off-by: Kun Liu <liukun11(a)huaqin.corp-partner.google.com>
---
M src/mainboard/google/rex/Kconfig
1 file changed, 0 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/95/78295/3
--
To view, visit https://review.coreboot.org/c/coreboot/+/78295?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: I00a8f43094d8b3674a4bbaeed24b96aab64b9b75
Gerrit-Change-Number: 78295
Gerrit-PatchSet: 3
Gerrit-Owner: Kun Liu <liukun11(a)huaqin.corp-partner.google.com>
Gerrit-Reviewer: Eran Mitrani <mitrani(a)google.com>
Gerrit-Reviewer: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Reviewer: Rui Zhou <zhourui(a)huaqin.corp-partner.google.com>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: Tarun <tstuli(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Eran Mitrani <mitrani(a)google.com>
Gerrit-Attention: Subrata Banik <subratabanik(a)google.com>
Gerrit-Attention: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Attention: Kun Liu <liukun11(a)huaqin.corp-partner.google.com>
Gerrit-Attention: Tarun <tstuli(a)gmail.com>
Gerrit-MessageType: newpatchset
Attention is currently required from: Martin Roth, Nikolai Vyssotski, Zheng Bao.
Hello Martin Roth, Nikolai Vyssotski, Zheng Bao, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/78274?usp=email
to look at the new patch set (#4).
Change subject: amdfwtool: Set the table size as the sum of entries and header
......................................................................
amdfwtool: Set the table size as the sum of entries and header
According to the PSP spec, the Table size doesn't include the FWs
themselves. And the table size is 4K aligned. The max entry is 256.
So sizes of both PSP and BIOS table are 4K.
Set the table size earlier to make sure no other data be put
into the table.
Change-Id: Id4fbc6d57d7ea070a9478649a96af92be9441289
Signed-off-by: Zheng Bao <fishbaozi(a)gmail.com>
---
M util/amdfwtool/amdfwtool.c
1 file changed, 14 insertions(+), 21 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/74/78274/4
--
To view, visit https://review.coreboot.org/c/coreboot/+/78274?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: Id4fbc6d57d7ea070a9478649a96af92be9441289
Gerrit-Change-Number: 78274
Gerrit-PatchSet: 4
Gerrit-Owner: Bao Zheng <fishbaozi(a)gmail.com>
Gerrit-Reviewer: Martin Roth <martin.roth(a)amd.corp-partner.google.com>
Gerrit-Reviewer: Nikolai Vyssotski <nikolai.vyssotski(a)amd.corp-partner.google.com>
Gerrit-Reviewer: Zheng Bao
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Zheng Bao
Gerrit-Attention: Nikolai Vyssotski <nikolai.vyssotski(a)amd.corp-partner.google.com>
Gerrit-Attention: Martin Roth <martin.roth(a)amd.corp-partner.google.com>
Gerrit-MessageType: newpatchset
Attention is currently required from: Eran Mitrani, Kapil Porwal, Kun Liu, Tarun, yunhao wang.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/78290?usp=email )
The change is no longer submittable: All-Comments-Resolved is unsatisfied now.
Change subject: mb/google/rex/var/screebo: Update DTT settings for thermal control
......................................................................
Patch Set 2:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/78290/comment/8f1c6dec_96d115f4 :
PS2, Line 9: update DTT settings for thermal control
Please elaborate, and describe the source and the changes.
--
To view, visit https://review.coreboot.org/c/coreboot/+/78290?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: I0e2ff6eea9fed71ad7680c1fac4921984b87aca5
Gerrit-Change-Number: 78290
Gerrit-PatchSet: 2
Gerrit-Owner: Kun Liu <liukun11(a)huaqin.corp-partner.google.com>
Gerrit-Reviewer: Eran Mitrani <mitrani(a)google.com>
Gerrit-Reviewer: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: Tarun <tstuli(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Reviewer: yunhao wang <wangyunhao(a)huaqin.corp-partner.google.com>
Gerrit-Reviewer: zhongtian wu <wuzhongtian(a)huaqin.corp-partner.google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: yunhao wang <wangyunhao(a)huaqin.corp-partner.google.com>
Gerrit-Attention: Eran Mitrani <mitrani(a)google.com>
Gerrit-Attention: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Attention: Kun Liu <liukun11(a)huaqin.corp-partner.google.com>
Gerrit-Attention: Tarun <tstuli(a)gmail.com>
Gerrit-Comment-Date: Mon, 09 Oct 2023 10:28:28 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Eran Mitrani, Kapil Porwal, Kun Liu, Subrata Banik, Tarun.
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/78295?usp=email )
Change subject: Revert "mb/google/rex/var/screebo: Enable GL9750 invert WP function"
......................................................................
Patch Set 2:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/78295/comment/b6455751_1cadfa29 :
PS2, Line 11: Added
Do you mean: In schematic X a sdcard write protection pull-down … was added, so …
https://review.coreboot.org/c/coreboot/+/78295/comment/7b2059b3_162d6ca2 :
PS2, Line 12: hardware,so
Please add a space after the comma.
--
To view, visit https://review.coreboot.org/c/coreboot/+/78295?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: I00a8f43094d8b3674a4bbaeed24b96aab64b9b75
Gerrit-Change-Number: 78295
Gerrit-PatchSet: 2
Gerrit-Owner: Kun Liu <liukun11(a)huaqin.corp-partner.google.com>
Gerrit-Reviewer: Eran Mitrani <mitrani(a)google.com>
Gerrit-Reviewer: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Reviewer: Rui Zhou <zhourui(a)huaqin.corp-partner.google.com>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: Tarun <tstuli(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Eran Mitrani <mitrani(a)google.com>
Gerrit-Attention: Subrata Banik <subratabanik(a)google.com>
Gerrit-Attention: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Attention: Kun Liu <liukun11(a)huaqin.corp-partner.google.com>
Gerrit-Attention: Tarun <tstuli(a)gmail.com>
Gerrit-Comment-Date: Mon, 09 Oct 2023 10:28:03 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Uwe Poeche has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/77409?usp=email )
Change subject: soc/intel/apollolake: Correct the logic for the legacy 8254 timer
......................................................................
Patch Set 2:
(1 comment)
Patchset:
PS2:
This patch let at least at one mainboard (siemens/mc_apl4) run the payload into problems. It seems that the patch only disables the legacy 8254 timer.
--
To view, visit https://review.coreboot.org/c/coreboot/+/77409?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: I2d6c859c0910b796d2ae5874a560ff9974578106
Gerrit-Change-Number: 77409
Gerrit-PatchSet: 2
Gerrit-Owner: Sean Rhodes <sean(a)starlabs.systems>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Matt DeVillier <matt.devillier(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: 9elements QA <hardwaretestrobot(a)gmail.com>
Gerrit-CC: Uwe Poeche <uwe.poeche(a)siemens.com>
Gerrit-Comment-Date: Mon, 09 Oct 2023 10:06:35 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Zheng Bao.
Hello Zheng Bao,
I'd like you to do a code review.
Please visit
https://review.coreboot.org/c/coreboot/+/78280?usp=email
to review the following change.
Change subject: amdfwtool: Change&Record the current table in integration function
......................................................................
amdfwtool: Change&Record the current table in integration function
Align with the function integrating PSP FWs.
TEST=Identical test on all AMD platform
Change-Id: I1a98614f3a5756a462b01085e9565b52cf9a9343
Signed-off-by: Zheng Bao <fishbaozi(a)gmail.com>
---
M util/amdfwtool/amdfwtool.c
1 file changed, 4 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/80/78280/1
diff --git a/util/amdfwtool/amdfwtool.c b/util/amdfwtool/amdfwtool.c
index caa8cda..99b0f9a 100644
--- a/util/amdfwtool/amdfwtool.c
+++ b/util/amdfwtool/amdfwtool.c
@@ -1206,7 +1206,6 @@
((bios_directory_hdr *) ptr)->additional_info = 0;
((bios_directory_hdr *) ptr)->additional_info_fields.address_mode = ctx->address_mode;
((bios_directory_hdr *) ptr)->additional_info_fields.dir_size = table_size / TABLE_ALIGNMENT;
- ctx->current_table = ctx->current;
adjust_current_pointer(ctx, table_size, 1);
return ptr;
}
@@ -1295,6 +1294,7 @@
int apob_idx;
uint32_t size;
uint64_t source;
+ uint32_t current_table_save;
/* This function can create a primary table, a secondary table, or a
* flattened table which contains all applicable types. These if-else
@@ -1312,6 +1312,8 @@
else
level = BDT_BOTH;
+ current_table_save = ctx->current_table;
+ ctx->current_table = (char *)biosdir - ctx->rom;
adjust_current_pointer(ctx, 0, TABLE_ALIGNMENT);
for (i = 0, count = 0; fw_table[i].type != AMD_BIOS_INVALID; i++) {
@@ -1512,6 +1514,7 @@
}
fill_dir_header(biosdir, count, cookie, ctx);
+ ctx->current_table = current_table_save;
}
enum {
--
To view, visit https://review.coreboot.org/c/coreboot/+/78280?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: I1a98614f3a5756a462b01085e9565b52cf9a9343
Gerrit-Change-Number: 78280
Gerrit-PatchSet: 1
Gerrit-Owner: Bao Zheng <fishbaozi(a)gmail.com>
Gerrit-Reviewer: Zheng Bao
Gerrit-Attention: Zheng Bao
Gerrit-MessageType: newchange