Refine new library instances according to review comments.

[mirror_edk2.git] / MdePkg / Library / UefiSalLib / UefiSalLib.c

diff --git a/MdePkg/Library/UefiSalLib/UefiSalLib.c b/MdePkg/Library/UefiSalLib/UefiSalLib.c
index e41cc0d980cb2458b12d11aece7b0eb9f87f695c..f4319ed11c55f3963e46b596acfd4a5327f754fe 100644 (file)
--- a/MdePkg/Library/UefiSalLib/UefiSalLib.c
+++ b/MdePkg/Library/UefiSalLib/UefiSalLib.c
@@ -1,5 +1,6 @@
 /** @file\r
-  SAL Library implementation built upon UEFI.\r
+  SAL Library implementation retrieving the SAL Entry Point from the SAL System Table\r
+  register in the EFI System Confguration Table.\r
 \r
   Copyright (c) 2007 - 2008, Intel Corporation All rights\r
   reserved. This program and the accompanying materials are\r
@@ -85,7 +86,7 @@ SalCall (
 \r
   The constructor function looks up the SAL System Table in the EFI System Configuration\r
   Table. Once the SAL System Table is found, the SAL Entry Point in the SAL System Table\r
-  will be derived and stored inot a global variable for library usage.\r
+  will be derived and stored into a global variable for library usage.\r
   It will ASSERT() if the SAL System Table cannot be found or the data in the SAL System\r
   Table is not the valid data.\r
 \r
@@ -114,9 +115,9 @@ UefiSalLibConstructor (
   ASSERT_EFI_ERROR (Status);\r
 \r
   //\r
-  // Move the SAL System Table point to the first Entry\r
-  // Due to the SAL Entry is in ascending order with the Entry type,\r
-  // the type 0 Entry should be the first if exist.\r
+  // Check the first entry of SAL System Table,\r
+  // because the SAL entry is in ascending order with the entry type,\r
+  // the type 0 entry should be the first if exist.\r
   //\r
   SalStEntryDes = (SAL_ST_ENTRY_POINT_DESCRIPTOR *)(SalSystemTable + 1);\r
 \r
@@ -128,7 +129,7 @@ UefiSalLibConstructor (
   mPlabel.EntryPoint = SalStEntryDes->SalProcEntry;\r
   mPlabel.GP = SalStEntryDes->SalGlobalDataPointer;\r
   //\r
-  // Make sure the EntryPoint has the real value\r
+  // Make sure the EntryPoint has the valid value\r
   //\r
   ASSERT ((mPlabel.EntryPoint != 0) && (mPlabel.GP != 0));\r
 \r