Hello Patrick Rudolph, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/19535
to look at the new patch set (#2).
Change subject: drivers/pc80/tpm: Fix missing tis_close() function and TPM deactivation.
......................................................................
drivers/pc80/tpm: Fix missing tis_close() function and TPM deactivation.
tis_close() must be called after tis_open() otherwise the locked
locality isn't released and …
[View More]the sessions hangs.
The deactivate command of the TPM should be executed after
the startup is done. So that we can be sure that the TPM
exists and is functional.
Change-Id: I1a06f6a29015708e4bc1de6e6678827c28b84e98
Signed-off-by: Philipp Deppenwiese <zaolin(a)das-labor.org>
---
M src/drivers/pc80/tpm/romstage.c
1 file changed, 4 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/35/19535/2
--
To view, visit https://review.coreboot.org/19535
To unsubscribe, visit https://review.coreboot.org/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I1a06f6a29015708e4bc1de6e6678827c28b84e98
Gerrit-PatchSet: 2
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>
[View Less]
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/19522
to look at the new patch set (#4).
Change subject: mb/gigabyte/ga-b75m-d3h: add libgfxinit support
......................................................................
mb/gigabyte/ga-b75m-d3h: add libgfxinit support
Currently native video init works on port HDMI1 (wired to the
on-board DVI-D socket) and HDMI3 (the on-board HDMI port), both text
mode and fb mode, but fails …
[View More]to work on the onboard vga port.
Every ports works on GNU/Linux.
Tested against an IVB cpu (i7-3770T).
Change-Id: If00a7247df0c32b3d1f489fb92d86baaa8fdf8ba
Signed-off-by: Bill XIE <persmule(a)gmail.com>
---
M src/mainboard/gigabyte/ga-b75m-d3h/Kconfig
M src/mainboard/gigabyte/ga-b75m-d3h/Makefile.inc
A src/mainboard/gigabyte/ga-b75m-d3h/gma-mainboard.ads
3 files changed, 17 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/22/19522/4
--
To view, visit https://review.coreboot.org/19522
To unsubscribe, visit https://review.coreboot.org/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If00a7247df0c32b3d1f489fb92d86baaa8fdf8ba
Gerrit-PatchSet: 4
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Bill XIE <persmule(a)gmail.com>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Bill XIE <persmule(a)gmail.com>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
[View Less]
Philipp Deppenwiese has posted comments on this change. ( https://review.coreboot.org/19535 )
Change subject: drivers/pc80/tpm: Fix missing tis_close() function and TPM deactivation.
......................................................................
Patch Set 1:
Now I see what you mean. In the coreboot implementation a temporary deactivate via the startup method is called. Not a physical presence deactivate command. I will move this out of this patch into an extra change.
--
To view, …
[View More]visit https://review.coreboot.org/19535
To unsubscribe, visit https://review.coreboot.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I1a06f6a29015708e4bc1de6e6678827c28b84e98
Gerrit-PatchSet: 1
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
[View Less]
Philipp Deppenwiese has posted comments on this change. ( https://review.coreboot.org/19535 )
Change subject: drivers/pc80/tpm: Fix missing tis_close() function and TPM deactivation.
......................................................................
Patch Set 1:
(2 comments)
@Patrick can you be more specific giving -2 for a change without any explanation and referenced line of code is somehow hard to understand
https://review.coreboot.org/#/c/19535/1//COMMIT_MSG
Commit Message:
PS1, …
[View More]Line 9: tis_close() must be called after tis_open() otherwise the locked
: locality isn't released and the sessions hangs.
> My impression was that the Linux Kernel works fine with the TPMs used in re
Yes the implementation is broken by not calling tis_close() after the command are issued. The deactivate should be done after tis_init() / tis_open() which probes the tpm is called
Line 15:
> Tested how?
Tested on a APU2 platform
--
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: 1
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: Yes
[View Less]
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/19500 )
Change subject: nb/intel/gm45: Set display backlight according to EDID string
......................................................................
Patch Set 19:
ok still works as expect: correct backlight when edid string in list, notice message when not.
--
To view, visit https://review.coreboot.org/19500
To unsubscribe, visit https://review.coreboot.org/settings
Gerrit-MessageType: comment
…
[View More]Gerrit-Change-Id: Id7bc0d453fac31e806852206ba2c895720b2c843
Gerrit-PatchSet: 19
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-HasComments: No
[View Less]
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/19500 )
Change subject: nb/intel/gm45: Set display backlight according to EDID string
......................................................................
Patch Set 19:
> >> When does _this_ code get tested?
> >
> > what do you mean? like the values for displays?
>
> No, I mean this exact commit. It obviously didn't compile, yet
> stated
> the added value is tested. So …
[View More]I presume you tested it with
> different
> code?
yes I modified a value and added another display a user tested, was a bit to hasty in doing so.
I'll do a last test now.
--
To view, visit https://review.coreboot.org/19500
To unsubscribe, visit https://review.coreboot.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Id7bc0d453fac31e806852206ba2c895720b2c843
Gerrit-PatchSet: 19
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-HasComments: No
[View Less]
Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/19500 )
Change subject: nb/intel/gm45: Set display backlight according to EDID string
......................................................................
Patch Set 19:
> When does _this_ code get tested?
what do you mean? like the values for displays?
--
To view, visit https://review.coreboot.org/19500
To unsubscribe, visit https://review.coreboot.org/settings
Gerrit-MessageType: comment
Gerrit-…
[View More]Change-Id: Id7bc0d453fac31e806852206ba2c895720b2c843
Gerrit-PatchSet: 19
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-Reviewer: Nico Huber <nico.h(a)gmx.de>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Reviewer: Paul Menzel <paulepanter(a)users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-HasComments: No
[View Less]