[Skiboot] [PATCH][RESEND] occ: Add pstate corresponding to base frequency to DT

Shilpasri G Bhat shilpa.bhat at linux.vnet.ibm.com
Tue Jun 18 15:31:33 AEST 2019


Unlike POWER8, nominal frequency is not the highest guaranteed
frequency of the POWER9 chip. In POWER9, the highest guaranteed
frequency is greater than the nominal frequency and is referred
to as base frequency. In POWER9 base frequency is the highest
frequency the processor will operate at when ALL cores are active
and in ANY operating condition. This patch exports the turbo pstate
as the base frequency as per OCC documentation.

Signed-off-by: Shilpasri G Bhat <shilpa.bhat at linux.vnet.ibm.com>
---
 doc/device-tree/ibm,opal/power-mgt.rst | 12 +++++++++++-
 hw/occ.c                               |  1 +
 2 files changed, 12 insertions(+), 1 deletion(-)

diff --git a/doc/device-tree/ibm,opal/power-mgt.rst b/doc/device-tree/ibm,opal/power-mgt.rst
index 8d9439d..c8c753d 100644
--- a/doc/device-tree/ibm,opal/power-mgt.rst
+++ b/doc/device-tree/ibm,opal/power-mgt.rst
@@ -116,7 +116,10 @@ ibm,pstate-ultra-turbo ibm,pstate-turbo
 ---------------------------------------
 
 These properties are added when ultra-turbo(WOF) is enabled. These properties
-give the max turbo and max ultra-turbo pstate.
+give the max turbo and max ultra-turbo pstate-id as specified in the
+ibm,pstate-ids file. The frequencies present in turbo to ultra-turbo range are
+referred to as boost/WOF frequencies and these are attained by the CPU under
+favourable environmental conditions, low workloads and low active core counts.
 
 Example:
 
@@ -134,3 +137,10 @@ ibm,pstate-core-max
 This property is added when ultra_turbo(WOF) is enabled. This property gives
 the list of max pstate for each 'n' number of active cores in the chip.
 
+ibm,pstate-base
+----------------
+
+This pstate points to the base frequency of the chip. POWER9 base frequency is
+the highest frequency that is guaranteed when ALL cores are active in ANY
+operating condition (ie. workloads, environmental conditions such as max
+ambient temperature, active core counts)
diff --git a/hw/occ.c b/hw/occ.c
index 4e97839..f42742d 100644
--- a/hw/occ.c
+++ b/hw/occ.c
@@ -720,6 +720,7 @@ static bool add_cpu_pstate_properties(struct dt_node *power_mgt,
 		dt_add_property(power_mgt, "ibm,pstate-core-max", dt_cmax,
 				nr_cores * sizeof(u32));
 
+		dt_add_property_cells(power_mgt, "ibm,pstate-base", pturbo);
 		free(dt_cmax);
 	}
 
-- 
1.8.3.1



More information about the Skiboot mailing list