Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/37823 )
Change subject: mb/google/hatch/var/kindred: Update PL2 power limit value
......................................................................
Patch Set 1:
Patch Set 1:
(3 comments)
Why does the value need to be specified in two places?
One is the initial value to be programmed by FSP (devicetree), the other is exposed to DPTF.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/37823
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Iaefab3011224d6dcf561b437bd34f54a62cf1287
Gerrit-Change-Number: 37823
Gerrit-PatchSet: 1
Gerrit-Owner: David Wu
david_wu@quanta.corp-partner.google.com
Gerrit-Reviewer: David Wu
david_wu@quanta.corp-partner.google.com
Gerrit-Reviewer: Furquan Shaikh
furquan@google.com
Gerrit-Reviewer: Paul Fagerburg
pfagerburg@chromium.org
Gerrit-Reviewer: Philip Chen
philipchen@google.com
Gerrit-Reviewer: Sumeet R Pawnikar
sumeet.r.pawnikar@intel.com
Gerrit-Reviewer: Tim Wawrzynczak
twawrzynczak@chromium.org
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-CC: Peter Ou
peter.ou@quanta.corp-partner.google.com
Gerrit-Comment-Date: Thu, 19 Dec 2019 16:21:50 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment