Attention is currently required from: Tarun Tuli, Kapil Porwal, John Zhao, Eric Lai.
John Zhao has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/67742 )
Change subject: mb/google/rex: Enable TCSS DisplayPort detection at preboot
......................................................................
Patch Set 7:
(1 comment)
File src/mainboard/google/rex/Kconfig:
https://review.coreboot.org/c/coreboot/+/67742/comment/cce691f9_9fb6056a
PS7, Line 30: RUN_FSP_GOP
> so we don't need the if, right?
yes, no need in terms of functionality. It was simply aligned with brya's setting. Do you plan to update brya?
--
To view, visit https://review.coreboot.org/c/coreboot/+/67742
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I1a8a13e937c7132696aa39d85c3c6b6fb2dd13a5
Gerrit-Change-Number: 67742
Gerrit-PatchSet: 7
Gerrit-Owner: John Zhao <john.zhao(a)intel.com>
Gerrit-Reviewer: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: Tarun Tuli <taruntuli(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Eric Lai <eric_lai(a)quanta.corp-partner.google.com>
Gerrit-CC: John Zhao <john.zhao(a)intel.corp-partner.google.com>
Gerrit-Attention: Tarun Tuli <taruntuli(a)google.com>
Gerrit-Attention: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Attention: John Zhao <john.zhao(a)intel.com>
Gerrit-Attention: Eric Lai <eric_lai(a)quanta.corp-partner.google.com>
Gerrit-Comment-Date: Fri, 18 Nov 2022 03:53:37 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: John Zhao <john.zhao(a)intel.corp-partner.google.com>
Comment-In-Reply-To: Eric Lai <eric_lai(a)quanta.corp-partner.google.com>
Gerrit-MessageType: comment
Attention is currently required from: Tarun Tuli, John Zhao, Kapil Porwal.
Eric Lai has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/67742 )
Change subject: mb/google/rex: Enable TCSS DisplayPort detection at preboot
......................................................................
Patch Set 7:
(1 comment)
File src/mainboard/google/rex/Kconfig:
https://review.coreboot.org/c/coreboot/+/67742/comment/04aee505_4b5918aa
PS7, Line 30: RUN_FSP_GOP
> It seems not.
so we don't need the if, right?
--
To view, visit https://review.coreboot.org/c/coreboot/+/67742
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I1a8a13e937c7132696aa39d85c3c6b6fb2dd13a5
Gerrit-Change-Number: 67742
Gerrit-PatchSet: 7
Gerrit-Owner: John Zhao <john.zhao(a)intel.com>
Gerrit-Reviewer: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: Tarun Tuli <taruntuli(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Eric Lai <eric_lai(a)quanta.corp-partner.google.com>
Gerrit-CC: John Zhao <john.zhao(a)intel.corp-partner.google.com>
Gerrit-Attention: Tarun Tuli <taruntuli(a)google.com>
Gerrit-Attention: John Zhao <john.zhao(a)intel.corp-partner.google.com>
Gerrit-Attention: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Attention: John Zhao <john.zhao(a)intel.com>
Gerrit-Comment-Date: Fri, 18 Nov 2022 03:40:41 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: John Zhao <john.zhao(a)intel.corp-partner.google.com>
Comment-In-Reply-To: Eric Lai <eric_lai(a)quanta.corp-partner.google.com>
Gerrit-MessageType: comment
Attention is currently required from: Tarun Tuli, Kapil Porwal, John Zhao, Eric Lai.
John Zhao has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/67742 )
Change subject: mb/google/rex: Enable TCSS DisplayPort detection at preboot
......................................................................
Patch Set 7:
(1 comment)
File src/mainboard/google/rex/Kconfig:
https://review.coreboot.org/c/coreboot/+/67742/comment/be11710a_d54cdc1a
PS7, Line 30: RUN_FSP_GOP
> Any condition we don't have RUN_FSP_GOP on Rex?
It seems not.
--
To view, visit https://review.coreboot.org/c/coreboot/+/67742
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I1a8a13e937c7132696aa39d85c3c6b6fb2dd13a5
Gerrit-Change-Number: 67742
Gerrit-PatchSet: 7
Gerrit-Owner: John Zhao <john.zhao(a)intel.com>
Gerrit-Reviewer: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-Reviewer: Tarun Tuli <taruntuli(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Eric Lai <eric_lai(a)quanta.corp-partner.google.com>
Gerrit-CC: John Zhao <john.zhao(a)intel.corp-partner.google.com>
Gerrit-Attention: Tarun Tuli <taruntuli(a)google.com>
Gerrit-Attention: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Attention: John Zhao <john.zhao(a)intel.com>
Gerrit-Attention: Eric Lai <eric_lai(a)quanta.corp-partner.google.com>
Gerrit-Comment-Date: Fri, 18 Nov 2022 03:39:11 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Eric Lai <eric_lai(a)quanta.corp-partner.google.com>
Gerrit-MessageType: comment
Attention is currently required from: Martin L Roth, Arthur Heymans, Felix Held.
Elyes Haouas has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/64103 )
Change subject: Revert "crossgcc: Upgrade IASL from 20211217 to 20220331"
......................................................................
Patch Set 2: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/64103
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I5c1b30585fdfab046a40d1721276d0fa40457a10
Gerrit-Change-Number: 64103
Gerrit-PatchSet: 2
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Elyes Haouas <ehaouas(a)noos.fr>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Attention: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Attention: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Comment-Date: Fri, 18 Nov 2022 02:52:14 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: Martin L Roth, Arthur Heymans, Felix Held.
Elyes Haouas has uploaded a new patch set (#2) to the change originally created by Arthur Heymans. ( https://review.coreboot.org/c/coreboot/+/64103 )
Change subject: Revert "crossgcc: Upgrade IASL from 20211217 to 20220331"
......................................................................
Revert "crossgcc: Upgrade IASL from 20211217 to 20220331"
This reverts commit 9a8d0a03dbccf95554153cd06eee6270518174f6.
Reason for revert: IASL now only throws errors on syntax errors.
Change-Id: I5c1b30585fdfab046a40d1721276d0fa40457a10
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
M Makefile.inc
M util/crossgcc/buildgcc
R util/crossgcc/patches/acpica-unix2-20211217_iasl.patch
A util/crossgcc/sum/acpica-unix2-20211217.tar.gz.cksum
D util/crossgcc/sum/acpica-unix2-20220331.tar.gz.cksum
5 files changed, 19 insertions(+), 2 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/03/64103/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/64103
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I5c1b30585fdfab046a40d1721276d0fa40457a10
Gerrit-Change-Number: 64103
Gerrit-PatchSet: 2
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Elyes Haouas <ehaouas(a)noos.fr>
Gerrit-Attention: Martin L Roth <gaumless(a)gmail.com>
Gerrit-Attention: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Attention: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-MessageType: newpatchset
Attention is currently required from: Jakub Czapiga, Yu-Ping Wu.
Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69762 )
Change subject: vboot/vboot_logic: Fix hash digest size and padding
......................................................................
Patch Set 1:
(1 comment)
File src/security/vboot/vboot_logic.c:
https://review.coreboot.org/c/coreboot/+/69762/comment/13739a42_5b414817
PS1, Line 344: uint8_t hash_digest[VBOOT_MAX_HASH_SIZE];
Should we do this in vboot_save_hash()? Seems like a weird quirk of the backing store that doesn't really belong in the common business logic code.
--
To view, visit https://review.coreboot.org/c/coreboot/+/69762
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: If6d46e0b58dbca86af56221b7ff2606ab2d1799a
Gerrit-Change-Number: 69762
Gerrit-PatchSet: 1
Gerrit-Owner: Jakub Czapiga <jacz(a)semihalf.com>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Yu-Ping Wu <yupingso(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Jakub Czapiga <jacz(a)semihalf.com>
Gerrit-Attention: Yu-Ping Wu <yupingso(a)google.com>
Gerrit-Comment-Date: Fri, 18 Nov 2022 02:45:00 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment