blackbird-hostboot/src/usr/hwas, branch 07-25-2019

Enhancements to default console output

2020-01-07T19:22:21+00:00

Added the following pieces of extra information to the default console output: -- Attribute Overrides -- 3.49165|**Found 24 attribute overrides in Tank FAPI(1) 3.49169|- PEC:n0:p0:c0 3.49171| ATTR 0EB604DF [1] = 00 3.49174| ATTR 085F1C7E [1] = 00 3.49177| ATTR 048A8902 [1] = 01 3.49180| ATTR 04D8DF8D [2] = 0000 3.49182|- PEC:n0:p0:c1 3.49185| ATTR 0EB604DF [1] = 00 3.49188| ATTR 085F1C7E [1] = 00 3.49191| ATTR 048A8902 [1] = 03 -- New part detection -- 4.97449|Detected new part : 00030002 (Physical:/Sys0/Node0/DIMM2) -- Applying GARD records -- 8.78767|HWAS|Applying GARD record for HUID=0x000E0000 (Physical:/Sys0/Node0/Pr oc0/XBUS0) due to 0x90000135 8.78912|HWAS|Deconfig HUID 0x000E0000, Physical:/Sys0/Node0/Proc0/XBUS0 Change-Id: Ic2ae04515453dbb49e359abff952c87c83e23b72 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/88388 Tested-by: Jenkins Server Reviewed-by: Nicholas E Bofferding Reviewed-by: Roland Veloz Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: William G Hoffa

Automatically include config.h

2019-12-06T16:28:47+00:00

Rather than having to remember to include config.h anywhere we reference a CONFIG variable (and usually forgetting), this adds it to the default compiler flags so that it gets included in every source file we build. Change-Id: I53622ab4d46c55d942e98cae6ec03049fd5b3d08 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/87475 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Zachary Clark Reviewed-by: Roland Veloz Reviewed-by: Christian R Geddes Reviewed-by: Nicholas E Bofferding

Allow dynmic i2c device addresses and set up PMIC targets to do this

2019-11-01T20:41:41+00:00

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 Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Daniel M Crowell Reviewed-by: Roland Veloz Reviewed-by: William G Hoffa

Add call to fapi wrapper of exp_getidec into hwasPlat code

2019-10-03T19:15:15+00:00

The file hwasPlat.C contains the code we use to lookup and compare the IDEC information of a given OCMB when read from the card's SPD versus what is read from the card's defined registers. Recently the code that looks up the IDEC from the card's registers was made into a HWP so it could be leveraged by multiple platforms. A new wrapper has been created on the hostboot side in src/usr/fapiwrap/ that provides an interface for the hostboot platform to call in order to leverage the new EKB hwp. Change-Id: I9155767b199c9b197c010521f33cc0d418901628 RTC: 214627 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/83109 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Ilya Smirnov Reviewed-by: Daniel M Crowell Reviewed-by: William G Hoffa

Add new DDR4 DDIMM keyword ENTIRE_SPD_WITHOUT_EFD

2019-09-27T22:29:55+00:00

With the DDR4 DDIMM SPD there are really two parts. There is the 640 bytes of information you typically see in a DIMM's SPD and then there are 2 KB - 640 bytes of EFD (extensible function descriptors) to give vendors space to add customized data. In many cases we want the ENTIRE_SPD (including the EFD area) but in other cases we want to save some time/space and only read in the first 640 bytes. This new keyword will give us that flexibility. Change-Id: I88ed58b37417b576f2ec4a779ee0c114791c6c21 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84070 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Matt Derksen Reviewed-by: Matthew Raybuck Reviewed-by: Daniel M Crowell

Disable OMI deconfig tests

2019-09-26T16:20:34+00:00

These tests cannot run as part of the standard test suite as they disrupt targeting expectations from other tests that are running in parallel. Currently we are seeing other tests cases that use the OMI/OCMB targets to fail intermittently because targets are missing. Change-Id: Id152e6f571465fee22bae5789eb9a92643738d4b Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/84250 Tested-by: Jenkins Server Reviewed-by: Matt Derksen Reviewed-by: Glenn Miles Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Roland Veloz Reviewed-by: Daniel M Crowell

Account for Explorer's DMB_REVISION format in its SPD

2019-09-11T15:52:05+00:00

We have been told that the field we are reading to figure out the EC level of the OCMB via its SPD (offset 0x200) is going to say 0xA0 for the first version of the card, at IBM we would call this EC 0x10. The Explorer manufacture has stated that they will increment the 1st nibble when they do match card revisions, and the 2nd nibble on minor revisions. This commit adds the logic to convert the manufacture's revision format into IBM's EC level format. It also updates the prebuilt cache we use in simics for axone to have the expected 0xA0 value for the explorer SPD stored in the prebuilt cache Change-Id: I10e9523a8fbea847e3124c256fab7bee850cb29d RTC: 214627 Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/82873 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Matt Derksen Reviewed-by: Daniel M Crowell

Lookup ID/EC register on Gemini OCBMs for comparison to SPD

2019-08-21T21:09:13+00:00

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 Tested-by: Jenkins OP Build CI Tested-by: FSP CI Jenkins Tested-by: Jenkins OP HW Reviewed-by: Matthew Raybuck Reviewed-by: Matt Derksen Reviewed-by: Daniel M Crowell

Fix NVDIMM update error log comments

2019-08-20T17:35:45+00:00

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 Tested-by: FSP CI Jenkins Reviewed-by: Nicholas E Bofferding Reviewed-by: Matt Derksen Reviewed-by: Daniel M Crowell

Set Gemini EC level to be 0x10 if 0x00 is found in SPD revision field

2019-08-09T16:19:54+00:00

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 Reviewed-by: Roland Veloz Reviewed-by: Matt Derksen Tested-by: FSP CI Jenkins Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Reviewed-by: Daniel M Crowell