From f066cf15b3091b6e84894ad5c3ba361b7bd0d737 Mon Sep 17 00:00:00 2001 From: Greg Still Date: Wed, 7 Dec 2016 15:28:47 -0600 Subject: Update pm_plat_attributes with defaults and better descriptions - Add 24x7 invocation time to hb_temp_defaults - Rename 24x7 attribute to not start with a number (HB CI) - Made iVRM deadzone attribute an override attribute - Addressed Gerrit comments(2) - Rebased - Added some addition indicators and other default values - Made PERF_24x7_INVOCATION default to 1 to resolve Gerrit comment Change-Id: I333ec30337b0801b041b8deec23ce7dd55ffd6eb RTC: 167295 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/34853 Dev-Ready: Gregory S. Still Tested-by: Jenkins Server Reviewed-by: Matt K. Light Tested-by: Hostboot CI Tested-by: PPE CI Reviewed-by: Jennifer A. Stofer Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/34859 Reviewed-by: Hostboot Team Tested-by: FSP CI Jenkins Reviewed-by: Sachin Gupta --- .../xml/attribute_info/pm_plat_attributes.xml | 182 ++++++++++++++++----- 1 file changed, 141 insertions(+), 41 deletions(-) (limited to 'src/import') diff --git a/src/import/chips/p9/procedures/xml/attribute_info/pm_plat_attributes.xml b/src/import/chips/p9/procedures/xml/attribute_info/pm_plat_attributes.xml index 4538a44f..dc5253cd 100644 --- a/src/import/chips/p9/procedures/xml/attribute_info/pm_plat_attributes.xml +++ b/src/import/chips/p9/procedures/xml/attribute_info/pm_plat_attributes.xml @@ -5,7 +5,7 @@ - + @@ -601,6 +601,7 @@ int8 + @@ -619,6 +620,7 @@ int8 + @@ -641,6 +643,7 @@ int8 + @@ -663,6 +666,7 @@ int8 + @@ -685,6 +689,7 @@ int8 + @@ -707,6 +712,7 @@ int8 + @@ -717,15 +723,16 @@ if OFF, treat STOP4 as STOP4 if ON, treat STOP4 as STOP2 - Producer: ??? + Producer: Work-around tools - Consumer: p8_hcode_image_build.C + Consumer: p9_hcode_image_build.C Platform default: OFF uint8 OFF=0, ON=1 + @@ -737,13 +744,14 @@ Producer: ??? - Consumer: p8_hcode_image_build.C + Consumer: p9_hcode_image_build.C Platform default: ON uint8 OFF=0, ON=1 + @@ -754,15 +762,16 @@ if OFF, treat STOP8 as STOP8 if ON, treat STOP8 as STOP4 - Producer: ??? + Producer: Work-around tools - Consumer: p8_hcode_image_build.C + Consumer: p9_hcode_image_build.C Platform default: OFF uint8 OFF=0, ON=1 + @@ -773,15 +782,16 @@ if OFF, treat STOP11 as STOP11 if ON, treat STOP11 as STOP8 - Producer: ??? + Producer: Work-around tools - Consumer: p8_hcode_image_build.C + Consumer: p9_hcode_image_build.C Platform default: OFF uint8 OFF=0, ON=1 + @@ -801,6 +811,7 @@ uint8 FALSE=0, TRUE=1 + @@ -819,6 +830,7 @@ uint8 FALSE=0, TRUE=1 + @@ -832,7 +844,7 @@ Consumers: p9_pm_pfet_init - Platform default: + Platform default:250 uint32 @@ -849,7 +861,7 @@ Consumers: p9_pm_pfet_init - Platform default: + latform default:250 uint32 @@ -865,7 +877,7 @@ Consumers: p9_pm_pfet_init - Platform default: + Platform default: ALLOFF uint8 @@ -893,7 +905,7 @@ Consumers: p9_pm_pfet_init - Platform default: + Platform default: ALLOFF uint8 @@ -961,8 +973,8 @@ uint8 - - + + ATTR_POUNDV_BUCKET_NUM @@ -1282,6 +1294,7 @@ OFF = 0x00,ON = 0x01 + @@ -1309,10 +1322,23 @@ ATTR_IVRM_DEADZONE_MV TARGET_TYPE_SYSTEM - Indicates the size of the deadzone where the iVRM cannot regulate - (binary in millivolts) + Override value for the value of the deadzone where the iVRM cannot regulate + (binary in millivolts). + + The deadzone is the difference between the voltage from the external VRM + after load line and other losses are removed and the request regulation + voltage. If this difference is smaller than the value of this attribute, + the iVRM is forced in to bypass to use the external voltage. + + Producer: MRWB. + + Consumer: p9_pstate_parameter_block -> + CME pstate parameter block + PGPE pstate parameter block + + Handling: p9_pstate_parameter_block reads this attribute. If zero, the + value of 50 (0x32) is used. If non-zero, this value is used. - Producer: MRWB. uint8 @@ -1323,67 +1349,107 @@ ATTR_IVRM_STRENGTH_LOOKUP TARGET_TYPE_SYSTEM + Lookup table used to as part of determining the PFET width to use based + on the voltage across the PFET header. + + Producer: MRWB via the iVRM team + + Consumer: p9_pstate_parameter_block -> + CME pstate parameter block + PGPE pstate parameter block - Producer: MRWB. uint8 64 + ATTR_IVRM_VIN_MULTIPLIER TARGET_TYPE_SYSTEM - + Multiplier used with the strength lookup to determine the IVRM + PFET width. - Producer: MRWB. + Producer: MRWB via the iVRM team + + Consumer: p9_pstate_parameter_block -> + CME pstate parameter block + PGPE pstate parameter block uint8 64 + ATTR_IVRM_VIN_MAX_MV TARGET_TYPE_SYSTEM - + Override voltage maximum that is used for the IVRM PFET width + calculation. Setting to 0 will use the default 1100mV. Setting this to a + non-zero value will cause this value to be used instead. - Producer: MRWB. + Producer: MRWB. + + Consumer: p9_pstate_parameter_block -> + CME pstate parameter block + PGPE pstate parameter block uint16 + ATTR_IVRM_STEP_DELAY_NS TARGET_TYPE_SYSTEM - + Time (in nanoseconds) to wait between IVRM steps that are + part of a larger transition to the ultimate destination voltage. The + value of 0 is valid to indicate that no delay is necessary. - Producer: MRWB. + Producer: MRWB. + + Consumer: p9_pstate_parameter_block -> + CME pstate parameter block + PGPE pstate parameter block uint16 + ATTR_IVRM_STABILIZATION_DELAY_NS TARGET_TYPE_SYSTEM + Time (in nanoseconds) to wait after the iVRMs indicate "done" in the event + extra time is required. - Producer: MRWB. + Producer: MRWB. + + Consumer: p9_pstate_parameter_block -> + CME pstate parameter block + PGPE pstate parameter block uint16 + ATTR_TDP_RDP_CURRENT_FACTOR TARGET_TYPE_PROC_CHIP TODO RTC 157943 -- Placeholder description + This overrides the RDP to TDP Scaling Factor IQ VPD field that is used for + Workload Optimized Frequency (WOF) voltage uplifting. + Consumers: p9_pstate_parameter_block + uint32 + @@ -1411,8 +1477,7 @@ Minimum delay (in nanoseconds) between clock grid management transition steps - Producer: MRWB - + Producer: MRWB from clock team Consumers: p9_build_pstate_datablock -> CME Quad Pstate Region (CQPR) for CM Quad Manager @@ -1420,6 +1485,7 @@ uint16 + @@ -1434,11 +1500,14 @@ ATTR_RESCLK_FREQ_REGIONS[2] LE Region 2 LT ATTR_RESCLK_FREQ_REGIONS[3] etc. + Producer: MRWB from clock team + Consumers: p9_pstate_parameter_block uint8 8 + @@ -1451,11 +1520,14 @@ ATTR_RESCLK_FREQ_REGIONS[i] and ATTR_RESCLK_FREQ_REGIONS[i+1] for 0 LE i LE 7. + Producer: MRWB from clock team + Consumers: p9_pstate_parameter_block uint8 8 + @@ -1464,11 +1536,14 @@ Array of Clock strength values that will we written in QACCR by CME Hcode + Producer: MRWB from clock team + Consumers: p9_pstate_parameter_block uint16 64 + @@ -1493,11 +1568,14 @@ ATTR_RESCLK_L3_VOLTAGE_THRESHOLD_MV and then steps 3->0 when going at or above the voltage defined by ATTR_RESCLK_L3_VOLTAGE_THRESHOLD_MV. + Producer: MRWB from clock team + Consumers: p9_pstate_parameter_block uint8 4 + @@ -1510,24 +1588,46 @@ strength values put in the hardware for this mode transtion are defined by ATTR_RESCLK_L3_VALUE. + Producer: MRWB from clock team + Consumers: p9_pstate_parameter_block uint16 - - + + + - ATTR_SYSTEM_RING_DBG_MODE - - Debug modes supported for CME/SGPE Scan layout in HOMER. - - TARGET_TYPE_SYSTEM - uint8 - - SCAN_RING_NO_DEBUG = 0x00, SCAN_RING_TRACE_DEBUG = 0x01 - - - + ATTR_SYSTEM_RING_DBG_MODE + + Debug modes supported for CME/SGPE Scan layout in HOMER. + + TARGET_TYPE_SYSTEM + uint8 + + SCAN_RING_NO_DEBUG = 0x00, SCAN_RING_TRACE_DEBUG = 0x01 + + + - + + + ATTR_PERF_24x7_INVOCATION_TIME_MS + + Time between invocations of the 24x7 performance collection function on + GPE1. The time (in milliseconds) is derived as + 2^PERF_24x7_INVOCATION_TIME_MS with 0 indicating the function is OFF. + + Consumer: p9_hcode_image_build.c -> + SGPE Header field + + Provided by the Machine Readable Workbook to tune the collection. + + Platform default: 1 + + TARGET_TYPE_SYSTEM + uint8 + + + -- cgit v1.2.1