HWPF: Add function pointer to HWP header files

Change-Id: I61cd4ac09b19ce585769b52a0657bb19de65ad3c
Reviewed-on: http://gfw160.austin.ibm.com:8080/gerrit/438
Tested-by: Jenkins Server
Reviewed-by: MIKE J. JONES <mjjones@us.ibm.com>

7 files changed, 86 insertions, 27 deletions

diff --git a/src/include/usr/hwpf/hwp/fapiTestHwp.H b/src/include/usr/hwpf/hwp/fapiTestHwp.H
index eb7e47cda..12349e737 100644
--- a/src/include/usr/hwpf/hwp/fapiTestHwp.H
+++ b/src/include/usr/hwpf/hwp/fapiTestHwp.H
@@ -33,16 +33,16 @@
  *                          mjjones     04/21/2011  Created.
  *                          mjjones     06/28/2011  Updated comment
  *                          mjjones     08/11/2011  Removed Clock HWP
- *
+ *                          mjjones     10/17/2011  Added func pointer
  */
 
-#ifndef FAPITESTHWPROC_H_
-#define FAPITESTHWPROC_H_
+#ifndef FAPITESTHW_H_
+#define FAPITESTHW_H_
 
 #include <fapi.H>
 
-// HWPs are defined as C functions because platforms may wish to package them
-// in linux shared libraries which are DL-Opened
+typedef fapi::ReturnCode (*hwpInitialTest_FP_t)(const fapi::Target &);
+
 extern "C"
 {
 
@@ -55,8 +55,6 @@ extern "C"
  */
 fapi::ReturnCode hwpInitialTest(const fapi::Target & i_chip);
 
+}
 
-
-} // extern "C"
-
-#endif // FAPITESTHWPROC_H_
+#endif
diff --git a/src/include/usr/hwpf/hwp/fapiTestHwpAnalyzeError.H b/src/include/usr/hwpf/hwp/fapiTestHwpAnalyzeError.H
new file mode 100644
index 000000000..85bb19975
--- /dev/null
+++ b/src/include/usr/hwpf/hwp/fapiTestHwpAnalyzeError.H
@@ -0,0 +1,59 @@
+//  IBM_PROLOG_BEGIN_TAG
+//  This is an automatically generated prolog.
+//
+//  $Source: src/include/usr/hwpf/hwp/fapiTestHwpAnalyzeError.H $
+//
+//  IBM CONFIDENTIAL
+//
+//  COPYRIGHT International Business Machines Corp. 2011
+//
+//  p1
+//
+//  Object Code Only (OCO) source materials
+//  Licensed Internal Code Source Materials
+//  IBM HostBoot Licensed Internal Code
+//
+//  The source code for this program is not published or other-
+//  wise divested of its trade secrets, irrespective of what has
+//  been deposited with the U.S. Copyright Office.
+//
+//  Origin: 30
+//
+//  IBM_PROLOG_END
+/**
+ *  @file fapiTestHwpAnalyzeError.H
+ *
+ *  @brief Defines a simple test Hardware Procedure that returns an error
+ */
+
+/*
+ * Change Log ******************************************************************
+ * Flag     Defect/Feature  User        Date        Description
+ * ------   --------------  ----------  ----------- ----------------------------
+ *                          mjjones     08/08/2011  Created. Moved from other file
+ *
+ */
+
+#ifndef FAPITESTHWPANALYZEERROR_H_
+#define FAPITESTHWPANALYZEERROR_H_
+
+#include <fapi.H>
+
+typedef fapi::ReturnCode (*hwpTestAnalyzeError_FP_t)(const fapi::Target &);
+
+extern "C"
+{
+
+/**
+ * @brief Simple HWP that analyzes an error and returns an error describing the
+ *        problem (RC_TEST_ERROR_B)
+ *
+ * @param[in] i_target Reference to target (unused by HWP)
+ *
+ * @return ReturnCode
+ */
+fapi::ReturnCode hwpTestAnalyzeError(const fapi::Target & i_target);
+
+}
+
+#endif
diff --git a/src/include/usr/hwpf/hwp/fapiTestHwpAttr.H b/src/include/usr/hwpf/hwp/fapiTestHwpAttr.H
index cdf7e6afb..1bb9481b6 100755
--- a/src/include/usr/hwpf/hwp/fapiTestHwpAttr.H
+++ b/src/include/usr/hwpf/hwp/fapiTestHwpAttr.H
@@ -32,6 +32,7 @@
  * ------   --------------  ----------  ----------- ----------------------------
  *                          mjjones     06/30/2011  Created.
  *                          mjjones     10/07/2011  Removed target param
+ *                          mjjones     10/17/2011  Added func pointer
  */
 
 #ifndef FAPITESTHWPATTR_H_
@@ -39,8 +40,8 @@
 
 #include <fapi.H>
 
-// HWPs are defined as C functions because platforms may wish to package them
-// in linux shared libraries which are DL-Opened
+typedef fapi::ReturnCode (*hwpTestAttributes_FP_t)();
+
 extern "C"
 {
 
@@ -51,6 +52,6 @@ extern "C"
  */
 fapi::ReturnCode hwpTestAttributes();
 
-} // extern "C"
+}
 
-#endif // FAPITESTHWPATTR_H_
+#endif
diff --git a/src/include/usr/hwpf/hwp/fapiTestHwpConfig.H b/src/include/usr/hwpf/hwp/fapiTestHwpConfig.H
index 4faec30ef..fd2fb4be2 100644
--- a/src/include/usr/hwpf/hwp/fapiTestHwpConfig.H
+++ b/src/include/usr/hwpf/hwp/fapiTestHwpConfig.H
@@ -32,7 +32,7 @@
  * Flag     Defect/Feature  User        Date        Description
  * ------   --------------  ----------  ----------- ----------------------------
  *                          mjjones     09/12/2011  Created.
- *
+ *                          mjjones     10/17/2011  Added func pointer
  */
 
 #ifndef FAPITESTHWCONFIG_H_
@@ -40,8 +40,8 @@
 
 #include <fapi.H>
 
-// HWPs are defined as C functions because platforms may wish to package them
-// in linux shared libraries which are DL-Opened
+typedef fapi::ReturnCode (*hwpTestConfig_FP_t)(const fapi::Target &);
+
 extern "C"
 {
 
@@ -54,6 +54,6 @@ extern "C"
  */
 fapi::ReturnCode hwpTestConfig(const fapi::Target & i_chip);
 
-} // extern "C"
+}
 
 #endif
diff --git a/src/include/usr/hwpf/hwp/fapiTestHwpError.H b/src/include/usr/hwpf/hwp/fapiTestHwpError.H
index 4a6bcdd2e..2c18ba0ec 100644
--- a/src/include/usr/hwpf/hwp/fapiTestHwpError.H
+++ b/src/include/usr/hwpf/hwp/fapiTestHwpError.H
@@ -31,7 +31,7 @@
  * Flag     Defect/Feature  User        Date        Description
  * ------   --------------  ----------  ----------- ----------------------------
  *                          mjjones     08/08/2011  Created.
- *
+ *                          mjjones     10/17/2011  Added func pointer
  */
 
 #ifndef FAPITESTHWPERROR_H_
@@ -39,8 +39,8 @@
 
 #include <fapi.H>
 
-// HWPs are defined as C functions because platforms may wish to package them
-// in linux shared libraries which are DL-Opened
+typedef fapi::ReturnCode (*hwpTestError_FP_t)(const fapi::Target &);
+
 extern "C"
 {
 
@@ -53,6 +53,6 @@ extern "C"
  */
 fapi::ReturnCode hwpTestError(const fapi::Target & i_target);
 
-} // extern "C"
+}
 
-#endif // FAPITESTHWPERROR_H_
+#endif
diff --git a/src/include/usr/hwpf/hwp/fapiTestHwpFfdc.H b/src/include/usr/hwpf/hwp/fapiTestHwpFfdc.H
index 84620b133..cde3042b7 100644
--- a/src/include/usr/hwpf/hwp/fapiTestHwpFfdc.H
+++ b/src/include/usr/hwpf/hwp/fapiTestHwpFfdc.H
@@ -31,7 +31,7 @@
  * Flag     Defect/Feature  User        Date        Description
  * ------   --------------  ----------  ----------- ----------------------------
  *                          mjjones     08/08/2011  Created.
- *
+ *                          mjjones     10/17/2011  Added func pointer
  */
 
 #ifndef FAPITESTHWPFFDC_H_
@@ -39,8 +39,8 @@
 
 #include <fapi.H>
 
-// HWPs are defined as C functions because platforms may wish to package them
-// in linux shared libraries which are DL-Opened
+typedef fapi::ReturnCode (*hwpTestFfdc1_FP_t)(const fapi::Target &, fapi::ReturnCode &);
+
 extern "C"
 {
 
@@ -55,6 +55,6 @@ extern "C"
 fapi::ReturnCode hwpTestFfdc1(const fapi::Target & i_target,
                               fapi::ReturnCode & o_rc);
 
-} // extern "C"
+}
 
-#endif // FAPITESTHWPFFDC_H_
+#endif
diff --git a/src/include/usr/hwpf/plat/fapiPlatHwpExecutor.H b/src/include/usr/hwpf/plat/fapiPlatHwpExecutor.H
index aeef2a485..b3441ea9b 100644
--- a/src/include/usr/hwpf/plat/fapiPlatHwpExecutor.H
+++ b/src/include/usr/hwpf/plat/fapiPlatHwpExecutor.H
@@ -36,6 +36,7 @@
 
 #include <fapiTestHwp.H>
 #include <fapiTestHwpError.H>
+#include <fapiTestHwpAnalyzeError.H>
 #include <fapiTestHwpFfdc.H>
 #include <fapiTestHwpConfig.H>
 #include <fapiTestHwpAttr.H>