Wonkyu Kim has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39413 )
Change subject: mb/google/volteer: configure L1Substate for PCIe
......................................................................
Patch Set 5:
(2 comments)
https://review.coreboot.org/c/coreboot/+/39413/4/src/mainboard/google/voltee...
File src/mainboard/google/volteer/variants/ripto/overridetree.cb:
https://review.coreboot.org/c/coreboot/+/39413/4/src/mainboard/google/voltee...
PS4, Line 4: # Limit L1.1(value:2) for RP9, RP11
Please add a space before the (.
Done
https://review.coreboot.org/c/coreboot/+/39413/4/src/mainboard/google/voltee...
PS4, Line 5: register "PcieRpL1Substates[8]" = "2"
What is the default value?
Default value is 0 (FSP default: L1.2)
--
To view, visit
https://review.coreboot.org/c/coreboot/+/39413
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ie85bf71c43427e326ef2ba674da4566f8f51495a
Gerrit-Change-Number: 39413
Gerrit-PatchSet: 5
Gerrit-Owner: Wonkyu Kim
wonkyu.kim@intel.com
Gerrit-Reviewer: Caveh Jalali
caveh@chromium.org
Gerrit-Reviewer: Furquan Shaikh
furquan@google.com
Gerrit-Reviewer: Nick Vaccaro
nvaccaro@google.com
Gerrit-Reviewer: Raj Astekar
raj.astekar@intel.com
Gerrit-Reviewer: Shaunak Saha
shaunak.saha@intel.com
Gerrit-Reviewer: Srinidhi N Kaushik
srinidhi.n.kaushik@intel.com
Gerrit-Reviewer: Wonkyu Kim
wonkyu.kim@intel.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-CC: Angel Pons
th3fanbus@gmail.com
Gerrit-CC: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-Comment-Date: Wed, 11 Mar 2020 21:40:57 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Paul Menzel
paulepanter@users.sourceforge.net
Gerrit-MessageType: comment