Julius Werner has posted comments on this change. ( https://review.coreboot.org/28374 )
Change subject: mainboard/google/kahlee: Enable EC firmware update screen
......................................................................
Patch Set 1: Code-Review+2
(1 comment)
https://review.coreboot.org/#/c/28374/1/src/mainboard/google/kahlee/Kconfig
File src/mainboard/google/kahlee/Kconfig:
https://review.coreboot.org/#/c/28374/1/src/mainboard/google/kahlee/Kconfig…
PS1, Line 101: if VBOOT_EC_SOFTWARE_SYNC
nit: VBOOT_EC_SLOW_UPDATE already has a 'depends on VBOOT_EC_SOFTWARE_SYNC', this if clause is pretty pointless.
--
To view, visit https://review.coreboot.org/28374
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: I95fc4d3430bac66c09f57a4d34abde08752e5f0e
Gerrit-Change-Number: 28374
Gerrit-PatchSet: 1
Gerrit-Owner: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Daniel Kurtz <djkurtz(a)google.com>
Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org>
Gerrit-Reviewer: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Tue, 28 Aug 2018 22:27:50 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: Yes
Hello Tom Hiller, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/28368
to look at the new patch set (#2).
Change subject: Documentation: add description for util/pmh7tool
......................................................................
Documentation: add description for util/pmh7tool
Change-Id: Iab5daf101a9ff27aa49b7849bf6bf39362b8db09
Signed-off-by: Evgeny Zinoviev <me(a)ch1p.com>
---
M Documentation/util.md
M util/README.md
A util/pmh7tool/description.md
3 files changed, 3 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/68/28368/2
--
To view, visit https://review.coreboot.org/28368
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: Iab5daf101a9ff27aa49b7849bf6bf39362b8db09
Gerrit-Change-Number: 28368
Gerrit-PatchSet: 2
Gerrit-Owner: Evgeny Zinoviev <me(a)ch1p.com>
Gerrit-Reviewer: Evgeny Zinoviev <me(a)ch1p.com>
Gerrit-Reviewer: Tom Hiller <thrilleratplay(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Evgeny Zinoviev has posted comments on this change. ( https://review.coreboot.org/28368 )
Change subject: Documentation: add util/pmh7tool to util.md
......................................................................
Patch Set 1:
> There is a script under util called "util_readme" that auto
> generates `Documentation/util.md` and `util/README.md` based on the
> `description.md` file under each util directory. Can you add a
> `description.md` for pmh7tool and run the script instead and submit
> those as changes?
Sorry, didn't know that. Sure.
--
To view, visit https://review.coreboot.org/28368
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: Iab5daf101a9ff27aa49b7849bf6bf39362b8db09
Gerrit-Change-Number: 28368
Gerrit-PatchSet: 1
Gerrit-Owner: Evgeny Zinoviev <me(a)ch1p.com>
Gerrit-Reviewer: Evgeny Zinoviev <me(a)ch1p.com>
Gerrit-Reviewer: Tom Hiller <thrilleratplay(a)gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Tue, 28 Aug 2018 21:57:02 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No
Felix Held has posted comments on this change. ( https://review.coreboot.org/28377 )
Change subject: Documentation/nb/intel/sandybridge/nri_registers.md: Fix mistake
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://review.coreboot.org/28377
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: I28fed4a9264b96f93b9e88325f547a5db512514c
Gerrit-Change-Number: 28377
Gerrit-PatchSet: 1
Gerrit-Owner: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Angel Pons <th3fanbus(a)gmail.com>
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: Patrick Rudolph <siro(a)das-labor.org>
Gerrit-Comment-Date: Tue, 28 Aug 2018 18:44:39 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes
Raul Rangel has posted comments on this change. ( https://review.coreboot.org/28374 )
Change subject: mainboard/google/kahlee: Enable EC firmware update screen
......................................................................
Patch Set 1: Code-Review+2
Ah, I see it sets the vboot flags
--
To view, visit https://review.coreboot.org/28374
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: I95fc4d3430bac66c09f57a4d34abde08752e5f0e
Gerrit-Change-Number: 28374
Gerrit-PatchSet: 1
Gerrit-Owner: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Daniel Kurtz <djkurtz(a)google.com>
Gerrit-Reviewer: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Tue, 28 Aug 2018 15:18:36 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes
Raul Rangel has posted comments on this change. ( https://review.coreboot.org/28374 )
Change subject: mainboard/google/kahlee: Enable EC firmware update screen
......................................................................
Patch Set 1:
Is coreboot updating the EC now?
--
To view, visit https://review.coreboot.org/28374
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: I95fc4d3430bac66c09f57a4d34abde08752e5f0e
Gerrit-Change-Number: 28374
Gerrit-PatchSet: 1
Gerrit-Owner: Martin Roth <martinroth(a)google.com>
Gerrit-Reviewer: Daniel Kurtz <djkurtz(a)google.com>
Gerrit-Reviewer: Raul Rangel <rrangel(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Comment-Date: Tue, 28 Aug 2018 15:08:24 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No