L3 draminit and mss_lib

Change-Id: If5cae63291864da0b87d6a1e82407da9358d62d2
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/43121
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Louis Stermole <stermole@us.ibm.com>
Reviewed-by: STEPHEN GLANCY <sglancy@us.ibm.com>
Reviewed-by: ANDRE A. MARIN <aamarin@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://ralgit01.raleigh.ibm.com/gerrit1/43278
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>

1 files changed, 18 insertions, 10 deletions

diff --git a/src/import/chips/p9/procedures/hwp/memory/lib/dimm/bcw_load.H b/src/import/chips/p9/procedures/hwp/memory/lib/dimm/bcw_load.H
index 6c94019f3..be5f1bfc5 100644
--- a/src/import/chips/p9/procedures/hwp/memory/lib/dimm/bcw_load.H
+++ b/src/import/chips/p9/procedures/hwp/memory/lib/dimm/bcw_load.H
@@ -28,9 +28,9 @@
 /// @brief Code to support bcw_loads
 ///
 // *HWP HWP Owner: Andre Marin <aamarin@us.ibm.com>
-// *HWP HWP Backup: Brian Silver <bsilver@us.ibm.com>
+// *HWP HWP Backup: Jacob Harvey <jlharvey@us.ibm.com>
 // *HWP Team: Memory
-// *HWP Level: 2
+// *HWP Level: 3
 // *HWP Consumed by: HB:FSP
 
 #ifndef _MSS_BCW_LOAD_H_
@@ -96,12 +96,12 @@ struct perform_bcw_load_overload< KIND_LRDIMM_DDR4 >
     static constexpr bool available = true;
 };
 
-///
-///  Define the default case for overloaded calls. enable_if states that
-/// if there is a DEFAULT_KIND overload for this TargetType, then this
-/// entry point will be defined. Note the general case below is enabled if
-/// there is no overload defined for this TargetType
-///
+//
+// Define the default case for overloaded calls. enable_if states that
+// if there is a DEFAULT_KIND overload for this TargetType, then this
+// entry point will be defined. Note the general case below is enabled if
+// there is no overload defined for this TargetType
+//
 
 ///
 /// @brief Perform the bcw_load operations
@@ -149,8 +149,9 @@ fapi2::ReturnCode perform_bcw_load<DEFAULT_KIND>( const fapi2::Target<fapi2::TAR
 /// @brief Perform the bcw_load operations
 /// @tparam K the kind of DIMM we're operating on (derived)
 /// @tparam B boolean that enables API from K dimm kind
+/// @param[in] i_kind the dimm kind struct for the i_target
 /// @param[in] i_target, a fapi2::Target<fapi2::TARGET_TYPE_DIMM>
-/// @param[in,out] a vector of CCS instructions we should add to
+/// @param[in,out] io_inst a vector of CCS instructions we should add to
 /// @return FAPI2_RC_SUCCESS if and only if ok
 ///
 template< kind_t K, bool B = perform_bcw_load_overload<K>::available >
@@ -170,7 +171,14 @@ inline fapi2::ReturnCode perform_bcw_load_dispatch( const kind_t& i_kind,
     return perform_bcw_load<K>(i_target, io_inst);
 }
 
-// DEFAULT_KIND is 0 so this is the end of the recursion
+///
+/// @brief Perform the bcw_load operations
+/// @param[in] i_kind the dimm kind struct for the i_target
+/// @param[in] i_target, a fapi2::Target<fapi2::TARGET_TYPE_DIMM>
+/// @param[in,out] io_inst a vector of CCS instructions we should add to
+/// @return FAPI2_RC_SUCCESS if and only if ok
+/// @note DEFAULT_KIND is 0 so this is the end of the recursion
+///
 template<>
 inline fapi2::ReturnCode perform_bcw_load_dispatch<DEFAULT_KIND>(const kind_t&,
         const fapi2::Target<fapi2::TARGET_TYPE_DIMM>& i_target,