summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Disable SBE console print as it significantly slows down SBE execution ↵07-25-2019Raptor Engineering Development Team2020-02-191-1/+1
| | | | Status information is already transmitted over Port 81h/82h to the host system
* Send IPL status codes to BMC over LPC port 81h/82hTimothy Pearson2020-02-199-0/+170
| | | | Signed-off-by: Timothy Pearson <tpearson@raptorengineering.com>
* Add SPD, ATTR, and code support for thermal sensor fieldsMark Pizzutillo2020-02-182-2/+47
| | | | | | | | | | | | | | | | | | | | | | | In order to not break systems with old DDIMMs or old Explorer FW, this commit includes a workaround to bypass the thermal field parsing if the SPD revision is less than 0.3, as the decoder will try to parse garbage. This can be replaced down the line if needed with a proper API. git-coreq:hostboot:I7400886b90b566bbfdf218bcedf11627acf1d2fd Change-Id: I7400886b90b566bbfdf218bcedf11627acf1d2fd Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/87999 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Dev-Ready: Mark Pizzutillo <mark.pizzutillo@ibm.com> Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com> Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com> Reviewed-by: Louis Stermole <stermole@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/88583 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Refactor PMIC attributes to be at OCMB levelMark Pizzutillo2020-02-171-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | eff_config can now put SPD attributes at OCMB level if the attribute is defined as such. It will ignore any values for DIMM 1 and only use those for DIMM 0 (attributes that will be the same, like PMIC fields, for example) Minor fix to pmic_status_tool_ut failing UTs due to mismatch in help screen git-coreq:hostboot:I5a66f0540ecc65434123c976d83103c99d9ae018 Change-Id: I5a66f0540ecc65434123c976d83103c99d9ae018 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/90584 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: Louis Stermole <stermole@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/90830 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Adjust idec algorithm for Explorer B.0 formatDan Crowell2020-02-071-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The update to B.0 doesn't allow for a change to the register that previously contained the MAJOR_REVISION value. Because of that we are revising the logic that determines the EC level to be a lookup based on a single number instead. Ignore the MAJOR_RELEASE field from CHIP_INFO entirely, it will always =1. Instead we will rely only on the EFUSE bits currently designated for MINOR_RELEASE. Since we lost the ability to have a common M.m we will just use a single incrementing value to represent the ordinal revision number, regardless of major/minor, i.e. the 1st revision is 0, the 2nd is 1, the 10th is 9, etc. So that gives us these values for EFUSE_IMAGE_OUT_3[13:10] A.0 = 0 = 0x10 in Host Firmware A.1 = 1 = 0x11 in Host Firmware B.0 = 2 = 0x20 in Host Firmware B.1 = 3 = 0x21 in Host Firmware (if it were to happen) Change-Id: I878491e8e004c231ce4dcd7abe887a947c7f3dc3 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/89764 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> 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/89793 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Add attribute for SPD_REVISION (byte 1)Mark Pizzutillo2020-02-071-1/+2
| | | | | | | | | | | | | | | | | | | | | Rename previous uses of Byte 192, the ddimm module spd section to avoid ambiguity. Renamed from spd_revision --> ddimm_module_spd_revision git-coreq:hostboot:I039173097619111ae5847660472e75ed9f2d5f1d git-coreq:sbe:I039173097619111ae5847660472e75ed9f2d5f1d git-coreq:hwsv:I039173097619111ae5847660472e75ed9f2d5f1d Change-Id: I039173097619111ae5847660472e75ed9f2d5f1d Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/90410 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: PPE CI <ppe-ci+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/90506 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Add TARGET_TYPE_GENERICI2CSLAVE and relevant attributesMark Pizzutillo2020-02-071-42/+43
| | | | | | | | | | | | | | | Change-Id: I7d7579f66aba39ab8a462215417e7ab7745f3a6b Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/90321 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: PPE CI <ppe-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/90331 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Update EnterMPIPL to collect architected dumps andMurulidhar Nataraju2020-02-071-2/+29
| | | | | | | | | | | stopClocks for BMC based system Change-Id: Iac0b82e12f048e593acb54c82815d890c2ae93a6 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/90183 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Use full UART TX buffer (shave 5s off booting)Stewart Smith2020-02-041-19/+15
| | | | | | | | | | | | | | | | | | | | | This patch makes us use the full TX buffer of the UART rather than waiting for each character to go through. This practically means we get a *massive* IPL speed up. Measured on my Raptor Blackbird with P9N DD2.21, from "Welcome to SBE" to "Booting Hostboot" (timed on a stopwatch): Before (with UART output): 15 seconds With this patch (with UART output): 10 seconds. A full 33% reduction in boot time for SBE is nothing to sneeze at! Change-Id: Iac50d4b820f3b998e55e5f6e0bff60b067c1df45 Signed-off-by: Stewart Smith <stewart@flamingspork.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/90587 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Andrew R Jeffery <andrewrj@au1.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Update the Simics release pathMurulidhar Nataraju2020-02-032-4/+4
| | | | | | | | Change-Id: I046e248d8815080f97a5f259315e8a6f7a9e9da5 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/90657 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* set OTL Err Rpt Mask Reg 0 bit 21, OTL TX ECC UE on CQ buffer read data.Ryan Black2020-01-211-1/+26
| | | | | | | | | | | | | | | Change-Id: Iacb879cf0aab5db14cee794701bdcf70df2b8dd1 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/89053 Reviewed-by: Thi N Tran <thi@us.ibm.com> Reviewed-by: Jenny Huynh <jhuynh@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: FSP CI Jenkins <fsp-CI-jenkins+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/89055 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* treewide: use print function instead of statementMarty E. Plummer2020-01-1568-585/+644
| | | | | | | | | | | | | | | | | | | | This enables better compat between python2.7 and python3.x Without this change, building with python3.x (python3.5 tested) as /usr/bin/python will result in the following error: File: "sbe/src/build/security/securityRegListGen.py", line 64 -v, --verbose enable verbose traces" ^ SyntaxError: Missing parentheses in call to 'print' Signed-off-by: Marty E. Plummer <hanetzer@startmail.com> Change-Id: Id617f54096fca5cc5fcd829767595a85350e343d Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/89618 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Setting the HRMOR value for OPAL based systems.vinaybs62020-01-104-15/+45
| | | | | | | | | Change-Id: I75f01f8996c6664641bca939855a7bcad6220a01 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/87387 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: MURULIDHAR NATARAJU <murulidhar@in.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* parsAndCutElf: convert line to strMarty E. Plummer2020-01-091-2/+2
| | | | | | | | | | | | | | | | | | | | Without this change, building with python3.x (python3.5 tested) as /usr/bin/python will result in the following error: Traceback (most recent call last): File "./parsAndCutElf.py", line 79, in <module> parserElf(sys.argv) File "./parsAndCutElf.py", line 54, in parserElf if( (line.find(firstSection) != -1) ): TypeError: a bytes-like object is required, not 'str' Change-Id: I0220d0457371349d0a77ac042484fd03f8a984da Signed-off-by: Marty E. Plummer <hanetzer@startmail.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/89385 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Srikantha S. Meesala <srikantha@in.ibm.com> Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* securityRegListGen: use io.open and read in text mode.Marty E. Plummer2020-01-091-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | io.open is compatible with python3.x's open builtin. Open in text mode (no 'b') as this is a plain csv file, and specify the encoding as utf8. Without this change, building with python3.x (python3.5 tested) as /usr/bin/python will result in the following error: File "sbe/src/build/security/securityRegListGen.py", line 695, in <module> main(sys.argv) File "sbe/src/build/security/securityRegListGen.py", line 592, in main for idx, row in enumerate(reader): File "/usr/lib/python3.5/csv.py", line 109, in __next__ self.fieldnames File "/usr/lib/python3.5/csv.py", line 96, in fieldnames self._fieldnames = next(self.reader) _csv.Error: iterator should return strings, not bytes (did you open the file in text mode?) Change-Id: Ib86ea774b40c89e83ab1c0474014222f2fe9ddb9 Signed-off-by: Marty E. Plummer <hanetzer@startmail.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/89386 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Srikantha S. Meesala <srikantha@in.ibm.com> Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* sbeCompression: use floor divisionMarty E. Plummer2020-01-091-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | python2 returns an integer when both division operands are integers. python3 will return a float in this situation. Use the floor division operator (//) which returns an integer on both versions. Without this change, building with python3.x (python3.5 tested) as /usr/bin/python will result in the following error: Traceback (most recent call last): File "sbe/src/boot/sbeCompression.py", line 198, in <module> main( sys.argv ) File "sbe/src/boot/sbeCompression.py", line 181, in main compress(imagePath + "/" + image + ".base", imagePath + "/" + image + ".base.compressed") File "sbe/src/boot/sbeCompression.py", line 58, in compress for i in range(0, os.stat(inputFile).st_size / 4 ): TypeError: 'float' object cannot be interpreted as an integer Change-Id: Ibd63e9d2739eb6cf23d79d739237537e05932b15 Signed-off-by: Marty E. Plummer <hanetzer@startmail.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/89371 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Srikantha S. Meesala <srikantha@in.ibm.com> Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* sbeCompression: add a python3 compatible fallbackMarty E. Plummer2020-01-091-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In python3.x dict.items() does (more or less) what dict.iteritems() did in python2.x, but dict.iteritems() was removed in python3.x entirely. Further, dict.items() in python2.7 is less efficient than dict.iteritems(), so we attempt to use dict.iteritems(), and if that throws an AttributeError, (which would happen on python3.x), fall back to dict.items(). Without this change, building with python3.x (python3.5 tested) as /usr/bin/python will result in the following error: Traceback (most recent call last): File "sbe/src/boot/sbeCompress ion.py", line 198, in <module> main( sys.argv ) File "sbe/src/boot/sbeCompression.py", line 181, in main compress(imagePath + "/" + image + ".base", imagePath + "/" + image + ".base.compressed") File "sbe/src/boot/sbeCompression.py", line 72, in compress sortedList = sorted(instDict.iteritems(), key=operator.itemgetter(1), reverse = True) AttributeError: 'dict' object has no attribute 'iteritems' Change-Id: Iea3109cb51266161629d6e09e79af63af0bbd08c Signed-off-by: Marty E. Plummer <hanetzer@startmail.com> Signed-off-by: vinaybs6 <vinaybs6@in.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/89370 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Srikantha S. Meesala <srikantha@in.ibm.com> Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* updateBuildTag: convert proc.stdout.read() to str.Marty E. Plummer2020-01-091-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | In python2.7, proc.stdout.read() returns a str type. On python3.x, it returns bytes. Without this change, building with python3.x (python3.5 tested) as /usr/bin/python will result in the following error: Traceback (most recent call last): File "./updateBuildTag.py", line 76, in <module> updateBuildTag(sys.argv) File "./updateBuildTag.py", line 62, in updateBuildTag cmd_build_tag = cmd_build_tag+" "+commit_id TypeError: Can't convert 'bytes' object to str implicitly Change-Id: Ib53e0012dd32883559c5fb9d82beb6f2d8881921 Signed-off-by: Marty E. Plummer <hanetzer@startmail.com> Signed-off-by: vinaybs6 <vinaybs6@in.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/89363 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Srikantha S. Meesala <srikantha@in.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* for defect number HW476620Sheng-Hao Huang2020-01-071-0/+17
| | | | | | | | | | | | | | | 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/83795 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Updating the customrc filevinaybs62020-01-071-1/+1
| | | | | | | | | | | Description:with this checkin build has been updated to b1218a_1951.940 Change-Id: I8aa196a1abf2b13e90325c421dbfdf63b038a962 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88909 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Support EXP_FW_LOG command to get Explorer FW logsMatt Derksen2019-12-191-2/+2
| | | | | | | | | | | | | | | | | | | Framework to send inband command and a proof of concept adding explorer error log data via collectFfdc RC. Change-Id: I41be5598eb6fb3239fd933e0a0563d1a99ed2fa0 RTC:205128 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82335 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> Tested-by: HWSV CI <hwsv-ci+hostboot@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/82796 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Add new DDIMM spd version 0_3 and update UTsMark Pizzutillo2019-12-161-1/+2
| | | | | | | | | | | | | | | | Change-Id: I0517bebd0514f0c073e424dc786be7352113c1f8 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88225 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: ANDRE A MARIN <aamarin@us.ibm.com> Dev-Ready: Mark Pizzutillo <mark.pizzutillo@ibm.com> Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88232 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Updated backing build in customrcRaja Das2019-12-101-1/+1
| | | | | | | | Change-Id: I28e49f266431efbc60ce813d07ac5ebd9ecc211f Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88339 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Add asserts for invalid states of enterprise & half dimm modeMark Pizzutillo2019-12-081-0/+33
| | | | | | | | | | | | | | | Change-Id: I995c333fc7f891ea7fa055097d4377a861a1e391 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/87345 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> Dev-Ready: 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/87599 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Add MCBIST infinite testsMark Pizzutillo2019-12-081-0/+11
| | | | | | | | | | | | | | | Change-Id: I54a042d0612a3debd616fa191f524d583fcc34e8 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86588 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: HWSV CI <hwsv-ci+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> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86676 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Add deconfigure of memory port if mss_freq can't find consensus freqLouis Stermole2019-11-121-3/+16
| | | | | | | | | | | | | | | | | | | | | 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/84777 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Add exp_i2c_scom driver that will be consumed by HB/SBE platformsChristian Geddes2019-11-121-0/+62
| | | | | | | | | | | | | | | | | | | | | | This commit adds a new exp_i2c_scom.H file which contains two functions, i2c_get_scom and i2c_put_scom. These functions will take in a fapi2 OCMB target, an address and a buffer that either contains scom data to write or space for scom data to be written to. The functions use the fapi2::puti2c / fapi2::geti2c interfaces to perform the scoms. Change-Id: I92ac697e9ec4d392e1f382e057fbbf8915b78212 Original-Change-Id: I4de680e187258cbfc57dd71f698dc1fc8760cefb RTC:196806 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/67949 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/86610 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Add assert for RSP extended data buffer size in exp_draminitMark Pizzutillo2019-11-121-1/+23
| | | | | | | | | | | | | | | 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/85789 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Run LPC Init to clean up errors on MPIPLDean Sanner2019-11-103-10/+24
| | | | | | | | | | | | | | | | | | | | | Recently HBBL added some error checking on LPC accesses Unfortunately OPAL leaves some errors in the accumulated status register of the LPC Host Controller. Since HBBL doesn't have the space to clear... have the SBE run lpc_init on MPIPL to clean up Keep a global alias for HRMOR Attribute in Pibmem Fetch the SPMode from Default SPMODE Attr if Mbox8 valid bit is not set, else fetch it from Mbox3 bit3. This is to support BMC scratch register setting which are all NULL Change-Id: I975311d9313ac8dbe70cf4595a96c43faaa98dd7 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/86434 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Update exp_draminit results display for new response structsMark Pizzutillo2019-11-051-0/+13
| | | | | | | | | | | | | | 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/82251 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Replaces NVDIMM flush sequence with CCSTsung Yeung2019-11-053-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/85880 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Add FAPI_ASSERT for unsupported SPD rank in RTT fetchNico Fajardo2019-10-301-0/+15
| | | | | | | | | | | | | | | | | 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/85447 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Add code and workarounds for *_omi_setup and *_omi_train for SwiftMark Pizzutillo2019-10-202-5/+55
| | | | | | | | | | | | | | | 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/84274 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Enable rolling counter and verification of req id for ExplorerDan Crowell2019-10-201-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | 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/84632 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* add function to turn on adaptation in bootconfigDevon Baughen2019-10-201-0/+42
| | | | | | | | | | | | | | | 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/84890 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Updates to Explorer FW upload toolsLouis Stermole2019-10-031-6/+7
| | | | | | | | | | | | | | | | 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/84337 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Optimized the Control Instruction Chip-opRaja Das2019-09-301-82/+97
| | | | | | | | | | | | | | | | - Simics doesn't handle core2 onwards properly for control Instruction. It fails with invalid clock state for core2 onwards. CecChip::isValidClocksState Invalid clocks state [0x00000000 0x00000001] to SCOM 0x22010a9c (clockBits [0xe0000000 0x00000001]) - Need to comeback on this to see why simics doesn't complain about core0/1. Change-Id: Id6480b041e4e1ddb29e448ce1653e0995887db2e Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82990 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* PM: Modified FFDC to avoid corruption of RC in error path.Prem Shanker Jha2019-09-251-11/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | On FSP platform, in case of some errors, HWP captures certain EX level registers. To ensure availability of such registers, FSP first attempts special wakeup. Special wakeup HWP uses FAPI_TRY in its code path. It is known that in FFDC collection path, use of FAPI_TRY leads to corruption of original FAPI2 RC. Commit removes such registers from the capture list. Key_Cronus_Test=PM_REGRESS Change-Id: I23ce8b1a8a4a61f71c7b33c1122fcfac905616c4 CQ: SW471606 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83863 Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Gregory S Still <stillgs@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> Tested-by: Cronus HW CI <cronushw-ci+hostboot@us.ibm.com> Reviewed-by: RANGANATHPRASAD G. BRAHMASAMUDRA <prasadbgr@in.ibm.com> Reviewed-by: AMIT J. TENDOLKAR <amit.tendolkar@in.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83956 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Clean up for p10 porting dependency errorsAndre Marin2019-09-252-12/+60
| | | | | | | | | | | | | | | | 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/82366 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Fixes explorer accessor bugsStephen Glancy2019-09-251-1/+1
| | | | | | | | | | | | | | | | | | Bugs fixed 1) updates the Update delay line command to be 0x22 2) updates number of lanes based upon if there is a spare byte Change-Id: I93f1c12c8111a752631ed7a274f5befd2e824453 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83833 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: Mark Pizzutillo <mark.pizzutillo@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/83953 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Add "Not Wired" option for OPTICS_CONFIG_MODE attrsChristian Geddes2019-09-091-2/+8
| | | | | | | | | | | | | | | | | | | | | For Swift systems obus1 is not wired to anything. This is causing issues when we attempt to enable the link. If we set it to NOT_WIRED then we should avoid the problem we are seeing currently where obus1 is having the NV link enabled which is getting parity errors as soon as we enabled the obus. Change-Id: I1380388cbf9a967380c6657102a1b1ae9b266014 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82766 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: Joseph J McGill <jmcgill@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/82793 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Introduce exp_getidec HWPChristian Geddes2019-08-281-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | This hardware procedure is used to read out the chipId and the EC level of a given explorer ocmb. The chipId is an IBM standard that provides information about what kind of chip it is and where it was made. The EC is the write level of the chip. The initial plan was to read all of this info from the CHIP_INFO register but we found that the 2nd nibble of the EC level , which represents the "minor" ec level, was not populated correctly so we must instead read that value from the EFUSE_OUT3 regsiter. Change-Id: Id3dca4449c010ad42f7f3d55428b4efb3f6dfbf0 RTC: 214627 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82712 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: Mark Pizzutillo <mark.pizzutillo@ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82783 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Axone int updatesAdam Hale2019-08-261-0/+17
| | | | | | | | | | | | | | | Change-Id: I6468addadd23f6f252ade88eacfd6a1fd2f47025 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82625 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: Mark Pizzutillo <mark.pizzutillo@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/82635 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* HB Implement Static Analysis with cppcheckLuis Fernandez2019-08-261-1/+2
| | | | | | | | | | | | | | | | | | | | | Inline cppcheck-suppressions for false positives. HB (which pulls in files edited here) will start having the option to implement cppcheck. Change-Id: I08adcf3a344295621c5730fc04516f806c6b85bd RTC: 201451 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82267 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: Ilya Smirnov <ismirno@us.ibm.com> Reviewed-by: Glenn Miles <milesg@ibm.com> Reviewed-by: Nicholas E Bofferding <bofferdn@us.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82297 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Update ocmb i2c driver to send address in read command sequenceChristian Geddes2019-08-261-29/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | As part of the register read operation to a ocmb card the spec tells us to perform 2 commands. The first command is a REGISTER_LATCH cmd which includes the address of the register you want to read. The 2nd cmd is a REGISTER_READ command which, according to the spec does not require the address to be sent again. We found during bringup that the gemini cards were expecting the address to be sent along with the REGISTER_READ cmd as well as with the REGISTER_LATCH cmd. Cronus was doing this in their code so they did not see this problem. As these address bits will be ignored by explorer cards (per the spec) it should be benign the include them with the REGISTER_READ cmd. Change-Id: Ifcade18a290355432df43c0c3e58d6575f1f25b5 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82249 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/82257 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* PM: Fix DB0 HangRahul Batra2019-08-261-0/+1
| | | | | | | | | | | | | | | | | | | Key_Cronus_Test=PM_REGRESS Change-Id: I706ec7b87e777b736153d5765ced0a3f6cea5d96 CQ: SW470688 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81266 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: PPE CI <ppe-ci+hostboot@us.ibm.com> Tested-by: Cronus HW CI <cronushw-ci+hostboot@us.ibm.com> Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com> Reviewed-by: YUE DU <daviddu@us.ibm.com> Reviewed-by: RANGANATHPRASAD G. BRAHMASAMUDRA <prasadbgr@in.ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/81560 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Adds explorer shmoo traitsStephen Glancy2019-08-261-0/+1
| | | | | | | | | | | | | | Change-Id: I099c3ab4d5a1b14936097d7d231682a0362a3be1 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82139 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: Mark Pizzutillo <mark.pizzutillo@ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82315 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Updates exp access delay regs to set delaysStephen Glancy2019-08-261-0/+1
| | | | | | | | | | | | | | Change-Id: I2d78d348aa64f40410b697aae9bdd44ebc8faf1b Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82086 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: Mark Pizzutillo <mark.pizzutillo@ibm.com> Reviewed-by: Jennifer A Stofer <stofer@us.ibm.com> Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82237 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* Add slbv, slbe extraction to p9_ram_core procedureJenny Huynh2019-08-251-1/+13
| | | | | | | | | | | | | Change-Id: I6efe5d4f8fbb9f893a2371acd108d9d1d3002ecd Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82496 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: Matt K Light <mklight@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/82502 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
* p9_proc_gettracearray -- updates for AxoneJoe McGill2019-08-252-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | p9_tracearray_defs.H: - adjust value of PROC_TB_LAST_AXONE_CHIP_TARGET, to cover NPU busses only - introduce PROC_TB_LAST_AXONE_MC_TARGET, to cover OMI busses that logically associate with MC pervasive targets p9_proc_gettracearray.H: - update proc_gettracearray_target_type to return TARGET_TYPE_MC for Axone OMI busses p9_sbe_tracearray.H: - update p9_sbe_tracearray_target_type to return TARGET_TYPE_PERV for Axone OMI busses p9_proc_gettracearray_wrap.C: - add eCMD looper to determine chip type - use chip type to swizzle target type returned by proc_gettracearray_target_type from MCBIST to MC, when running on Axone or Cumulus Change-Id: I5c729385c685ed3b1aac02f1f63b2c81f3e2f0e0 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82308 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> Dev-Ready: Joseph J McGill <jmcgill@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/82361 Reviewed-by: RAJA DAS <rajadas2@in.ibm.com>
OpenPOWER on IntegriCloud