coreboot-gerrit May 2017

coreboot-gerrit@coreboot.org

1 participants
3888 discussions

Change in coreboot[master]: mainboard/pcengines/apu2: Add LPC TPM support
by build bot (Jenkins) (Code Review) 04 May '17

04 May '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/18523 ) Change subject: mainboard/pcengines/apu2: Add LPC TPM support ...................................................................... Patch Set 11: Build Successful https://qa.coreboot.org/job/coreboot-gerrit/53193/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/9019/ : SUCCESS -- To view, visit https://review.coreboot.org/18523 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: If9312370a5071ffbeb6d83888c75fa69a0c27819 Gerrit-PatchSet: 11 Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Owner: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com> Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-HasComments: No

1 0

Change in coreboot[master]: amd/pi: Add AMD fam16h TPM ACPI path support
by build bot (Jenkins) (Code Review) 04 May '17

04 May '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/18522 ) Change subject: amd/pi: Add AMD fam16h TPM ACPI path support ...................................................................... Patch Set 7: Build Successful https://qa.coreboot.org/job/coreboot-gerrit/53192/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/9018/ : SUCCESS -- To view, visit https://review.coreboot.org/18522 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5322d731a0dc655f2da14b87fa6cbc1e54b5abd5 Gerrit-PatchSet: 7 Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Owner: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com> Gerrit-Reviewer: Kyösti Mälkki <kyosti.malkki(a)gmail.com> Gerrit-Reviewer: Marshall Dawson <marshalldawson3rd(a)gmail.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Philippe Mathieu-Daudé <philippe.mathieu.daude(a)gmail.com> Gerrit-Reviewer: Timothy Pearson <tpearson(a)raptorengineering.com> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-HasComments: No

1 0

Change in coreboot[master]: drivers/pc80/tpm: Fix missing tis_close() function
by build bot (Jenkins) (Code Review) 04 May '17

04 May '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/19535 ) Change subject: drivers/pc80/tpm: Fix missing tis_close() function ...................................................................... Patch Set 9: Build Successful https://qa.coreboot.org/job/coreboot-gerrit/53191/ : SUCCESS https://qa.coreboot.org/job/coreboot-checkpatch/9017/ : SUCCESS -- To view, visit https://review.coreboot.org/19535 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1a06f6a29015708e4bc1de6e6678827c28b84e98 Gerrit-PatchSet: 9 Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Owner: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com> Gerrit-Reviewer: Furquan Shaikh <furquan(a)google.com> Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com> Gerrit-Reviewer: Tobias Diedrich <ranma+coreboot(a)tdiedrich.de> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-HasComments: No

1 0

Change in coreboot[master]: drivers/pc80/tpm: Refactor init_tpm() implementation
by Philipp Deppenwiese (Code Review) 04 May '17

04 May '17

Hello build bot (Jenkins), I'd like you to reexamine a change. Please visit https://review.coreboot.org/19555 to look at the new patch set (#4). Change subject: drivers/pc80/tpm: Refactor init_tpm() implementation ...................................................................... drivers/pc80/tpm: Refactor init_tpm() implementation Move the TPM deactivate command after the tis_init() function in order to ensure that there is a TPM which can be used. Add additional doc for CONFIG_NO_TPM_RESUME kconfig option which is mainly used by chromebooks. Add TPM_PcrRead command for TPM state retrieval by calling it. Now the TPM state is logged. Also if TPM deactivate is enforced and something is wrong it will jump directly to the error handling. Change-Id: I2c51ce402f43466e211bc1990335fd320d685829 Signed-off-by: Philipp Deppenwiese <zaolin(a)das-labor.org> --- M src/drivers/pc80/tpm/romstage.c 1 file changed, 62 insertions(+), 32 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/55/19555/4 -- To view, visit https://review.coreboot.org/19555 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I2c51ce402f43466e211bc1990335fd320d685829 Gerrit-PatchSet: 4 Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Owner: Philipp Deppenwiese <zaolin.daisuki(a)gmail.com> Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in coreboot[master]: mb/gigabyte/ga-b75m-d3h: Add tpm support for its onboard tpm...
by Philipp Deppenwiese (Code Review) 04 May '17

04 May '17

Philipp Deppenwiese has posted comments on this change. ( https://review.coreboot.org/19521 ) Change subject: mb/gigabyte/ga-b75m-d3h: Add tpm support for its onboard tpm socket ...................................................................... Patch Set 5: Code-Review+1 (1 comment) https://review.coreboot.org/#/c/19521/5/src/mainboard/gigabyte/ga-b75m-d3h/… File src/mainboard/gigabyte/ga-b75m-d3h/Kconfig: Line 21: select MAINBOARD_HAS_LPC_TPM add select TPM because we have two implementation using the same driver tpm 1.2 and tpm 2.0 -- To view, visit https://review.coreboot.org/19521 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id0dee74d945bae5d77eb669d8b9d468a67aee508 Gerrit-PatchSet: 5 Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Owner: Bill XIE <persmule(a)gmail.com> Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org> 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-HasComments: Yes

1 0

Change in coreboot[master]: intelmetool: Print strerror() results for mmap errors
by Philipp Deppenwiese (Code Review) 04 May '17

04 May '17

Philipp Deppenwiese has posted comments on this change. ( https://review.coreboot.org/19560 ) Change subject: intelmetool: Print strerror() results for mmap errors ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://review.coreboot.org/19560 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I21352a00b583163472ccd3302a83adf1f8396c61 Gerrit-PatchSet: 1 Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Owner: Paul Wise (Debian) <pabs(a)debian.org> 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-HasComments: No

1 0

Change in coreboot[master]: intelmetool: free sb pci_dev struct allocated by pci_get_dev()
by Philipp Deppenwiese (Code Review) 04 May '17

04 May '17

Philipp Deppenwiese has posted comments on this change. ( https://review.coreboot.org/19561 ) Change subject: intelmetool: free sb pci_dev struct allocated by pci_get_dev() ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://review.coreboot.org/19561 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I011b2f96122d8f88aed121352afe3f0d41edef60 Gerrit-PatchSet: 1 Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Owner: Paul Wise (Debian) <pabs(a)debian.org> 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-HasComments: No

1 0

Change in coreboot[master]: intelmetool: handle failures to mmap MEI memory
by Philipp Deppenwiese (Code Review) 04 May '17

04 May '17

Philipp Deppenwiese has posted comments on this change. ( https://review.coreboot.org/19562 ) Change subject: intelmetool: handle failures to mmap MEI memory ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://review.coreboot.org/19562 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5becc0c2870dd97297c4e8d1b101b95b31792ca7 Gerrit-PatchSet: 1 Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Owner: Paul Wise (Debian) <pabs(a)debian.org> 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-HasComments: No

1 0

Change in coreboot[master]: blobtool/ifd-x200.set: Fix flashmap0 NR
by build bot (Jenkins) (Code Review) 04 May '17

04 May '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/19565 ) Change subject: blobtool/ifd-x200.set: Fix flashmap0 NR ...................................................................... Patch Set 2: Build Successful https://qa.coreboot.org/job/coreboot-gerrit/53190/ : SUCCESS -- To view, visit https://review.coreboot.org/19565 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3f4dcb0d41718dd176982679f8e045681fd3f486 Gerrit-PatchSet: 2 Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-HasComments: No

1 0

Change in coreboot[master]: blobtool/ifd-x200.set: Fix flashmap0 NR
by Arthur Heymans (Code Review) 04 May '17

04 May '17

Hello build bot (Jenkins), Nico Huber, I'd like you to reexamine a change. Please visit https://review.coreboot.org/19565 to look at the new patch set (#2). Change subject: blobtool/ifd-x200.set: Fix flashmap0 NR ...................................................................... blobtool/ifd-x200.set: Fix flashmap0 NR NR indicates the last non empty region, which in this case is GbE (region3). Needed for flashrom ifd layout support. Change-Id: I3f4dcb0d41718dd176982679f8e045681fd3f486 Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz> --- M util/blobtool/ifd-x200.set 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/65/19565/2 -- To view, visit https://review.coreboot.org/19565 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I3f4dcb0d41718dd176982679f8e045681fd3f486 Gerrit-PatchSet: 2 Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz> Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

coreboot-gerrit May 2017