Philipp Deppenwiese has posted comments on this change. ( https://review.coreboot.org/29026 )
Change subject: tpm/tspi: clean up tpm_setup function flow
......................................................................
Patch Set 5: Code-Review+2
--
To view, visit https://review.coreboot.org/29026
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibea44880683a301e82ee2ba049003c36fcb44eba
Gerrit-Change-Number: 29026
Gerrit-PatchSet: 5
Gerrit-Owner: Joel Kitching <kitching(a)google.com>
Gerrit-Reviewer: Aaron Durbin <adurbin(a)chromium.org>
Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com>
Gerrit-Reviewer: Joel Kitching <kitching(a)google.com>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Comment-Date: Thu, 18 Oct 2018 16:22:33 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes
Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/29105 )
Change subject: cpu/x86/smm: Fix non-local header treated as local
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://review.coreboot.org/29105
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I15dfe0332fd87db61d692a94bf1fd5d00dfb83d4
Gerrit-Change-Number: 29105
Gerrit-PatchSet: 1
Gerrit-Owner: Elyes HAOUAS <ehaouas(a)noos.fr>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Thu, 18 Oct 2018 16:21:19 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes
Hello Werner Zeh, Patrick Rudolph, build bot (Jenkins), Joel Kitching, Vadim Bendebury, Patrick Georgi,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/29181
to look at the new patch set (#3).
Change subject: arch/x86/acpi: Add TPM2 table support
......................................................................
arch/x86/acpi: Add TPM2 table support
* Distinguish between TPM 1.2 and 2.0
ACPI table support
* Add TPM2 table support for TIS interface only
Change-Id: I030c7ea744bcfe61ebef8d66d1295273b5dccda5
Signed-off-by: Philipp Deppenwiese <zaolin(a)das-labor.org>
---
M src/arch/x86/acpi.c
M src/arch/x86/include/arch/acpi.h
2 files changed, 55 insertions(+), 7 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/81/29181/3
--
To view, visit https://review.coreboot.org/29181
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I030c7ea744bcfe61ebef8d66d1295273b5dccda5
Gerrit-Change-Number: 29181
Gerrit-PatchSet: 3
Gerrit-Owner: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com>
Gerrit-Reviewer: Joel Kitching <kitching(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: Vadim Bendebury <vbendeb(a)chromium.org>
Gerrit-Reviewer: Werner Zeh <werner.zeh(a)siemens.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Hello Werner Zeh, Patrick Rudolph, build bot (Jenkins), Joel Kitching, Vadim Bendebury, Patrick Georgi,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/29181
to look at the new patch set (#2).
Change subject: arch/x86/acpi: Add TPM2 table support
......................................................................
arch/x86/acpi: Add TPM2 table support
* Distinguish between TPM 1.2 and 2.0
ACPI table support
* Add TPM2 table support for TIS interface only
Change-Id: I030c7ea744bcfe61ebef8d66d1295273b5dccda5
Signed-off-by: Philipp Deppenwiese <zaolin(a)das-labor.org>
---
M src/arch/x86/acpi.c
M src/arch/x86/include/arch/acpi.h
2 files changed, 55 insertions(+), 7 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/81/29181/2
--
To view, visit https://review.coreboot.org/29181
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I030c7ea744bcfe61ebef8d66d1295273b5dccda5
Gerrit-Change-Number: 29181
Gerrit-PatchSet: 2
Gerrit-Owner: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com>
Gerrit-Reviewer: Joel Kitching <kitching(a)google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi(a)google.com>
Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph(a)9elements.com>
Gerrit-Reviewer: Vadim Bendebury <vbendeb(a)chromium.org>
Gerrit-Reviewer: Werner Zeh <werner.zeh(a)siemens.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Philipp Hug has posted comments on this change. ( https://review.coreboot.org/29165 )
Change subject: soc/lowrisc: Remove the remains of a LowRISC soc
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://review.coreboot.org/29165
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I03c5860b27d04b6e1d7868ba8ea7b52d1075aa6a
Gerrit-Change-Number: 29165
Gerrit-PatchSet: 1
Gerrit-Owner: Peter Lemenkov <lemenkov(a)gmail.com>
Gerrit-Reviewer: Jonathan Neuschäfer <j.neuschaefer(a)gmx.net>
Gerrit-Reviewer: Philipp Hug <philipp(a)hug.cx>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Thu, 18 Oct 2018 12:24:22 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes
chris wang has posted comments on this change. ( https://review.coreboot.org/29179 )
Change subject: mb/google/kahlee/variants/liara: enable 2T mode for liara.
......................................................................
Patch Set 2:
Hi Martin,
I revert the NB P-state disabling and use 2T mode instead.
To enable 2T mode will help this not happened. It might be a
workaround but can increase the power saving on liara.
Do you have any suggestion ?
--
To view, visit https://review.coreboot.org/29179
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3a96276d88ffb70530d72b15c07b59a01cc6209a
Gerrit-Change-Number: 29179
Gerrit-PatchSet: 2
Gerrit-Owner: chris wang <Chris.Wang(a)amd.com>
Gerrit-Reviewer: Chris Wang <chris.wang(a)amd.corp-partner.google.com>
Gerrit-Reviewer: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Richard Spiegel <richard.spiegel(a)silverbackltd.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Reviewer: chris wang <Chris.Wang(a)amd.com>
Gerrit-Comment-Date: Thu, 18 Oct 2018 11:46:08 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No