diff options
| author | Sudheendra K Srivathsa <sudheendraks@in.ibm.com> | 2016-11-08 13:52:19 -0500 |
|---|---|---|
| committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2016-11-15 23:40:02 -0500 |
| commit | a633d3b4885dbec7f8633fb990b1e8bc38a9bea9 (patch) | |
| tree | a26584c5679c41abd311d37ce57f8ed673204e7f /src/import/chips/p9/procedures/hwp/pm | |
| parent | 231dd65379bd560be939266fccd43efa053be264 (diff) | |
| download | talos-hostboot-a633d3b4885dbec7f8633fb990b1e8bc38a9bea9.tar.gz talos-hostboot-a633d3b4885dbec7f8633fb990b1e8bc38a9bea9.zip | |
p9_setup_evid and p9_avsbus_lib lab updates
Change-Id: If5f6baf417ff41f07332423fb36587a70b2cde75
Original-Change-Id: I738ac23483e01e323a33f770cdc9da0179fe3507
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/32382
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Gregory S. Still <stillgs@us.ibm.com>
Reviewed-by: Brian T. Vanderpool <vanderp@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/32645
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/import/chips/p9/procedures/hwp/pm')
| -rw-r--r-- | src/import/chips/p9/procedures/hwp/pm/p9_avsbus_voltage.C | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/src/import/chips/p9/procedures/hwp/pm/p9_avsbus_voltage.C b/src/import/chips/p9/procedures/hwp/pm/p9_avsbus_voltage.C index 58752f7c6..3f9247236 100644 --- a/src/import/chips/p9/procedures/hwp/pm/p9_avsbus_voltage.C +++ b/src/import/chips/p9/procedures/hwp/pm/p9_avsbus_voltage.C @@ -134,11 +134,17 @@ p9_avsbus_voltage_read( const fapi2::Target<fapi2::TARGET_TYPE_PROC_CHIP>& i_tar attrs.RailBusNum, BRIDGE_NUMBER), "Initializing avsBus Num %d, bridge %d", attrs.RailBusNum, BRIDGE_NUMBER); + + FAPI_INF("Sending an Idle frame before Voltage reads"); + FAPI_TRY(avsIdleFrame(i_target, attrs.RailBusNum, BRIDGE_NUMBER)); + FAPI_INF("Reading the specified voltage rail value"); FAPI_TRY(avsVoltageRead(i_target, attrs.RailBusNum, BRIDGE_NUMBER, - attrs.RailSelect, &voltage_read_data.o_voltage), + attrs.RailSelect, voltage_read_data.o_voltage), "AVS Voltage read transaction failed"); + o_voltage_data.o_voltage = voltage_read_data.o_voltage; + fapi_try_exit: return fapi2::current_err; } // Procedure @@ -162,6 +168,9 @@ p9_avsbus_voltage_write( const fapi2::Target<fapi2::TARGET_TYPE_PROC_CHIP>& i_ta attrs.RailBusNum, BRIDGE_NUMBER), "Initializing avsBus Num %d, bridge %d", attrs.RailBusNum, BRIDGE_NUMBER); + FAPI_INF("Sending an Idle frame before Voltage writes"); + FAPI_TRY(avsIdleFrame(i_target, attrs.RailBusNum, BRIDGE_NUMBER)); + // Set the required voltage FAPI_INF("Setting the specified voltage rail value"); FAPI_TRY(avsVoltageWrite(i_target, attrs.RailBusNum, BRIDGE_NUMBER, |
