diff options
author | Jaymes Wilks <mjwilks@us.ibm.com> | 2018-07-19 12:57:03 -0500 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2018-08-13 10:40:27 -0500 |
commit | 8b8b8de8870034b121aad1aa1f571bb9e3d2dbc5 (patch) | |
tree | e91d5c057b9c4776f96a189511929e39273e1b1c /src/usr/i2c/i2c.C | |
parent | ce0ea22d0d9d93b1bf1cc491d93f7907c05185dc (diff) | |
download | talos-hostboot-8b8b8de8870034b121aad1aa1f571bb9e3d2dbc5.tar.gz talos-hostboot-8b8b8de8870034b121aad1aa1f571bb9e3d2dbc5.zip |
Implement generic i2c device callouts for FSP
Creates a new type of callout for i2c devices which populates
the error log user details section with the address, port, engine,
and i2c master of the device being called out. The change only
affects the FSP path.
Change-Id: I92acc092f105ee21346594f5e3733957e15ad768
RTC:94872
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/63223
Reviewed-by: Michael Baiocchi <mbaiocch@us.ibm.com>
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: 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: MURULIDHAR NATARAJU <murulidhar@in.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/usr/i2c/i2c.C')
-rwxr-xr-x | src/usr/i2c/i2c.C | 18 |
1 files changed, 18 insertions, 0 deletions
diff --git a/src/usr/i2c/i2c.C b/src/usr/i2c/i2c.C index db93bb2d7..4045ab26c 100755 --- a/src/usr/i2c/i2c.C +++ b/src/usr/i2c/i2c.C @@ -56,6 +56,8 @@ #include <i2c/eepromif.H> #include <i2c/tpmddif.H> +// TODO RTC 94872 Remove the following include in a future commit +#include <initservice/initserviceif.H> // ---------------------------------------------- // Globals @@ -4442,6 +4444,22 @@ void addHwCalloutsI2c(errlHndl_t i_err, break; } } + // For the FSP, non-TPM case add an I2c Device callout. + // TODO RTC 94872 In a future commit, we will want FSP to handle + // the TPM case also, but we won't add it here until FSP fully + // supports the new i2c callout type. The reasoning is that we still need + // TPMs to be called out as before in the interim. The other i2c devices + // (that were never called out to begin with) are the only ones being added + // as the new type, because FSP will treat the new type as unknown until + // support for the new type is added. + if (l_devFound == false && INITSERVICE::spBaseServicesEnabled()) + { + i_err->addI2cDeviceCallout(i_target, + i_args.engine, + i_args.port, + i_args.devAddr, + HWAS::SRCI_PRIORITY_HIGH); + } // Could also be an issue with Processor or its bus // -- both on the same FRU |