diff options
author | Chris Cain <cjcain@us.ibm.com> | 2017-05-16 08:31:17 -0500 |
---|---|---|
committer | Christopher J. Cain <cjcain@us.ibm.com> | 2017-05-17 12:56:56 -0400 |
commit | b1525a6c98780e442f772dc068872bafb94c0cc1 (patch) | |
tree | f1155a59342080ba57a0911a29c6389b22490966 /src/usr/htmgt | |
parent | 9fe37b817a8862347166fc89fc4d2f7248174783 (diff) | |
download | talos-hostboot-b1525a6c98780e442f772dc068872bafb94c0cc1.tar.gz talos-hostboot-b1525a6c98780e442f772dc068872bafb94c0cc1.zip |
Update attribute names and prevent sending empty config data
CQ: SW388150
Change-Id: I01ebe5981f77de67bef5efef110b59c37a776d15
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/40557
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Reviewed-by: Martha Broyles <mbroyles@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: Sheldon R. Bailey <baileysh@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/usr/htmgt')
-rw-r--r-- | src/usr/htmgt/htmgt_cfgdata.C | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/src/usr/htmgt/htmgt_cfgdata.C b/src/usr/htmgt/htmgt_cfgdata.C index 96c41cf58..a105259d2 100644 --- a/src/usr/htmgt/htmgt_cfgdata.C +++ b/src/usr/htmgt/htmgt_cfgdata.C @@ -54,7 +54,6 @@ namespace HTMGT } uint8_t cmdData[OCC_MAX_DATA_LENGTH] = {0}; - uint64_t cmdDataLen = OCC_MAX_DATA_LENGTH; const occCfgDataTable_t* start = &occCfgDataTable[0]; const occCfgDataTable_t* end = @@ -125,7 +124,7 @@ namespace HTMGT if (sendData) { - cmdDataLen = OCC_MAX_DATA_LENGTH; + uint64_t cmdDataLen = 0; switch(format) { case OCC_CFGDATA_FREQ_POINT: @@ -182,7 +181,6 @@ namespace HTMGT TMGT_ERR("sendOccConfigData: Unsupported" " format type 0x%02X", format); - cmdDataLen = 0; } if (cmdDataLen > 0) @@ -931,7 +929,11 @@ void getFrequencyPointMessageData(uint8_t* o_data, turbo = sys->getAttr<ATTR_FREQ_CORE_MAX>(); //Ultra Turbo Frequency in MHz - const uint16_t wofSupported = sys->getAttr<ATTR_SYSTEM_WOF_ENABLED>(); + ATTR_SYSTEM_WOF_ENABLED_type wofSupported; + if (!sys->tryGetAttr<ATTR_SYSTEM_WOF_ENABLED>(wofSupported)) + { + wofSupported = 0; + } if (0 != wofSupported) { ultra = sys->getAttr<ATTR_ULTRA_TURBO_FREQ_MHZ>(); @@ -990,8 +992,8 @@ void getApssMessageData(uint8_t* o_data, ATTR_ADC_CHANNEL_FUNC_IDS_type function; sys->tryGetAttr<ATTR_ADC_CHANNEL_FUNC_IDS>(function); - ATTR_ADC_CHANNEL_GROUNDS_type ground; - sys->tryGetAttr<ATTR_ADC_CHANNEL_GROUNDS>(ground); + ATTR_ADC_CHANNEL_GNDS_type ground; + sys->tryGetAttr<ATTR_ADC_CHANNEL_GNDS>(ground); ATTR_ADC_CHANNEL_GAINS_type gain; sys->tryGetAttr<ATTR_ADC_CHANNEL_GAINS>(gain); |