summaryrefslogtreecommitdiffstats
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* Update ATTR_MEM_VPD_DQ_MAPs default to a 1-to-1 mappingCaleb Palmer2019-11-131-2/+8
| | | | | | | | | | | | | | | | | | CMVC-Coreq: 1096221 Change-Id: I4c771180f3bcc8e64959976be07ce8cf222ae688 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86144 Reviewed-by: Louis Stermole <stermole@us.ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@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> Tested-by: HWSV CI <hwsv-ci+hostboot@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/86264 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>
* Fix odt RD/WR fieldsMark Pizzutillo2019-11-136-34/+170
| | | | | | | | | | | | | | | | | Change-Id: I4aaa23af53a72e4f90218daedbed80d8721ff337 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86280 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> Dev-Ready: STEPHEN GLANCY <sglancy@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/86542 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>
* Uncomment the remainder of hwp_wrappers.HCaleb Palmer2019-11-121-8/+6
| | | | | | | | | | | | | | | | | Change-Id: Idf567c5cddbf4b5f4e4683ef411fd9d060b88c2f Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86473 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: Zane C Shelley <zshelle@us.ibm.com> Reviewed-by: Brian J Stegmiller <bjs@us.ibm.com> Reviewed-by: Paul Greenwood <paul.greenwood@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/86485 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>
* Adds NVDIMM RCW drive strength workaroundStephen Glancy2019-11-123-0/+135
| | | | | | | | | | | | | | | | | | | | | | | | An NVDIMM supplier updated the drive strength values for for the RCW's command/address/control signals. The new drive strength causes the parts to fail writes/reads. This commit restores the old drive strength values. Change-Id: If4d44f08cff2522de463b9a2ac57853950aa3356 CQ:SW478491 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85229 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Louis Stermole <stermole@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Dev-Ready: STEPHEN GLANCY <sglancy@us.ibm.com> Reviewed-by: JEREMY R NEATON <jrneaton@us.ibm.com> Reviewed-by: TSUNG K YEUNG <tyeung@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85426 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>
* Reverse mcbist address buffer fieldsMark Pizzutillo2019-11-122-1/+45
| | | | | | | | | | | | | | | Change-Id: Ib1b68cd1a21c9190db170d95e6d528bebfbc2539 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86044 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: Louis Stermole <stermole@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86054 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>
* MDIA: Uncomment HWP callsCaleb Palmer2019-11-123-27/+46
| | | | | | | | | | | | | | Change-Id: I23f0abb7b6d26c2fa6d197ceaa0cc276ec461594 RTC: 201293 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86476 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: Brian J Stegmiller <bjs@us.ibm.com> Reviewed-by: Benjamen G Tyner <ben.tyner@ibm.com> Reviewed-by: Paul Greenwood <paul.greenwood@ibm.com> Reviewed-by: Zane C Shelley <zshelle@us.ibm.com>
* Improper element erase during loop causing post-restore being skipped on nvdimmTsung Yeung2019-11-111-18/+21
| | | | | | | | | | | | | Change-Id: Iec89586a9ac019960a11b2c4924046c17cec3b06 CQ:SW474826 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86265 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> Reviewed-by: Corey V Swenson <cswenson@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>
* Add 100ms wait to disable write protection for BPMMatthew Raybuck2019-11-111-0/+2
| | | | | | | | | | | | | | | | | | There was an issue where without waiting after unlocking the BPM registers we'd fail out of the BPM update. This commit adds a 100ms delay after writing the unlock sequence before reading to verify that the registers were unlocked. Change-Id: Ic8cfeb8dba181d4071bb2d4dccaf367d88840cc2 CQ:SW480382 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86707 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: TSUNG K YEUNG <tyeung@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Create submodule in hostboot repo for mctpJet Li2019-11-111-0/+0
| | | | | | | | | | | | | | | | | This commit does the following... -Adds libmctp repo as submodule of hostboot repo -Modify .gitmodule to make submodule updates RTC: 215965 Change-Id: I4bd498b15b86f0bc0e9fdd9c495c390d56308f76 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85889 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> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Increase size of default printk buffer in error logsDan Crowell2019-11-111-2/+4
| | | | | | | | | | | | | | | | | | | Recent changes to add more data to our exception path have made the printk output we get in our error logs less useful than it should be. It currently takes at minimum 1076 bytes for a single exception. That number would increase if the backtrace got very large. This change resizes the buffer to allow for 17 stack frames within a single exception (1280 bytes). Change-Id: I029283ee39877c3ace812fd85c0fbc26cdecf230 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86596 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: Zachary Clark <zach@ibm.com> Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com>
* Move HBRT reserved memory off of NVDIMMsDan Crowell2019-11-083-1/+194
| | | | | | | | | | | | | | | | | | On a NVDIMM system we need to make sure that we don't use the NV memory for the HOMER (or other reserved memory). Depending on the specific memory layout the NV memory could be placed at the top of memory where we would normally land. Change-Id: Ie4407c63a4944cc5be13d2573d06fbed09d6a11f CQ: SW470564 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86480 Reviewed-by: Ilya Smirnov <ismirno@us.ibm.com> Reviewed-by: Corey V Swenson <cswenson@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> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Always position HOMER in HV space for istep 16Dean Sanner2019-11-082-74/+93
| | | | | | | | | | | | | | | | | | | | | | | | P9N/P9C always inits to DD2.x mode for hostboot which doesn't support SMF. This means that the HCODE non secure transition to HCODE secure won't work. Because of that, we now force istep 16 to always place HOMER in non SMF memory (in UV mode if SMF active) and transition into UV there. Then in istep 21 the HOMER is rebuilt correctly for SMF. Note that all the changes are in istep 15 as istep 21 uses the ATTR setup there. Change-Id: I566903dc373667a410ddcc0a1d2cc91bcd94b08d Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81202 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: Dean Sanner <dsanner@us.ibm.com> Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Fix timestamp parser for OpenBMC ESELsDan Crowell2019-11-081-10/+34
| | | | | | | | | | | | | | | | | | | | Evidently the OpenBMC code changed their timestamp format at some point so our parser isn't grabbing it anymore. It will now handle data in a format like this: "raweSEL": "00 00 df 00 00 00 00 20 00 04 ... "timestamp": "2019-10-31 10:20:50" Change-Id: Ieeae63505ce0497a3b3fcf1fd0a92455c8319e29 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86538 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: Corey V Swenson <cswenson@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Modify INTRP Initialization Order to route LSI interrupts over PSHIHBBill Hoffa2019-11-081-7/+7
| | | | | | | | | | | | | | | | | - Currently the code sets up routing over PSIHIB and then a second function call overwrites that setting which forces things back on the local wire Change-Id: I7d3b44d48c66f994e8a8758eb75568038492fdaf CQ: SW464174 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86669 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: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Fixes to PMIC presence detectionChristian Geddes2019-11-074-24/+81
| | | | | | | | | | | | | | | | | | | | | While performing presence detection on the PMIC targets we need to first read the parent OCMB's SPD to see what device address the PMIC is on. There was a bug where we were attempting to read the parent OCMB's spd without first checking if the OCMB is present itself. This commit adds a check to ensure we dont attempt i2c reads on devices that are not present. Also this commit adds a check to make sure we do not attempt presence detection on GEMINI ocmbs Change-Id: I999189b3b97210bb37b7ba1fdb2d86658d770e36 CQ: SW480414 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86564 Reviewed-by: Ilya Smirnov <ismirno@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>
* Avoid commiting errors inside fapi2_i2c device driverChristian Geddes2019-11-071-13/+34
| | | | | | | | | | | | | | | | | | | | | | | | There are instances where we expect NACK errors returned from i2c requests, for example when waiting for the ocmbs to declare themselves "ready". In these cases we do not want to flood the error log with expected I2C nack errors. To avoid this, instead of committing retriable errors inside of the fapi2_i2c device driver when the MAX RETRY ATTEMPTS is reached we will instead just add traces to the err that gets returned to the caller that explain previous retryable errs occured. In the case where we hit retryable errors and eventually hit and non-retryable error the retryable error is committed in the driver and the non-retryable error is returned to the caller. Also this commit adds some timings to the error log so we can tell how long our attempts took. Change-Id: I9a5ee0bfe088444aa7fecd974f61514c40b320a6 CQ: SW480155 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86419 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@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>
* Fix fapi spd testcasesDan Crowell2019-11-062-170/+123
| | | | | | | | | | | | | | | Fixed a few places where the Axone path was getting skipped incorrectly, then fixed the resulting latent errors. Change-Id: I917b9a0b6f4ff1491ff384dc924e29f688548873 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85688 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> Reviewed-by: Zachary Clark <zach@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: William G Hoffa <wghoffa@us.ibm.com>
* Remove excessive NVDIMM tracing from normal pathMatt Derksen2019-11-061-20/+52
| | | | | | | | | | | | | | | | | | Tracing is causing a bottleneck when running NVDIMM updates. Should only trace the real failure and add what is needed for debug into the error log. Change-Id: Ifa75f85e1129efd9642a3fef0b842fc1f60cd182 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/75667 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Glenn Miles <milesg@ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85499 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 ipmifruinv calls for dimms in swift systemsChristian Geddes2019-11-061-0/+2
| | | | | | | | | | | | | | | | | | We had a bug in the VPD map for the the DDR4_DDIMMs where we were missing fields that were expected in the data collection for the fruinventory. This commit adds the field we were missing and also adds the PART_NUMBER field which the BMC is currently just leaving blank. Change-Id: I65dbfb0b79bef9d3a9a3bf2d6aa135c5df5ece56 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86420 Reviewed-by: William G Hoffa <wghoffa@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>
* Adjust deadman timeout request time to be 10.5 secondsChristian Geddes2019-11-061-2/+3
| | | | | | | | | | | | | | | | | | | | | We see intermittent deadman timeouts occuring in our testing. Upon further inspection it was determined that value we were passing to the SBE was being cast to a uint32_t and multiplied by up to a factor of 400000 , which was resulting in an overflow of the uint32_t. This was giving us a timeout lower than desired. This commit lowers the requested timeout to be 10.5 seconds which will not overflow the uint32_t. Change-Id: I747db46a5d4a99d273b5604971d3db51f077be50 CQ: SW477557 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86513 Reviewed-by: Glenn Miles <milesg@ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@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>
* Slot check after firmware updateMatt Derksen2019-11-055-11/+147
| | | | | | | | | | | | | | | | | | Verify every NVDIMM is running from slot 1 and is at the latest firmware version. Create a predictive error for those that are not running latest code level. Change-Id: I46766d267f4a1293739a49c989ef5ab7dde585b5 CQ:SW478561 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85656 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> Reviewed-by: TSUNG K YEUNG <tyeung@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>
* Add new Key Clear Request AttributeMike Baiocchi2019-11-052-0/+78
| | | | | | | | | | | | | | | | | | This commit adds a new Key Clear Request Attribute and Enumeration to be shared by the FSP and Hostboot code for future Key Clear support. Change-Id: I74bb9c12ed1d3fdcf8f55c77c76d4001b15e3b0f RTC:210301 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86409 Reviewed-by: Christopher J Engel <cjengel@us.ibm.com> Reviewed-by: Ilya Smirnov <ismirno@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> Reviewed-by: Nicholas E Bofferding <bofferdn@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>
* NVDIMM FW update region write retryMatt Derksen2019-11-052-16/+61
| | | | | | | | | | | | | | | | | | Retry writing a region of code if the checksums do not match. Retry up to 3 times per region. Second of two commits for this defect. CQ:SW469894 Change-Id: Id3cbd71a2930a2dc19532b3b5c87f692dde447cd Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/80360 Reviewed-by: TSUNG K YEUNG <tyeung@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: Corey V Swenson <cswenson@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* NVDIMM update - Clear FW Ops Status verificationMatt Derksen2019-11-053-7/+117
| | | | | | | | | | | | | | | | | | | SMART is recommending after 0x41=0x02 (Clear FW Ops Status) that the IBM utility read 0x71 to verify that the OPS_SUCCESS and OPS_ERROR bits have been cleared and that only Bit 2 (FIRMWARE_UPDATE_MODE) is set. First of two fixes for SW469894 Change-Id: Id9e9d7fa5abd5bcd3bae806b27332513f48956b2 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83946 Reviewed-by: TSUNG K YEUNG <tyeung@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: Corey V Swenson <cswenson@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Adding obj_handle values for attentions to explorer.scom.initfileBen Gass2019-11-042-0/+90
| | | | | | | | | | | | | | | | Change-Id: Ifb4928893d78ab399614c0ef5aa30a1074118903 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/66683 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Dev-Ready: Benjamin Gass <bgass@us.ibm.com> Reviewed-by: Louis Stermole <stermole@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: Benjamin Gass <bgass@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85236 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>
* Allow dynmic i2c device addresses and set up PMIC targets to do thisChristian Geddes2019-11-0110-17/+242
| | | | | | | | | | | | | | | | | | | | | | Depending on which vendor made a given OCMB the i2c device address of the PMIC targets on the OCMB will be different. To account for this we have added a new DYNAMIC_DEVICE_ADDRESS attribute. This attribute is filled out on the PMIC target by looking at the SPD on parent OCMB chip. This means that we must do presence detection on the OCMB prior to the the PMIC targets. While doing i2c operations if a given target has the DYNAMIC_DEVICE_ADDRESS we will use that over the devAddr in the any complex i2c attribute for that target. Change-Id: I22a185a65c064a1514751dd5828547c57af98df1 RTC: 209714 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85394 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> Reviewed-by: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Replaces NVDIMM flush sequence with CCSTsung Yeung2019-11-013-188/+181
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The current design relies on the power control logic to put the nvdimm into STR. There have been several defects opened on the nvdimm failing to save due to STR not entered but no indication of the function failing to execute. Therefore, the decision has been made to leverage CCS to issue STR and assert RESETn. This gives us full control of what goes onto the bus and not have to worry about STR getting exit due to unwanted mainline traffic. The same CCS sequence has already been exercised numerous times on AC powerloss path. Change-Id: Idd422beea72ee5902674562f5834c1ac9e79fe00 CQ:SW477735 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85831 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Dev-Ready: STEPHEN GLANCY <sglancy@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: 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/85879 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>
* Only call memdiags on Explorer ocmb chipsChristian Geddes2019-10-311-9/+15
| | | | | | | | | | | | | | | | | | We should only call memdiags on the Explorer ocmb chips because it will fail if we attempt to call it on the Gemini chips. Gemini chips will boot fine without this step being called. Change-Id: I50645db29b17d4dc93de9c1e5f7b419e5dea6f0d Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86159 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: Glenn Miles <milesg@ibm.com> Reviewed-by: Ilya Smirnov <ismirno@us.ibm.com> Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com> Reviewed-by: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* update include path to resolve compiler warning in cronusSteven Janssen2019-10-301-1/+1
| | | | | | | | | | | | | | | Change-Id: Ic79d5ee09622ed6310b579a088d7dfff8996854b Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85618 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> 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/85646 Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Add assert for RSP extended data buffer size in exp_draminitMark Pizzutillo2019-10-303-7/+119
| | | | | | | | | | | | | | | | | Change-Id: If924286f31c31bb66b5191d51afce43c8a1e50d2 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85774 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> Reviewed-by: Louis Stermole <stermole@us.ibm.com> Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com> Dev-Ready: STEPHEN GLANCY <sglancy@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85790 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 deconfigure of memory port if mss_freq can't find consensus freqLouis Stermole2019-10-303-7/+28
| | | | | | | | | | | | | | | | | | | | | | | Fixes condition where MSS_PORT_DOES_NOT_SUPPORT_MAJORITY_FREQ caused the IPL to halt. Fixed procedure code to deconfigure correct port's DIMMs, and updated error XML to call out all DIMMs in freq domain. Also updated error description to be more helpful. Change-Id: Ia08357c1de28f5f03d41b95d81a910d6c2e32451 CQ:SW473836 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84733 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: Mark Pizzutillo <mark.pizzutillo@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: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84776 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 FAPI_ASSERT for unsupported SPD rank in RTT fetchNico Fajardo2019-10-302-6/+39
| | | | | | | | | | | | | | | | | | | Change-Id: I10523ec048e04eebe35e55894949e54a796a2aa5 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85430 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Dev-Ready: STEPHEN GLANCY <sglancy@us.ibm.com> Reviewed-by: Mark Pizzutillo <mark.pizzutillo@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: 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/85448 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>
* Create FW interfaces to execute memory diagnosticsDan Crowell2019-10-304-85/+374
| | | | | | | | | | | | | | | | | | | Created 2 files, each encapsulating the functionality to drive memdiags for either Nimbus or Explorer. Change-Id: Ice58b4eb43c51b30ed5ba22c7b5d042eea5e4bfe Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85588 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Dev-Ready: STEPHEN GLANCY <sglancy@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Reviewed-by: ANDRE A MARIN <aamarin@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/85592 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>
* Do not crash if we have no functional nodesDan Crowell2019-10-301-43/+50
| | | | | | | | | | | | | | | | | | | We hit a case where i2c errors caused the node target to get marked as non-present. This led to an assert crashing Hostboot because it assumed we would always have a functional node0. The change here is to allow this case to flow through so that we end up with more recognizable error handling for the real failures during boot. Change-Id: Iaaaa4e531b677bda442da892cfbd6094e0897beb Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85169 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: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Corey V Swenson <cswenson@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* New Optional Argument to PnorUtils.pm’s loadPnorLayoutLuis Fernandez2019-10-294-21/+43
| | | | | | | | | | | | | | | | | | New Optional Argument to PnorUtils.pm’s loadPnorLayout to declare where to place the newly created layout file. If argument is not given, file will not be saved. Edited scripts and makefiles that use this function. Change-Id: I7b9ee0f7ad33ad88be58598ddf594765eabe29ea Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85916 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> Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Fix linker Object initCorey Swenson2019-10-291-2/+4
| | | | | | | | | | | | | | | | Builds on Ubuntu are failing with hbibl.bin size error. Traced back to linker struct Object constructor missing the init of sfpr section. Change-Id: I9ad7ab90f02d403dc0237831736cc615e73738e6 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86066 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Zachary Clark <zach@ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Support scoms to MEM_PORT targetsDan Crowell2019-10-282-10/+145
| | | | | | | | | | | | | | | | | | Some of the memory HWPs require MEM_PORT scom support for Explorer. There is no address translation required but we need to allow the scoms to occur. Added testcase paths to scom MEM_PORT targets. Change-Id: I00b29e8dda6a105e994a5dcd009fd4b8aebdbdef Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85690 Tested-by: Jenkins Server <pfd-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> 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>
* Update exp_draminit results display for new response structsMark Pizzutillo2019-10-2811-335/+1231
| | | | | | | | | | | | | | | | Change-Id: I65df040404585fd5cb3b9b79258e697333079dd8 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81318 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> Reviewed-by: Louis Stermole <stermole@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/82252 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>
* pmic status tool updatesMichael Pardeik2019-10-283-14/+80
| | | | | | | | | | | | | | | | | | | | | | added checks for when items are applicable added in power reporting added in voltage reporting added in ADC data reporting added in temperature reporting Change-Id: I764d28c93bf097369d1e060a09178bb672b8002e Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85181 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: Mark Pizzutillo <mark.pizzutillo@ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85186 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>
* HW508063: Fix Mirrored BAR setup for SMF and Holes,set map_mode when flippedAdam Hale2019-10-282-110/+184
| | | | | | | | | | | | | | | | Change-Id: I58a721d5371d5d5a10094ab46e15f9ab4083227d Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84996 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: Thi N Tran <thi@us.ibm.com> Reviewed-by: Benjamin Gass <bgass@us.ibm.com> Reviewed-by: Adam S Hale <adam.samuel.hale@ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85185 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>
* Add printout of ECID in exp_getecidMark Pizzutillo2019-10-281-0/+5
| | | | | | | | | | | | | | | | Change-Id: Ib005e5a76365247aa19bd623038d64f95d779075 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85655 Dev-Ready: 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> 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/85663 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>
* Adds eff config blank workaround filesStephen Glancy2019-10-282-0/+48
| | | | | | | | | | | | | | | | | Change-Id: I8cdb884b7b72554a551ad77fbb3286357be601a6 CQ:SW478491 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85228 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Louis Stermole <stermole@us.ibm.com> Reviewed-by: TSUNG K YEUNG <tyeung@us.ibm.com> Reviewed-by: JEREMY R NEATON <jrneaton@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85265 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>
* Don't consider processor's OMI freq if it is 0 when freq restrictionsChristian Geddes2019-10-251-3/+5
| | | | | | | | | | | | | | | | | | We have been told that processors must run at the same OMI frequency. During bringup we saw that if the first boot was done with one of the processors guarded out, and that processor was later unguarded. We would fail the check that ensures these frequencies are the same, because the proc that was guarded/unguarded would still be 0 where the other would be set. This change will allow us to ignore processors with unset frequency so we can boot far enough to set it. Later on the rule will be enforced as well on subsequent boots. Change-Id: I23950371f6f65750aacb07b28317ba11a3a4e8ab Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85832 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> Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com>
* Re-enable NV_STATUS notification for OpalCorey Swenson2019-10-251-9/+3
| | | | | | | | | | | | | | Change-Id: Ie577faad36e6799a347e24c8ff32b80f0e2c7719 RTC:213292 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85100 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: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Call cronus version of memdiags for explorer until prd version worksChristian Geddes2019-10-252-0/+35
| | | | | | | | | | | | | | | | | | There are some delays in getting the attention driven memory diagnostics code up and running. In the mean time we will use the memory diagnostics HWP that cronus uses. This will allow us to init the system without any bootscripts. Eventually, after we get the correct fir settings, we will enable PRD's version of memory diagnostics and remove the cronus version. Change-Id: I6b9b5ff8f32983487a7b811a82a515d9da9502d9 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85863 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: Caleb N Palmer <cnpalmer@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com>
* Update Part Number field side for DDR4 DDIMM spd layoutChristian Geddes2019-10-251-1/+1
| | | | | | | | | | | | | | | | Originally we had this set to 0x30 (48) bytes but now it seems the spec says its only 0x1E (30) bytes. This commit update to the new value that the spec is indicating. Change-Id: I3ba643411a51fdd0df731119dce371bd16435944 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85432 Reviewed-by: Roland Veloz <rveloz@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@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>
* Add register file dump to kernel backtrace printout for crashed tasksZach Clark2019-10-241-0/+12
| | | | | | | | | | | | | | Add a printout for a crashed task's general-purpose registers to the kernel backtrace printout as a debugging aid. Change-Id: I84d81cc6af679dfc8618145e1f143db7b4b6d9db Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/85925 Reviewed-by: Michael Baiocchi <mbaiocch@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>
* 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>
OpenPOWER on IntegriCloud