summaryrefslogtreecommitdiffstats
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* Add plug-rule callout to memory vpd lookup failureDan Crowell2019-10-241-5/+12
| | | | | | | | | | | | | | | | | | | | | | If there is a mismatch in the dimm ranks behind a memory port that will result in a VPD lookup error. It would be better if we got a list of the dimms but that isn't how the code is currently designed. In lieu of a perfect fix, we will add a procedure callout that points a memory plugging issue. Change-Id: I036f002c2145fd1debd957db8f46bc2e23d298f6 CQ: SW477820 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85737 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: Louis Stermole <stermole@us.ibm.com> Reviewed-by: Marc Gollub <gollub@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85778 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Revert "Remove ibm prefix from reserved memory spaces"Daniel M Crowell2019-10-231-12/+12
| | | | | | | | | | | | | | | | Turns out OPAL wasn't quite ready... This reverts commit e9c4e6ad68f9ac55345180bd77a7dd97b364c384. Change-Id: Ie826083788af599cda5eae6506ac0c95a591a7f4 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85695 Reviewed-by: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-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>
* Make explorer endian attributes platInitMark Pizzutillo2019-10-221-2/+2
| | | | | | | | | | | | | | | | | | Change-Id: I2c8f3ca7d73db586d39ec7fd1490e70baf9a18da Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85284 Reviewed-by: Louis Stermole <stermole@us.ibm.com> Reviewed-by: Steven B Janssen <janssens@us.ibm.com> Dev-Ready: Steven B Janssen <janssens@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85303 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Set endianess attributes in hb_customized_attrs instead of in codeChristian Geddes2019-10-222-13/+20
| | | | | | | | | | | | When we first detected we needed to have these attributes differerent than what the EKB had we set them in the call_mss_eff_config istep code. Really these should be set in the hb_customized_attrs xml. Change-Id: I7166a25c53330bf90e423b0e4de718b95344bfb7 RTC: 202491 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85443 Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com> Tested-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Add attribute to explicitly force OCMB Firmware update behaviorDan Crowell2019-10-213-3/+77
| | | | | | | | | | | | | | | | | | Allow us to override the default behavior (update when versions don't match) via an attribute. The options will be: - Update if versions do not match - Never update - Always update - Do the version check but don't actually update the code Change-Id: I5fdaa3cd2adf4cfb96f7ad7568de5d2c083d276c Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85180 Reviewed-by: Glenn Miles <milesg@ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Increase default PRBS_TIME for FWMark Pizzutillo2019-10-181-1/+2
| | | | | | | | | | | | | | | | | | Cronus looked fine with 0x100, but FW appeared to need about 0x400 for consistent results Change-Id: I9fc0eec9f4936bdd7852610af9af0feb0f6911ce Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85333 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Reviewed-by: Devon A Baughen <devon.baughen1@ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: Louis Stermole <stermole@us.ibm.com> Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85365 Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Remove unused procedure p9a_mss_eff_config_thermalLouis Stermole2019-10-185-572/+293
| | | | | | | | | | | | | | | | Change-Id: Ib848efb56534b484abfd7f5ebedde0895c941519 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82289 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Mark Pizzutillo <mark.pizzutillo@ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: ANDRE A MARIN <aamarin@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82343 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Reorganize omi training steps accounting for new omi training hwpsChristian Geddes2019-10-174-99/+201
| | | | | | | | | | | | | | | | | | | | | | | | A few new steps have been added that were found to be required for explorer power on. The correct steps of the OMI training need to be dmi_pre_trainadv: - exp_omi_setup - p9a_omi_setup dmi_io_run_training: - exp_omi_train (skip on gemini) - p9a_omi_train dmi_post_trainadv: - exp_omi_train_check (skip on gemini) - p9a_omi_train_check Change-Id: I90a3c2dfb1bec8983d38e0d4810753154651e452 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85289 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Add code and workarounds for *_omi_setup and *_omi_train for SwiftMark Pizzutillo2019-10-1721-154/+771
| | | | | | | | | | | | | | | | Change-Id: I139357a553e621b25b46bee6303357c712b67be2 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83848 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Dev-Ready: Steven B Janssen <janssens@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: Louis Stermole <stermole@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83905 Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com> Tested-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Automatically set VPD cache flags based on EEPROM cache settingDan Crowell2019-10-172-26/+17
| | | | | | | | | | | | | | | | Added more logic to the vpd/HBconfig file to force the correct (use hardware) values for the VPD code if the EECACHE flag (SUPPORT_EEPROM_CACHING) is set. This allows the system config files to not include the unused VPD flags anymore. Change-Id: I87f7c5f3e51e3121c081b3007164dbf21cbafba8 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84757 Tested-by: Jenkins Server <pfd-jenkins+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: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: Corey V Swenson <cswenson@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Remove ibm prefix from reserved memory spacesDan Crowell2019-10-171-12/+12
| | | | | | | | | | | | | | | | | Removing the unnecessary "ibm," prefix from the HBRT reserved memory spaces. Change-Id: Ib044e9dd719061e32b7332c50b08d6f7de320cf3 RTC: 198923 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83852 Reviewed-by: Corey V Swenson <cswenson@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Roland Veloz <rveloz@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Increase timeout for OCMB inband communication doorbellChristian Geddes2019-10-161-5/+5
| | | | | | | | | | | | | | | | | | | | | During explorer bringup on firmware we noticed that we are not waiting nearly long enough for draminit training for complete. We have found that we needed to poll 18+ times longer with firmware than we did for cronus. The new max timeout is 4 seconds , we expect on average to poll for about .36 seconds. Change-Id: I5bf29b5989761a8afac8a65bde3bec9609b6532b Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85296 Reviewed-by: Mark Pizzutillo <mark.pizzutillo@ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Louis Stermole <stermole@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85318 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Clean up for p10 porting dependency errorsAndre Marin2019-10-1611-121/+147
| | | | | | | | | | | | | | | | | | | CMVC-Coreq: 1094420 Change-Id: I02b8dfab08121d4ac9ffbb7dedd37f7083bf527f Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81543 Dev-Ready: STEPHEN GLANCY <sglancy@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Reviewed-by: Louis Stermole <stermole@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82367 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* for defect number HW476620Sheng-Hao Huang2019-10-162-0/+19
| | | | | | | | | | | | | | | | | Change-Id: I5188183869da97f4d4f0bb9105020d562948c031 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83724 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com> Reviewed-by: Adam S Hale <adam.samuel.hale@ibm.com> Reviewed-by: SHELTON LEUNG <sleung@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83797 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Remove unmask of MCBISTFIRQ[12]Louis Stermole2019-10-163-30/+0
| | | | | | | | | | | | | | | | | | | | | | | | This FIR bit was only used for a DD1.x workaround, and the PRD code to handle it had already been removed from FW. We thought we would need this FIR to implement the symbol mark performance workaround in commit 70081 (CQ SW453101) but we didn't end up needing it. As a result of unmasking this, there have been erroneous error logs appearing in manufacturing tests. Change-Id: I9ce1d563280b889609d2b0a6301ed82bd8540ba0 CQ: SW475386 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83429 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Dev-Ready: Louis Stermole <stermole@us.ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Reviewed-by: Mark Pizzutillo <mark.pizzutillo@ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83435 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Dynamically generate ocmb cmd/rsp seq idChen Du2019-10-164-1/+76
| | | | | | | | | | | | | | | | | | | The current procedure code is using the sequence id portion of the FW message to indicate the type of the command rather than the command number. This makes error handling very difficult because we can't detect which ocmb we are accessing when we detect a failure. Create a function backed ekb attribute to hold the sequence id so that every time we retrieve this attribute, we increment the valuwe on the next get. Change-Id: I93e0e7062d01ddf7ba6a69f89411a5b6bcf01471 RTC: 210371 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/78767 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-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>
* Enable rolling counter and verification of req id for ExplorerDan Crowell2019-10-167-40/+97
| | | | | | | | | | | | | | | | | | | | | | | | | | Rather than using a constant value based on which command is being sent to the OCMB, we will instead define an attribute that is set by the platform in some unique way to differentiate this particular command/response from all others going on. This will allow the code to catch bugs where commands erroneously are sent to the wrong target for some reason. Change-Id: I3162a8ac040d8487e9311f770a0466ee8500b415 RTC: 210371 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83851 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Dev-Ready: STEPHEN GLANCY <sglancy@us.ibm.com> Reviewed-by: ANDRE A MARIN <aamarin@us.ibm.com> Reviewed-by: Matt K Light <mklight@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84633 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Fix invalid use of vector iteratorGlenn Miles2019-10-161-3/+6
| | | | | | | | | | | | | | | | | | | Once an element of a vector is erased using an iterator, that iterator should be considered invalid and not used. The accepted practice is to update the iterator with the return value of the erase method and not to increment the iterator. CQ: SW465848 Change-Id: If6ec7887d8c2b6c6ad44a9608c474b9c118bd117 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85220 Reviewed-by: Zachary Clark <zach@ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com>
* Update OMI DS template supportBen Gass2019-10-152-16/+17
| | | | | | | | | | | | | | | | | | | | - Allow DS template 1 to be turned off - Leave DS template 0 only bit on if no other templates enabled - Remove overly restrictive DS template checks Change-Id: I7d5d02b949043d1cbae340c392ce587cd5bac26f Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84969 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: Adam S Hale <adam.samuel.hale@ibm.com> Dev-Ready: Benjamin Gass <bgass@us.ibm.com> Reviewed-by: Thi N Tran <thi@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84973 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com>
* Do nothing with error logs after shutdown event is received.Glenn Miles2019-10-152-0/+33
| | | | | | | | | | | | | | | | | | Disable processing of error logs by the error log server after receiving the shutdown event. This is to prevent the possibility of bad things from happening once other services are shutdown. CQ: SW465848 Change-Id: I59c4951182d43a78f5704b4b4c3379f10dd03182 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85278 Tested-by: Jenkins Server <pfd-jenkins+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> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* add function to turn on adaptation in bootconfigDevon Baughen2019-10-143-0/+61
| | | | | | | | | | | | | | | | | Change-Id: Ia5602d413250f1fbbb23d50b113f245526e3bf54 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84744 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com> Dev-Ready: Devon A Baughen <devon.baughen1@ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Reviewed-by: Louis Stermole <stermole@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84891 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com>
* PRD: OCMB handling for checkstop analysis pathCaleb Palmer2019-10-143-35/+54
| | | | | | | | | | | | | | Change-Id: I708d59f8bbf732edac4ee77e9885bf287cdf03d8 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85020 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Benjamen G Tyner <ben.tyner@ibm.com> Reviewed-by: Brian J Stegmiller <bjs@us.ibm.com> Reviewed-by: Paul Greenwood <paul.greenwood@ibm.com> Reviewed-by: Zane C Shelley <zshelle@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85204 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
* Add OMI bus support to calloutsDan Crowell2019-10-103-0/+9
| | | | | | | | | | | | | | | | Fill in a few spots where we need to handle OMI busses for proper error processing and display. Change-Id: Ibf236a7f6cb894d1516dbbe49b57e5be7dcceca2 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84952 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Corey V Swenson <cswenson@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* HCBI-239 Enable static analysisDan Larson2019-10-092-21/+27
| | | | | | | | | | | | | Change-Id: Id07e25aeb7337060baab8eaa2bbf4e3e8df238cd Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84660 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Luis P Fernandez <luis.fernandez@ibm.com> Reviewed-by: Camvan T Nguyen <ctnguyen@us.ibm.com> Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Prevent pre-3.0 level NVDIMMs from being in systemsMatt Derksen2019-10-091-3/+7
| | | | | | | | | | | | | | | Do not want these non-updateable NVDIMMs to be used. Change-Id: Ia045257d101fc249d65b2d0af6194f8f3aa1e9dc CQ: SW477628 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84729 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Corey V Swenson <cswenson@us.ibm.com> Reviewed-by: TSUNG K YEUNG <tyeung@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* NVDIMM Erase error handle not working in IPL Arm handlingMatt Derksen2019-10-092-12/+3
| | | | | | | | | | | | | | | | | | | | NVDIMM erase error handling is failing because nvdimmDisarm() checks ATTR_NVDIMM_ARMED before disarming the nvdimm. Disarming an nvdimm that is already disarmed should not be harmful. Also erase error should correctly set the error status as 0x01 (memory unable to preserve future content) Change-Id: I8a710343478691c87f1570fd30074c2543d3373c CQ: SW477541 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84662 Reviewed-by: TSUNG K YEUNG <tyeung@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Corey V Swenson <cswenson@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> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com>
* Add RC A2, B3, and C3 for 16Gb RDIMMsAndre Marin2019-10-082-3/+31
| | | | | | | | | | | | | | | | | Change-Id: I0fc7abc092ca3935519bc768f5ffbef8770dbf83 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84137 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Dev-Ready: STEPHEN GLANCY <sglancy@us.ibm.com> Reviewed-by: Louis Stermole <stermole@us.ibm.com> Reviewed-by: Michael D Pardeik <pardeik@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84149 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Skip running EXP_FW_TEMP_SENSOR_CONFIG_INTERVAL_READ in exp_mss_thermal_initLouis Stermole2019-10-081-1/+3
| | | | | | | | | | | | | | | | | This command is not yet working properly in Explorer FW Change-Id: I1ee9322bec2d451cd12769b636cec2c0efd98021 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84427 Reviewed-by: Devon A Baughen <devon.baughen1@ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84455 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Fix OCC sandbox base address overlapped with SMF in FLIP mode.Thi Tran2019-10-081-2/+4
| | | | | | | | | | | | | | | | | This is an AXON only issue. Change-Id: I3ba9ccdcce69be80e93141c93fd823288b51ad0c Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84618 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: Adam S Hale <adam.samuel.hale@ibm.com> Reviewed-by: Jenny Huynh <jhuynh@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84626 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Rename p9a_mss_memdiag to exp_mss_memdiagLouis Stermole2019-10-081-15/+10
| | | | | | | | | | | | | | | | | | | | The memdiag process operates entirely on Explorer, so the procedure name and location should reflect that for porting purposes. Change-Id: I3f43eae5c0476ef4bc5140f43d8625f05fff558d Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84508 Reviewed-by: ANDRE A MARIN <aamarin@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Dev-Ready: Steven B Janssen <janssens@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Reviewed-by: Devon A Baughen <devon.baughen1@ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84546 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Add MDS attribute & draminit supportMark Pizzutillo2019-10-083-6/+17
| | | | | | | | | | | | | | | Change-Id: I0496a52f5c7dd72779ed097c26fc8b305a9e00b0 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84431 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Reviewed-by: Louis Stermole <stermole@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84454 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Disable & clear PMIC status codes in beginning of pmic_enableMark Pizzutillo2019-10-085-0/+76
| | | | | | | | | | | | | | Change-Id: I64b025443395c842f68dbaa1fb58909a4695f31a Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84680 Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Reviewed-by: Louis Stermole <stermole@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84792 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Check PMIC error and enable statuses after pmic_enableMark Pizzutillo2019-10-087-53/+519
| | | | | | | | | | | | | | Change-Id: I26bc700a3b368c9bd110ee62b1037a6c6d14491e Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83886 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: Louis Stermole <stermole@us.ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83904 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* HW508066 - interleave granularity not setup for group of 3/6 in AxoneAdam Hale2019-10-081-32/+6
| | | | | | | | | | | | | | | Change-Id: I14e82293d1526f35652c08412fc6be32f8a0e37a Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84769 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: SHELTON LEUNG <sleung@us.ibm.com> Reviewed-by: Mark Pizzutillo <mark.pizzutillo@ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84795 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Add support for DMI-MEMBUF bus failures to fapiDan Crowell2019-10-071-0/+7
| | | | | | | | | | | | | | | | | | | I noticed this error message the other day - processEIBusCallouts - Bus between target types not known (0x00000027:0x00000004) The effect of this is that we don't end up with the bus error procedure getting added to the error log like it should be. Change-Id: Iecc5f6df2f9c6812d70c5ea66ad0f46dc9bb2694 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83671 Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: Corey V Swenson <cswenson@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Update dimmspd used by explorers in axone simicsChristian Geddes2019-10-072-2/+2
| | | | | | | | | | | | | | We want the explorer's SPD to contain valid i2c addresses for the PMIC for simics. Currently simics has these as 0x90 and 0xC0. Normally we ould just update simics directly and then update the simics relase but there has been issues in the last few weeks getting onto a newer simics release and we need these changes to test functionality that is being added to aid with hardware bringup. Change-Id: I1e29205cef59da986092bc8b609ebe27a89c5611 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84682 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Create error for unsuspected target in configureHbrtHypIdsMatt Derksen2019-10-072-8/+49
| | | | | | | | | | | | | | | | | Specifically check for all expected types. Create an error if an unsuspected type is encountered. Also use the const for error logs which just add software callout. Change-Id: Ie1b35945ee7c580dd70d3c97c68184a22fb6ced5 RTC:210321 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82225 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Corey V Swenson <cswenson@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* PRD: Axone Two Phase Scrub (TPS) supportCaleb Palmer2019-10-042-1054/+196
| | | | | | | | | | | | | | Change-Id: If2540907b7ceb51612b0a68142d54836b89f1e13 RTC: 208263 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84537 Reviewed-by: Benjamen G Tyner <ben.tyner@ibm.com> Reviewed-by: Brian J Stegmiller <bjs@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Zane C Shelley <zshelle@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84783 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
* PRD: Axone VCM UpdatesCaleb Palmer2019-10-043-174/+84
| | | | | | | | | | | | | | Change-Id: I9c54c9d8ae01186785bf4a773f6a9ea5d0d311fb RTC: 208262 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84233 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Benjamen G Tyner <ben.tyner@ibm.com> Reviewed-by: Brian J Stegmiller <bjs@us.ibm.com> Reviewed-by: Zane C Shelley <zshelle@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84782 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
* PRD: Axone UE/CE threshold for background scrubCaleb Palmer2019-10-044-20/+100
| | | | | | | | | | | | | Change-Id: I2872ee94a385d84b10a6e0aaf7f3c0a95c496aa0 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84551 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Benjamen G Tyner <ben.tyner@ibm.com> Reviewed-by: Brian J Stegmiller <bjs@us.ibm.com> Reviewed-by: Zane C Shelley <zshelle@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84781 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
* PRD: Add additional FFDC to runtime NVDIMM analysisCaleb Palmer2019-10-043-5/+74
| | | | | | | | | | | | | | Change-Id: I7472a883a27c30315c01f2722f98fe40d42c1070 CQ: SW476683 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84181 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Benjamen G Tyner <ben.tyner@ibm.com> Reviewed-by: Brian J Stegmiller <bjs@us.ibm.com> Reviewed-by: Zane C Shelley <zshelle@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84182 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
* Fix to SW473350: p9_xip_customize Vpd core fill bugClaus Michael Olsen2019-10-041-4/+8
| | | | | | | | | | | | | | | | | | This commit fixes the returning of RC_XIPC_IMAGE_WOULD_OVERFLOW even though the minimum requires cores has been filled. Change-Id: Iac1031c56b60ce58ec1b9c1463566af04be5c3be Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84619 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84630 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
* Add nvdimm arm retry logic in case of glitchesTsung Yeung2019-10-042-88/+194
| | | | | | | | | | | | | | | | | Random BPM glitches could cause temporary persistency lost and failing the arm command. Allow arm retry if glitch only persisted for a second or less. Change-Id: I28e65b05e482129f6fea34580064a825923aaaf3 CQ:SW477211 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84668 Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Dean Sanner <dsanner@us.ibm.com>
* PRD: Fix returning port position in dynamic mem deallocCaleb Palmer2019-10-031-4/+4
| | | | | | | | | | | | | Change-Id: Ib128e038b84c6a9aafd692c43611cadf25b582b8 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84602 Reviewed-by: Brian J Stegmiller <bjs@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Benjamen G Tyner <ben.tyner@ibm.com> Reviewed-by: Zane C Shelley <zshelle@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84657 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
* Add call to fapi wrapper of exp_getidec into hwasPlat codeChristian Geddes2019-10-032-85/+73
| | | | | | | | | | | | | | | | | | | | | | The file hwasPlat.C contains the code we use to lookup and compare the IDEC information of a given OCMB when read from the card's SPD versus what is read from the card's defined registers. Recently the code that looks up the IDEC from the card's registers was made into a HWP so it could be leveraged by multiple platforms. A new wrapper has been created on the hostboot side in src/usr/fapiwrap/ that provides an interface for the hostboot platform to call in order to leverage the new EKB hwp. Change-Id: I9155767b199c9b197c010521f33cc0d418901628 RTC: 214627 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83109 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Ilya Smirnov <ismirno@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Verify manufacture date (MB) from PVPD before usinglixg2019-10-031-35/+57
| | | | | | | | | | | | | | | | | | If the contents of the MB keyword is invalid, we could end up doing some illegal math and causing a crash. This change will add some simple verification of the data before we use it. Resolves #183 Change-Id: I2f3ead87be24e909a671f54f6005f3fffdbafd52 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84441 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* errl: Free memory allocated for parsersArtem Senichev2019-10-031-1/+3
| | | | | | | | | | | | | | | | | Errl's factory creates new instance of User Defined Data parser on every call, but doesn't free it after usage. Resolves #184 Signed-off-by: Artem Senichev <a.senichev@yadro.com> Change-Id: I8e8bd54e7aaa4ddb813cec3485404b6ba7a29ffa Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84518 Reviewed-by: Corey V Swenson <cswenson@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* errl: Fix data reading from unaligned pointersArtem Senichev2019-10-036-26/+117
| | | | | | | | | | | | | | | | | | | | | | Some architectures (like an ARM used in OpenBMC) do not support unaligned memory access. On these systems, reading an integral value from an unaligned pointer leads to undefined behavior. This patch doesn't change existing functionality, but forces the compiler to generate extra instructions to satisfy architectural alignment requirements. Resolves #185 Signed-off-by: Artem Senichev <a.senichev@yadro.com> Change-Id: I6e75044b93c26cca76336d17bb3886fab403253a Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84520 Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-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>
* Updates to Explorer FW upload toolsLouis Stermole2019-10-033-148/+234
| | | | | | | | | | | | | | | | | | Change-Id: I4b3034af0dc076fa392efbef1f4f287967bd91f1 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81619 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Reviewed-by: Mark Pizzutillo <mark.pizzutillo@ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com> Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: Glenn Miles <milesg@ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84077 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com>
* Compile Hostboot with -OsDean Sanner2019-10-036-10/+27
| | | | | | | | | | | | | | | | | | | Hostboot currently spends a significant percentage of time loading data from PNOR across a relatively slow LPC bus. This commit enables compilation of Hostboot with -Os, optimized for space to reduce the amount of data required to be transferred across the LPC bus Rough measurements show a ~30% size reduction of the hostboot image and a ~10% reduction (improvement) in boot time Change-Id: Ib73863019595a24cc7d10e924bc5c92cd64af1ed Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84436 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
OpenPOWER on IntegriCloud