[coreboot-gerrit] New patch to review for coreboot: soc/intel/apollolake: Update PL1 value in RAPL MMIO register

[Sumeet R Pawnikar (sumeet.r.pawnikar@intel.com)]

Sumeet R Pawnikar (sumeet.r.pawnikar at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/16595

-gerrit

commit ef5ea332b693a22dfc9be9a62153e615c51ca6e9
Author: Sumeet Pawnikar <sumeet.r.pawnikar at intel.com>
Date:   Tue Aug 23 11:20:20 2016 +0530

    soc/intel/apollolake: Update PL1 value in RAPL MMIO register
    
    Due to incorrect value set for power limit PL1, not able to
    leverage full TDP capacity of the system. RAPL MMIO register
    sets the PL1 to 6W which is limiting Package power.
    This RAPL MMIO register is a physically separate instance
    from RAPL MSR register. This patch sets PL1 value to 15W
    in RAPL MMIO register.
    
    BUG=chrome-os-partner:56524
    TEST=Built, Booted on reef and verifed the package power
    with heavy workload.
    
    Change-Id: Ib344247cd8d98ccce7c403e778cd87c13f168ce0
    Signed-off-by: Sumeet Pawnikar <sumeet.r.pawnikar at intel.com>
---
 src/soc/intel/apollolake/chip.c                    | 17 +++++++++++++++++
 src/soc/intel/apollolake/include/soc/cpu.h         |  1 +
 src/soc/intel/apollolake/include/soc/northbridge.h |  2 ++
 3 files changed, 20 insertions(+)

diff --git a/src/soc/intel/apollolake/chip.c b/src/soc/intel/apollolake/chip.c
index 78c669d..6a4d7fa 100644
--- a/src/soc/intel/apollolake/chip.c
+++ b/src/soc/intel/apollolake/chip.c
@@ -34,6 +34,7 @@
 #include <spi-generic.h>
 #include <soc/pm.h>
 #include <soc/p2sb.h>
+#include <soc/northbridge.h>
 
 #include "chip.h"
 
@@ -189,6 +190,19 @@ static void pcie_override_devicetree_after_silicon_init(void)
 	pcie_update_device_tree(PCIEB0_DEVFN, 2);
 }
 
+static void rapl_update(void)
+{
+	uint32_t *rapl_reg = (uint32_t*) (MCH_BASE_ADDR + MCHBAR_RAPL_PPL);
+	uint32_t val;
+	const uint32_t power_mw = 15000;
+
+	/* Due to incorrect value set for RAPL power limit PL1, not able
+	 * to leverage full TDP capacity of the system. Setting RAPL PL1
+	 * in Bits[14:0] to 15W in RAPL MMIO register. */
+	val = (power_mw << (rdmsr(MSR_PKG_POWER_SKU_UNIT).lo & 0xf)) / 1000;
+	write32(rapl_reg, (read32(rapl_reg) & ~0x7fff) | val);
+}
+
 static void soc_init(void *data)
 {
 	struct global_nvs_t *gnvs;
@@ -218,6 +232,9 @@ static void soc_init(void *data)
 
 	/* Allocate ACPI NVS in CBMEM */
 	gnvs = cbmem_add(CBMEM_ID_ACPI_GNVS, sizeof(*gnvs));
+
+	/* Update RAPL package power limit */
+	rapl_update();
 }
 
 static void soc_final(void *data)
diff --git a/src/soc/intel/apollolake/include/soc/cpu.h b/src/soc/intel/apollolake/include/soc/cpu.h
index 8887c17..22412af 100644
--- a/src/soc/intel/apollolake/include/soc/cpu.h
+++ b/src/soc/intel/apollolake/include/soc/cpu.h
@@ -38,6 +38,7 @@ void apollolake_init_cpus(struct device *dev);
 #define   PREFETCH_L1_DISABLE	(1 << 0)
 #define   PREFETCH_L2_DISABLE	(1 << 2)
 
+#define MSR_PKG_POWER_SKU_UNIT	0x606
 
 #define MSR_L2_QOS_MASK(reg)		(0xd10 + reg)
 #define MSR_IA32_PQR_ASSOC		0xc8f
diff --git a/src/soc/intel/apollolake/include/soc/northbridge.h b/src/soc/intel/apollolake/include/soc/northbridge.h
index 0f61674..04e369e 100644
--- a/src/soc/intel/apollolake/include/soc/northbridge.h
+++ b/src/soc/intel/apollolake/include/soc/northbridge.h
@@ -34,5 +34,7 @@
 #define MCH_IMR_PITCH		0x20
 #define MCH_NUM_IMRS		20
 
+/* RAPL Package Power Limit register under MCHBAR. */
+#define MCHBAR_RAPL_PPL		0x70A8
 
 #endif /* _SOC_APOLLOLAKE_NORTHBRIDGE_H_ */