diff options
| author | Rahul Batra <rbatra@us.ibm.com> | 2017-02-20 08:47:45 -0600 |
|---|---|---|
| committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2017-03-17 11:32:26 -0400 |
| commit | 6f56050a5315f15710e9eff1dd70746b39843ab8 (patch) | |
| tree | e1fb74a90f336e8dc6f074fa4323f3a2a35f1c17 /src | |
| parent | c02404bc31d5e1772ed2688f22535633f11f7356 (diff) | |
| download | talos-hostboot-6f56050a5315f15710e9eff1dd70746b39843ab8.tar.gz talos-hostboot-6f56050a5315f15710e9eff1dd70746b39843ab8.zip | |
WOF/PGPE Bug Fixes(Set 2)
-Fixes Inter-CME interrupt issue
-Fixed back to back CLIP Updt issue
-reverses CLIP_UPDT clip_min and clip_max(as per OCC)
-Fixes AVS_Driver issues
-Moved VPD points from GeneratedPstateInfo to Global Parm
-Added nest frequency field to OCCParmBlock
-Picking up slopes from Global Parm
Change-Id: I7578a8a51efde59f0678b9c3f44be77bf7a15603
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/36741
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: RANGANATHPRASAD G. BRAHMASAMUDRA <prasadbgr@in.ibm.com>
Reviewed-by: Douglas R. Gilbert <dgilbert@us.ibm.com>
Reviewed-by: Gregory S. Still <stillgs@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/37213
Reviewed-by: Hostboot Team <hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src')
4 files changed, 8 insertions, 11 deletions
diff --git a/src/import/chips/p9/procedures/hwp/lib/p9_pstates_common.h b/src/import/chips/p9/procedures/hwp/lib/p9_pstates_common.h index 944faa430..72fa4b230 100644 --- a/src/import/chips/p9/procedures/hwp/lib/p9_pstates_common.h +++ b/src/import/chips/p9/procedures/hwp/lib/p9_pstates_common.h @@ -120,8 +120,8 @@ #define VPD_SLOPES_RAW 0 #define VPD_SLOPES_BIASED 1 #define VPD_NUM_SLOPES_REGION 3 -#define REGION_POWERSAVE_NOMINAL 1 -#define REGION_NOMINAL_TURBO 0 +#define REGION_POWERSAVE_NOMINAL 0 +#define REGION_NOMINAL_TURBO 1 #define REGION_TURBO_ULTRA 2 // Different points considered for calculating slopes @@ -130,6 +130,8 @@ #define VPD_PT_SET_SYSP 1 #define VPD_PT_SET_BIASED 2 #define VPD_PT_SET_BIASED_SYSP 3 +#define VPD_PT_SET_ORDER {VPD_PT_SET_RAW, VPD_PT_SET_SYSP, VPD_PT_SET_BIASED, VPD_PT_SET_BIASED_SYSP} +#define VPD_PT_SET_ORDER_STR {"Raw ", "SysParam ","Biased ", "Biased/SysParam"} #define EVID_SLOPE_FP_SHIFT 13 diff --git a/src/import/chips/p9/procedures/hwp/lib/p9_pstates_occ.h b/src/import/chips/p9/procedures/hwp/lib/p9_pstates_occ.h index 67be858be..fbcc909f0 100644 --- a/src/import/chips/p9/procedures/hwp/lib/p9_pstates_occ.h +++ b/src/import/chips/p9/procedures/hwp/lib/p9_pstates_occ.h @@ -185,6 +185,8 @@ typedef struct // Minimum Pstate; Maximum is always 0. uint32_t pstate_min; // Comes from PowerSave #V point after biases + /// Nest frequency in Mhz. This is used by FIT interrupt + uint32_t nest_frequency_mhz; } __attribute__((aligned(128))) OCCPstateParmBlock; #ifdef __cplusplus diff --git a/src/import/chips/p9/procedures/hwp/lib/p9_pstates_pgpe.h b/src/import/chips/p9/procedures/hwp/lib/p9_pstates_pgpe.h index 7106bf9d0..a6cd8b943 100644 --- a/src/import/chips/p9/procedures/hwp/lib/p9_pstates_pgpe.h +++ b/src/import/chips/p9/procedures/hwp/lib/p9_pstates_pgpe.h @@ -241,6 +241,8 @@ typedef struct /// Precalculated Voltage-Pstates Slopes uint16_t VPsSlopes[VPD_NUM_SLOPES_SET][VPD_NUM_SLOPES_REGION]; + /// All operating points + VpdOperatingPoint operating_points_set[NUM_VPD_PTS_SET][VPD_PV_POINTS]; // @todo DPLL Droop Settings. These need communication to SGPE for STOP diff --git a/src/import/chips/p9/procedures/hwp/lib/p9_pstates_table.h b/src/import/chips/p9/procedures/hwp/lib/p9_pstates_table.h index ede135996..956d6a2cf 100644 --- a/src/import/chips/p9/procedures/hwp/lib/p9_pstates_table.h +++ b/src/import/chips/p9/procedures/hwp/lib/p9_pstates_table.h @@ -122,15 +122,6 @@ typedef struct /// Note: if all bias attributes are 0, this content will be the same /// as the raw_pstates content. PstateTable biased_pstates[MAX_PSTATE_TABLE_ENTRIES]; - - ///VPD Operating points generated after applying biases - VpdOperatingPoint operating_points_biased[VPD_PV_POINTS]; - - ///VPD Operating points generated after applying system parameters - VpdOperatingPoint operating_points_sysp[VPD_PV_POINTS]; - - ///VPD Operating points generated after applying system parameters and biases - VpdOperatingPoint operating_points_biased_sysp[VPD_PV_POINTS]; } GeneratedPstateInfo; |
