summaryrefslogtreecommitdiffstats
path: root/src/usr/i2c
Commit message (Collapse)AuthorAgeFilesLines
* Fix up some i2c testcasesDan Crowell2017-05-164-22/+47
| | | | | | | | | | | | | Enable a few of the i2c testcases and fix up a broken one Change-Id: Ic7111579e058a31bc1f37a5cb3d21a71d91d8128 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/39940 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Martin Gloff <mgloff@us.ibm.com> Reviewed-by: Elizabeth K. Liner <eliner@us.ibm.com> Reviewed-by: Matthew A. Ploetz <maploetz@us.ibm.com>
* ProcessMRW changes for dynamic i2c devicesElizabeth Liner2017-05-041-7/+5
| | | | | | | | | | | | | | | | | | HDAT has a requirement from hostboot to provide I2C information from the connections in the mrw. This change pulls the changes out of the mrw and stores them under the master proc target. Each index in the arrays created is a separate connection. The information is then pulled out in i2c.C getDeviceInfo() Change-Id: I725278bc482131a65b8cfb565e08ff0a37f4cb30 RTC:165485 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/39737 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Prachi Gupta <pragupta@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* Remove Write Cycle Time work aroundMarty Gloff2017-04-261-5/+0
| | | | | | | | | | | | | | | Remove code to force the the write cycle time for the SBE SEEPROM to a minimum value of 10. Change-Id: I61c8331cf56154095135951063b40956ba8f3ca6 RTC: 164392 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/39382 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Corey V. Swenson <cswenson@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Converge shadow TPM object into targeting modelNick Bofferding2017-04-101-10/+4
| | | | | | | | | | | | | | | | | | | | - Added TPM attributes - Linked TPM attributes to XML models - Updated TpmTarget as alias for TARGETING::Target - Trace all states in TPM targeting target Change-Id: Ic0e6cf974aa82f0273523bdada1081b625cb0ae4 RTC: 168781 CMVC-Coreq: 1021028 CMVC-Prereq: 1021177 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/38337 Tested-by: Jenkins Server <pfd-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> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Stephen M. Cprek <smcprek@us.ibm.com> Reviewed-by: Christopher J. Engel <cjengel@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Enum value fix for HDAT i2c data.nagurram-in2017-03-271-3/+3
| | | | | | | | | | | Change-Id: Ie38e800a8f77d59d84f0d99e129c27cad3b38a16 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/38053 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Jayashankar Padath <jayashankar.padath@in.ibm.com> Reviewed-by: VENKATESH SAINATH <venkatesh.sainath@in.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* Add TPM device driver support for DRTM PCR reset sequenceChris Engel2017-03-082-31/+109
| | | | | | | | | | | Change-Id: Ief02e10fc85d09b837e20dfb529186e2da4269fd Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/35935 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> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Support DRTM RIT protectionNick Bofferding2017-03-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Added mailbox scratch register 7 definition - Added DRTM functions - Added set/clear security switch register functions - Added additional security switch bit definitions - Added secureboot extended library to host DRTM functions - Inhibited TPM start command in DRTM flow - Added new config options for DRTM and DRTM RIT protection - Added new DRTM attribute to indicate if DRTM is active - Added new DRTM attribute to hold DRTM payload address - Added new DRTM attribute to initiate DRTM in lieu of loading payload - Updated target service init to determine DRTM settings - Updated host start payload step to initiate DRTM if conditions are met - Updated host MPIPL service to verify DRTM payload and clean up DRTM HW state - Updated host gard step to verify DRTM HW state - Rerouted PCR extensions to PCR 17 in DRTM boot - Use locality 2 for all PCR extensions in DRTM boot - Inhibit extension logging (for now) in DRTM boot - Only extend seperator to PCR 17 in DRTM boot Change-Id: Id52c36c3a64ca002571396d605caa308d9dc0199 RTC: 157140 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/35633 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Stephen M. Cprek <smcprek@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Timothy R. Block <block@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Adding interface changes to provide I2C information to HDATElizabeth Liner2017-02-212-2/+216
| | | | | | | | | | | | | | | | This is the first of two commits, this includes the I2C information that hostboot is already aware of (EEProm and TPM). There will be a later commit that pulls in the new information from the MRW. Change-Id: I33b0cd7784b242de3b68a3e7b07527604d819c23 RTC:161649 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/32437 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Martin Gloff <mgloff@us.ibm.com> Reviewed-by: NAGENDRA K. GURRAM <nagendra.g@in.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Don't reset/setup I2C master 0 as SBE owns itDean Sanner2017-02-211-0/+8
| | | | | | | | | | | | | | | Hostboot shouldn't be messing with I2C master 0 as the SBE is actively using it. Reseting or setting it up will cause SBE I2C errors the next time it accesses it Change-Id: I12b16d158a18b09dadff1de4534f3ea986a532a6 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/36569 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> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Corey V. Swenson <cswenson@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* Revert "i2c: only send 1 master stop in FORCE_UNLOCK reset mode"Daniel M. Crowell2017-02-171-6/+5
| | | | | | | | | | | This reverts commit 28bfab63a26982fe32b7711729ee79f58aaa623a. Change-Id: I02060167fc113dcd81da59742e5cf5d6c8c5a766 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/36573 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* i2c: only send 1 master stop in FORCE_UNLOCK reset modeRobert Lippert2017-02-141-5/+7
| | | | | | | | | | | | | | | | | | The existing code path would send 2 master stops, one in i2cForceResetAndUnlock() and another in i2cSendSlaveStop(). The one in i2cSendSlaveStop() can cause IPL failures if the VSPD voltage is not online for a centaur I2C port because it requires checking that the I2C bus is "free" (pulled up) before reporting success. Change-Id: I435f4e738d1c000e23167221a42f6747f0dce93b Signed-off-by: Robert Lippert <rlippert@google.com> Signed-off-by: William A. Kennington III <wak@google.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/36263 Tested-by: Jenkins Server <pfd-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>
* eepromdd: retry reads on I2C arbitration lost errorsWilliam A. Kennington III2017-02-111-77/+92
| | | | | | | | | | | | | | | | These "arbitration lost only" errors seem to be intermittent and in many cases works to retry them when reading SPDs. Helps to avoid deconfiguring a bunch of memory unnecessarily. Change-Id: I05dd93f96545adc65c895295b0d50c2a545aa9e7 Signed-off-by: William A. Kennington III <wak@google.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/36300 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Corey V. Swenson <cswenson@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Add base support for TPM locality 2Nick Bofferding2017-02-081-10/+14
| | | | | | | | | | | | | | | | | | | | | - Added locality 2 enum - Updated check for localities to include locality 2 - Updated MRW generator to add locality 2 - Updated standalone config files to include locality 2 - Updated TPM_INFO to include locality 2 Change-Id: I385f331875c59568def895c1b89fcdc14dda6234 RTC: 157140 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/35916 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> Reviewed-by: Marshall J. Wilks <mjwilks@us.ibm.com> Reviewed-by: Stephen M. Cprek <smcprek@us.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Timothy R. Block <block@us.ibm.com> Reviewed-by: Christopher J. Engel <cjengel@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Add mising pieces to read DVPD from HWPrachi Gupta2017-02-071-1/+5
| | | | | | | | | | | | | - Added support for MCS target to access EEPROM device - Added EEPROM_VPD_PRIMARY_INFO attribute for MCS target - Fixup caching of DVPD in PNOR Change-Id: I9fb2de82b16eb017517f5addf361c4a90b3edf30 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/35767 Reviewed-by: Dean Sanner <dsanner@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* Add TPM4 locality supportChris Engel2017-01-272-17/+41
| | | | | | | | | | | | | Change-Id: I9f16fe77ee18f3d8839d0a06f9322ca1b1e47d93 RTC: 134415 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/35271 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> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com> Reviewed-by: Timothy R. Block <block@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* Handle fsi i2c correctly on second socketDean Sanner2017-01-171-4/+41
| | | | | | | | | | | | | | - Security registers prevent usage of diag mode - FSI to host engine mapping changes in P9 Change-Id: I1b88e73cc1cdb6c01a24710916fdfe71cd664b48 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/34926 Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@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> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* I2C diag mode can't be used on master 2, port 0Dean Sanner2017-01-141-2/+19
| | | | | | | | | | | | | | | -This is the typical port where the TPM is connected, and the extra security features for locality 4 interfere with the diag mode reset Change-Id: I5dddc8263d6a75e246ce3e2043037ab9f69555ff Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/34544 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com> Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com> Reviewed-by: Dean Sanner <dsanner@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* Fix i2c scom address math for p9Prachi Gupta2016-12-051-1/+4
| | | | | | | | | | Change-Id: I7dd49b51f233a7862907f236813f2ed26da455c5 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/32616 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Martin Gloff <mgloff@us.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* SBE Update fixes and patches for Bring-upMarty Gloff2016-12-021-0/+5
| | | | | | | | | | | | | | | | | Set NO_SBE_UPDATE in fsprelease.config. Patch eepromdd.C to make write cycle time a minimum of 10 msec. Find .hbbl section before attempting to append new HBBL to SBE image and delete it if it already exists. Use malloc rather than stack space for ring section buffer. Fix i_maxImgSize value passed to procCustomizeSbeImg. Fix SBE_ECC_IMG_MAX_SIZE calculation to include pad bytes. Change-Id: I632e17851830acb1b365abc92438b0356232487c Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/32487 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Matt Derksen <v2cibmd@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com> Tested-by: William G. Hoffa <wghoffa@us.ibm.com>
* Enable TPM in compile configurations; disable TPM_REQUIRED for HWMike Baiocchi2016-10-171-2/+2
| | | | | | | | | | | | | | | | | Enable the CONFIG_TPMDD and the CONFIG_TPM_NUVOTON compile flags. Changed TPM_REQUIRED attribute to default to 'no' for HW while keeping it "yes" for simics configurations. Change-Id: I1e18d528174cbea3f843a0a6acd777501b8dba16 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/31019 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Christopher J. Engel <cjengel@us.ibm.com> Reviewed-by: Stephen M. Cprek <smcprek@us.ibm.com> Reviewed-by: Timothy R. Block <block@us.ibm.com> Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com> Reviewed-by: Marshall J. Wilks <mjwilks@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* Add interfaces for HDAT to fetch new dataDan Crowell2016-10-161-0/+12
| | | | | | | | | | | | | | | | | | New interfaces added: I2C::getDeviceInfo() IPMI::getBmcInfo() CONSOLE::getUartInfo() New attributes: SYSTEM_FAMILY SYSTEM_TYPE Change-Id: I7d1a018c01067a19b8a66f853e11922c910fcc54 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/30405 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Reviewed-by: Elizabeth K. Liner <eliner@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Add call to update SBE image if nest freq changed in istep 7.3Andres Lugo-Reyes2016-10-051-1/+1
| | | | | | | | | | | Change-Id: I529056bf1380238f5064f18b39bc09ec97e7a112 RTC: 157659 Depends-on: Idea9e3ae8d08052e960c00c225522bbe8da3ea5e Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/29521 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 <v2cibmd@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Changes for P9 SBE - General clean-upMarty Gloff2016-09-251-9/+3
| | | | | | | | | | | | | | | This story is for general clean-up after the SBE Update work. It will be used to turn off unit test traces, make changes that were deferred from prior reviews, finalize selection of best cores and handling of cores for p9_xip_customize, and turn off performUpdateActions() call in sbeupdatetest. Change-Id: I04747bd3897d84a6b3c3c6c7791b28a0708842f9 RTC:161074 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/29955 Tested-by: Jenkins Server <pfd-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>
* Changes for P9 SBE - Enable/Remove Istep calls/processingMarty Gloff2016-09-034-60/+128
| | | | | | | | | | | | | | | | | | | | | | 3) Clean up TODO's in isteps 08, 09, and 10 In istep08 enable resolveProcessorSbeSeeproms call from call_host_slave_sbe_config.C and remove findSBEInPnor call and other processing from call_proc_check_slave_sbe_seeprom_complete.C. In istep09 remove updateProcessorSbeSeeproms call and nest frequency processing from call_fabric_erepair.C. In istep10 enable updateProcessorSbeSeeproms call from call_host_slave_sbe_update.C and enable loop to set use of xscom in call_proc_build_smp.C. Change-Id: I79237f530738e3088d1b3aedafdc6ad1139d21a8 RTC: 156597 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/26801 Tested-by: Jenkins Server <pfd-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>
* Trustedboot support for using TPM targetChris Engel2016-08-223-435/+288
| | | | | | | | | | Change-Id: I362085fd81663b9b8ec56ed9e0670cf71fc851e4 RTC: 153386 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/28482 Reviewed-by: Christopher J. Engel <cjengel@us.ibm.com> Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* Support for TPM Required attribute to allow system to IPL without a TPMChris Engel2016-07-272-13/+22
| | | | | | | | | | | Change-Id: I53e841036dfff75c6ed7d04ee55292b1285a6bee RTC: 125287 ForwardPort: yes Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/27454 Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com> Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com> Reviewed-by: Christopher J. Engel <cjengel@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com>
* Add explicit delays to centaur sensor cache enable and disableDan Crowell2016-06-171-1/+13
| | | | | | | | | | | | | | | | | | | | | | We discovered that if multiple Centaur i2c operations are running while the chip is trying to read the dimm temps that we will eventually time out the OCC due to collisions. TO fix this we need to add explicit delays between enable->disable (30ms) and also between disable->enable (2ms) to allow the sensor cache to get valid data. Change-Id: I16c96482f4827097d9af7b1e08dcd42375ae8d30 CQ: SW354148 ForwardPort: yes Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/25723 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Matthew A. Ploetz <maploetz@us.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/25982
* Trustedboot add TPM and associated i2c master to the devtreeChris Engel2016-06-082-3/+43
| | | | | | | | | | | Change-Id: Ic2edee549d23669f046a6e78f0cfae838faaec2d RTC: 125287 ForwardPort: yes Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/25470 Reviewed-by: Marshall J. Wilks <mjwilks@us.ibm.com> Tested-by: Jenkins Server Tested-by: FSP CI Jenkins Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* TPM default attributes for nimbus modelChris Engel2016-06-071-1/+1
| | | | | | | | | | | | | | Enable trustedboot support by default for P9 Change-Id: Ibe5e4428100225d78ee9d107da3769ab35578c2b RTC: 125287 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/25187 Tested-by: Jenkins Server Tested-by: FSP CI Jenkins Reviewed-by: Timothy R. Block <block@us.ibm.com> Reviewed-by: Stephen M. Cprek <smcprek@us.ibm.com> Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Fix error log memory leaks in tpmddChris Engel2016-05-271-5/+14
| | | | | | | | | | | | | | | | Change-Id: I03bf0184affa6020697403aa0a64f8dd0f229220 RTC: 125287 ForwardPort: yes Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/25002 Tested-by: Jenkins Server Reviewed-by: Nicholas E. Bofferding <bofferdn@us.ibm.com> Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Stephen M. Cprek <smcprek@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/25097 Reviewed-by: Christopher J. Engel <cjengel@us.ibm.com>
* P9 i2c master supportChris Engel2016-05-171-8/+10
| | | | | | | | | | | | | | | Updated i2c master code to support 4 engines in P9 and scom offset changes Updated attributes to support 4 engines and to match existing simics models Change-Id: I29aeb98274407777a59f11404f45d08ecadbeab2 RTC: 153481 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/24211 Tested-by: Jenkins Server Tested-by: FSP CI Jenkins Reviewed-by: Andres A. Lugo-Reyes <aalugore@us.ibm.com> Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Fix memory leak by committing error log in eepromPresence()Andres Lugo-Reyes2016-05-031-0/+6
| | | | | | | | | | | | | | | Change-Id: I487276ddd7a15f61f48a69a239d61beacfe4886f ForwardPort: yes Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/23852 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Matthew A. Ploetz <maploetz@us.ibm.com> Reviewed-by: Prachi Gupta <pragupta@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/24030
* Optimize EEPROM page switching to remember current pageAndres Lugo-Reyes2016-04-211-22/+121
| | | | | | | | | | Change-Id: I8355693b0ee16dfbaef5361006b564c4b59b714d RTC:147385 ForwardPort: yes Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/22859 Tested-by: Jenkins Server Tested-by: FSP CI Jenkins Reviewed-by: Matthew A. Ploetz <maploetz@us.ibm.com>
* DDR4 - Allow SPD writesaalugore2016-04-214-331/+1370
| | | | | | | | | | | | | | | | | | | -DDR4 has 512-byte EEPROM with 2 256-byte pages. This commit contains the necessary page switching logic to support this. Change-Id: Iaa8e3e344def98b71d6a9e9387c5e0d9137a0397 RTC:137707 ForwardPort: yes Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/797 Tested-by: Jenkins Server Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Tested-by: Jenkins OP Build CI Reviewed-by: William G. Hoffa <wghoffa@us.ibm.com> Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Matthew A. Ploetz <maploetz@us.ibm.com> Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/871
* Re-enable functionality turned off during p9 bringupcrgeddes2016-03-031-2/+2
| | | | | | | | Change-Id: I39e1d7fa7854be7425831e657c82a4a3ebdeebf2 RTC:137627 Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/777 Tested-by: Jenkins Server Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
* Establish a working P9 Hostboot and Simics baseDean Sanner2015-12-111-0/+3
| | | | | | | | | | | | | | | Includes changes for nimbus.por Making recent Simics usable by Hostboot Removing portions of code not yet ready Basic LPC read/write Change-Id: Ic40a9613934fab7bb6a28a8100685496246bb5ea RTC:132170 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/21931 Tested-by: Jenkins Server Reviewed-by: WILLIAM G. HOFFA <wghoffa@us.ibm.com> Reviewed-by: Christian Geddes <crgeddes@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Trustedboot: Additional commands in trusted boot init sequenceChris Engel2015-12-113-164/+24
| | | | | | | | | | | | | | - getCap FW Version - TPM Command marshal/unmarshal code Change-Id: Ia9a90b1160c9c3b5d818318771bff21eb013bdf4 RTC: 125287 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/20056 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Reviewed-by: STEPHEN M. CPREK <smcprek@us.ibm.com> Reviewed-by: Timothy R. Block <block@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Trustedboot: Remove TPM 1.2 supportChris Engel2015-11-194-158/+6
| | | | | | | | | | | | Change-Id: I2a8635f901e1473ffb81e92861a4b25a5d479095 RTC:125288 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/22088 Tested-by: Jenkins Server Reviewed-by: Timothy R. Block <block@us.ibm.com> Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Trustedboot: Move data to Hostboot base and add pcrExtend stub functionChris Engel2015-11-181-11/+0
| | | | | | | | | | | | Change-Id: Id488af88c7f1796606434a062a9001f31f413ac0 RTC: 125288 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/21475 Tested-by: Jenkins Server Reviewed-by: Timothy R. Block <block@us.ibm.com> Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Trusted boot support for performing TPM_STARTUP during istepsChris Engel2015-11-054-133/+336
| | | | | | | | | | | | | | | | | Support added for TPM DD2.0 Added call to host_update_master_tpm during host_discover_targets istep host_update_master_tpm istep performs TPM_STARTUP on TPM Change-Id: Ie9c232ed6ecf72da58c40df726fe1deaec5af053 RTC: 125287 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/18075 Tested-by: Jenkins Server Reviewed-by: STEPHEN M. CPREK <smcprek@us.ibm.com> Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Timothy R. Block <block@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* I2C Error Log User Data ImprovementMike Baiocchi2015-09-103-60/+180
| | | | | | | | | | | | | | | | | This commit adds more information into the two uint64_t user data sections for certain error logs, including I2C Master HUID, engine, port, and bus speed. Change-Id: Ia2c91795c22619931c37b52fa4ff2091cb5d5ef3 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/20060 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Tested-by: FSP CI Jenkins Tested-by: Jenkins OP HW Reviewed-by: Corey V. Swenson <cswenson@us.ibm.com> Reviewed-by: WILLIAM G. HOFFA <wghoffa@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Nuvoton TPM 1.2 Device DriverChris Engel2015-08-318-5/+3422
| | | | | | | | | | | | | | This adds support for the Nuvoton TPM 1.2 in simics Change-Id: I24973cb9824e31967aae3fb97e86a631e514b2ff RTC: 125287 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/16535 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Tested-by: FSP CI Jenkins Reviewed-by: Michael Baiocchi <baiocchi@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* SW316014: INITPROC: updates to implement SBE interrupt serviceThi Tran2015-08-241-41/+36
| | | | | | | | | | | | | | CMVC-Coreq: 968951 CQ:SW316014 Change-Id: I31c015962f39342d0d1d9d575fd0c4569496918c Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/19475 Reviewed-by: Thi N. Tran <thi@us.ibm.com> Tested-by: Thi N. Tran <thi@us.ibm.com> Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/19478 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Tested-by: Jenkins OP HW Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Generate error when I2C writePageSize is zeroCorey Swenson2015-07-171-0/+26
| | | | | | | | | | Change-Id: Iabfcc2ae99af78c392b5e2edaeec26150fe27a05 RTC:123567 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/18604 Tested-by: Jenkins Server Tested-by: Jenkins OP Build CI Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Remove engine1 from the I2C Master list for Centaur in devtreeMike Baiocchi2015-07-021-0/+10
| | | | | | | | | | | | | | | While the processor and centaur I2C code is common, the centaur only has one I2c master/engine (engine 0), where as the processor has two. This commit removes engine 1 from the I2C Master list for Centaurs in the devtree. Change-Id: Icd4da919a126372949d30aa4b07d7b87ecf151d4 RTC: 129126 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/18046 Tested-by: Jenkins Server Reviewed-by: William H. Schwartz <whs@us.ibm.com> Reviewed-by: Corey V. Swenson <cswenson@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Missing I2C Devices for Centaur in devtreeBill Schwartz2015-06-081-1/+1
| | | | | | | | | | | Fix the missing busses and devices in the Centaur devtree Change-Id: Ic973c22de8791fb827bf3749fbfa1cc9357acdcb RTC: 127529 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/18094 Tested-by: Jenkins Server Reviewed-by: Michael Baiocchi <baiocchi@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Add support for backplane VPDBill Schwartz2015-05-211-0/+5
| | | | | | | | | | | Replace the temporary use of mem buf to access planar vpd with new planar vpd interfaces. Change-Id: I24cda4d713806330a9f61d588006d63966f92550 RTC: 118373 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/16326 Tested-by: Jenkins Server Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Improve I2C Master Setup By Checking Master StateMike Baiocchi2015-05-121-1/+64
| | | | | | | | | | | | | | | Before setting up the I2C Masters for PHYP there is now a check that the engine is in a good state. If it is in a bad state, an I2C reset is issued in an attempt to recover the master before setting it up. Change-Id: Ib023366fc416bb9d5de9fcd0b885c879f22b918f CQ: SW295172 Backport: release-fips830 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/17524 Tested-by: Jenkins Server Reviewed-by: STEPHEN M. CPREK <smcprek@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Disable OCC Sensor Cache during Centaur I2C access at RuntimeCorey Swenson2015-05-012-4/+137
| | | | | | | | | Change-Id: I4520421f3d3e1c7cb7f58874bd4076acced7ddd3 RTC: 125538 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/16653 Tested-by: Jenkins Server Reviewed-by: Michael Baiocchi <baiocchi@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
* Allow I2C Resets on Processor Engine 1Mike Baiocchi2015-04-241-10/+1
| | | | | | | | | | | | | | | | | | This commit removes a workaround that limited all I2C resets to just engine 0 on processors. (Centaurs only use engine 0). Now, if there is a device off of one of the buses connected to engine 1 (port 0, 1, or 2), then a reset will be issued on the bus. Change-Id: Id2091b44d931be5321420c091fe91a803cc5df65 CQ: SW300339 Backport: release-fips830 CMVC-Prereq: 955865 Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/17309 Tested-by: Jenkins Server Reviewed-by: STEPHEN M. CPREK <smcprek@us.ibm.com> Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com> Reviewed-by: WILLIAM G. HOFFA <wghoffa@us.ibm.com> Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>
OpenPOWER on IntegriCloud