Gaggery Tsai has posted comments on this change. ( https://review.coreboot.org/23633 )
Change subject: mb/google/fizz: Set Pmax to 120 for all SKUs
......................................................................
Patch Set 1: Code-Review+1
That will be great if you can add up some contents in the commit message. For example, the Pmax is calcuated from MAX(Psku1, Psku2), where Psku1, Psku2 are estimated Pmax power of U42 and U22 skus. For U42 sku, the Pmax is PL4 (71W) + ROPmax (49W) = 120W; for U22 SKU, the Pmax is PL4 (43W) + ROPmax (49W) = 92W. So Pmax is set to MAX(120W, 92W) = 120W.
--
To view, visit
https://review.coreboot.org/23633
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: Ic27fef87c869094b20438e6ee0e1eb0b35122b8d
Gerrit-Change-Number: 23633
Gerrit-PatchSet: 1
Gerrit-Owner: Shelley Chen
shchen@google.com
Gerrit-Reviewer: Furquan Shaikh
furquan@google.com
Gerrit-Reviewer: Gaggery Tsai
gaggery.tsai@intel.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Comment-Date: Wed, 07 Feb 2018 05:31:58 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: Yes