SW261374: Intermittent (DMIFIR[12]) Too Many Bus Errors (HWP -1)

CQ:SW261374
Change-Id: Ie789fe4eead51d13320c663027482b8925d932ba
Reviewed-on: http://gfw160.aus.stglabs.ibm.com:8080/gerrit/11808
Tested-by: Jenkins Server
Reviewed-by: A. Patrick Williams III <iawillia@us.ibm.com>

1 files changed, 14 insertions, 8 deletions

diff --git a/src/usr/hwpf/hwp/bus_training/io_cleanup.H b/src/usr/hwpf/hwp/bus_training/io_cleanup.H
index 214060283..bfae99447 100644
--- a/src/usr/hwpf/hwp/bus_training/io_cleanup.H
+++ b/src/usr/hwpf/hwp/bus_training/io_cleanup.H
@@ -21,35 +21,41 @@
 /*                                                                        */
 /* IBM_PROLOG_END_TAG                                                     */
 

-//$Id: io_cleanup.H,v 1.2 2013/12/10 09:22:24 varkeykv Exp $

+//$Id: io_cleanup.H,v 1.3 2014/06/06 19:43:49 steffen Exp $

 #ifndef IO_CLEANUP_H_

 #define IO_CLEANUP_H_

 

 #include <fapi.H>

+#include <p8_scom_addresses.H>

 #include "io_clear_firs.H"

 

 using namespace fapi;

 

+	CONST_UINT64_T( CENTAUR_MBI_CFG_0x0201080A                     , ULL(0x0201080A) );

+	CONST_UINT64_T( CENTAUR_MBI_FIR_0x02010800                     , ULL(0x02010800) );

+	CONST_UINT64_T( CENTAUR_MBI_CRCSYN_0x0201080C                  , ULL(0x0201080C) );

+	CONST_UINT64_T( CENTAUR_CEN_DMIFIR_0x02010400                  , ULL(0x02010400) );

+

 /**

- * io_cleanup func pointer Typedef for hostboot 

+ * io_cleanup func pointer Typedef for hostboot

  *

  */

-typedef fapi::ReturnCode (*io_cleanup_FP_t)(const fapi::Target &,const fapi::Target &);

+typedef fapi::ReturnCode (*io_cleanup_FP_t)(const fapi::Target &);

 

 extern "C"

 {

 

 /**

- * io_cleanup 

+ * io_cleanup

  *

- * master_target is the master side of a bus ..p8.mcs in a DMI .. or a p8.abus/p8.xbus in fabric 

+ * master_target is the master side of a bus ..p8.mcs in a DMI .. or a p8.abus/p8.xbus in fabric

  * slave_target - slave side of the bus .. Centaur in DMI , p8.xbus or p8.abus for fabric

- * while these are called master or slave... I actually do a check in the code to see 

+ * while these are called master or slave... I actually do a check in the code to see

  * whether these are actually master chips by reading a GCR master_mode bit

- * and accordingly will perform a target swap if required 

+ * and accordingly will perform a target swap if required

  * @return ReturnCode

  */

-fapi::ReturnCode io_cleanup(const fapi::Target &master_target,const fapi::Target & slave_target);

+fapi::ReturnCode io_cleanup(const fapi::Target &master_target);

 

 } // extern "C"