summaryrefslogtreecommitdiffstats
path: root/src/usr
Commit message (Collapse)AuthorAgeFilesLines
* Implement an HBRT interface to log a gard event from PHYP/OPALRoland Veloz2019-08-281-0/+136
| | | | | | | | | | | | | | | | Added a firmware notify interface, gard_event_t, to accept a gard event message, from PHYP/OPAL, to log that event. Change-Id: I9bcf684f0850c9a07ab7d46635aa07a2c1e9917c RTC: 210201 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82199 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: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Add more FFDC to BPM Update Error logsMatthew Raybuck2019-08-281-6/+144
| | | | | | | | | | | | | | | | | | Adds part callouts for all BPM related error logs and collects BPM traces for any errors that are returned from interactions with the NVDIMM to provide context. Additionally, if there is an error during the config updates the code will erase the firmware section to force an update on any subsequent attempt. Change-Id: I045b4ba1feea2fbc1ddb5e418bf60b86d6c11bb1 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82946 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: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: Corey V Swenson <cswenson@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Fix lingering error log issues for BPM updatesMatthew Raybuck2019-08-282-77/+260
| | | | | | | | | | | | | | | | | There was a lingering issue where an error that was retryable was thought to be non-retryable. This propagated out of blockWrite() causing the update to think it had failed when it fact it had not. Change-Id: I536e79ddd4ac476fdf1aa0b370f81f4b0b198f4c RTC: 212448 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82614 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: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Updated the HBRT reserved trace buffer code to compensate for relocationRoland Veloz2019-08-284-69/+363
| | | | | | | | | | | | | | | | | | | | | | This change fixes an issue where the HBRT reserved trace buffer can be placed in a new location than it's origination. When HB crashes, the reserved trace buffer is persisted until next IPL. With OPAL, the reserved trace buffer may be in a different memory location than where the buffer was when it crashed, therefore all internal pointers will be invalid. This change will detect the change in buffer location and realign all internal pointers to be valid once again. This is not an issue with PHYP, only with OPAL. Change-Id: I476845550062433fba190294b0bd2bbcf8dad658 RTC: 206137 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82094 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: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: Glenn Miles <milesg@ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Added security bit and fir handling, gard and trace fixesMATTHEW I. HICKMAN2019-08-277-113/+273
| | | | | | | | | | | Change-Id: I8160d9662859232dfd612b3a2f5c1c522c02e308 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82534 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: FSP CI Jenkins <fsp-CI-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 call to exp_mss_thermal_init to step 14.2Andre Marin2019-08-272-7/+14
| | | | | | | | | | | Change-Id: I85b34168b8a93ea8c744b6a79f207ce924cdf50f RTC:195557 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/79326 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: Daniel M Crowell <dcrowell@us.ibm.com>
* PRD: updates to IOOLFIR based on recent OpenCAPI FIR reviewZane Shelley2019-08-278-327/+444
| | | | | | | | | | | | | | | | Change-Id: I0f721760eebfd805cfd6b939d75d42ec2d46cdc2 CQ: SW469988 Backport: release-op920 Backport: release-op930 Backport: release-op940 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82785 Reviewed-by: Caleb N Palmer <cnpalmer@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/82881 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: fix OBUS workaround for non-SMP modeZane Shelley2019-08-271-4/+20
| | | | | | | | | | | | | | | | | | There were a couple functions that checked for non-SMP mode that could be called for types other than OBUS. This would cause an assert when calling the SMP mode check. Change-Id: I044108e40c305cd87861989c0dd69bda3bd79480 CQ: SW473849 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82851 Reviewed-by: Caleb N Palmer <cnpalmer@us.ibm.com> Reviewed-by: Paul Greenwood <paul.greenwood@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/82880 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: Don't dealloc NVDIMMs set to NO_GARD in a predictive logCaleb Palmer2019-08-271-1/+10
| | | | | | | | | | | | | Change-Id: I0dd124c1d797baa50de78f01f01c29b6af524a32 CQ: SW473646 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82702 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/82761 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>
* Add Encryption Enabled bit to NV_STATUS_FLAGCorey Swenson2019-08-272-13/+48
| | | | | | | | | | | | | | | | | | - 0x10: Contents are encrypted Set when encryption is enabled Clear when encryption is disabled Attribute comments should match HDAT Change-Id: I52805f6c8c3f303ce29cf8b8534304dfd0a09821 RTC:214625 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82638 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: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Replaces nvdimmWrite() to reset controller with nvdimmResetController()Tsung Yeung2019-08-261-3/+3
| | | | | | | | | | | | | | | nvdimmResetController() will call nvdimm_encrypt_unlock() to unlock encryption if needed. Without it nvdimm ops could fail on runtime Change-Id: Iad4c4d354d3b9522ee0544b5c8de7484dcf89c6c CQ:SW474050 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82838 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: Daniel M Crowell <dcrowell@us.ibm.com>
* Fix restore fail due to restore attempt on empty flashTsung Yeung2019-08-264-71/+87
| | | | | | | | | | | | | | | -fix NVDIMM FFDC misalignment -correct NV_STATUS enum naming Change-Id: Ib0084f5fb95ce8a93ee5e85a0790878469065acb CQ:SW473934 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82775 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> Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* PRD: removed redundant header iipbits.hZane Shelley2019-08-2612-49/+15
| | | | | | | | | | | | | Change-Id: I88903ba637a1f89910fcd226325a220c98169945 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82617 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Caleb N Palmer <cnpalmer@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/82786 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>
* NVDIMM: Load warning thresholds, after FW_UPDATE, on every IPLCorey Swenson2019-08-262-0/+112
| | | | | | | | | | | | | | | | See defect for register list and hard-coded values Change-Id: Ibc32ddfcb4719388dd744aa946df62e27066a05c CQ:SW472435 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82348 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Glenn Miles <milesg@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: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Fix OMIC MRW processingDan Crowell2019-08-232-36/+43
| | | | | | | | | | | | | | | | | | | | Missing a few things in our mrw xml processing for OMIC targets. - Added a maxInstance value for the OMIC targets. This prevents the HUIDs from repeating the same value behind each proc. - Modified OMIC_PARENT calculation to use existing PATHs instead of calculating (incorrectly) from FAPI_POS. - Added correct unit_pos calculation Change-Id: I4e0dee4ec1aa9a080f2ce49922659c4c7b5ca548 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82629 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: Chen Du <duchen@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com> Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com>
* Update SBE In Istep7.5 In SMF ModeIlya Smirnov2019-08-231-1/+31
| | | | | | | | | | | | | | | | | | | | | A bug was discovered on OpenPOWER systems wherein if SMF is enabled, we would checkstop in istep10.1 when trying to XSCOM to the slave proc. The issue was that SBE at that time didn't know we were booting in SMF mode and so didn't set up the XSCOM BAR for the slave proc appropriately. The solution is to force SBE update in istep7.5 if SMF is enabled in the MRW (ATTR_SMF_CONFIG) but SBE doesn't yet know about that (bit 16 in scratch reg6 is not set). Change-Id: Ibaa2d1e5b441f378528443aa84e5cd61789792dd Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81950 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: Nicholas E Bofferding <bofferdn@us.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* MDIA: Update mdiatestmba.H test caseCaleb Palmer2019-08-221-46/+67
| | | | | | | | | | | Change-Id: I47d4612fa063cd43e3d49e4cc47e466620add3c9 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82514 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: Benjamen G Tyner <ben.tyner@ibm.com> Reviewed-by: Zane C Shelley <zshelle@us.ibm.com>
* Add subsystem translations for NVDIMM part calloutsDan Crowell2019-08-222-1/+55
| | | | | | | | | | | | | | | | | | | A few new part callouts were added to support the NVDIMM function but a corresponding error log subsystem was never added. This change also adds a testcase to check for missing subsystems going forward. Change-Id: I523840615fe734553dd30af8e608c1aa214fd2dc Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82591 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: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: ARAVIND T. NAIR <aravindnair@in.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Lookup ID/EC register on Gemini OCBMs for comparison to SPDChristian Geddes2019-08-211-39/+52
| | | | | | | | | | | | | | | | | | The ID/EC values of Gemini card are written to SPD contain on its eeprom device but we need to validate that the SPD matches what the contents of the actual regisers are when we get up to a point where we can read these registers. Change-Id: I11874573e8978e3b0ebb652ded800f2d68d1d40b RTC:209353 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82118 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: Matthew Raybuck <matthew.raybuck@ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* In regards to health check errors, removed the software calloutRoland Veloz2019-08-211-4/+4
| | | | | | | | | | | | | | | | | Software callouts, ADD_SW_CALLOUT, where added to the error logs for errors associated with the health check status when they should not have. Changed them to NO_SW_CALLOUT. Change-Id: Iccf5289a38053aa3eccaa59b200a99fa1f14f83b CQ: SW473391 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82540 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Corey V Swenson <cswenson@us.ibm.com> Reviewed-by: Christian R Geddes <crgeddes@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: Daniel M Crowell <dcrowell@us.ibm.com>
* Remove verbose option from handle_duplicates callMatt Derksen2019-08-211-4/+2
| | | | | | | | | | | | | | | | | | verbose is not a parameter for handle_duplicates.pl so it should not be called from makefile. Results in two cluttering/misleading traces: Unknown option: verbose Change-Id: I6a8f1a383ec3351c50b18080f7215635e657cc76 CQ:SW473214 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82452 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: Christian R Geddes <crgeddes@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: Uncomment OMI bus callout and cleanup getSlaveRanksCaleb Palmer2019-08-212-20/+8
| | | | | | | | | | | | Change-Id: Ia5e6c86124b1a5ecece87eed43721fc38486ae88 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82216 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/82566 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 MemoryMru UpdatesCaleb Palmer2019-08-215-35/+115
| | | | | | | | | | | | | | Change-Id: I925e06e59ecab97d4f951ea4eafca6d27238312d RTC: 199037 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82156 Tested-by: Jenkins Server <pfd-jenkins+hostboot@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> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82565 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: removed unused MopRegisterAccess codeZane Shelley2019-08-2113-440/+39
| | | | | | | | | | | | | | Change-Id: Ifc379d6180fc85f784397d66cfc5a6aed8c2670d Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81992 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Paul Greenwood <paul.greenwood@ibm.com> Reviewed-by: Caleb N Palmer <cnpalmer@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/82030 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>
* Enable BPM UpdatesMatthew Raybuck2019-08-202-395/+881
| | | | | | | | | | | | | | | | This commit turns on BPM updates and fixes several issues that were preventing them from occuring successfully. Change-Id: I2aa544c59f04fcc06bf7f10bdff5026579d7f053 RTC:212448 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82487 Reviewed-by: Roland Veloz <rveloz@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>
* Fix NVDIMM update error log commentsCorey Swenson2019-08-206-65/+68
| | | | | | | | | | | | | | First line of error log comment must include /*@ or error log will not be documented. Also fixed in other HB files. Change-Id: Ifa5eba6d6abd7f8565a4cc8d62a25a2b833725f2 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82497 Tested-by: Jenkins Server <pfd-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: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* PRD: NVDIMM don't check threshold unless no errors foundCaleb Palmer2019-08-201-1/+1
| | | | | | | | | | | | | Change-Id: Ibe98c4b9f56efa7fc2edd75cab4b11f13e5226b4 CQ: SW473078 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82347 Reviewed-by: Benjamen G Tyner <ben.tyner@ibm.com> Reviewed-by: Zane C Shelley <zshelle@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82441 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>
* Test runtime scom support in AxoneMatt Derksen2019-08-206-11/+21
| | | | | | | | | | | | | | | | Needed to disable i2c explorer command tests during runtime. Currently just support mmio operations for runtime. Change-Id: Ifd2888b1deb03b76408b03516a9dd030896e1ee1 RTC:210321 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/80236 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Christian R Geddes <crgeddes@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: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Fix retry failure in nvdimmWriteData()Matt Derksen2019-08-191-3/+6
| | | | | | | | | | | | | | | | Need to create a local variable for i_dataLen, because deviceOp will alter the i_dataLen value on failure (ie NACK). Change-Id: If73efb77a1ba918fbc1bb7decd8fb46a5d2014b9 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82344 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: Matthew Raybuck <matthew.raybuck@ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Add attribute to force/skip BPM updatesMatthew Raybuck2019-08-193-7/+131
| | | | | | | | | | | | | | | | | Adds a new attribute ATTR_BPM_UPDATE_OVERRIDE to override the default update behavior. Updates can be forced, skipped, and the firmware and config portion of the updates can be individually forced or skipped. Change-Id: I1d50ab60d18a0511466f1551108755ef9f70cf35 RTC:212448 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82263 Reviewed-by: Roland Veloz <rveloz@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Christian R Geddes <crgeddes@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>
* Correct BPM update flow to match SMARTs flowMatthew Raybuck2019-08-192-170/+540
| | | | | | | | | | | | | | | | | | Corrects the update flow for the BPM to match the flow given by SMART. This commit also pulls out the firmware and config update procedures into their own functions as well as fix a few bugs in the verifyBlockWrite function. Change-Id: Ia7950078112e1ad54ef63594efa010b06460513a RTC:212448 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82219 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: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Add code to run BPM update procedure from nvdimm_update.CMatthew Raybuck2019-08-192-21/+187
| | | | | | | | | | | | | | | | | Adds the necessary code to hook the existing BPM update code into istep 21. The BPM update will be called in nvdimm_update.C after the nvdimm updates have occurred. Change-Id: If968313433bfdfbbbb929874de6eb0bf4d21b241 RTC: 212448 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/79640 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: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Added error checking and handling to nvdimm iplMatthew Hickman2019-08-1912-233/+2372
| | | | | | | | | | | | Change-Id: I153cf39cc674b49441b5f41f7b96cd667b2a265b Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/77543 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: TSUNG K YEUNG <tyeung@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* HBRT command from SBE to trigger NVDIMM operationsCorey Swenson2019-08-194-4/+84
| | | | | | | | | | | | | | | | | Add support for calling NVDIMM operations through the SBE message passing interface Change-Id: Iae94166a411f134771f461dd3a63806721904368 RTC:211725 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/80856 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: Glenn Miles <milesg@ibm.com> Reviewed-by: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Skip Alternate PNOR Validation on alt-master proc if lpc init failsBill Hoffa2019-08-191-7/+12
| | | | | | | | | | | | | | | | | | - It is expected that the alt pnor validation step will fail if lpc init fails so this step should be skipped. The errors from lpc init are committed so that is already handled. Change-Id: I07ab852abd986d743bad86040ea9b13479926501 CQ: SW465693 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/78345 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: Ilya Smirnov <ismirno@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com> Reviewed-by: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: William G Hoffa <wghoffa@us.ibm.com>
* Fix MMIO Addressing and Attributes on Non-Master ProcBill Hoffa2019-08-162-6/+17
| | | | | | | | | | | | | | | | | | | | - MMIOs to the MC/OMI space to a non-master proc were incorrectly using a base address pointing to proc0 - The OMI_INBAND_BAR_BASE_ADDR_OFFSET attribute MRW processing was incorrect for MC and OMI targets behind the 2nd proc Change-Id: Ife440541642463e11d4146157d568f8cd4440877 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82234 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: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Fix calling eepromddtest.HMatt Derksen2019-08-161-1/+1
| | | | | | | | | | | | | Makefile change overwrote TESTS and lost adding of eepromddtest.H Change-Id: I01adcc3958ce181909589b159a86a54bceb65257 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82293 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: Christian R Geddes <crgeddes@us.ibm.com>
* Runtime scom support for OCMBsMatt Derksen2019-08-158-64/+333
| | | | | | | | | | | | | | | | | | | | | fapi2::getScom on OCMB targets eventually resolves to a mmio call at runtime. This commit registers a runtime version of mmioScomPerformOp to the IBSCOM device ops for OCMB targets. During runtime, hostboot will rely on the hypervisor (opal/phyp) to issue the inband commands to perform the operation on the desired OCMB. Change-Id: I5ec25d0401267125419cd579a3a2d752f14234a4 RTC:210321 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/80235 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: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Add OSYS record MM/SS keyword checks for PVPDChristian Geddes2019-08-151-6/+75
| | | | | | | | | | | | | | | | | | | For planar VPD (PVPD) there is an additional PN/SN keyword pair we must check when determining if the hardware matches what we have cached in the EECACHE section of pnor. This commit adds this extra check for NODE targets which PVPD eeproms are associated with. Change-Id: I257045144597d250683e67c24e240bd6be468218 RTC: 210350 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/80562 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: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Call new p9a_throttle_sync procedureMatt Derksen2019-08-142-9/+26
| | | | | | | | | | | | | | | Should be called just like the old p9_throttle_sync but specific to Axone. Change-Id: I08c39ecae6453e527482f7e498e44131b4222894 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/80237 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: Christian R Geddes <crgeddes@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Use REL_POS of OMI and not OCMB for calculating hdat DIMM ID'sChristian Geddes2019-08-141-3/+3
| | | | | | | | | | | | | | | | | | | | There is a bug in the code where we are attempting to lookup the REL_POS of an ocmb target while trying to generate the id hdat will use to uniquely identify a given dimm target. OCMB targets do not have the REL_POS attribute and if they did it would always be 0, or it would match the associated DIMM's REL_POS (for single dimm ocmbs at least). Instead we will use the OMI's REL_POS which should work because OMI's are 1-1 with OCMB targets. Change-Id: Ib22307c7f4c2d97a5368fd507fe7b4965135c76e Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82120 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Jayashankar Padath <jayashankar.padath@in.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: William G Hoffa <wghoffa@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Do not unload modules loaded for unit testsChristian Geddes2019-08-136-136/+9
| | | | | | | | | | | | | | | | | | | | | | | | For some of the unit tests we must load in .so modules that were unloaded during the boot. Modules must get loaded and unloaded during the boot, especially before we expand from cache containment to over come size limitations. We were seeing issues were test case A and B both relied on a module and attempted to load it when the test case was instantiated then tried to unload the module when the test case was completed. This was causing issues if two tests were using the same loaded module and one test finished early and unloaded it. Test cases are run on simics after memory is expanded so there is no reason unload the extra modules we load in so we will leave them loaded. Change-Id: Ia41d38da11400f54ee2e59e497b9610ac24f1629 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82099 Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Glenn Miles <milesg@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>
* Increased the BMC timeout from 5 seconds to 30 secondsRoland Veloz2019-08-121-2/+2
| | | | | | | | | | | | | | | | | | | | | On a zaius machine, the BMC was consistently hitting a checkstop with the system shutting down with error status 0x613 following istep 13.11 - mss_draminit_training. The theory was that it was hitting a timeout issue. Increasing the timeout from 5 seconds to 30 seconds abated the situation. It does appear that it was timeout issue. Change-Id: I825f461855b2a4a70c732f0dda308b2579d60a7d CQ:SW466685 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82034 Reviewed-by: Glenn Miles <milesg@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>
* Move exp_inband to lib directoryMark Pizzutillo2019-08-117-6/+10
| | | | | | | | | | | | | | | Change-Id: If494c330ce20f6ec96ded6194d705e908198cf8b Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81882 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: ANDRE A MARIN <aamarin@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81886 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: Temp fix to ignore lane repair attns in OpenCAPI modeZane Shelley2019-08-094-0/+164
| | | | | | | | | | | | | | | | | | | The hardware team reused several bits in the IOOLFIR for OpenCAPI mode. Previously, we had expected these bits only to be reported in SMP mode. Removed assert to allow for bring-up while waiting for actual design changes to be finalized. Change-Id: Iad4015f372c37257fd7d5efef25865d006484cd3 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81274 Reviewed-by: Caleb N Palmer <cnpalmer@us.ibm.com> Reviewed-by: Paul Greenwood <paul.greenwood@ibm.com> 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> 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: Zane C Shelley <zshelle@us.ibm.com>
* Only compare a proc's omi freq if it has functional OCMB childrenChristian Geddes2019-08-092-78/+32
| | | | | | | | | | | | | | | | | | There are checks we do during the memory intialization where we check to make sure that both processors have the same frequence set for their OMI traffic. If a processor has no OCMB targets then its OMI frequence will not get set so we should ignore those processors for this comparison. Change-Id: Id00f01756b431695e92a15872ebe00fd18306c4f Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81462 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> 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: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Add workaround to ignore MC channel hangs for Swift bringupChristian Geddes2019-08-092-2/+70
| | | | | | | | | | | | | | | | | Using the gemini cards we have been detecting channel hangs which are actually just a result of the gemini cards being slow. To work around this for now we will disable the channel hang checks for the memory controller. Change-Id: Ic21dca4c207140fab8d91b5f1f1a20e6670494cc Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81471 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: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@us.ibm.com> Reviewed-by: Daniel M Crowell <dcrowell@us.ibm.com>
* Set Gemini EC level to be 0x10 if 0x00 is found in SPD revision fieldChristian Geddes2019-08-091-1/+1
| | | | | | | | | | | | | | | | The SPD for ocmb cards does not have the EC level but instead has the "revision". Revision 0x00 for the gemini should be EC level 0x10. If we make the gemini EC level to be 0x00 the EC attributes will not work. Change-Id: I4eb0025a7e9758cb0b5238a349bc7e6d48ac86ff Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81930 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Roland Veloz <rveloz@us.ibm.com> Reviewed-by: Matt Derksen <mderkse1@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: Fix issue where a deviceRead of a register is returning bad dataRoland Veloz2019-08-094-4/+117
| | | | | | | | | | | | | | | | | | | The reason the deviceRead is returning bad data, from an NVDIMM register, is because the register data is segmented across several pages. One must first set the page correctly before attempting to do a read of a register. This fix will set the page up correctly before doing a read/write to an NVDIMM register. Change-Id: Ic63c8671c0083f1ca243f255a29d1ef4606b7ce5 CQ: SW465917 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81542 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Matthew Raybuck <matthew.raybuck@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: Daniel M Crowell <dcrowell@us.ibm.com>
* Fix to not send a NVDIMM protection state change to OPALRoland Veloz2019-08-091-1/+6
| | | | | | | | | | | | | | | | | | | There is a firmware request call, within the notifyNvdimmProtectionChange method, that needs to be guarded such that the message is NOT sent to OPAL. A guard has been placed in the call so the notification is only sent to PHYP and not OPAL. Change-Id: Ib17dd8f3bb20d4d40bf163760da4b4b3c835ff70 CQ:SW471511 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81976 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>
OpenPOWER on IntegriCloud