[coreboot-gerrit] New patch to review for coreboot: kunimitsu: Update the PL1 value as TDP

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Wed Jan 13 18:44:35 CET 2016


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/12934

-gerrit

commit 2f7c875ba0251a2ab079cff0807f1ff3958910f6
Author: Sumeet Pawnikar <sumeet.r.pawnikar at intel.com>
Date:   Thu Nov 26 13:33:13 2015 +0530

    kunimitsu: Update the PL1 value as TDP
    
    Currently, the Power Limit 1 (PL1) value is 6W which is
    low for high performance KPIs. This patch updates PL1
    value as TDP. SKL-U has 15W TDP.
    
    BRANCH=None
    BUG=None
    TEST=Built and boot on kunimitsu. Check for the PL1 value over
    sysfs interface
    "/sys/class/powercap/intel-rapl:0/constraint_0_power_limit_uw".
    Load the system with Aquarium 1000 Fish, average FPS should be
    meeting target 60 with this change.
    
    Change-Id: I8e083192e8018edc2cf8b88530df1e05ede10bde
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: eb9aa00a4271875b5471c33883aa7da022f1cb0e
    Original-Change-Id: I0c61fe1a9f76a9cf9a306240fb66d4c081d2bb5e
    Original-Signed-off-by: Sumeet Pawnikar <sumeet.r.pawnikar at intel.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/314416
    Original-Commit-Ready: Sumeet R Pawnikar <sumeet.r.pawnikar at intel.com>
    Original-Tested-by: Sumeet R Pawnikar <sumeet.r.pawnikar at intel.com>
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    Original-Reviewed-by: Rizwan Qureshi <rizwan.qureshi at intel.com>
    Original-Reviewed-by: Sumeet R Pawnikar <sumeet.r.pawnikar at intel.com>
---
 src/mainboard/intel/kunimitsu/acpi/dptf.asl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mainboard/intel/kunimitsu/acpi/dptf.asl b/src/mainboard/intel/kunimitsu/acpi/dptf.asl
index 2eea804..e011c97 100644
--- a/src/mainboard/intel/kunimitsu/acpi/dptf.asl
+++ b/src/mainboard/intel/kunimitsu/acpi/dptf.asl
@@ -110,7 +110,7 @@ Name (MPPC, Package ()
 	Package () {	/* Power Limit 1 */
 		0,	/* PowerLimitIndex, 0 for Power Limit 1 */
 		1600,	/* PowerLimitMinimum */
-		6000,	/* PowerLimitMaximum */
+		15000,	/* PowerLimitMaximum */
 		1000,	/* TimeWindowMinimum */
 		1000,	/* TimeWindowMaximum */
 		200	/* StepSize */



More information about the coreboot-gerrit mailing list