UefiCpuPkg/RegisterCpuFeaturesLib: Fix ECC issues.

Changes include:
1. Remove extra white space at the end of line.
2. Add comments for the new add function parameter.
3. Update IN OUT tag for function parameter.

Cc: Dandan Bi <dandan.bi@intel.com>
Cc: Ruiyu Ni <ruiyu.ni@intel.com>
Cc: Laszlo Ersek <lersek@redhat.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Eric Dong <eric.dong@intel.com>
Reviewed-by: Ruiyu Ni <ruiyu.ni@intel.com>

diff --git a/UefiCpuPkg/Library/RegisterCpuFeaturesLib/PeiRegisterCpuFeaturesLib.c b/UefiCpuPkg/Library/RegisterCpuFeaturesLib/PeiRegisterCpuFeaturesLib.c
index a688e03152bd07a75b671d65a9a2f5772e85527c..4bab2837cc68c38dbd0d8fa980e09af023a0bddd 100644 (file)
--- a/UefiCpuPkg/Library/RegisterCpuFeaturesLib/PeiRegisterCpuFeaturesLib.c
+++ b/UefiCpuPkg/Library/RegisterCpuFeaturesLib/PeiRegisterCpuFeaturesLib.c
@@ -146,6 +146,8 @@ GetProcessorInformation (
 \r
   @param[in]  Procedure               A pointer to the function to be run on\r
                                       enabled APs of the system.\r
+  @param[in]  MpEvent                 The Event used to sync the result.\r
+\r
 **/\r
 VOID\r
 StartupAPsWorker (\r

diff --git a/UefiCpuPkg/Library/RegisterCpuFeaturesLib/RegisterCpuFeaturesLib.c b/UefiCpuPkg/Library/RegisterCpuFeaturesLib/RegisterCpuFeaturesLib.c
index 5b49bc450494cca3815894525b74192664bb8ff1..b6e108b8ad143ecaaaee3b2d9bd7537b3edb4904 100644 (file)
--- a/UefiCpuPkg/Library/RegisterCpuFeaturesLib/RegisterCpuFeaturesLib.c
+++ b/UefiCpuPkg/Library/RegisterCpuFeaturesLib/RegisterCpuFeaturesLib.c
@@ -160,7 +160,7 @@ DetectFeatureScope (
 /**\r
   Clear dependence for the specified type.\r
 \r
-  @param[in]  CurrentFeature     Cpu feature need to clear.\r
+  @param[in]  CpuFeature         Cpu feature need to clear.\r
   @param[in]  Before             Before or after dependence relationship.\r
 \r
 **/\r
@@ -202,14 +202,14 @@ ClearFeatureScope (
 /**\r
   Base on dependence relationship to asjust feature dependence.\r
 \r
-  ONLY when the feature before(or after) the find feature also has \r
+  ONLY when the feature before(or after) the find feature also has\r
   dependence with the find feature. In this case, driver need to base\r
   on dependce relationship to decide how to insert current feature and\r
   adjust the feature dependence.\r
 \r
-  @param[in]  PreviousFeature    CPU feature current before the find one.\r
-  @param[in]  CurrentFeature     Cpu feature need to adjust.\r
-  @param[in]  Before             Before or after dependence relationship.\r
+  @param[in, out]  PreviousFeature    CPU feature current before the find one.\r
+  @param[in, out]  CurrentFeature     Cpu feature need to adjust.\r
+  @param[in]       Before             Before or after dependence relationship.\r
 \r
   @retval   TRUE   means the current feature dependence has been adjusted.\r
 \r
@@ -239,7 +239,7 @@ AdjustFeaturesDependence (
   }\r
 \r
   //\r
-  // If both feature have dependence, keep the one which needs use more \r
+  // If both feature have dependence, keep the one which needs use more\r
   // processors and clear the dependence for the other one.\r
   //\r
   if (PreDependType >= CurrentDependType) {\r
@@ -254,10 +254,10 @@ AdjustFeaturesDependence (
 /**\r
   Base on dependence relationship to asjust feature order.\r
 \r
-  @param[in]  FeatureList        Pointer to CPU feature list\r
-  @param[in]  FindEntry          The entry this feature depend on.\r
-  @param[in]  CurrentEntry       The entry for this feature.\r
-  @param[in]  Before             Before or after dependence relationship.\r
+  @param[in]       FeatureList        Pointer to CPU feature list\r
+  @param[in, out]  FindEntry          The entry this feature depend on.\r
+  @param[in, out]  CurrentEntry       The entry for this feature.\r
+  @param[in]       Before             Before or after dependence relationship.\r
 \r
 **/\r
 VOID\r
@@ -279,8 +279,8 @@ AdjustEntry (
   // base on dependence type of feature A and B to update the logic here.\r
   // For example, feature A has package type dependence and feature B has core type dependence,\r
   // because package type dependence need to wait for more processors which has strong dependence\r
-  // than core type dependence. So driver will adjust the feature order to B -> A -> C. and driver \r
-  // will remove the feature dependence in feature B. \r
+  // than core type dependence. So driver will adjust the feature order to B -> A -> C. and driver\r
+  // will remove the feature dependence in feature B.\r
   // Driver just needs to make sure before feature C been executed, feature A has finished its task\r
   // in all all thread. Feature A finished in all threads also means feature B have finshed in all\r
   // threads.\r