Fix the <HelpText> for PCD used by EdkNt32Pkg and EdkModulePkg.

git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@891 6f19259b-4bc3-4df7-8a09-765794883524

diff --git a/EdkModulePkg/EdkModulePkg.spd b/EdkModulePkg/EdkModulePkg.spd
index 2d2b2a8e1cd9c23e6b8ba0aef836b03568c33bd7..fbfac345171d9382925722beaca6ca1ef934ed57 100644 (file)
--- a/EdkModulePkg/EdkModulePkg.spd
+++ b/EdkModulePkg/EdkModulePkg.spd
@@ -594,64 +594,14 @@
       <DefaultValue>0</DefaultValue>\r
       <HelpText>HELP FIX ME!</HelpText>\r
     </PcdEntry>\r
-    <PcdEntry>\r
-      <C_Name>PcdMaxPcdCallBackNumber</C_Name>\r
-      <Token>0x0001000f</Token>\r
-      <TokenSpaceGuidCName>gEfiEdkModulePkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <DatumType>UINT32</DatumType>\r
-      <ValidUsage>FIXED_AT_BUILD</ValidUsage>\r
-      <DefaultValue>0</DefaultValue>\r
-      <HelpText>HELP FIX ME!</HelpText>\r
-    </PcdEntry>\r
-    <PcdEntry>\r
-      <C_Name>PcdTestDynamicUint8</C_Name>\r
-      <Token>0x00011000</Token>\r
-      <TokenSpaceGuidCName>gEfiEdkModulePkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <DatumType>UINT8</DatumType>\r
-      <ValidUsage>DYNAMIC</ValidUsage>\r
-      <DefaultValue>0x01</DefaultValue>\r
-      <HelpText>HELP FIX ME!</HelpText>\r
-    </PcdEntry>\r
-    <PcdEntry>\r
-      <C_Name>PcdTestDynamicUint16</C_Name>\r
-      <Token>0x00011001</Token>\r
-      <TokenSpaceGuidCName>gEfiEdkModulePkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <DatumType>UINT16</DatumType>\r
-      <ValidUsage>DYNAMIC</ValidUsage>\r
-      <DefaultValue>0x1234</DefaultValue>\r
-      <HelpText>HELP FIX ME!</HelpText>\r
-    </PcdEntry>\r
-    <PcdEntry>\r
-      <C_Name>PcdTestDynamicUint32</C_Name>\r
-      <Token>0x00011002</Token>\r
-      <TokenSpaceGuidCName>gEfiEdkModulePkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <DatumType>UINT32</DatumType>\r
-      <ValidUsage>DYNAMIC</ValidUsage>\r
-      <HelpText>HELP FIX ME!</HelpText>\r
-    </PcdEntry>\r
-    <PcdEntry>\r
-      <C_Name>PcdTestDynamicUint64</C_Name>\r
-      <Token>0x00011003</Token>\r
-      <TokenSpaceGuidCName>gEfiEdkModulePkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <DatumType>UINT64</DatumType>\r
-      <ValidUsage>DYNAMIC</ValidUsage>\r
-      <HelpText>HELP FIX ME!</HelpText>\r
-    </PcdEntry>\r
-    <PcdEntry>\r
-      <C_Name>PcdTestDynamicBoolean</C_Name>\r
-      <Token>0x00011004</Token>\r
-      <TokenSpaceGuidCName>gEfiEdkModulePkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <DatumType>BOOLEAN</DatumType>\r
-      <ValidUsage>DYNAMIC</ValidUsage>\r
-      <HelpText>HELP FIX ME!</HelpText>\r
-    </PcdEntry>\r
     <PcdEntry>\r
       <C_Name>PcdMaxPeiPcdCallBackNumberPerPcdEntry</C_Name>\r
       <Token>0x0001000f</Token>\r
       <TokenSpaceGuidCName>gEfiEdkModulePkgTokenSpaceGuid</TokenSpaceGuidCName>\r
       <DatumType>UINT32</DatumType>\r
       <ValidUsage>FIXED_AT_BUILD</ValidUsage>\r
-      <HelpText>HELP FIX ME!</HelpText>\r
+      <HelpText>The maximum number of callback function, which will be triggered when \r
+        a PCD entry is been set, can be registered for a single PCD entry in PEI phase.</HelpText>\r
     </PcdEntry>\r
     <PcdEntry>\r
       <C_Name>PcdVpdBaseAddress</C_Name>\r
@@ -659,7 +609,8 @@
       <TokenSpaceGuidCName>gEfiEdkModulePkgTokenSpaceGuid</TokenSpaceGuidCName>\r
       <DatumType>UINT32</DatumType>\r
       <ValidUsage>FIXED_AT_BUILD</ValidUsage>\r
-      <HelpText>HELP FIX ME!</HelpText>\r
+      <HelpText>The base address of the VPD (Vital Product Data) region. It is \r
+        normally a region reserved on flash.</HelpText>\r
     </PcdEntry>\r
     <PcdEntry>\r
       <C_Name>PcdSupportUpdateCapsuleRest</C_Name>\r
@@ -692,7 +643,8 @@
       <DatumType>BOOLEAN</DatumType>\r
       <ValidUsage>FEATURE_FLAG</ValidUsage>\r
       <DefaultValue>TRUE</DefaultValue>\r
-      <HelpText>This feature flag can be used to enable or disable the Pcd PEIM database traverse capability. Disable it can reduce the size of final image generated.</HelpText>\r
+      <HelpText>This feature flag can be used to enable or disable the Pcd PEIM database \r
+        traverse capability. Disable it can reduce the size of final image generated.</HelpText>\r
     </PcdEntry>\r
     <PcdEntry>\r
       <C_Name>PcdDxePcdDatabaseTraverseEnabled</C_Name>\r
@@ -701,7 +653,8 @@
       <DatumType>BOOLEAN</DatumType>\r
       <ValidUsage>FEATURE_FLAG</ValidUsage>\r
       <DefaultValue>TRUE</DefaultValue>\r
-      <HelpText>This feature flag can be used to enable or disable the Pcd DXE database traverse capability. Disable it can reduce the size of final image generated.</HelpText>\r
+      <HelpText>This feature flag can be used to enable or disable the Pcd DXE database \r
+        traverse capability. Disable it can reduce the size of final image generated.</HelpText>\r
     </PcdEntry>\r
   </PcdDeclarations>\r
 </PackageSurfaceArea>
\ No newline at end of file

diff --git a/EdkModulePkg/Universal/PCD/Dxe/Pcd.msa b/EdkModulePkg/Universal/PCD/Dxe/Pcd.msa
index ed41cb85ce9e7c269700c7911f314b692bd1083a..c64dcf3c3d7db51e066da5ec5dc160b731f57fd7 100644 (file)
--- a/EdkModulePkg/Universal/PCD/Dxe/Pcd.msa
+++ b/EdkModulePkg/Universal/PCD/Dxe/Pcd.msa
@@ -98,12 +98,14 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.-->
     <PcdEntry PcdItemType="FIXED_AT_BUILD">\r
       <C_Name>PcdVpdBaseAddress</C_Name>\r
       <TokenSpaceGuidCName>gEfiEdkModulePkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <HelpText/>\r
+      <HelpText>The base address of the VPD (Vital Product Data) region. It is \r
+        normally a region reserved on flash.</HelpText>\r
     </PcdEntry>\r
     <PcdEntry PcdItemType="FEATURE_FLAG">\r
       <C_Name>PcdDxePcdDatabaseTraverseEnabled</C_Name>\r
       <TokenSpaceGuidCName>gEfiEdkModulePkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <HelpText/>\r
+      <HelpText>This feature flag can be used to enable or disable the Pcd DXE database \r
+        traverse capability. Disable it can reduce the size of final image generated.</HelpText>\r
     </PcdEntry>\r
   </PcdCoded>\r
 </ModuleSurfaceArea>
\ No newline at end of file

diff --git a/EdkModulePkg/Universal/PCD/Pei/Pcd.msa b/EdkModulePkg/Universal/PCD/Pei/Pcd.msa
index d169434fc378fd8c084a2b68e7b628e0b5b99ff7..1cd780170c9372b249b8d22870332ab3d69c6ed0 100644 (file)
--- a/EdkModulePkg/Universal/PCD/Pei/Pcd.msa
+++ b/EdkModulePkg/Universal/PCD/Pei/Pcd.msa
@@ -95,17 +95,20 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.-->
     <PcdEntry PcdItemType="FIXED_AT_BUILD">\r
       <C_Name>PcdMaxPeiPcdCallBackNumberPerPcdEntry</C_Name>\r
       <TokenSpaceGuidCName>gEfiEdkModulePkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <HelpText/>\r
+      <HelpText>The maximum number of callback function, which will be triggered when \r
+        a PCD entry is been set, can be registered for a single PCD entry in PEI phase.</HelpText>\r
     </PcdEntry>\r
     <PcdEntry PcdItemType="FIXED_AT_BUILD">\r
       <C_Name>PcdVpdBaseAddress</C_Name>\r
       <TokenSpaceGuidCName>gEfiEdkModulePkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <HelpText/>\r
+      <HelpText>The base address of the VPD (Vital Product Data) region. It is \r
+        normally a region reserved on flash.</HelpText>\r
     </PcdEntry>\r
     <PcdEntry PcdItemType="FEATURE_FLAG">\r
       <C_Name>PcdPeiPcdDatabaseTraverseEnabled</C_Name>\r
       <TokenSpaceGuidCName>gEfiEdkModulePkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <HelpText/>\r
+      <HelpText>This feature flag can be used to enable or disable the Pcd PEIM database \r
+        traverse capability. Disable it can reduce the size of final image generated.</HelpText>\r
     </PcdEntry>\r
   </PcdCoded>\r
 </ModuleSurfaceArea>
\ No newline at end of file

diff --git a/EdkNt32Pkg/Dxe/WinNtThunk/Bus/WinNtBusDriver/WinNtBusDriver.msa b/EdkNt32Pkg/Dxe/WinNtThunk/Bus/WinNtBusDriver/WinNtBusDriver.msa
index e09323e5bfbc285d120aa767e7c4fde6edc7c380..672dc83824056553058e86e5de8263920cc2af83 100644 (file)
--- a/EdkNt32Pkg/Dxe/WinNtThunk/Bus/WinNtBusDriver/WinNtBusDriver.msa
+++ b/EdkNt32Pkg/Dxe/WinNtThunk/Bus/WinNtBusDriver/WinNtBusDriver.msa
@@ -128,67 +128,59 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.-->
     <PcdEntry PcdItemType="DYNAMIC">\r
       <C_Name>PcdWinNtConsole</C_Name>\r
       <TokenSpaceGuidCName>gEfiEdkNt32PkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <HelpText/>\r
+      <HelpText>This PCD declares the title string of the text console window. \r
+        such as "My EFI Console".\r
+        The item type of this PCD can only be "DYNAMIC".</HelpText>\r
     </PcdEntry>\r
     <PcdEntry PcdItemType="DYNAMIC">\r
       <C_Name>PcdWinNtUga</C_Name>\r
       <TokenSpaceGuidCName>gEfiEdkNt32PkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <HelpText/>\r
+      <HelpText>This PCD declares the resolutions for the UGA windows.\r
+        The item type of this PCD can only be "DYNAMIC".</HelpText>\r
     </PcdEntry>\r
     <PcdEntry PcdItemType="DYNAMIC">\r
       <C_Name>PcdWinNtSerialPort</C_Name>\r
       <TokenSpaceGuidCName>gEfiEdkNt32PkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <HelpText/>\r
+      <HelpText>This Pcd declares two serial port for simulated environment.\r
+        The item type of this PCD can only be "DYNAMIC".</HelpText>\r
     </PcdEntry>\r
     <PcdEntry PcdItemType="DYNAMIC">\r
       <C_Name>PcdWinNtFileSystem</C_Name>\r
       <TokenSpaceGuidCName>gEfiEdkNt32PkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <HelpText/>\r
+      <HelpText>This PCD defines the windows directory who will be mounted as \r
+        harddisk in simulator.\r
+        The item type of this PCD can only be "DYNAMIC".</HelpText>\r
     </PcdEntry>\r
     <PcdEntry PcdItemType="DYNAMIC">\r
       <C_Name>PcdWinNtVirtualDisk</C_Name>\r
       <TokenSpaceGuidCName>gEfiEdkNt32PkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <HelpText/>\r
+      <HelpText>This PCD defines the file attribute which will be simulated as \r
+        a harddisk in simulator.\r
+        The item type if this PCD can only be "DYNAMIC".</HelpText>\r
     </PcdEntry>\r
     <PcdEntry PcdItemType="DYNAMIC">\r
       <C_Name>PcdWinNtPhysicalDisk</C_Name>\r
       <TokenSpaceGuidCName>gEfiEdkNt32PkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <HelpText/>\r
+      <HelpText>This PCD defines physical disk which will be simualted as a \r
+        harddisk in simulator.\r
+        The item type of this PCD can only be "DYNAMIC".</HelpText>\r
     </PcdEntry>\r
     <PcdEntry PcdItemType="DYNAMIC">\r
       <C_Name>PcdWinNtCpuModel</C_Name>\r
       <TokenSpaceGuidCName>gEfiEdkNt32PkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <HelpText/>\r
+      <HelpText>This PCD defines simulated CPU model string.\r
+        The item type of this PCD can only be "DYNAMIC".</HelpText>\r
     </PcdEntry>\r
     <PcdEntry PcdItemType="DYNAMIC">\r
       <C_Name>PcdWinNtCpuSpeed</C_Name>\r
       <TokenSpaceGuidCName>gEfiEdkNt32PkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <HelpText/>\r
+      <HelpText>This PCD defines simulated CPU speed string.</HelpText>\r
     </PcdEntry>\r
     <PcdEntry PcdItemType="DYNAMIC">\r
       <C_Name>PcdWinNtMemorySize</C_Name>\r
       <TokenSpaceGuidCName>gEfiEdkNt32PkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <HelpText/>\r
-    </PcdEntry>\r
-    <PcdEntry PcdItemType="PATCHABLE_IN_MODULE">\r
-      <C_Name>PcdWinNtBinaryPatch1</C_Name>\r
-      <TokenSpaceGuidCName>gEfiEdkNt32PkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <HelpText/>\r
-    </PcdEntry>\r
-    <PcdEntry PcdItemType="PATCHABLE_IN_MODULE">\r
-      <C_Name>PcdWinNtBinaryPatch2</C_Name>\r
-      <TokenSpaceGuidCName>gEfiEdkNt32PkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <HelpText/>\r
-    </PcdEntry>\r
-    <PcdEntry PcdItemType="FEATURE_FLAG">\r
-      <C_Name>PcdWinNtFeatureFlag1</C_Name>\r
-      <TokenSpaceGuidCName>gEfiEdkNt32PkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <HelpText/>\r
-    </PcdEntry>\r
-    <PcdEntry PcdItemType="DYNAMIC">\r
-      <C_Name>PcdWinNtDynamicUINT32</C_Name>\r
-      <TokenSpaceGuidCName>gEfiEdkNt32PkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <HelpText/>\r
+      <HelpText>This PCD defines the size of simulated memory size.\r
+        The item type of this PCD can only be "DYNAMIC".</HelpText>\r
     </PcdEntry>\r
   </PcdCoded>\r
 </ModuleSurfaceArea>
\ No newline at end of file

diff --git a/EdkNt32Pkg/EdkNt32Pkg.spd b/EdkNt32Pkg/EdkNt32Pkg.spd
index f52a1ac30fe91bb46e85455410484ecf1cc3cfce..13e446ee3e2b0b3f97e5c04ecffc29942b154513 100644 (file)
--- a/EdkNt32Pkg/EdkNt32Pkg.spd
+++ b/EdkNt32Pkg/EdkNt32Pkg.spd
@@ -266,7 +266,9 @@
       <DatumType>VOID*</DatumType>\r
       <ValidUsage>DYNAMIC</ValidUsage>\r
       <DefaultValue>L"FW;40960;512"</DefaultValue>\r
-      <HelpText>HELP FIX ME!</HelpText>\r
+      <HelpText>This PCD defines physical disk which will be simualted as a \r
+        harddisk in simulator.\r
+        The item type of this PCD can only be "DYNAMIC".</HelpText>\r
     </PcdEntry>\r
     <PcdEntry>\r
       <C_Name>PcdWinNtVirtualDisk</C_Name>\r
@@ -275,7 +277,9 @@
       <DatumType>VOID*</DatumType>\r
       <ValidUsage>DYNAMIC</ValidUsage>\r
       <DefaultValue>L"FW;40960;512"</DefaultValue>\r
-      <HelpText>HELP FIX ME!</HelpText>\r
+      <HelpText>This PCD defines the file attribute which will be simulated as \r
+        a harddisk in simulator.\r
+        The item type if this PCD can only be "DYNAMIC".</HelpText>\r
     </PcdEntry>\r
     <PcdEntry>\r
       <C_Name>PcdWinNtSerialPort</C_Name>\r
@@ -284,7 +288,8 @@
       <DatumType>VOID*</DatumType>\r
       <ValidUsage>DYNAMIC</ValidUsage>\r
       <DefaultValue>L"COM1!COM2"</DefaultValue>\r
-      <HelpText>HELP FIX ME!</HelpText>\r
+      <HelpText>This Pcd declares two serial port for simulated environment.\r
+        The item type of this PCD can only be "DYNAMIC".</HelpText>\r
     </PcdEntry>\r
     <PcdEntry>\r
       <C_Name>PcdWinNtUga</C_Name>\r
@@ -293,7 +298,8 @@
       <DatumType>VOID*</DatumType>\r
       <ValidUsage>DYNAMIC</ValidUsage>\r
       <DefaultValue>L"UGA Window 1!UGA Window 2"</DefaultValue>\r
-      <HelpText>HELP FIX ME!</HelpText>\r
+      <HelpText>This PCD declares the resolutions for the UGA windows.\r
+        The item type of this PCD can only be "DYNAMIC".</HelpText>\r
     </PcdEntry>\r
     <PcdEntry>\r
       <C_Name>PcdWinNtFileSystem</C_Name>\r
@@ -302,7 +308,9 @@
       <DatumType>VOID*</DatumType>\r
       <ValidUsage>FIXED_AT_BUILD</ValidUsage>\r
       <DefaultValue>L".!C:\\D\\work\\Remodel\\mdk\\EdkShellBinPkg\\bin\\ia32\\Apps"</DefaultValue>\r
-      <HelpText>HELP FIX ME!</HelpText>\r
+      <HelpText>This PCD defines the windows directory who will be mounted as \r
+        harddisk in simulator.\r
+        The item type of this PCD can only be "DYNAMIC".</HelpText>\r
     </PcdEntry>\r
     <PcdEntry>\r
       <C_Name>PcdWinNtMemorySize</C_Name>\r
@@ -311,7 +319,8 @@
       <DatumType>VOID*</DatumType>\r
       <ValidUsage>FIXED_AT_BUILD</ValidUsage>\r
       <DefaultValue>L"64!64"</DefaultValue>\r
-      <HelpText>HELP FIX ME!</HelpText>\r
+      <HelpText>This PCD defines the size of simulated memory size.\r
+        The item type of this PCD can only be "DYNAMIC".</HelpText>\r
     </PcdEntry>\r
     <PcdEntry>\r
       <C_Name>PcdWinNtMemorySizeForSecMain</C_Name>\r
@@ -320,7 +329,9 @@
       <DatumType>VOID*</DatumType>\r
       <ValidUsage>FIXED_AT_BUILD</ValidUsage>\r
       <DefaultValue>L"64!64"</DefaultValue>\r
-      <HelpText>HELP FIX ME!</HelpText>\r
+      <HelpText>This PCD defines the memory size of simulated machine. Simulator will allocate\r
+        the size of PcdWinNtMemorySizeForSecMain in windows platform.\r
+        The item type of this PCD can only be "FIXED_AT_BUILD".</HelpText>\r
     </PcdEntry>\r
     <PcdEntry>\r
       <C_Name>PcdWinNtBootMode</C_Name>\r
@@ -329,7 +340,20 @@
       <DatumType>UINT32</DatumType>\r
       <ValidUsage>FIXED_AT_BUILD</ValidUsage>\r
       <DefaultValue>1</DefaultValue>\r
-      <HelpText>HELP FIX ME!</HelpText>\r
+      <HelpText>This PCD defines the boot mode for simualtor. \r
+        The boot mode can be set as following value:\r
+          0x0:  Boot with full configuration.\r
+          0x1:  Boot with minimal configuration.\r
+          0x2:  Boot assume no configuration changes.\r
+          0x3:  Boot with full configuration plus diagnostics.\r
+          0x4:  Boot with default settings.\r
+          0x5:  Boot on S4 resume.\r
+          0x6:  Boot on S5 resume.\r
+          0x10: Boot on S2 resume. \r
+          0x11: Boot on S3 resume.\r
+          0x12: Boot on flash update.\r
+          0x20: Boot in reovery mode.\r
+        The item type of this PCD can only be "FIXED_AT_BUILD".</HelpText>\r
     </PcdEntry>\r
     <PcdEntry>\r
       <C_Name>PcdWinNtCpuModel</C_Name>\r
@@ -338,7 +362,8 @@
       <DatumType>VOID*</DatumType>\r
       <ValidUsage>FIXED_AT_BUILD</ValidUsage>\r
       <DefaultValue>L"Intel(R) Processor Model"</DefaultValue>\r
-      <HelpText>HELP FIX ME!</HelpText>\r
+      <HelpText>This PCD defines simulated CPU model string.\r
+        The item type of this PCD can only be "DYNAMIC".</HelpText>\r
     </PcdEntry>\r
     <PcdEntry>\r
       <C_Name>PcdWinNtCpuSpeed</C_Name>\r
@@ -347,7 +372,7 @@
       <DatumType>VOID*</DatumType>\r
       <ValidUsage>FIXED_AT_BUILD</ValidUsage>\r
       <DefaultValue>L"3000"</DefaultValue>\r
-      <HelpText>HELP FIX ME!</HelpText>\r
+      <HelpText>This PCD defines simulated CPU speed string.</HelpText>\r
     </PcdEntry>\r
     <PcdEntry>\r
       <C_Name>PcdWinNtFirmwareVolume</C_Name>\r
@@ -356,7 +381,8 @@
       <DatumType>VOID*</DatumType>\r
       <ValidUsage>FIXED_AT_BUILD</ValidUsage>\r
       <DefaultValue>L"..\\..\\Fv\\Fv_Recovery.fd"</DefaultValue>\r
-      <HelpText>HELP FIX ME!</HelpText>\r
+      <HelpText>This PCD defines the FD file windows path string. Simulator will load the FD file and execute.\r
+        The item type of this PCD can only be "FIXED_AT_BUILD".</HelpText>\r
     </PcdEntry>\r
     <PcdEntry>\r
       <C_Name>PcdWinNtConsole</C_Name>\r
@@ -365,7 +391,9 @@
       <DatumType>VOID*</DatumType>\r
       <ValidUsage>FIXED_AT_BUILD</ValidUsage>\r
       <DefaultValue>L"Bus Driver Console Window"</DefaultValue>\r
-      <HelpText>HELP FIX ME!</HelpText>\r
+      <HelpText>This PCD declares the title string of the text console window. \r
+        such as "My EFI Console".\r
+        The item type of this PCD can only be "DYNAMIC".</HelpText>\r
     </PcdEntry>\r
   </PcdDeclarations>\r
 </PackageSurfaceArea>

diff --git a/EdkNt32Pkg/Sec/SecMain.msa b/EdkNt32Pkg/Sec/SecMain.msa
index 1e7efe38d32f2b83e08a4e4348ef0dc5808fac57..23f059511d785fd14457fa004b6f21d7622ba570 100644 (file)
--- a/EdkNt32Pkg/Sec/SecMain.msa
+++ b/EdkNt32Pkg/Sec/SecMain.msa
@@ -93,17 +93,33 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.-->
     <PcdEntry PcdItemType="FIXED_AT_BUILD">\r
       <C_Name>PcdWinNtMemorySizeForSecMain</C_Name>\r
       <TokenSpaceGuidCName>gEfiEdkNt32PkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <HelpText/>\r
+      <HelpText>This PCD defines the memory size of simulated machine. Simulator will allocate\r
+        the size of PcdWinNtMemorySizeForSecMain in windows platform.\r
+        The item type of this PCD can only be "FIXED_AT_BUILD".</HelpText>\r
     </PcdEntry>\r
     <PcdEntry PcdItemType="FIXED_AT_BUILD">\r
       <C_Name>PcdWinNtFirmwareVolume</C_Name>\r
       <TokenSpaceGuidCName>gEfiEdkNt32PkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <HelpText/>\r
+      <HelpText>This PCD defines the FD file windows path string. Simulator will load the FD file and execute.\r
+        The item type of this PCD can only be "FIXED_AT_BUILD".</HelpText>\r
     </PcdEntry>\r
     <PcdEntry PcdItemType="FIXED_AT_BUILD">\r
       <C_Name>PcdWinNtBootMode</C_Name>\r
       <TokenSpaceGuidCName>gEfiEdkNt32PkgTokenSpaceGuid</TokenSpaceGuidCName>\r
-      <HelpText/>\r
+      <HelpText>This PCD defines the boot mode for simualtor. \r
+        The boot mode can be set as following value:\r
+          0x0: Boot with full configuration.\r
+          0x1: Boot with minimal configuration.\r
+          0x2: Boot assume no configuration changes.\r
+          0x3: Boot with full configuration plus diagnostics.\r
+          0x4: Boot with default settings.\r
+          0x5: Boot on S4 resume.\r
+          0x6: Boot on S5 resume.\r
+          0x10: Boot on S2 resume. \r
+          0x11: Boot on S3 resume.\r
+          0x12: Boot on flash update.\r
+          0x20: Boot in reovery mode.\r
+        The item type of this PCD can only be "FIXED_AT_BUILD".</HelpText>\r
     </PcdEntry>\r
   </PcdCoded>\r
 </ModuleSurfaceArea>
\ No newline at end of file