public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Wang, Jian J" <jian.j.wang@intel.com>
To: "Kinney, Michael D" <michael.d.kinney@intel.com>,
	"devel@edk2.groups.io" <devel@edk2.groups.io>
Cc: Sean Brogan <sean.brogan@microsoft.com>,
	"Yao, Jiewen" <jiewen.yao@intel.com>,
	"Zhang, Chao B" <chao.b.zhang@intel.com>
Subject: Re: [Patch] SecurityPkg: Fix spelling errors
Date: Tue, 22 Oct 2019 04:39:13 +0000	[thread overview]
Message-ID: <D827630B58408649ACB04F44C510003625996C40@SHSMSX107.ccr.corp.intel.com> (raw)
In-Reply-To: <20191018190138.25544-1-michael.d.kinney@intel.com>


Reviewed-by: Jian J Wang <jian.j.wang@intel.com>

> -----Original Message-----
> From: Kinney, Michael D <michael.d.kinney@intel.com>
> Sent: Saturday, October 19, 2019 3:02 AM
> To: devel@edk2.groups.io
> Cc: Sean Brogan <sean.brogan@microsoft.com>; Yao, Jiewen
> <jiewen.yao@intel.com>; Wang, Jian J <jian.j.wang@intel.com>; Zhang, Chao B
> <chao.b.zhang@intel.com>
> Subject: [Patch] SecurityPkg: Fix spelling errors
> 
> From: Sean Brogan <sean.brogan@microsoft.com>
> 
> https://bugzilla.tianocore.org/show_bug.cgi?id=2265
> 
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Chao Zhang <chao.b.zhang@intel.com>
> Signed-off-by: Michael D Kinney <michael.d.kinney@intel.com>
> ---
>  SecurityPkg/FvReportPei/FvReportPei.c         |  4 ++--
>  SecurityPkg/Hash2DxeCrypto/Driver.c           |  6 ++---
>  SecurityPkg/Hash2DxeCrypto/Driver.h           |  4 ++--
>  SecurityPkg/HddPassword/HddPasswordDxe.c      | 10 ++++----
>  SecurityPkg/HddPassword/HddPasswordDxe.h      |  2 +-
>  .../HddPassword/HddPasswordStrings.uni        |  2 +-
>  .../Guid/AuthenticatedVariableFormat.h        |  2 +-
>  .../Include/Library/Tcg2PhysicalPresenceLib.h |  4 ++--
>  .../Include/Library/TcgStorageCoreLib.h       | 12 +++++-----
>  SecurityPkg/Include/Library/Tpm2CommandLib.h  |  2 +-
>  SecurityPkg/Include/Library/TpmCommLib.h      |  2 +-
>  .../Ppi/FirmwareVolumeInfoPrehashedFV.h       |  4 ++--
>  .../Library/AuthVariableLib/AuthService.c     |  4 ++--
>  .../AuthVariableLib/AuthServiceInternal.h     |  2 +-
>  .../Library/AuthVariableLib/AuthVariableLib.c |  4 ++--
>  .../DxeImageAuthenticationStatusLib.c         |  2 +-
>  .../DxeImageVerificationLib.c                 | 10 ++++----
>  .../DxeRsa2048Sha256GuidedSectionExtractLib.c |  4 ++--
>  ...xeRsa2048Sha256GuidedSectionExtractLib.inf |  2 +-
>  ...xeRsa2048Sha256GuidedSectionExtractLib.uni |  2 +-
>  .../DxeTpm2MeasureBootLib.c                   |  4 ++--
>  .../DxeTpmMeasureBootLib.c                    |  4 ++--
>  .../DxeTpmMeasurementLib.c                    |  2 +-
>  .../HashInstanceLibSha1/HashInstanceLibSha1.c |  2 +-
>  .../HashInstanceLibSha256.c                   |  2 +-
>  .../HashInstanceLibSha384.c                   |  2 +-
>  .../HashInstanceLibSha512.c                   |  2 +-
>  SecurityPkg/Library/HashLibTpm2/HashLibTpm2.c |  2 +-
>  .../PeiRsa2048Sha256GuidedSectionExtractLib.c |  4 ++--
>  ...eiRsa2048Sha256GuidedSectionExtractLib.inf |  2 +-
>  ...eiRsa2048Sha256GuidedSectionExtractLib.uni |  2 +-
>  .../TcgStorageCoreLib/TcgStorageCore.c        | 10 ++++----
>  .../TcgStorageCoreLib/TcgStorageUtil.c        |  2 +-
>  .../TcgStorageOpalLib/TcgStorageOpalUtil.c    |  6 ++---
>  .../Library/Tpm12CommandLib/Tpm12NvStorage.c  |  2 +-
>  .../Library/Tpm12DeviceLibDTpm/Tpm12Tis.c     |  2 +-
>  .../Library/Tpm2CommandLib/Tpm2Capability.c   |  4 ++--
>  .../Library/Tpm2CommandLib/Tpm2Hierarchy.c    |  2 +-
>  .../Tpm2DeviceLibDTpm/Tpm2DeviceLibDTpm.c     |  2 +-
>  .../Tpm2DeviceLibDTpm/Tpm2InstanceLibDTpm.c   |  2 +-
>  .../Library/Tpm2DeviceLibDTpm/Tpm2Ptp.c       |  4 ++--
>  .../Library/Tpm2DeviceLibDTpm/Tpm2Tis.c       |  4 ++--
>  SecurityPkg/Library/TpmCommLib/CommonHeader.h |  2 +-
>  SecurityPkg/Library/TpmCommLib/TisPc.c        |  2 +-
>  .../Pkcs7VerifyDxe/Pkcs7VerifyDxe.c           | 18 +++++++-------
>  .../RandomNumberGenerator/RngDxe/RdRand.c     |  2 +-
>  SecurityPkg/SecurityPkg.dec                   | 18 +++++++-------
>  SecurityPkg/SecurityPkg.dsc                   |  2 +-
>  SecurityPkg/SecurityPkg.uni                   | 12 +++++-----
>  .../Tcg/MemoryOverwriteControl/TcgMor.c       |  6 ++---
>  .../Tcg/MemoryOverwriteControl/TcgMor.inf     |  2 +-
>  .../Tcg/MemoryOverwriteControl/TcgMor.uni     |  2 +-
>  .../TcgMorLock.c                              |  4 ++--
>  .../TcgMorLock.h                              |  2 +-
>  .../TcgMorLock.uni                            |  4 ++--
>  .../TcgMorLockSmm.inf                         |  2 +-
>  .../Tcg/Opal/OpalPassword/OpalDriver.c        |  6 ++---
>  .../Tcg/Opal/OpalPassword/OpalDriver.h        |  6 ++---
>  SecurityPkg/Tcg/Opal/OpalPassword/OpalHii.c   | 10 ++++----
>  SecurityPkg/Tcg/Opal/OpalPassword/OpalHii.h   |  8 +++----
>  .../PhysicalPresencePei/PhysicalPresencePei.c |  2 +-
>  SecurityPkg/Tcg/Tcg2Config/Tcg2Config.vfr     |  6 ++---
>  SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigDriver.c |  2 +-
>  SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigPeim.c   |  4 ++--
>  SecurityPkg/Tcg/Tcg2Dxe/MeasureBootPeCoff.c   |  2 +-
>  SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c             |  2 +-
>  SecurityPkg/Tcg/Tcg2Pei/Tcg2Pei.c             | 16 ++++++-------
>  SecurityPkg/Tcg/Tcg2Smm/Tcg2Smm.c             | 22 ++++++++---------
>  SecurityPkg/Tcg/Tcg2Smm/Tpm.asl               |  6 ++---
>  .../Tcg/TcgConfigDxe/TcgConfigDriver.c        |  2 +-
>  SecurityPkg/Tcg/TcgDxe/TcgDxe.c               |  6 ++---
>  SecurityPkg/Tcg/TcgPei/TcgPei.c               | 18 +++++++-------
>  SecurityPkg/Tcg/TcgSmm/Tpm.asl                |  6 ++---
>  .../SecureBootConfigDriver.c                  |  2 +-
>  .../SecureBootConfigDxe.inf                   |  2 +-
>  .../SecureBootConfigDxe.uni                   |  2 +-
>  .../SecureBootConfigImpl.c                    | 24 +++++++++----------
>  .../SecureBootConfigImpl.h                    | 10 ++++----
>  .../SecureBootConfigMisc.c                    |  8 +++----
>  .../SecureBootConfigNvData.h                  |  4 ++--
>  .../SecureBootConfigStrings.uni               |  4 ++--
>  81 files changed, 205 insertions(+), 205 deletions(-)
> 
> diff --git a/SecurityPkg/FvReportPei/FvReportPei.c
> b/SecurityPkg/FvReportPei/FvReportPei.c
> index d24470af6b..d709760ea3 100644
> --- a/SecurityPkg/FvReportPei/FvReportPei.c
> +++ b/SecurityPkg/FvReportPei/FvReportPei.c
> @@ -145,7 +145,7 @@ VerifyHashedFv (
>    ASSERT (HashValue != NULL);
> 
>    //
> -  // Calcuate hash value for each FV first.
> +  // Calculate hash value for each FV first.
>    //
>    FvHashValue = HashValue;
>    for (FvIndex = 0; FvIndex < FvNumber; ++FvIndex) {
> @@ -281,7 +281,7 @@ ReportHashedFv (
>    @param[in] StoredHashFvPpi  Pointer to PPI containing hash information.
>    @param[in] BootMode         Current boot mode.
> 
> -  @retval Pointer to structure containning valid hash information for current
> boot mode.
> +  @retval Pointer to structure containing valid hash information for current
> boot mode.
>    @retval NULL if there's no hash associated with current boot mode.
>  **/
>  STATIC
> diff --git a/SecurityPkg/Hash2DxeCrypto/Driver.c
> b/SecurityPkg/Hash2DxeCrypto/Driver.c
> index 6aa88f8b3b..4627061a28 100644
> --- a/SecurityPkg/Hash2DxeCrypto/Driver.c
> +++ b/SecurityPkg/Hash2DxeCrypto/Driver.c
> @@ -22,7 +22,7 @@ EFI_SERVICE_BINDING_PROTOCOL
> mHash2ServiceBindingProtocol = {
>                                       it is not NULL, then the I/O services are added
>                                       to the existing child handle.
> 
> -  @retval EFI_SUCCES                 The protocol was added to ChildHandle.
> +  @retval EFI_SUCCESS                The protocol was added to ChildHandle.
>    @retval EFI_INVALID_PARAMETER      ChildHandle is NULL.
>    @retval EFI_OUT_OF_RESOURCES       There are not enough resources
> available to
>                                       create the child.
> @@ -99,7 +99,7 @@ Hash2ServiceBindingCreateChild (
>                                   instance.
>    @param[in]  ChildHandle        Handle of the child to destroy.
> 
> -  @retval EFI_SUCCES             The protocol was removed from ChildHandle.
> +  @retval EFI_SUCCESS            The protocol was removed from ChildHandle.
>    @retval EFI_UNSUPPORTED        ChildHandle does not support the protocol
> that
>                                   is being removed.
>    @retval EFI_INVALID_PARAMETER  ChildHandle is NULL.
> @@ -193,7 +193,7 @@ Hash2ServiceBindingDestroyChild (
>    @param[in]  ImageHandle  The image handle of the driver.
>    @param[in]  SystemTable  The system table.
> 
> -  @retval EFI_SUCCES       The service binding protocols is successfully installed.
> +  @retval EFI_SUCCESS      The service binding protocols is successfully installed.
>    @retval Others           Other errors as indicated.
> 
>  **/
> diff --git a/SecurityPkg/Hash2DxeCrypto/Driver.h
> b/SecurityPkg/Hash2DxeCrypto/Driver.h
> index 162279e469..7b8996912a 100644
> --- a/SecurityPkg/Hash2DxeCrypto/Driver.h
> +++ b/SecurityPkg/Hash2DxeCrypto/Driver.h
> @@ -79,7 +79,7 @@ typedef struct {
>                                       it is not NULL, then the I/O services are added
>                                       to the existing child handle.
> 
> -  @retval EFI_SUCCES                 The protocol was added to ChildHandle.
> +  @retval EFI_SUCCESS                The protocol was added to ChildHandle.
>    @retval EFI_INVALID_PARAMETER      ChildHandle is NULL.
>    @retval EFI_OUT_OF_RESOURCES       There are not enough resources
> available to
>                                       create the child.
> @@ -104,7 +104,7 @@ Hash2ServiceBindingCreateChild (
>                                   instance.
>    @param[in]  ChildHandle        Handle of the child to destroy.
> 
> -  @retval EFI_SUCCES             The protocol was removed from ChildHandle.
> +  @retval EFI_SUCCESS            The protocol was removed from ChildHandle.
>    @retval EFI_UNSUPPORTED        ChildHandle does not support the protocol
> that
>                                   is being removed.
>    @retval EFI_INVALID_PARAMETER  ChildHandle is NULL.
> diff --git a/SecurityPkg/HddPassword/HddPasswordDxe.c
> b/SecurityPkg/HddPassword/HddPasswordDxe.c
> index 051e64091d..a25b3471d0 100644
> --- a/SecurityPkg/HddPassword/HddPasswordDxe.c
> +++ b/SecurityPkg/HddPassword/HddPasswordDxe.c
> @@ -120,7 +120,7 @@ BuildHddPasswordDeviceInfo (
> 
>      //
>      // 1. Handle device which already set password.
> -    // 2. When request to send freeze comamnd, driver also needs to handle
> device
> +    // 2. When request to send freeze command, driver also needs to handle
> device
>      //    which support security feature.
>      //
>      if ((!PasswordIsFullZero (ConfigFormEntry->Password)) ||
> @@ -2008,9 +2008,9 @@ SaveHddPasswordRequest (
>  }
> 
>  /**
> -  Get the HDD Password configuration form entry by the index of the goto
> opcode actived.
> +  Get the HDD Password configuration form entry by the index of the goto
> opcode activated.
> 
> -  @param[in]  Index The 0-based index of the goto opcode actived.
> +  @param[in]  Index The 0-based index of the goto opcode activated.
> 
>    @return The HDD Password configuration form entry found.
>  **/
> @@ -2758,7 +2758,7 @@ HddPasswordConfigFormInit (
>    @param ImageHandle     Image handle this driver.
>    @param SystemTable     Pointer to SystemTable.
> 
> -  @retval EFI_SUCESS     This function always complete successfully.
> +  @retval EFI_SUCCESS     This function always complete successfully.
> 
>  **/
>  EFI_STATUS
> @@ -2806,7 +2806,7 @@ HddPasswordDxeInit (
>    ASSERT_EFI_ERROR (Status);
> 
>    //
> -  // Make HDD_PASSWORD_VARIABLE_NAME varible read-only.
> +  // Make HDD_PASSWORD_VARIABLE_NAME variable read-only.
>    //
>    Status = gBS->LocateProtocol (&gEdkiiVariableLockProtocolGuid, NULL, (VOID
> **) &VariableLock);
>    if (!EFI_ERROR (Status)) {
> diff --git a/SecurityPkg/HddPassword/HddPasswordDxe.h
> b/SecurityPkg/HddPassword/HddPasswordDxe.h
> index f695ca9454..87db587eb6 100644
> --- a/SecurityPkg/HddPassword/HddPasswordDxe.h
> +++ b/SecurityPkg/HddPassword/HddPasswordDxe.h
> @@ -88,7 +88,7 @@ typedef struct _HDD_PASSWORD_DXE_PRIVATE_DATA {
>  #define HDD_PASSWORD_DXE_PRIVATE_FROM_THIS(a)  CR (a,
> HDD_PASSWORD_DXE_PRIVATE_DATA, ConfigAccess,
> HDD_PASSWORD_DXE_PRIVATE_SIGNATURE)
> 
>  //
> -//Iterate through the doule linked list. NOT delete safe
> +//Iterate through the double linked list. NOT delete safe
>  //
>  #define EFI_LIST_FOR_EACH(Entry, ListHead)    \
>    for (Entry = (ListHead)->ForwardLink; Entry != (ListHead); Entry = Entry-
> >ForwardLink)
> diff --git a/SecurityPkg/HddPassword/HddPasswordStrings.uni
> b/SecurityPkg/HddPassword/HddPasswordStrings.uni
> index 188c40f349..b6fddffced 100644
> --- a/SecurityPkg/HddPassword/HddPasswordStrings.uni
> +++ b/SecurityPkg/HddPassword/HddPasswordStrings.uni
> @@ -16,7 +16,7 @@
>  #string STR_SECURITY_HDD_BANNER_ONE           #language en-US "Allows
> Access to Set, Modify and Clear HardDisk User and"
>  #string STR_SECURITY_HDD_BANNER_TWO           #language en-US "Master
> Passwords."
>  #string STR_SECURITY_HDD_BANNER_THREE         #language en-US "User
> Password need to be installed for Enabling Security."
> -#string STR_SECURITY_HDD_BANNER_FOUR          #language en-US "Master
> Password can be modified only when succesfully"
> +#string STR_SECURITY_HDD_BANNER_FOUR          #language en-US "Master
> Password can be modified only when successfully"
>  #string STR_SECURITY_HDD_BANNER_FIVE          #language en-US "unlocked
> with User Password in POST."
> 
>  #string STR_HDD_SECURITY_HD                   #language en-US "HDD Password"
> diff --git a/SecurityPkg/Include/Guid/AuthenticatedVariableFormat.h
> b/SecurityPkg/Include/Guid/AuthenticatedVariableFormat.h
> index 2a097826a0..44ceebc488 100644
> --- a/SecurityPkg/Include/Guid/AuthenticatedVariableFormat.h
> +++ b/SecurityPkg/Include/Guid/AuthenticatedVariableFormat.h
> @@ -54,7 +54,7 @@ extern EFI_GUID gEfiVendorKeysNvGuid;
> 
>  ///
>  ///  "VendorKeysNv" variable to record the out of band secure boot keys
> modification.
> -///  This variable is a read-only NV varaible that indicates whether someone
> other than
> +///  This variable is a read-only NV variable that indicates whether someone
> other than
>  ///  the platform vendor has used a mechanism not defined by the UEFI
> Specification to
>  ///  transition the system to setup mode or to update secure boot keys.
>  ///
> diff --git a/SecurityPkg/Include/Library/Tcg2PhysicalPresenceLib.h
> b/SecurityPkg/Include/Library/Tcg2PhysicalPresenceLib.h
> index 8da3deaf86..39febcb655 100644
> --- a/SecurityPkg/Include/Library/Tcg2PhysicalPresenceLib.h
> +++ b/SecurityPkg/Include/Library/Tcg2PhysicalPresenceLib.h
> @@ -29,8 +29,8 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
>  //
>  // UEFI TCG2 library definition bit of the BIOS Information Flags
>  //
> -#define
> TCG2_BIOS_INFORMATION_FLAG_HIERACHY_CONTROL_STORAGE_DISABLE
> BIT8
> -#define
> TCG2_BIOS_INFORMATION_FLAG_HIERACHY_CONTROL_ENDORSEMENT_DISA
> BLE  BIT9
> +#define
> TCG2_BIOS_INFORMATION_FLAG_HIERARCHY_CONTROL_STORAGE_DISABLE
> BIT8
> +#define
> TCG2_BIOS_INFORMATION_FLAG_HIERARCHY_CONTROL_ENDORSEMENT_DIS
> ABLE  BIT9
> 
>  //
>  // UEFI TCG2 library definition bit of the BIOS Storage Management Flags
> diff --git a/SecurityPkg/Include/Library/TcgStorageCoreLib.h
> b/SecurityPkg/Include/Library/TcgStorageCoreLib.h
> index 190065b63e..c25a56a14c 100644
> --- a/SecurityPkg/Include/Library/TcgStorageCoreLib.h
> +++ b/SecurityPkg/Include/Library/TcgStorageCoreLib.h
> @@ -696,7 +696,7 @@ TcgGetTokenUINT64(
>    Get next specify value.
> 
>    @param    ParseStruct   Input parse structure.
> -  @param    Value         Return vlaue.
> +  @param    Value         Return value.
> 
>    @retval   return the action result.
> 
> @@ -713,7 +713,7 @@ TcgGetNextUINT8(
>    Get next specify value.
> 
>    @param    ParseStruct   Input parse structure.
> -  @param    Value         Return vlaue.
> +  @param    Value         Return value.
> 
>    @retval   return the action result.
> 
> @@ -729,7 +729,7 @@ TcgGetNextUINT16(
>    Get next specify value.
> 
>    @param    ParseStruct   Input parse structure.
> -  @param    Value         Return vlaue.
> +  @param    Value         Return value.
> 
>    @retval   return the action result.
> 
> @@ -745,7 +745,7 @@ TcgGetNextUINT32(
>    Get next specify value.
> 
>    @param    ParseStruct   Input parse structure.
> -  @param    Value         Return vlaue.
> +  @param    Value         Return value.
> 
>    @retval   return the action result.
> 
> @@ -761,7 +761,7 @@ TcgGetNextUINT64(
>    Get next specify value.
> 
>    @param    ParseStruct   Input parse structure.
> -  @param    Value         Return vlaue.
> +  @param    Value         Return value.
> 
>    @retval   return the action result.
> 
> @@ -1261,7 +1261,7 @@ TcgEnumLevel0Discovery(
>    Get Feature code from the header.
> 
>    @param     DiscoveryHeader    The discovery header.
> -  @param     FeatureCode        reutrn the Feature code.
> +  @param     FeatureCode        return the Feature code.
>    @param     FeatureSize        return the Feature size.
> 
>    @retval    return the Feature code data.
> diff --git a/SecurityPkg/Include/Library/Tpm2CommandLib.h
> b/SecurityPkg/Include/Library/Tpm2CommandLib.h
> index d78d5ee1ba..ce381e786b 100644
> --- a/SecurityPkg/Include/Library/Tpm2CommandLib.h
> +++ b/SecurityPkg/Include/Library/Tpm2CommandLib.h
> @@ -585,7 +585,7 @@ Tpm2PcrAllocateBanks (
>    NOTE:
>    To simplify this function, leave returned CapabilityData for caller to unpack
> since there are
>    many capability categories and only few categories will be used in firmware. It
> means the caller
> -  need swap the byte order for the feilds in CapabilityData.
> +  need swap the byte order for the fields in CapabilityData.
> 
>    @param[in]  Capability         Group selection; determines the format of the
> response.
>    @param[in]  Property           Further definition of information.
> diff --git a/SecurityPkg/Include/Library/TpmCommLib.h
> b/SecurityPkg/Include/Library/TpmCommLib.h
> index b9aca60b82..7e102a6a11 100644
> --- a/SecurityPkg/Include/Library/TpmCommLib.h
> +++ b/SecurityPkg/Include/Library/TpmCommLib.h
> @@ -214,7 +214,7 @@ TisPcWaitRegisterBits (
>    in the time of default TIS_TIMEOUT_D.
> 
>    @param[in]  TisReg                Pointer to TIS register.
> -  @param[out] BurstCount            Pointer to a buffer to store the got BurstConut.
> +  @param[out] BurstCount            Pointer to a buffer to store the got
> BurstCount.
> 
>    @retval     EFI_SUCCESS           Get BurstCount.
>    @retval     EFI_INVALID_PARAMETER TisReg is NULL or BurstCount is NULL.
> diff --git a/SecurityPkg/Include/Ppi/FirmwareVolumeInfoPrehashedFV.h
> b/SecurityPkg/Include/Ppi/FirmwareVolumeInfoPrehashedFV.h
> index 2fc95537dc..1b7320e916 100644
> --- a/SecurityPkg/Include/Ppi/FirmwareVolumeInfoPrehashedFV.h
> +++ b/SecurityPkg/Include/Ppi/FirmwareVolumeInfoPrehashedFV.h
> @@ -26,10 +26,10 @@ typedef struct _HASH_INFO {
>  //
>  // This PPI carries prehashsed data for one FV.  Platform should ensure 1:1
> mapping between pre-hashed PPI and corresponding FV.
>  // The Count field in PPI is followed by Count number of FV hash info entries,
> which can be extended to PCR and logged to TCG event log directly by TCG
> modules.
> -// TCG module checks TPM required hash algorithms(PcdTpm2HashMask) with
> each pre-hased PPIs
> +// TCG module checks TPM required hash algorithms(PcdTpm2HashMask) with
> each pre-hashed PPIs
>  //   For each pre-hashed PPI
>  //     If PPI carries hash generated by equivalent or larger algorithm set than
> TPM required, directly use PPI
> -//     else, drops PPI data and cacluate all hash again
> +//     else, drops PPI data and calculate all hash again
>  //
>  typedef struct {
>    UINT32                                     FvBase;
> diff --git a/SecurityPkg/Library/AuthVariableLib/AuthService.c
> b/SecurityPkg/Library/AuthVariableLib/AuthService.c
> index 486df55bed..2f60331f2c 100644
> --- a/SecurityPkg/Library/AuthVariableLib/AuthService.c
> +++ b/SecurityPkg/Library/AuthVariableLib/AuthService.c
> @@ -425,7 +425,7 @@ CheckSignatureListFormat(
>    RsaContext = NULL;
> 
>    //
> -  // Walk throuth the input signature list and check the data format.
> +  // Walk through the input signature list and check the data format.
>    // If any signature is incorrectly formed, the whole check will fail.
>    //
>    while ((SigDataSize > 0) && (SigDataSize >= SigList->SignatureListSize)) {
> @@ -1069,7 +1069,7 @@ AuthServiceInternalCompareTimeStamp (
>    @param[out] Sha256Digest       Sha256 digest calculated.
> 
>    @return EFI_ABORTED          Digest process failed.
> -  @return EFI_SUCCESS          SHA256 Digest is succesfully calculated.
> +  @return EFI_SUCCESS          SHA256 Digest is successfully calculated.
> 
>  **/
>  EFI_STATUS
> diff --git a/SecurityPkg/Library/AuthVariableLib/AuthServiceInternal.h
> b/SecurityPkg/Library/AuthVariableLib/AuthServiceInternal.h
> index da2c40bc37..2bec637f75 100644
> --- a/SecurityPkg/Library/AuthVariableLib/AuthServiceInternal.h
> +++ b/SecurityPkg/Library/AuthVariableLib/AuthServiceInternal.h
> @@ -159,7 +159,7 @@ DeleteCertsFromDb (
>  /**
>    Clean up signer's certificates for common authenticated variable
>    by corresponding VariableName and VendorGuid from "certdb".
> -  Sytem may break down during Timebased Variable update & certdb update,
> +  System may break down during Timebased Variable update & certdb update,
>    make them inconsistent,  this function is called in AuthVariable Init to ensure
>    consistency
> 
> diff --git a/SecurityPkg/Library/AuthVariableLib/AuthVariableLib.c
> b/SecurityPkg/Library/AuthVariableLib/AuthVariableLib.c
> index e80fadbf05..122b3b0bf4 100644
> --- a/SecurityPkg/Library/AuthVariableLib/AuthVariableLib.c
> +++ b/SecurityPkg/Library/AuthVariableLib/AuthVariableLib.c
> @@ -96,7 +96,7 @@ VOID **mAuthVarAddressPointer[9];
>  AUTH_VAR_LIB_CONTEXT_IN *mAuthVarLibContextIn = NULL;
> 
>  /**
> -  Initialization for authenticated varibale services.
> +  Initialization for authenticated variable services.
>    If this initialization returns error status, other APIs will not work
>    and expect to be not called then.
> 
> @@ -382,7 +382,7 @@ AuthVariableLibInitialize (
>    @retval EFI_INVALID_PARAMETER     Invalid parameter.
>    @retval EFI_WRITE_PROTECTED       Variable is write-protected.
>    @retval EFI_OUT_OF_RESOURCES      There is not enough resource.
> -  @retval EFI_SECURITY_VIOLATION    The variable is with
> EFI_VARIABLE_TIME_BASED_AUTHENTICATED_WRITE_ACESS
> +  @retval EFI_SECURITY_VIOLATION    The variable is with
> EFI_VARIABLE_TIME_BASED_AUTHENTICATED_WRITE_ACCESS
>                                      set, but the AuthInfo does NOT pass the validation
>                                      check carried out by the firmware.
>    @retval EFI_UNSUPPORTED           Unsupported to process authenticated
> variable.
> diff --git
> a/SecurityPkg/Library/DxeImageAuthenticationStatusLib/DxeImageAuthenticati
> onStatusLib.c
> b/SecurityPkg/Library/DxeImageAuthenticationStatusLib/DxeImageAuthenticati
> onStatusLib.c
> index 38448f0c8c..e4ddff21b5 100644
> ---
> a/SecurityPkg/Library/DxeImageAuthenticationStatusLib/DxeImageAuthenticati
> onStatusLib.c
> +++
> b/SecurityPkg/Library/DxeImageAuthenticationStatusLib/DxeImageAuthenticati
> onStatusLib.c
> @@ -49,7 +49,7 @@ DxeImageAuthenticationStatusHandler (
> 
> 
>  /**
> -  Register image authenticaion status check handler.
> +  Register image authentication status check handler.
> 
>    @param  ImageHandle   ImageHandle of the loaded driver.
>    @param  SystemTable   Pointer to the EFI System Table.
> diff --git
> a/SecurityPkg/Library/DxeImageVerificationLib/DxeImageVerificationLib.c
> b/SecurityPkg/Library/DxeImageVerificationLib/DxeImageVerificationLib.c
> index fe4cdccedd..a0a12b50dd 100644
> --- a/SecurityPkg/Library/DxeImageVerificationLib/DxeImageVerificationLib.c
> +++ b/SecurityPkg/Library/DxeImageVerificationLib/DxeImageVerificationLib.c
> @@ -745,7 +745,7 @@ AddImageExeInfo (
>    if (ImageExeInfoTable != NULL) {
>      //
>      // The table has been found!
> -    // We must enlarge the table to accomodate the new exe info entry.
> +    // We must enlarge the table to accommodate the new exe info entry.
>      //
>      ImageExeInfoTableSize = GetImageExeInfoTableSize (ImageExeInfoTable);
>    } else {
> @@ -947,7 +947,7 @@ Done:
> 
>    @param[in]  VariableName        Name of database variable that is searched in.
>    @param[in]  Signature           Pointer to signature that is searched for.
> -  @param[in]  CertType            Pointer to hash algrithom.
> +  @param[in]  CertType            Pointer to hash algorithm.
>    @param[in]  SignatureSize       Size of Signature.
> 
>    @return TRUE                    Found the signature in the variable database.
> @@ -992,7 +992,7 @@ IsSignatureFoundInDatabase (
>      goto Done;
>    }
>    //
> -  // Enumerate all signature data in SigDB to check if executable's signature
> exists.
> +  // Enumerate all signature data in SigDB to check if signature exists for
> executable.
>    //
>    CertList = (EFI_SIGNATURE_LIST *) Data;
>    while ((DataSize > 0) && (DataSize >= CertList->SignatureListSize)) {
> @@ -1844,7 +1844,7 @@ DxeImageVerificationHandler (
> 
>    if (OffSet != (SecDataDir->VirtualAddress + SecDataDir->Size)) {
>      //
> -    // The Size in Certificate Table or the attribute certicate table is corrupted.
> +    // The Size in Certificate Table or the attribute certificate table is corrupted.
>      //
>      VerifyStatus = EFI_ACCESS_DENIED;
>    }
> @@ -1855,7 +1855,7 @@ DxeImageVerificationHandler (
>      Status = EFI_ACCESS_DENIED;
>      if (Action == EFI_IMAGE_EXECUTION_AUTH_SIG_FAILED || Action ==
> EFI_IMAGE_EXECUTION_AUTH_SIG_FOUND) {
>        //
> -      // Get image hash value as executable's signature.
> +      // Get image hash value as signature of executable.
>        //
>        SignatureListSize = sizeof (EFI_SIGNATURE_LIST) + sizeof
> (EFI_SIGNATURE_DATA) - 1 + mImageDigestSize;
>        SignatureList     = (EFI_SIGNATURE_LIST *) AllocateZeroPool
> (SignatureListSize);
> diff --git
> a/SecurityPkg/Library/DxeRsa2048Sha256GuidedSectionExtractLib/DxeRsa2048
> Sha256GuidedSectionExtractLib.c
> b/SecurityPkg/Library/DxeRsa2048Sha256GuidedSectionExtractLib/DxeRsa2048
> Sha256GuidedSectionExtractLib.c
> index 0059328950..28807d4d98 100644
> ---
> a/SecurityPkg/Library/DxeRsa2048Sha256GuidedSectionExtractLib/DxeRsa2048
> Sha256GuidedSectionExtractLib.c
> +++
> b/SecurityPkg/Library/DxeRsa2048Sha256GuidedSectionExtractLib/DxeRsa2048
> Sha256GuidedSectionExtractLib.c
> @@ -2,7 +2,7 @@
> 
>    This library registers RSA 2048 SHA 256 guided section handler
>    to parse RSA 2048 SHA 256 encapsulation section and extract raw data.
> -  It uses the BaseCrypyLib based on OpenSSL to authenticate the signature.
> +  It uses the BaseCryptLib based on OpenSSL to authenticate the signature.
> 
>  Copyright (c) 2013 - 2018, Intel Corporation. All rights reserved.<BR>
>  SPDX-License-Identifier: BSD-2-Clause-Patent
> @@ -213,7 +213,7 @@ Rsa2048Sha256GuidedSectionHandler (
>    }
> 
>    //
> -  // All paths from here return EFI_SUCESS and result is returned in
> AuthenticationStatus
> +  // All paths from here return EFI_SUCCESS and result is returned in
> AuthenticationStatus
>    //
>    Status = EFI_SUCCESS;
> 
> diff --git
> a/SecurityPkg/Library/DxeRsa2048Sha256GuidedSectionExtractLib/DxeRsa2048
> Sha256GuidedSectionExtractLib.inf
> b/SecurityPkg/Library/DxeRsa2048Sha256GuidedSectionExtractLib/DxeRsa2048
> Sha256GuidedSectionExtractLib.inf
> index 0822fd0856..9c426edace 100644
> ---
> a/SecurityPkg/Library/DxeRsa2048Sha256GuidedSectionExtractLib/DxeRsa2048
> Sha256GuidedSectionExtractLib.inf
> +++
> b/SecurityPkg/Library/DxeRsa2048Sha256GuidedSectionExtractLib/DxeRsa2048
> Sha256GuidedSectionExtractLib.inf
> @@ -3,7 +3,7 @@
>  #  ExtractGuidedSectionLib service to register an RSA 2048 SHA 256 guided
> section handler
>  #  that parses RSA 2048 SHA 256 encapsulation section and extracts raw data.
>  #
> -#  It uses the BaseCrypyLib based on OpenSSL to authenticate the signature.
> +#  It uses the BaseCryptLib based on OpenSSL to authenticate the signature.
>  #
>  # Copyright (c) 2013 - 2018, Intel Corporation. All rights reserved.<BR>
>  #
> diff --git
> a/SecurityPkg/Library/DxeRsa2048Sha256GuidedSectionExtractLib/DxeRsa2048
> Sha256GuidedSectionExtractLib.uni
> b/SecurityPkg/Library/DxeRsa2048Sha256GuidedSectionExtractLib/DxeRsa2048
> Sha256GuidedSectionExtractLib.uni
> index 484bde0afc..f0294a68f5 100644
> ---
> a/SecurityPkg/Library/DxeRsa2048Sha256GuidedSectionExtractLib/DxeRsa2048
> Sha256GuidedSectionExtractLib.uni
> +++
> b/SecurityPkg/Library/DxeRsa2048Sha256GuidedSectionExtractLib/DxeRsa2048
> Sha256GuidedSectionExtractLib.uni
> @@ -4,7 +4,7 @@
>  // ExtractGuidedSectionLib service to register an RSA 2048 SHA 256 guided
> section handler
>  // that parses RSA 2048 SHA 256 encapsulation section and extracts raw data.
>  //
> -// It uses the BaseCrypyLib based on OpenSSL to authenticate the signature.
> +// It uses the BaseCryptLib based on OpenSSL to authenticate the signature.
>  //
>  // Copyright (c) 2013 - 2018, Intel Corporation. All rights reserved.<BR>
>  //
> diff --git
> a/SecurityPkg/Library/DxeTpm2MeasureBootLib/DxeTpm2MeasureBootLib.c
> b/SecurityPkg/Library/DxeTpm2MeasureBootLib/DxeTpm2MeasureBootLib.c
> index 543635b2e4..04b9b0d7fb 100644
> --- a/SecurityPkg/Library/DxeTpm2MeasureBootLib/DxeTpm2MeasureBootLib.c
> +++
> b/SecurityPkg/Library/DxeTpm2MeasureBootLib/DxeTpm2MeasureBootLib.c
> @@ -362,7 +362,7 @@ Tcg2MeasurePeImage (
>    if (Status == EFI_VOLUME_FULL) {
>      //
>      // Volume full here means the image is hashed and its result is extended to
> PCR.
> -    // But the event log cann't be saved since log area is full.
> +    // But the event log can't be saved since log area is full.
>      // Just return EFI_SUCCESS in order not to block the image load.
>      //
>      Status = EFI_SUCCESS;
> @@ -541,7 +541,7 @@ DxeTpm2MeasureBootHandler (
>      }
>      //
>      // The PE image from unmeasured Firmware volume need be measured
> -    // The PE image from measured Firmware volume will be mearsured
> according to policy below.
> +    // The PE image from measured Firmware volume will be measured
> according to policy below.
>      //   If it is driver, do not measure
>      //   If it is application, still measure.
>      //
> diff --git
> a/SecurityPkg/Library/DxeTpmMeasureBootLib/DxeTpmMeasureBootLib.c
> b/SecurityPkg/Library/DxeTpmMeasureBootLib/DxeTpmMeasureBootLib.c
> index a2d3923ca6..1f2eed29a1 100644
> --- a/SecurityPkg/Library/DxeTpmMeasureBootLib/DxeTpmMeasureBootLib.c
> +++ b/SecurityPkg/Library/DxeTpmMeasureBootLib/DxeTpmMeasureBootLib.c
> @@ -649,7 +649,7 @@ TcgMeasurePeImage (
>    if (Status == EFI_OUT_OF_RESOURCES) {
>      //
>      // Out of resource here means the image is hashed and its result is extended
> to PCR.
> -    // But the event log cann't be saved since log area is full.
> +    // But the event log can't be saved since log area is full.
>      // Just return EFI_SUCCESS in order not to block the image load.
>      //
>      Status = EFI_SUCCESS;
> @@ -838,7 +838,7 @@ DxeTpmMeasureBootHandler (
>      }
>      //
>      // The PE image from unmeasured Firmware volume need be measured
> -    // The PE image from measured Firmware volume will be mearsured
> according to policy below.
> +    // The PE image from measured Firmware volume will be measured
> according to policy below.
>      //   If it is driver, do not measure
>      //   If it is application, still measure.
>      //
> diff --git
> a/SecurityPkg/Library/DxeTpmMeasurementLib/DxeTpmMeasurementLib.c
> b/SecurityPkg/Library/DxeTpmMeasurementLib/DxeTpmMeasurementLib.c
> index adf03bb226..061136ee78 100644
> --- a/SecurityPkg/Library/DxeTpmMeasurementLib/DxeTpmMeasurementLib.c
> +++ b/SecurityPkg/Library/DxeTpmMeasurementLib/DxeTpmMeasurementLib.c
> @@ -56,7 +56,7 @@ Tpm12MeasureAndLogData (
>    TcgEvent = NULL;
> 
>    //
> -  // Tpm active/deactive state is checked in HashLogExtendEvent
> +  // Tpm activation state is checked in HashLogExtendEvent
>    //
>    Status = gBS->LocateProtocol (&gEfiTcgProtocolGuid, NULL, (VOID **)
> &TcgProtocol);
>    if (EFI_ERROR(Status)){
> diff --git a/SecurityPkg/Library/HashInstanceLibSha1/HashInstanceLibSha1.c
> b/SecurityPkg/Library/HashInstanceLibSha1/HashInstanceLibSha1.c
> index 9ca784a873..52521b6444 100644
> --- a/SecurityPkg/Library/HashInstanceLibSha1/HashInstanceLibSha1.c
> +++ b/SecurityPkg/Library/HashInstanceLibSha1/HashInstanceLibSha1.c
> @@ -128,7 +128,7 @@ HASH_INTERFACE  mSha1InternalHashInstance = {
>  /**
>    The function register SHA1 instance.
> 
> -  @retval EFI_SUCCESS   SHA1 instance is registered, or system dose not
> surpport registr SHA1 instance
> +  @retval EFI_SUCCESS   SHA1 instance is registered, or system does not
> support register SHA1 instance
>  **/
>  EFI_STATUS
>  EFIAPI
> diff --git
> a/SecurityPkg/Library/HashInstanceLibSha256/HashInstanceLibSha256.c
> b/SecurityPkg/Library/HashInstanceLibSha256/HashInstanceLibSha256.c
> index 01fce3fd2b..760e20bae0 100644
> --- a/SecurityPkg/Library/HashInstanceLibSha256/HashInstanceLibSha256.c
> +++ b/SecurityPkg/Library/HashInstanceLibSha256/HashInstanceLibSha256.c
> @@ -128,7 +128,7 @@ HASH_INTERFACE  mSha256InternalHashInstance = {
>  /**
>    The function register SHA256 instance.
> 
> -  @retval EFI_SUCCESS   SHA256 instance is registered, or system dose not
> surpport registr SHA256 instance
> +  @retval EFI_SUCCESS   SHA256 instance is registered, or system does not
> support register SHA256 instance
>  **/
>  EFI_STATUS
>  EFIAPI
> diff --git
> a/SecurityPkg/Library/HashInstanceLibSha384/HashInstanceLibSha384.c
> b/SecurityPkg/Library/HashInstanceLibSha384/HashInstanceLibSha384.c
> index 29f6c8a828..99e2416a8e 100644
> --- a/SecurityPkg/Library/HashInstanceLibSha384/HashInstanceLibSha384.c
> +++ b/SecurityPkg/Library/HashInstanceLibSha384/HashInstanceLibSha384.c
> @@ -128,7 +128,7 @@ HASH_INTERFACE  mSha384InternalHashInstance = {
>  /**
>    The function register SHA384 instance.
> 
> -  @retval EFI_SUCCESS   SHA384 instance is registered, or system dose not
> surpport registr SHA384 instance
> +  @retval EFI_SUCCESS   SHA384 instance is registered, or system does not
> support register SHA384 instance
>  **/
>  EFI_STATUS
>  EFIAPI
> diff --git
> a/SecurityPkg/Library/HashInstanceLibSha512/HashInstanceLibSha512.c
> b/SecurityPkg/Library/HashInstanceLibSha512/HashInstanceLibSha512.c
> index d5389e1e22..b047791e02 100644
> --- a/SecurityPkg/Library/HashInstanceLibSha512/HashInstanceLibSha512.c
> +++ b/SecurityPkg/Library/HashInstanceLibSha512/HashInstanceLibSha512.c
> @@ -127,7 +127,7 @@ HASH_INTERFACE  mSha512InternalHashInstance = {
>  /**
>    The function register SHA512 instance.
> 
> -  @retval EFI_SUCCESS   SHA512 instance is registered, or system dose not
> surpport registr SHA512 instance
> +  @retval EFI_SUCCESS   SHA512 instance is registered, or system does not
> support register SHA512 instance
>  **/
>  EFI_STATUS
>  EFIAPI
> diff --git a/SecurityPkg/Library/HashLibTpm2/HashLibTpm2.c
> b/SecurityPkg/Library/HashLibTpm2/HashLibTpm2.c
> index f6819793ab..bd24958505 100644
> --- a/SecurityPkg/Library/HashLibTpm2/HashLibTpm2.c
> +++ b/SecurityPkg/Library/HashLibTpm2/HashLibTpm2.c
> @@ -29,7 +29,7 @@ TPM2_HASH_MASK mTpm2HashMask[] = {
>  };
> 
>  /**
> -  The function get algorith from hash mask info.
> +  The function get algorithm from hash mask info.
> 
>    @return Hash algorithm
>  **/
> diff --git
> a/SecurityPkg/Library/PeiRsa2048Sha256GuidedSectionExtractLib/PeiRsa2048Sh
> a256GuidedSectionExtractLib.c
> b/SecurityPkg/Library/PeiRsa2048Sha256GuidedSectionExtractLib/PeiRsa2048Sh
> a256GuidedSectionExtractLib.c
> index df16715120..a759183d20 100644
> ---
> a/SecurityPkg/Library/PeiRsa2048Sha256GuidedSectionExtractLib/PeiRsa2048Sh
> a256GuidedSectionExtractLib.c
> +++
> b/SecurityPkg/Library/PeiRsa2048Sha256GuidedSectionExtractLib/PeiRsa2048Sh
> a256GuidedSectionExtractLib.c
> @@ -2,7 +2,7 @@
> 
>    This library registers RSA 2048 SHA 256 guided section handler
>    to parse RSA 2048 SHA 256 encapsulation section and extract raw data.
> -  It uses the BaseCrypyLib based on OpenSSL to authenticate the signature.
> +  It uses the BaseCryptLib based on OpenSSL to authenticate the signature.
> 
>  Copyright (c) 2013 - 2018, Intel Corporation. All rights reserved.<BR>
>  SPDX-License-Identifier: BSD-2-Clause-Patent
> @@ -197,7 +197,7 @@ Rsa2048Sha256GuidedSectionHandler (
>    }
> 
>    //
> -  // All paths from here return EFI_SUCESS and result is returned in
> AuthenticationStatus
> +  // All paths from here return EFI_SUCCESS and result is returned in
> AuthenticationStatus
>    //
>    Status = EFI_SUCCESS;
> 
> diff --git
> a/SecurityPkg/Library/PeiRsa2048Sha256GuidedSectionExtractLib/PeiRsa2048Sh
> a256GuidedSectionExtractLib.inf
> b/SecurityPkg/Library/PeiRsa2048Sha256GuidedSectionExtractLib/PeiRsa2048Sh
> a256GuidedSectionExtractLib.inf
> index 3fd5e52430..dac97a050e 100644
> ---
> a/SecurityPkg/Library/PeiRsa2048Sha256GuidedSectionExtractLib/PeiRsa2048Sh
> a256GuidedSectionExtractLib.inf
> +++
> b/SecurityPkg/Library/PeiRsa2048Sha256GuidedSectionExtractLib/PeiRsa2048Sh
> a256GuidedSectionExtractLib.inf
> @@ -3,7 +3,7 @@
>  #  ExtractGuidedSectionLib service to register an RSA 2048 SHA 256 guided
> section handler
>  #  that parses RSA 2048 SHA 256 encapsulation section and extracts raw data.
>  #
> -#  It uses the BaseCrypyLib based on OpenSSL to authenticate the signature.
> +#  It uses the BaseCryptLib based on OpenSSL to authenticate the signature.
>  #
>  # Copyright (c) 2013 - 2018, Intel Corporation. All rights reserved.<BR>
>  #
> diff --git
> a/SecurityPkg/Library/PeiRsa2048Sha256GuidedSectionExtractLib/PeiRsa2048Sh
> a256GuidedSectionExtractLib.uni
> b/SecurityPkg/Library/PeiRsa2048Sha256GuidedSectionExtractLib/PeiRsa2048Sh
> a256GuidedSectionExtractLib.uni
> index 484bde0afc..f0294a68f5 100644
> ---
> a/SecurityPkg/Library/PeiRsa2048Sha256GuidedSectionExtractLib/PeiRsa2048Sh
> a256GuidedSectionExtractLib.uni
> +++
> b/SecurityPkg/Library/PeiRsa2048Sha256GuidedSectionExtractLib/PeiRsa2048Sh
> a256GuidedSectionExtractLib.uni
> @@ -4,7 +4,7 @@
>  // ExtractGuidedSectionLib service to register an RSA 2048 SHA 256 guided
> section handler
>  // that parses RSA 2048 SHA 256 encapsulation section and extracts raw data.
>  //
> -// It uses the BaseCrypyLib based on OpenSSL to authenticate the signature.
> +// It uses the BaseCryptLib based on OpenSSL to authenticate the signature.
>  //
>  // Copyright (c) 2013 - 2018, Intel Corporation. All rights reserved.<BR>
>  //
> diff --git a/SecurityPkg/Library/TcgStorageCoreLib/TcgStorageCore.c
> b/SecurityPkg/Library/TcgStorageCoreLib/TcgStorageCore.c
> index 1c072fac23..f9c57a13f2 100644
> --- a/SecurityPkg/Library/TcgStorageCoreLib/TcgStorageCore.c
> +++ b/SecurityPkg/Library/TcgStorageCoreLib/TcgStorageCore.c
> @@ -1249,7 +1249,7 @@ TcgGetTokenByteSequence(
>    Get next specify value.
> 
>    @param    ParseStruct   Input parse structure.
> -  @param    Value         Return vlaue.
> +  @param    Value         Return value.
> 
>    @retval   return the action result.
> 
> @@ -1282,7 +1282,7 @@ TcgGetNextUINT8(
>    Get next specify value.
> 
>    @param    ParseStruct   Input parse structure.
> -  @param    Value         Return vlaue.
> +  @param    Value         Return value.
> 
>    @retval   return the action result.
> 
> @@ -1315,7 +1315,7 @@ TcgGetNextUINT16(
>    Get next specify value.
> 
>    @param    ParseStruct   Input parse structure.
> -  @param    Value         Return vlaue.
> +  @param    Value         Return value.
> 
>    @retval   return the action result.
> 
> @@ -1348,7 +1348,7 @@ TcgGetNextUINT32(
>    Get next specify value.
> 
>    @param    ParseStruct   Input parse structure.
> -  @param    Value         Return vlaue.
> +  @param    Value         Return value.
> 
>    @retval   return the action result.
> 
> @@ -1370,7 +1370,7 @@ TcgGetNextUINT64(
>    Get next specify value.
> 
>    @param    ParseStruct   Input parse structure.
> -  @param    Value         Return vlaue.
> +  @param    Value         Return value.
> 
>    @retval   return the action result.
> 
> diff --git a/SecurityPkg/Library/TcgStorageCoreLib/TcgStorageUtil.c
> b/SecurityPkg/Library/TcgStorageCoreLib/TcgStorageUtil.c
> index d06ad62c91..b6d5cf9890 100644
> --- a/SecurityPkg/Library/TcgStorageCoreLib/TcgStorageUtil.c
> +++ b/SecurityPkg/Library/TcgStorageCoreLib/TcgStorageUtil.c
> @@ -810,7 +810,7 @@ TcgFindFeatureCallback(
>    Get Feature code from the header.
> 
>    @param     DiscoveryHeader    The discovery header.
> -  @param     FeatureCode        reutrn the Feature code.
> +  @param     FeatureCode        return the Feature code.
>    @param     FeatureSize        return the Feature size.
> 
>    @retval    return the Feature code data.
> diff --git a/SecurityPkg/Library/TcgStorageOpalLib/TcgStorageOpalUtil.c
> b/SecurityPkg/Library/TcgStorageOpalLib/TcgStorageOpalUtil.c
> index bc52cceabd..b4927a0872 100644
> --- a/SecurityPkg/Library/TcgStorageOpalLib/TcgStorageOpalUtil.c
> +++ b/SecurityPkg/Library/TcgStorageOpalLib/TcgStorageOpalUtil.c
> @@ -11,7 +11,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
>  #include <Library/TcgStorageOpalLib.h>
>  #include "TcgStorageOpalLibInternal.h"
> 
> -#define OPAL_MSID_LENGHT        128
> +#define OPAL_MSID_LENGTH        128
> 
>  /**
>    Creates a session with OPAL_UID_ADMIN_SP as
> OPAL_ADMIN_SP_PSID_AUTHORITY, then reverts device using Admin SP Revert
> method.
> @@ -1095,7 +1095,7 @@ GetRevertTimeOut (
>    OPAL_DISK_SUPPORT_ATTRIBUTE  SupportedAttributes;
>    UINT16                       BaseComId;
>    UINT32                       MsidLength;
> -  UINT8                        Msid[OPAL_MSID_LENGHT];
> +  UINT8                        Msid[OPAL_MSID_LENGTH];
>    UINT32                       RemovalMechanishLists[ResearvedMechanism];
>    UINT8                        ActiveDataRemovalMechanism;
> 
> @@ -1104,7 +1104,7 @@ GetRevertTimeOut (
>      return 0;
>    }
> 
> -  TcgResult = OpalUtilGetMsid (Session, Msid, OPAL_MSID_LENGHT,
> &MsidLength);
> +  TcgResult = OpalUtilGetMsid (Session, Msid, OPAL_MSID_LENGTH,
> &MsidLength);
>    if (TcgResult != TcgResultSuccess) {
>      return 0;
>    }
> diff --git a/SecurityPkg/Library/Tpm12CommandLib/Tpm12NvStorage.c
> b/SecurityPkg/Library/Tpm12CommandLib/Tpm12NvStorage.c
> index 609186fb7c..8295a1d267 100644
> --- a/SecurityPkg/Library/Tpm12CommandLib/Tpm12NvStorage.c
> +++ b/SecurityPkg/Library/Tpm12CommandLib/Tpm12NvStorage.c
> @@ -218,7 +218,7 @@ Tpm12NvWriteValue (
>    if (EFI_ERROR (Status)) {
>      return Status;
>    }
> -  DEBUG ((DEBUG_INFO, "Tpm12NvWritedValue - ReturnCode = %x\n",
> SwapBytes32 (Response.returnCode)));
> +  DEBUG ((DEBUG_INFO, "Tpm12NvWriteValue - ReturnCode = %x\n",
> SwapBytes32 (Response.returnCode)));
>    switch (SwapBytes32 (Response.returnCode)) {
>    case TPM_SUCCESS:
>      return EFI_SUCCESS;
> diff --git a/SecurityPkg/Library/Tpm12DeviceLibDTpm/Tpm12Tis.c
> b/SecurityPkg/Library/Tpm12DeviceLibDTpm/Tpm12Tis.c
> index 9c5b3d5740..1caa076203 100644
> --- a/SecurityPkg/Library/Tpm12DeviceLibDTpm/Tpm12Tis.c
> +++ b/SecurityPkg/Library/Tpm12DeviceLibDTpm/Tpm12Tis.c
> @@ -125,7 +125,7 @@ Tpm12TisPcWaitRegisterBits (
>    in the time of default TIS_TIMEOUT_D.
> 
>    @param[in]  TisReg                Pointer to TIS register.
> -  @param[out] BurstCount            Pointer to a buffer to store the got BurstConut.
> +  @param[out] BurstCount            Pointer to a buffer to store the got
> BurstCount.
> 
>    @retval     EFI_SUCCESS           Get BurstCount.
>    @retval     EFI_INVALID_PARAMETER TisReg is NULL or BurstCount is NULL.
> diff --git a/SecurityPkg/Library/Tpm2CommandLib/Tpm2Capability.c
> b/SecurityPkg/Library/Tpm2CommandLib/Tpm2Capability.c
> index 4e9f030e77..85b11c7715 100644
> --- a/SecurityPkg/Library/Tpm2CommandLib/Tpm2Capability.c
> +++ b/SecurityPkg/Library/Tpm2CommandLib/Tpm2Capability.c
> @@ -53,7 +53,7 @@ typedef struct {
>    NOTE:
>    To simplify this function, leave returned CapabilityData for caller to unpack
> since there are
>    many capability categories and only few categories will be used in firmware. It
> means the caller
> -  need swap the byte order for the feilds in CapabilityData.
> +  need swap the byte order for the fields in CapabilityData.
> 
>    @param[in]  Capability         Group selection; determines the format of the
> response.
>    @param[in]  Property           Further definition of information.
> @@ -119,7 +119,7 @@ Tpm2GetCapability (
>    //
>    *MoreData = RecvBuffer.MoreData;
>    //
> -  // Does not unpack all possiable property here, the caller should unpack it and
> note the byte order.
> +  // Does not unpack all possible property here, the caller should unpack it and
> note the byte order.
>    //
>    CopyMem (CapabilityData, &RecvBuffer.CapabilityData, RecvBufferSize -
> sizeof (TPM2_RESPONSE_HEADER) - sizeof (UINT8));
> 
> diff --git a/SecurityPkg/Library/Tpm2CommandLib/Tpm2Hierarchy.c
> b/SecurityPkg/Library/Tpm2CommandLib/Tpm2Hierarchy.c
> index 8e05596763..dc36963c0e 100644
> --- a/SecurityPkg/Library/Tpm2CommandLib/Tpm2Hierarchy.c
> +++ b/SecurityPkg/Library/Tpm2CommandLib/Tpm2Hierarchy.c
> @@ -435,7 +435,7 @@ Tpm2HierarchyChangeAuth (
>    WriteUnaligned16 ((UINT16 *)Buffer, SwapBytes16(NewAuth->size));
>    Buffer += sizeof(UINT16);
> 
> -  // New Authorizeation
> +  // New Authorization
>    CopyMem(Buffer, NewAuth->buffer, NewAuth->size);
>    Buffer += NewAuth->size;
> 
> diff --git a/SecurityPkg/Library/Tpm2DeviceLibDTpm/Tpm2DeviceLibDTpm.c
> b/SecurityPkg/Library/Tpm2DeviceLibDTpm/Tpm2DeviceLibDTpm.c
> index 53b9bc42fb..42e1ecbce9 100644
> --- a/SecurityPkg/Library/Tpm2DeviceLibDTpm/Tpm2DeviceLibDTpm.c
> +++ b/SecurityPkg/Library/Tpm2DeviceLibDTpm/Tpm2DeviceLibDTpm.c
> @@ -137,7 +137,7 @@ Tpm2RegisterTpm2DeviceLib (
>  /**
>    The function caches current active TPM interface type.
> 
> -  @retval EFI_SUCCESS   DTPM2.0 instance is registered, or system dose not
> surpport registr DTPM2.0 instance
> +  @retval EFI_SUCCESS   DTPM2.0 instance is registered, or system does not
> support register DTPM2.0 instance
>  **/
>  EFI_STATUS
>  EFIAPI
> diff --git a/SecurityPkg/Library/Tpm2DeviceLibDTpm/Tpm2InstanceLibDTpm.c
> b/SecurityPkg/Library/Tpm2DeviceLibDTpm/Tpm2InstanceLibDTpm.c
> index 435cbf6298..691eaa40c0 100644
> --- a/SecurityPkg/Library/Tpm2DeviceLibDTpm/Tpm2InstanceLibDTpm.c
> +++ b/SecurityPkg/Library/Tpm2DeviceLibDTpm/Tpm2InstanceLibDTpm.c
> @@ -93,7 +93,7 @@ TPM2_DEVICE_INTERFACE  mDTpm2InternalTpm2Device
> = {
>  /**
>    The function register DTPM2.0 instance and caches current active TPM
> interface type.
> 
> -  @retval EFI_SUCCESS   DTPM2.0 instance is registered, or system dose not
> surpport registr DTPM2.0 instance
> +  @retval EFI_SUCCESS   DTPM2.0 instance is registered, or system does not
> support register DTPM2.0 instance
>  **/
>  EFI_STATUS
>  EFIAPI
> diff --git a/SecurityPkg/Library/Tpm2DeviceLibDTpm/Tpm2Ptp.c
> b/SecurityPkg/Library/Tpm2DeviceLibDTpm/Tpm2Ptp.c
> index 8596143b53..b671cef04e 100644
> --- a/SecurityPkg/Library/Tpm2DeviceLibDTpm/Tpm2Ptp.c
> +++ b/SecurityPkg/Library/Tpm2DeviceLibDTpm/Tpm2Ptp.c
> @@ -327,7 +327,7 @@ PtpCrbTpmCommand (
> 
>  GoReady_Exit:
>    //
> -  // Goto Ready State if command is completed succesfully and TPM support
> IdleBypass
> +  // Goto Ready State if command is completed successfully and TPM support
> IdleBypass
>    // If not supported. flow down to GoIdle
>    //
>    if (PcdGet8(PcdCRBIdleByPass) == 1) {
> @@ -347,7 +347,7 @@ GoIdle_Exit:
>    MmioWrite32((UINTN)&CrbReg->CrbControlRequest,
> PTP_CRB_CONTROL_AREA_REQUEST_GO_IDLE);
> 
>    //
> -  // Only enforce Idle state transition if execution fails when CRBIndleBypass==1
> +  // Only enforce Idle state transition if execution fails when CRBIdleBypass==1
>    // Leave regular Idle delay at the beginning of next command execution
>    //
>    if (PcdGet8(PcdCRBIdleByPass) == 1){
> diff --git a/SecurityPkg/Library/Tpm2DeviceLibDTpm/Tpm2Tis.c
> b/SecurityPkg/Library/Tpm2DeviceLibDTpm/Tpm2Tis.c
> index d2ce8d1f09..bbebc94b4d 100644
> --- a/SecurityPkg/Library/Tpm2DeviceLibDTpm/Tpm2Tis.c
> +++ b/SecurityPkg/Library/Tpm2DeviceLibDTpm/Tpm2Tis.c
> @@ -81,7 +81,7 @@ TisPcWaitRegisterBits (
>    in the time of default TIS_TIMEOUT_D.
> 
>    @param[in]  TisReg                Pointer to TIS register.
> -  @param[out] BurstCount            Pointer to a buffer to store the got BurstConut.
> +  @param[out] BurstCount            Pointer to a buffer to store the got
> BurstCount.
> 
>    @retval     EFI_SUCCESS           Get BurstCount.
>    @retval     EFI_INVALID_PARAMETER TisReg is NULL or BurstCount is NULL.
> @@ -303,7 +303,7 @@ Tpm2TisTpmCommand (
>                 TIS_TIMEOUT_B
>                 );
>      //
> -    // Do not clear CANCEL bit here bicoz Writes of 0 to this bit are ignored
> +    // Do not clear CANCEL bit here because Writes of 0 to this bit are ignored
>      //
>      if (EFI_ERROR (Status)) {
>        //
> diff --git a/SecurityPkg/Library/TpmCommLib/CommonHeader.h
> b/SecurityPkg/Library/TpmCommLib/CommonHeader.h
> index 386c9fab44..aa2e412a11 100644
> --- a/SecurityPkg/Library/TpmCommLib/CommonHeader.h
> +++ b/SecurityPkg/Library/TpmCommLib/CommonHeader.h
> @@ -1,5 +1,5 @@
>  /** @file
> -  The intenal header file for TpmCommLib.
> +  The internal header file for TpmCommLib.
> 
>  Copyright (c) 2006 - 2018, Intel Corporation. All rights reserved.<BR>
>  SPDX-License-Identifier: BSD-2-Clause-Patent
> diff --git a/SecurityPkg/Library/TpmCommLib/TisPc.c
> b/SecurityPkg/Library/TpmCommLib/TisPc.c
> index 162e883d21..4a14a2bf81 100644
> --- a/SecurityPkg/Library/TpmCommLib/TisPc.c
> +++ b/SecurityPkg/Library/TpmCommLib/TisPc.c
> @@ -64,7 +64,7 @@ TisPcWaitRegisterBits (
>    in the time of default TIS_TIMEOUT_D.
> 
>    @param[in]  TisReg                Pointer to TIS register.
> -  @param[out] BurstCount            Pointer to a buffer to store the got BurstConut.
> +  @param[out] BurstCount            Pointer to a buffer to store the got
> BurstCount.
> 
>    @retval     EFI_SUCCESS           Get BurstCount.
>    @retval     EFI_INVALID_PARAMETER TisReg is NULL or BurstCount is NULL.
> diff --git a/SecurityPkg/Pkcs7Verify/Pkcs7VerifyDxe/Pkcs7VerifyDxe.c
> b/SecurityPkg/Pkcs7Verify/Pkcs7VerifyDxe/Pkcs7VerifyDxe.c
> index a113df3718..a96862f937 100644
> --- a/SecurityPkg/Pkcs7Verify/Pkcs7VerifyDxe/Pkcs7VerifyDxe.c
> +++ b/SecurityPkg/Pkcs7Verify/Pkcs7VerifyDxe/Pkcs7VerifyDxe.c
> @@ -494,7 +494,7 @@ IsValidTimestamp (
>      TsaCertSize = SigList->SignatureSize - sizeof (EFI_GUID);
> 
>      //
> -    // Each TSA Certificate will normally be in a seperate EFI_SIGNATURE_LIST
> +    // Each TSA Certificate will normally be in a separate EFI_SIGNATURE_LIST
>      // Leverage ImageTimestampVerify interface for Timestamp
> counterSignature Verification
>      //
>      if (ImageTimestampVerify (SignedData, SignedDataSize, TsaCert, TsaCertSize,
> &SigningTime)) {
> @@ -514,7 +514,7 @@ IsValidTimestamp (
>  /**
>    Check whether the PKCS7 signedData is revoked by verifying with the revoked
>    certificates database, and if the signedData is timestamped, the embedded
> timestamp
> -  couterSignature will be checked with the supplied timestamp database.
> +  counterSignature will be checked with the supplied timestamp database.
> 
>    @param[in]  SignedData      Pointer to buffer containing ASN.1 DER-encoded
> PKCS7
>                                signature.
> @@ -657,8 +657,8 @@ P7CheckRevocationByHash (
>        Status = EFI_SUCCESS;
>        if (IsValidTimestamp (SignedData, SignedDataSize, TimeStampDb,
> &RevocationTime)) {
>          //
> -        // Use EFI_NOT_READY to identify the P7Data is not reovked, because the
> timestamping
> -        // occured prior to the time of certificate revocation.
> +        // Use EFI_NOT_READY to identify the P7Data is not revoked, because the
> timestamping
> +        // occurred prior to the time of certificate revocation.
>          //
>          Status = EFI_NOT_READY;
>        }
> @@ -679,7 +679,7 @@ _Exit:
>  /**
>    Check whether the PKCS7 signedData is revoked by verifying with the revoked
>    certificates database, and if the signedData is timestamped, the embedded
> timestamp
> -  couterSignature will be checked with the supplied timestamp database.
> +  counterSignature will be checked with the supplied timestamp database.
> 
>    @param[in]  SignedData      Pointer to buffer containing ASN.1 DER-encoded
> PKCS7
>                                signature.
> @@ -822,8 +822,8 @@ P7CheckRevocation (
>        Status = EFI_SUCCESS;
>        if (IsValidTimestamp (SignedData, SignedDataSize, TimeStampDb,
> &RevocationTime)) {
>          //
> -        // Use EFI_NOT_READY to identify the P7Data is not reovked, because the
> timestamping
> -        // occured prior to the time of certificate revocation.
> +        // Use EFI_NOT_READY to identify the P7Data is not revoked, because the
> timestamping
> +        // occurred prior to the time of certificate revocation.
>          //
>          Status = EFI_NOT_READY;
>        }
> @@ -1261,7 +1261,7 @@ VerifyBuffer (
>                 );
>      if (!EFI_ERROR (Status)) {
>        //
> -      // The PKCS7 SignedData is reovked
> +      // The PKCS7 SignedData is revoked
>        //
>        Status = EFI_SECURITY_VIOLATION;
>        goto _Exit;
> @@ -1413,7 +1413,7 @@ VerifySignature (
> 
>      if (!EFI_ERROR (Status)) {
>        //
> -      // The PKCS7 SignedData is reovked
> +      // The PKCS7 SignedData is revoked
>        //
>        return EFI_SECURITY_VIOLATION;
>      }
> diff --git a/SecurityPkg/RandomNumberGenerator/RngDxe/RdRand.c
> b/SecurityPkg/RandomNumberGenerator/RngDxe/RdRand.c
> index 59789533b0..e7dd5ab181 100644
> --- a/SecurityPkg/RandomNumberGenerator/RngDxe/RdRand.c
> +++ b/SecurityPkg/RandomNumberGenerator/RngDxe/RdRand.c
> @@ -79,7 +79,7 @@ RdRandGetSeed128 (
>    UINT32      Index2;
> 
>    //
> -  // Chose an arbitary key and zero the feed_forward_value (FFV)
> +  // Chose an arbitrary key and zero the feed_forward_value (FFV)
>    //
>    for (Index = 0; Index < 16; Index++) {
>      Key[Index] = (UINT8) Index;
> diff --git a/SecurityPkg/SecurityPkg.dec b/SecurityPkg/SecurityPkg.dec
> index 8249bf0220..cac36caf0a 100644
> --- a/SecurityPkg/SecurityPkg.dec
> +++ b/SecurityPkg/SecurityPkg.dec
> @@ -68,11 +68,11 @@ [LibraryClasses]
>    #
>    Tcg2PhysicalPresenceLib|Include/Library/Tcg2PhysicalPresenceLib.h
> 
> -  ## @libraryclass  Provides interfaces about TCG storage generic commond.
> +  ## @libraryclass  Provides interfaces about TCG storage generic command.
>    #
>    TcgStorageCoreLib|Include/Library/TcgStorageCoreLib.h
> 
> -  ## @libraryclass  Provides interfaces about TCG storage Opal generic
> commond.
> +  ## @libraryclass  Provides interfaces about TCG storage Opal generic
> command.
>    #
>    TcgStorageOpalLib|Include/Library/TcgStorageOpalLib.h
> 
> @@ -369,9 +369,9 @@ [PcdsFixedAtBuild, PcdsPatchableInModule,
> PcdsDynamic, PcdsDynamicEx]
>    # @Prompt TPM device address.
> 
> gEfiSecurityPkgTokenSpaceGuid.PcdTpmBaseAddress|0xFED40000|UINT64|0x0
> 0010012
> 
> -  ## This PCR means the OEM configurated number of PCR banks.
> +  ## This PCR means the OEM configured number of PCR banks.
>    #  0 means dynamic get from supported HASH algorithm
> -  # @Prompt OEM configurated number of PCR banks.
> +  # @Prompt OEM configured number of PCR banks.
> 
> gEfiSecurityPkgTokenSpaceGuid.PcdTcg2NumberOfPCRBanks|0x0|UINT32|0x0
> 0010015
> 
>    ## Provides one or more SHA 256 Hashes of the RSA 2048 public keys used to
> verify Recovery and Capsule Update images
> @@ -469,7 +469,7 @@ [PcdsDynamic, PcdsDynamicEx]
> 
> gEfiSecurityPkgTokenSpaceGuid.PcdTcg2HashAlgorithmBitmap|0xFFFFFFFF|UIN
> T32|0x00010016
> 
>    ## This PCD indicates current active TPM interface type.
> -  #  Accodingt to TCG PTP spec 1.3, there are 3 types defined in
> TPM2_PTP_INTERFACE_TYPE.<BR>
> +  #  According to TCG PTP spec 1.3, there are 3 types defined in
> TPM2_PTP_INTERFACE_TYPE.<BR>
>    #  0x00 - FIFO interface as defined in TIS 1.3 is active.<BR>
>    #  0x01 - FIFO interface as defined in PTP for TPM 2.0 is active.<BR>
>    #  0x02 - CRB interface is active.<BR>
> @@ -478,14 +478,14 @@ [PcdsDynamic, PcdsDynamicEx]
>    # @Prompt current active TPM interface type.
> 
> gEfiSecurityPkgTokenSpaceGuid.PcdActiveTpmInterfaceType|0xFF|UINT8|0x00
> 01001E
> 
> -  ## This PCD records IdleByass status supported by current active TPM
> interface.
> -  #  Accodingt to TCG PTP spec 1.3, TPM with CRB interface can skip idle state
> and
> -  #  diretcly move to CmdReady state. <BR>
> +  ## This PCD records IdleBypass status supported by current active TPM
> interface.
> +  #  According to TCG PTP spec 1.3, TPM with CRB interface can skip idle state
> and
> +  #  directly move to CmdReady state. <BR>
>    #  0x00 - Do not support IdleByPass.<BR>
>    #  0x01 - Support IdleByPass.<BR>
>    #  0xFF - IdleByPass State is not synced with TPM hardware.<BR>
>    #
> -  # @Prompt IdleByass status supported by current active TPM interface.
> +  # @Prompt IdleBypass status supported by current active TPM interface.
>    gEfiSecurityPkgTokenSpaceGuid.PcdCRBIdleByPass|0xFF|UINT8|0x0001001F
> 
>    ## This PCD records LAML field in TPM2 ACPI table.
> diff --git a/SecurityPkg/SecurityPkg.dsc b/SecurityPkg/SecurityPkg.dsc
> index aef2ac3a20..9a254087a3 100644
> --- a/SecurityPkg/SecurityPkg.dsc
> +++ b/SecurityPkg/SecurityPkg.dsc
> @@ -70,7 +70,7 @@ [LibraryClasses]
>  [LibraryClasses.ARM]
>    #
>    # It is not possible to prevent the ARM compiler for generic intrinsic functions.
> -  # This library provides the instrinsic functions generate by a given compiler.
> +  # This library provides the intrinsic functions generate by a given compiler.
>    # And NULL mean link this library into all ARM images.
>    #
>    NULL|ArmPkg/Library/CompilerIntrinsicsLib/CompilerIntrinsicsLib.inf
> diff --git a/SecurityPkg/SecurityPkg.uni b/SecurityPkg/SecurityPkg.uni
> index ee79b98f0d..68587304d7 100644
> --- a/SecurityPkg/SecurityPkg.uni
> +++ b/SecurityPkg/SecurityPkg.uni
> @@ -80,7 +80,7 @@
> 
>  #string
> STR_gEfiSecurityPkgTokenSpaceGuid_PcdTpmPhysicalPresence_PROMPT
> #language en-US "Physical presence of the platform operator."
> 
> -#string STR_gEfiSecurityPkgTokenSpaceGuid_PcdTpmPhysicalPresence_HELP
> #language en-US "Indicates the presence or absence of the platform operator
> during firmware booting. If platform operator is not physical presnece during
> boot. TPM will be locked and the TPM commands that required operator
> physical presence can not run.<BR><BR>\n"
> +#string STR_gEfiSecurityPkgTokenSpaceGuid_PcdTpmPhysicalPresence_HELP
> #language en-US "Indicates the presence or absence of the platform operator
> during firmware booting. If platform operator is not physical presence during
> boot. TPM will be locked and the TPM commands that required operator
> physical presence can not run.<BR><BR>\n"
>                                                                                         "TRUE  - The platform operator
> is physically present.<BR>\n"
>                                                                                         "FALSE - The platform operator
> is not physically present.<BR>"
> 
> @@ -176,9 +176,9 @@
>  #string
> STR_gEfiSecurityPkgTokenSpaceGuid_PcdRsa2048Sha256PublicKeyBuffer_HELP
> #language en-US "Provides one or more SHA 256 Hashes of the RSA 2048 public
> keys used to verify Recovery and Capsule Update images\n"
>                                                                                                  "WARNING: The default
> value is treated as test key. Please do not use default value in the production."
> 
> -#string
> STR_gEfiSecurityPkgTokenSpaceGuid_PcdTcg2NumberOfPCRBanks_PROMPT
> #language en-US "OEM configurated number of PCR banks."
> +#string
> STR_gEfiSecurityPkgTokenSpaceGuid_PcdTcg2NumberOfPCRBanks_PROMPT
> #language en-US "OEM configured number of PCR banks."
> 
> -#string
> STR_gEfiSecurityPkgTokenSpaceGuid_PcdTcg2NumberOfPCRBanks_HELP
> #language en-US "This PCR means the OEM configurated number of PCR
> banks.\n"
> +#string
> STR_gEfiSecurityPkgTokenSpaceGuid_PcdTcg2NumberOfPCRBanks_HELP
> #language en-US "This PCR means the OEM configured number of PCR banks.\n"
>                                                                                          "0 means dynamic get from
> supported HASH algorithm"
> 
>  #string
> STR_gEfiSecurityPkgTokenSpaceGuid_PcdTcg2HashAlgorithmBitmap_PROMPT
> #language en-US "Hash Algorithm bitmap."
> @@ -256,10 +256,10 @@
>                                                                                            "0x02 - CRB interface is
> active.<BR>\n"
>                                                                                            "0xFF - Contains no current
> active TPM interface type<BR>"
> 
> -#string STR_gEfiSecurityPkgTokenSpaceGuid_PcdCRBIdleByPass_PROMPT
> #language en-US "IdleByass status supported by current active TPM interface."
> +#string STR_gEfiSecurityPkgTokenSpaceGuid_PcdCRBIdleByPass_PROMPT
> #language en-US "IdleBypass status supported by current active TPM interface."
> 
> -#string STR_gEfiSecurityPkgTokenSpaceGuid_PcdCRBIdleByPass_HELP
> #language en-US "This PCD records IdleByass status supported by current active
> TPM interface.\n"
> -                                                                                          "Accodingt to TCG PTP spec
> 1.3, TPM with CRB interface can skip idle state and diretcly move to CmdReady
> state. <BR>"
> +#string STR_gEfiSecurityPkgTokenSpaceGuid_PcdCRBIdleByPass_HELP
> #language en-US "This PCD records IdleBypass status supported by current
> active TPM interface.\n"
> +                                                                                          "According to TCG PTP spec
> 1.3, TPM with CRB interface can skip idle state and directly move to CmdReady
> state. <BR>"
>                                                                                            "0x01 - Do not support
> IdleByPass.<BR>\n"
>                                                                                            "0x02 - Support
> IdleByPass.<BR>\n"
>                                                                                            "0xFF - IdleByPass State is not
> synced with TPM hardware.<BR>"
> diff --git a/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.c
> b/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.c
> index fe2bd749fb..97e819edf4 100644
> --- a/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.c
> +++ b/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.c
> @@ -1,7 +1,7 @@
>  /** @file
>    TCG MOR (Memory Overwrite Request) Control Driver.
> 
> -  This driver initilize MemoryOverwriteRequestControl variable. It
> +  This driver initialize MemoryOverwriteRequestControl variable. It
>    will clear MOR_CLEAR_MEMORY_BIT bit if it is set. It will also do TPer Reset
> for
>    those encrypted drives through
> EFI_STORAGE_SECURITY_COMMAND_PROTOCOL at EndOfDxe.
> 
> @@ -298,7 +298,7 @@ TPerResetAtEndOfDxe (
>    @param[in] ImageHandle  Image handle of this driver.
>    @param[in] SystemTable  A Pointer to the EFI System Table.
> 
> -  @retval EFI_SUCEESS
> +  @retval EFI_SUCCESS
>    @return Others          Some error occurs.
>  **/
>  EFI_STATUS
> @@ -341,7 +341,7 @@ MorDriverEntryPoint (
>      //
>      // Create a Ready To Boot Event and Clear the MorControl bit in the call back
> function.
>      //
> -    DEBUG ((EFI_D_INFO, "TcgMor: Create ReadyToBoot Event for MorControl
> Bit cleanning!\n"));
> +    DEBUG ((DEBUG_INFO, "TcgMor: Create ReadyToBoot Event for MorControl
> Bit cleaning!\n"));
>      Status = EfiCreateEventReadyToBootEx (
>                 TPL_CALLBACK,
>                 OnReadyToBoot,
> diff --git a/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.inf
> b/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.inf
> index de624423a6..906bafa04a 100644
> --- a/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.inf
> +++ b/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.inf
> @@ -1,5 +1,5 @@
>  ## @file
> -#  Initilizes MemoryOverwriteRequestControl variable
> +#  initializes MemoryOverwriteRequestControl variable
>  #
>  #  This module will clear MOR_CLEAR_MEMORY_BIT bit if it is set. It will also do
>  #  TPer Reset for those encrypted drives through
> EFI_STORAGE_SECURITY_COMMAND_PROTOCOL
> diff --git a/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.uni
> b/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.uni
> index 0c499e7d1a..6a1df10394 100644
> --- a/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.uni
> +++ b/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.uni
> @@ -1,5 +1,5 @@
>  // /** @file
> -// Initilizes MemoryOverwriteRequestControl variable
> +// initializes MemoryOverwriteRequestControl variable
>  //
>  // This module will clear MOR_CLEAR_MEMORY_BIT bit if it is set.
>  //
> diff --git
> a/SecurityPkg/Tcg/MemoryOverwriteRequestControlLock/TcgMorLock.c
> b/SecurityPkg/Tcg/MemoryOverwriteRequestControlLock/TcgMorLock.c
> index ef49fd928e..aa230eeefa 100644
> --- a/SecurityPkg/Tcg/MemoryOverwriteRequestControlLock/TcgMorLock.c
> +++ b/SecurityPkg/Tcg/MemoryOverwriteRequestControlLock/TcgMorLock.c
> @@ -1,7 +1,7 @@
>  /** @file
>    TCG MOR (Memory Overwrite Request) Lock Control Driver.
> 
> -  This driver initilize MemoryOverwriteRequestControlLock variable.
> +  This driver initializes MemoryOverwriteRequestControlLock variable.
>    This module will add Variable Hook and allow
> MemoryOverwriteRequestControlLock variable set only once.
> 
>  Copyright (c) 2015 - 2018, Intel Corporation. All rights reserved.<BR>
> @@ -167,7 +167,7 @@ SetVariableCheckHandlerMor (
>    @param[in] ImageHandle  Image handle of this driver.
>    @param[in] SystemTable  A Pointer to the EFI System Table.
> 
> -  @retval EFI_SUCEESS
> +  @retval EFI_SUCCESS
>    @return Others          Some error occurs.
>  **/
>  EFI_STATUS
> diff --git
> a/SecurityPkg/Tcg/MemoryOverwriteRequestControlLock/TcgMorLock.h
> b/SecurityPkg/Tcg/MemoryOverwriteRequestControlLock/TcgMorLock.h
> index 2ef194f92f..5a6658c158 100644
> --- a/SecurityPkg/Tcg/MemoryOverwriteRequestControlLock/TcgMorLock.h
> +++ b/SecurityPkg/Tcg/MemoryOverwriteRequestControlLock/TcgMorLock.h
> @@ -119,7 +119,7 @@ SetVariableCheckHandlerMor (
>    @param[in] ImageHandle  Image handle of this driver.
>    @param[in] SystemTable  A Pointer to the EFI System Table.
> 
> -  @retval EFI_SUCEESS
> +  @retval EFI_SUCCESS
>    @return Others          Some error occurs.
>  **/
>  EFI_STATUS
> diff --git
> a/SecurityPkg/Tcg/MemoryOverwriteRequestControlLock/TcgMorLock.uni
> b/SecurityPkg/Tcg/MemoryOverwriteRequestControlLock/TcgMorLock.uni
> index 639a3edcba..711b37d866 100644
> --- a/SecurityPkg/Tcg/MemoryOverwriteRequestControlLock/TcgMorLock.uni
> +++ b/SecurityPkg/Tcg/MemoryOverwriteRequestControlLock/TcgMorLock.uni
> @@ -1,5 +1,5 @@
>  // /** @file
> -// Initilizes MemoryOverwriteRequestControlLock variable
> +// Initializes MemoryOverwriteRequestControlLock variable
>  //
>  // This module will add Variable Hook and allow
> MemoryOverwriteRequestControlLock variable set only once.
>  //
> @@ -10,7 +10,7 @@
>  // **/
> 
> 
> -#string STR_MODULE_ABSTRACT             #language en-US "Initilizes
> MemoryOverwriteRequestControlLock variable"
> +#string STR_MODULE_ABSTRACT             #language en-US "Initializes
> MemoryOverwriteRequestControlLock variable"
> 
>  #string STR_MODULE_DESCRIPTION          #language en-US "This module will
> add Variable Hook and allow MemoryOverwriteRequestControlLock variable set
> only once."
> 
> diff --git
> a/SecurityPkg/Tcg/MemoryOverwriteRequestControlLock/TcgMorLockSmm.inf
> b/SecurityPkg/Tcg/MemoryOverwriteRequestControlLock/TcgMorLockSmm.inf
> index 67d6bda269..875c1e5f3a 100644
> ---
> a/SecurityPkg/Tcg/MemoryOverwriteRequestControlLock/TcgMorLockSmm.inf
> +++
> b/SecurityPkg/Tcg/MemoryOverwriteRequestControlLock/TcgMorLockSmm.inf
> @@ -1,5 +1,5 @@
>  ## @file
> -#  Initilizes MemoryOverwriteRequestControlLock variable
> +#  Initializes MemoryOverwriteRequestControlLock variable
>  #
>  #  This module will add Variable Hook and allow
> MemoryOverwriteRequestControlLock variable set only once.
>  #
> diff --git a/SecurityPkg/Tcg/Opal/OpalPassword/OpalDriver.c
> b/SecurityPkg/Tcg/Opal/OpalPassword/OpalDriver.c
> index e14fa32354..77905d2bf9 100644
> --- a/SecurityPkg/Tcg/Opal/OpalPassword/OpalDriver.c
> +++ b/SecurityPkg/Tcg/Opal/OpalPassword/OpalDriver.c
> @@ -2583,7 +2583,7 @@ OpalDriverGetDriverDeviceName(
>    @param ImageHandle     Image Handle this driver.
>    @param SystemTable     Pointer to SystemTable.
> 
> -  @retval EFI_SUCESS     This function always complete successfully.
> +  @retval EFI_SUCCESS     This function always complete successfully.
>  **/
>  EFI_STATUS
>  EFIAPI
> @@ -2637,7 +2637,7 @@ EfiDriverEntryPoint(
>    Tests to see if this driver supports a given controller.
> 
>    This function checks to see if the controller contains an instance of the
> -  EFI_STORAGE_SECURITY_COMMAND_PROTOCOL and the
> EFI_BLOCK_IO_PROTOCL
> +  EFI_STORAGE_SECURITY_COMMAND_PROTOCOL and the
> EFI_BLOCK_IO_PROTOCOL
>    and returns EFI_SUCCESS if it does.
> 
>    @param[in]  This                  A pointer to the EFI_DRIVER_BINDING_PROTOCOL
> instance.
> @@ -2741,7 +2741,7 @@ OpalEfiDriverBindingSupported(
>    "controller", which is a child Handle, contains the
> EF_STORAGE_SECURITY_COMMAND protocols.
>    This function will complete the other necessary checks, such as verifying the
> device supports
>    the correct version of Opal.  Upon verification, it will add the device to the
> -  Opal HII list in order to expose Opal managmeent options.
> +  Opal HII list in order to expose Opal management options.
> 
>    @param[in]  This                  A pointer to the EFI_DRIVER_BINDING_PROTOCOL
> instance.
>    @param[in]  ControllerHandle      The Handle of the controller to start. This
> Handle
> diff --git a/SecurityPkg/Tcg/Opal/OpalPassword/OpalDriver.h
> b/SecurityPkg/Tcg/Opal/OpalPassword/OpalDriver.h
> index beeabb1c0a..c19d78218e 100644
> --- a/SecurityPkg/Tcg/Opal/OpalPassword/OpalDriver.h
> +++ b/SecurityPkg/Tcg/Opal/OpalPassword/OpalDriver.h
> @@ -61,7 +61,7 @@ extern EFI_DRIVER_BINDING_PROTOCOL
> gOpalDriverBinding;
>  extern EFI_COMPONENT_NAME_PROTOCOL   gOpalComponentName;
>  extern EFI_COMPONENT_NAME2_PROTOCOL  gOpalComponentName2;
> 
> -#define OPAL_MSID_LENGHT        128
> +#define OPAL_MSID_LENGTH        128
> 
>  #define MAX_PASSWORD_TRY_COUNT  5
> 
> @@ -131,7 +131,7 @@ typedef struct {
>  //
>  typedef struct {
>    UINT32                                          MsidLength;             // Byte length of MSID Pin
> for device
> -  UINT8                                           Msid[OPAL_MSID_LENGHT]; // MSID Pin for
> device
> +  UINT8                                           Msid[OPAL_MSID_LENGTH]; // MSID Pin for
> device
>    EFI_STORAGE_SECURITY_COMMAND_PROTOCOL           *Sscp;
>    UINT32                                          MediaId;                // MediaId is used by Ssc
> Protocol.
>    EFI_DEVICE_PATH_PROTOCOL                        *OpalDevicePath;
> @@ -309,7 +309,7 @@ OpalEfiDriverBindingSupported(
>    "controller", which is a child handle, contains the
> EF_STORAGE_SECURITY_COMMAND protocols.
>    This function will complete the other necessary checks, such as verifying the
> device supports
>    the correct version of Opal.  Upon verification, it will add the device to the
> -  Opal HII list in order to expose Opal managmeent options.
> +  Opal HII list in order to expose Opal management options.
> 
>    @param[in]  This                  A pointer to the EFI_DRIVER_BINDING_PROTOCOL
> instance.
>    @param[in]  ControllerHandle      The handle of the controller to start. This
> handle
> diff --git a/SecurityPkg/Tcg/Opal/OpalPassword/OpalHii.c
> b/SecurityPkg/Tcg/Opal/OpalPassword/OpalHii.c
> index 18b65da756..ef117d3d03 100644
> --- a/SecurityPkg/Tcg/Opal/OpalPassword/OpalHii.c
> +++ b/SecurityPkg/Tcg/Opal/OpalPassword/OpalHii.c
> @@ -1090,7 +1090,7 @@ OpalHiiSetBrowserData (
> 
>  /**
> 
> -  Populate the hii_g_Configuraton with the browser Data.
> +  Populate the hii_g_Configuration with the browser Data.
> 
>  **/
>  VOID
> @@ -1164,7 +1164,7 @@ HiiSetFormString(
> 
>    @param Dev                  The Opal device.
> 
> -  @retval EFI_SUCESS          Initialize the device success.
> +  @retval EFI_SUCCESS          Initialize the device success.
>    @retval EFI_DEVICE_ERROR    Get info from device failed.
> 
>  **/
> @@ -1193,7 +1193,7 @@ OpalDiskInitialize (
>    }
>    Session.OpalBaseComId = Dev->OpalDisk.OpalBaseComId;
> 
> -  TcgResult = OpalUtilGetMsid (&Session, Dev->OpalDisk.Msid,
> OPAL_MSID_LENGHT, &Dev->OpalDisk.MsidLength);
> +  TcgResult = OpalUtilGetMsid (&Session, Dev->OpalDisk.Msid,
> OPAL_MSID_LENGTH, &Dev->OpalDisk.MsidLength);
>    if (TcgResult != TcgResultSuccess) {
>      return EFI_DEVICE_ERROR;
>    }
> @@ -1220,7 +1220,7 @@ OpalDiskInitialize (
> 
>    @param OpalDisk                The Opal device.
> 
> -  @retval EFI_SUCESS             Get ownership success.
> +  @retval EFI_SUCCESS             Get ownership success.
>    @retval EFI_ACCESS_DENIED      Has send BlockSID command, can't change
> ownership.
>    @retval EFI_INVALID_PARAMETER  Not get Msid info before get ownership
> info.
> 
> @@ -1254,7 +1254,7 @@ OpalDiskUpdateOwnerShip (
> 
>    @param OpalDisk                The Opal device.
> 
> -  @retval EFI_SUCESS             Initialize the device success.
> +  @retval EFI_SUCCESS             Initialize the device success.
>    @retval EFI_DEVICE_ERROR       Get info from device failed.
>    @retval EFI_INVALID_PARAMETER  Not get Msid info before get ownership
> info.
>    @retval EFI_ACCESS_DENIED      Has send BlockSID command, can't change
> ownership.
> diff --git a/SecurityPkg/Tcg/Opal/OpalPassword/OpalHii.h
> b/SecurityPkg/Tcg/Opal/OpalPassword/OpalHii.h
> index 89c709df99..0543a2ce49 100644
> --- a/SecurityPkg/Tcg/Opal/OpalPassword/OpalHii.h
> +++ b/SecurityPkg/Tcg/Opal/OpalPassword/OpalHii.h
> @@ -156,7 +156,7 @@ OpalHiiSetBrowserData (
> 
>  /**
> 
> -  Populate the hii_g_Configuraton with the browser Data.
> +  Populate the hii_g_Configuration with the browser Data.
> 
>  **/
>  VOID
> @@ -247,7 +247,7 @@ GetDiskNameStringId(
> 
>    @param OpalDisk                The Opal device.
> 
> -  @retval EFI_SUCESS             Initialize the device success.
> +  @retval EFI_SUCCESS             Initialize the device success.
>    @retval EFI_DEVICE_ERROR       Get info from device failed.
>    @retval EFI_INVALID_PARAMETER  Not get Msid info before get ownership
> info.
> 
> @@ -348,7 +348,7 @@ HiiUninstall(
> 
>    @param Dev                  The Opal device.
> 
> -  @retval EFI_SUCESS          Initialize the device success.
> +  @retval EFI_SUCCESS          Initialize the device success.
>    @retval EFI_DEVICE_ERROR    Get info from device failed.
> 
>  **/
> @@ -362,7 +362,7 @@ OpalDiskInitialize (
> 
>    @param OpalDisk                The Opal device.
> 
> -  @retval EFI_SUCESS             Get ownership success.
> +  @retval EFI_SUCCESS             Get ownership success.
>    @retval EFI_ACCESS_DENIED      Has send BlockSID command, can't change
> ownership.
>    @retval EFI_INVALID_PARAMETER  Not get Msid info before get ownership
> info.
> 
> diff --git a/SecurityPkg/Tcg/PhysicalPresencePei/PhysicalPresencePei.c
> b/SecurityPkg/Tcg/PhysicalPresencePei/PhysicalPresencePei.c
> index f25481044f..83c3ad50d1 100644
> --- a/SecurityPkg/Tcg/PhysicalPresencePei/PhysicalPresencePei.c
> +++ b/SecurityPkg/Tcg/PhysicalPresencePei/PhysicalPresencePei.c
> @@ -31,7 +31,7 @@ LockTpmPhysicalPresence (
>    );
> 
>  //
> -// Gobal defintions for lock physical presence PPI and its descriptor.
> +// Global defintions for lock physical presence PPI and its descriptor.
>  //
>  PEI_LOCK_PHYSICAL_PRESENCE_PPI    mLockPhysicalPresencePpi = {
>    LockTpmPhysicalPresence
> diff --git a/SecurityPkg/Tcg/Tcg2Config/Tcg2Config.vfr
> b/SecurityPkg/Tcg/Tcg2Config/Tcg2Config.vfr
> index 12d8adb7ea..91a463997c 100644
> --- a/SecurityPkg/Tcg/Tcg2Config/Tcg2Config.vfr
> +++ b/SecurityPkg/Tcg/Tcg2Config/Tcg2Config.vfr
> @@ -16,19 +16,19 @@ formset
> 
>    efivarstore TCG2_CONFIGURATION_INFO,
>      varid = TCG2_CONFIGURATION_INFO_VARSTORE_ID,
> -    attribute = 0x02,  // EFI variable attribures
> EFI_VARIABLE_BOOTSERVICE_ACCESS
> +    attribute = 0x02,  // EFI variable attributes
> EFI_VARIABLE_BOOTSERVICE_ACCESS
>      name  = TCG2_CONFIGURATION_INFO,
>      guid  = TCG2_CONFIG_FORM_SET_GUID;
> 
>    efivarstore TCG2_CONFIGURATION,
>      varid = TCG2_CONFIGURATION_VARSTORE_ID,
> -    attribute = 0x03,  // EFI variable attribures
> EFI_VARIABLE_BOOTSERVICE_ACCESS | EFI_VARIABLE_NON_VOLATILE
> +    attribute = 0x03,  // EFI variable attributes
> EFI_VARIABLE_BOOTSERVICE_ACCESS | EFI_VARIABLE_NON_VOLATILE
>      name  = TCG2_CONFIGURATION,
>      guid  = TCG2_CONFIG_FORM_SET_GUID;
> 
>    efivarstore TCG2_VERSION,
>      varid = TCG2_VERSION_VARSTORE_ID,
> -    attribute = 0x03,  // EFI variable attribures
> EFI_VARIABLE_BOOTSERVICE_ACCESS | EFI_VARIABLE_NON_VOLATILE
> +    attribute = 0x03,  // EFI variable attributes
> EFI_VARIABLE_BOOTSERVICE_ACCESS | EFI_VARIABLE_NON_VOLATILE
>      name  = TCG2_VERSION,
>      guid  = TCG2_CONFIG_FORM_SET_GUID;
> 
> diff --git a/SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigDriver.c
> b/SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigDriver.c
> index 9052eced75..10d1c5cc78 100644
> --- a/SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigDriver.c
> +++ b/SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigDriver.c
> @@ -240,7 +240,7 @@ InitializeTcg2VersionInfo (
> 
>    @retval EFI_ALREADY_STARTED    The driver already exists in system.
>    @retval EFI_OUT_OF_RESOURCES   Fail to execute entry point due to lack of
> resources.
> -  @retval EFI_SUCCES             All the related protocols are installed on the driver.
> +  @retval EFI_SUCCESS            All the related protocols are installed on the
> driver.
>    @retval Others                 Fail to install protocols as indicated.
> 
>  **/
> diff --git a/SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigPeim.c
> b/SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigPeim.c
> index a15919685e..e0d0a207e1 100644
> --- a/SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigPeim.c
> +++ b/SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigPeim.c
> @@ -57,7 +57,7 @@ DetectTpmDevice (
>    @param  FileHandle  Handle of the file being invoked.
>    @param  PeiServices Describes the list of possible PEI Services.
> 
> -  @retval EFI_SUCCES             Convert variable to PCD successfully.
> +  @retval EFI_SUCCESS            Convert variable to PCD successfully.
>    @retval Others                 Fail to convert variable to PCD.
>  **/
>  EFI_STATUS
> @@ -118,7 +118,7 @@ Tcg2ConfigPeimEntryPoint (
> 
>    //
>    // Convert variable to PCD.
> -  // This is work-around because there is no gurantee DynamicHiiPcd can return
> correct value in DXE phase.
> +  // This is work-around because there is no guarantee DynamicHiiPcd can
> return correct value in DXE phase.
>    // Using DynamicPcd instead.
>    //
>    // NOTE: Tcg2Configuration variable contains the desired TpmDevice type,
> diff --git a/SecurityPkg/Tcg/Tcg2Dxe/MeasureBootPeCoff.c
> b/SecurityPkg/Tcg/Tcg2Dxe/MeasureBootPeCoff.c
> index 4e67ac533f..347c6ab702 100644
> --- a/SecurityPkg/Tcg/Tcg2Dxe/MeasureBootPeCoff.c
> +++ b/SecurityPkg/Tcg/Tcg2Dxe/MeasureBootPeCoff.c
> @@ -86,7 +86,7 @@ Tcg2DxeImageRead (
>    @param[in]  PCRIndex       TPM PCR index
>    @param[in]  ImageAddress   Start address of image buffer.
>    @param[in]  ImageSize      Image size
> -  @param[out] DigestList     Digeest list of this image.
> +  @param[out] DigestList     Digest list of this image.
> 
>    @retval EFI_SUCCESS            Successfully measure image.
>    @retval EFI_OUT_OF_RESOURCES   No enough resource to measure image.
> diff --git a/SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c
> b/SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c
> index 85f2e0ae38..3cd16c2fa3 100644
> --- a/SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c
> +++ b/SecurityPkg/Tcg/Tcg2Dxe/Tcg2Dxe.c
> @@ -127,7 +127,7 @@ EFI_HANDLE mImageHandle;
>    @param[in]  PCRIndex       TPM PCR index
>    @param[in]  ImageAddress   Start address of image buffer.
>    @param[in]  ImageSize      Image size
> -  @param[out] DigestList     Digeest list of this image.
> +  @param[out] DigestList     Digest list of this image.
> 
>    @retval EFI_SUCCESS            Successfully measure image.
>    @retval EFI_OUT_OF_RESOURCES   No enough resource to measure image.
> diff --git a/SecurityPkg/Tcg/Tcg2Pei/Tcg2Pei.c
> b/SecurityPkg/Tcg/Tcg2Pei/Tcg2Pei.c
> index f28f33fdec..1565d4e402 100644
> --- a/SecurityPkg/Tcg/Tcg2Pei/Tcg2Pei.c
> +++ b/SecurityPkg/Tcg/Tcg2Pei/Tcg2Pei.c
> @@ -79,7 +79,7 @@ UINT32 mMeasuredMaxChildFvIndex = 0;
>  UINT32 mMeasuredChildFvIndex = 0;
> 
>  /**
> -  Measure and record the Firmware Volum Information once FvInfoPPI install.
> +  Measure and record the Firmware Volume Information once FvInfoPPI install.
> 
>    @param[in] PeiServices       An indirect pointer to the EFI_PEI_SERVICES table
> published by the PEI Foundation.
>    @param[in] NotifyDescriptor  Address of the notification descriptor data
> structure.
> @@ -91,14 +91,14 @@ UINT32 mMeasuredChildFvIndex = 0;
>  **/
>  EFI_STATUS
>  EFIAPI
> -FirmwareVolmeInfoPpiNotifyCallback (
> +FirmwareVolumeInfoPpiNotifyCallback (
>    IN EFI_PEI_SERVICES              **PeiServices,
>    IN EFI_PEI_NOTIFY_DESCRIPTOR     *NotifyDescriptor,
>    IN VOID                          *Ppi
>    );
> 
>  /**
> -  Record all measured Firmware Volum Information into a Guid Hob
> +  Record all measured Firmware Volume Information into a Guid Hob
> 
>    @param[in] PeiServices       An indirect pointer to the EFI_PEI_SERVICES table
> published by the PEI Foundation.
>    @param[in] NotifyDescriptor  Address of the notification descriptor data
> structure.
> @@ -120,12 +120,12 @@ EFI_PEI_NOTIFY_DESCRIPTOR           mNotifyList[] = {
>    {
>      EFI_PEI_PPI_DESCRIPTOR_NOTIFY_CALLBACK,
>      &gEfiPeiFirmwareVolumeInfoPpiGuid,
> -    FirmwareVolmeInfoPpiNotifyCallback
> +    FirmwareVolumeInfoPpiNotifyCallback
>    },
>    {
>      EFI_PEI_PPI_DESCRIPTOR_NOTIFY_CALLBACK,
>      &gEfiPeiFirmwareVolumeInfo2PpiGuid,
> -    FirmwareVolmeInfoPpiNotifyCallback
> +    FirmwareVolumeInfoPpiNotifyCallback
>    },
>    {
>      (EFI_PEI_PPI_DESCRIPTOR_NOTIFY_CALLBACK |
> EFI_PEI_PPI_DESCRIPTOR_TERMINATE_LIST),
> @@ -136,7 +136,7 @@ EFI_PEI_NOTIFY_DESCRIPTOR           mNotifyList[] = {
> 
> 
>  /**
> -  Record all measured Firmware Volum Information into a Guid Hob
> +  Record all measured Firmware Volume Information into a Guid Hob
>    Guid Hob payload layout is
> 
>       UINT32 *************************** FIRMWARE_BLOB number
> @@ -688,7 +688,7 @@ MeasureMainBios (
>  }
> 
>  /**
> -  Measure and record the Firmware Volum Information once FvInfoPPI install.
> +  Measure and record the Firmware Volume Information once FvInfoPPI install.
> 
>    @param[in] PeiServices       An indirect pointer to the EFI_PEI_SERVICES table
> published by the PEI Foundation.
>    @param[in] NotifyDescriptor  Address of the notification descriptor data
> structure.
> @@ -700,7 +700,7 @@ MeasureMainBios (
>  **/
>  EFI_STATUS
>  EFIAPI
> -FirmwareVolmeInfoPpiNotifyCallback (
> +FirmwareVolumeInfoPpiNotifyCallback (
>    IN EFI_PEI_SERVICES               **PeiServices,
>    IN EFI_PEI_NOTIFY_DESCRIPTOR      *NotifyDescriptor,
>    IN VOID                           *Ppi
> diff --git a/SecurityPkg/Tcg/Tcg2Smm/Tcg2Smm.c
> b/SecurityPkg/Tcg/Tcg2Smm/Tcg2Smm.c
> index 14e4ed0ee1..bd786bf479 100644
> --- a/SecurityPkg/Tcg/Tcg2Smm/Tcg2Smm.c
> +++ b/SecurityPkg/Tcg/Tcg2Smm/Tcg2Smm.c
> @@ -269,7 +269,7 @@ UpdatePPVersion (
> 
>  /**
>    Patch interrupt resources returned by TPM _PRS. ResourceTemplate to patch is
> determined by input
> -  interrupt buffer size. BufferSize, PkgLength and interrupt descirptor in ByteList
> need to be patched
> +  interrupt buffer size. BufferSize, PkgLength and interrupt descriptor in ByteList
> need to be patched
> 
>    @param[in, out] Table            The TPM item in ACPI table.
>    @param[in]      IrqBuffer        Input new IRQ buffer.
> @@ -304,7 +304,7 @@ UpdatePossibleResource (
>    // to patch TPM ACPI object _PRS returned ResourceTemplate() containing 2
> resource descriptors and an auto appended End Tag
>    //
>    //  AML data is organized by following rule.
> -  //  Code need to patch BufferSize and PkgLength and interrupt descirptor in
> ByteList
> +  //  Code need to patch BufferSize and PkgLength and interrupt descriptor in
> ByteList
>    //
>    // =============  Buffer ====================
>    //           DefBuffer := BufferOp PkgLength BufferSize ByteList
> @@ -321,8 +321,8 @@ UpdatePossibleResource (
>    //                               <bit 3-0: Least significant package length nybble>
>    //
>    //==============BufferSize==================
> -  //        BufferSize := Integar
> -  //           Integar := ByteConst|WordConst|DwordConst....
> +  //        BufferSize := Integer
> +  //           Integer := ByteConst|WordConst|DwordConst....
>    //
>    //           ByteConst := BytePrefix ByteData
>    //
> @@ -371,7 +371,7 @@ UpdatePossibleResource (
>        }
> 
>        //
> -      // Include Memory32Fixed Descritor (12 Bytes) + Interrupt Descriptor
> header(5 Bytes) + End Tag(2 Bytes)
> +      // Include Memory32Fixed Descriptor (12 Bytes) + Interrupt Descriptor
> header(5 Bytes) + End Tag(2 Bytes)
>        //
>        NewPkgLength += 19 + IrqBuffserSize;
>        if (NewPkgLength > 63) {
> @@ -389,7 +389,7 @@ UpdatePossibleResource (
>        *DataPtr = (UINT8)NewPkgLength;
> 
>        //
> -      // 1.2 Patch BufferSize = sizeof(Memory32Fixed Descritor + Interrupt
> Descriptor + End Tag).
> +      // 1.2 Patch BufferSize = sizeof(Memory32Fixed Descriptor + Interrupt
> Descriptor + End Tag).
>        //      It is Little endian. So only patch lowest byte of BufferSize due to current
> interrupt number limit.
>        //
>        *(DataPtr + 2) = (UINT8)(IrqBuffserSize + 19);
> @@ -445,7 +445,7 @@ UpdatePossibleResource (
>          }
> 
>          //
> -        // Include Memory32Fixed Descritor (12 Bytes) + Interrupt Descriptor
> header(5 Bytes) + End Tag(2  Bytes)
> +        // Include Memory32Fixed Descriptor (12 Bytes) + Interrupt Descriptor
> header(5 Bytes) + End Tag(2  Bytes)
>          //
>          NewPkgLength += 19 + IrqBuffserSize;
> 
> @@ -461,7 +461,7 @@ UpdatePossibleResource (
>          *(DataPtr + 1) = (UINT8)((NewPkgLength & 0xFF0) >> 4);
> 
>          //
> -        // 2.2 Patch BufferSize = sizeof(Memory32Fixed Descritor + Interrupt
> Descriptor + End Tag).
> +        // 2.2 Patch BufferSize = sizeof(Memory32Fixed Descriptor + Interrupt
> Descriptor + End Tag).
>          //     It is Little endian. Only patch lowest byte of BufferSize due to current
> interrupt number limit.
>          //
>          *(DataPtr + 2 + ((*DataPtr & (BIT7|BIT6)) >> 6)) = (UINT8)(IrqBuffserSize +
> 19);
> @@ -485,7 +485,7 @@ UpdatePossibleResource (
>    //
>    DataPtr += NewPkgLength - (5 + IrqBuffserSize + 2);
>    //
> -  //   3.1 Patch Length bit[7:0] of Interrupt descirptor patch interrupt descriptor
> +  //   3.1 Patch Length bit[7:0] of Interrupt descriptor patch interrupt descriptor
>    //
>    *(DataPtr + 1) = (UINT8)(2 + IrqBuffserSize);
>    //
> @@ -498,7 +498,7 @@ UpdatePossibleResource (
>    CopyMem(DataPtr + 5, IrqBuffer, IrqBuffserSize);
> 
>    //
> -  // 4. Jump over Interrupt descirptor and Patch END Tag, set Checksum field to
> 0
> +  // 4. Jump over Interrupt descriptor and Patch END Tag, set Checksum field to
> 0
>    //
>    DataPtr       += 5 + IrqBuffserSize;
>    *DataPtr       = ACPI_END_TAG_DESCRIPTOR;
> @@ -715,7 +715,7 @@ PublishAcpiTable (
>    mTcgNvs->IsShortFormPkgLength = IsShortFormPkgLength;
> 
>    //
> -  // Publish the TPM ACPI table. Table is re-checksumed.
> +  // Publish the TPM ACPI table. Table is re-checksummed.
>    //
>    Status = gBS->LocateProtocol (&gEfiAcpiTableProtocolGuid, NULL, (VOID **)
> &AcpiTable);
>    ASSERT_EFI_ERROR (Status);
> diff --git a/SecurityPkg/Tcg/Tcg2Smm/Tpm.asl
> b/SecurityPkg/Tcg/Tcg2Smm/Tpm.asl
> index 9c4b9afa46..7a91708a1a 100644
> --- a/SecurityPkg/Tcg/Tcg2Smm/Tpm.asl
> +++ b/SecurityPkg/Tcg/Tcg2Smm/Tpm.asl
> @@ -76,17 +76,17 @@ DefinitionBlock (
>        Field (TNVS, AnyAcc, NoLock, Preserve)
>        {
>          PPIN,   8,  //   Software SMI for Physical Presence Interface
> -        PPIP,   32, //   Used for save physical presence paramter
> +        PPIP,   32, //   Used for save physical presence parameter
>          PPRP,   32, //   Physical Presence request operation response
>          PPRQ,   32, //   Physical Presence request operation
>          PPRM,   32, //   Physical Presence request operation parameter
>          LPPR,   32, //   Last Physical Presence request operation
>          FRET,   32, //   Physical Presence function return code
>          MCIN,   8,  //   Software SMI for Memory Clear Interface
> -        MCIP,   32, //   Used for save the Mor paramter
> +        MCIP,   32, //   Used for save the Mor parameter
>          MORD,   32, //   Memory Overwrite Request Data
>          MRET,   32, //   Memory Overwrite function return code
> -        UCRQ,   32, //   Phyical Presence request operation to Get User
> Confirmation Status
> +        UCRQ,   32, //   Physical Presence request operation to Get User
> Confirmation Status
>          IRQN,   32, //   IRQ Number for _CRS
>          SFRB,   8   //   Is shortformed Pkglength for resource buffer
>        }
> diff --git a/SecurityPkg/Tcg/TcgConfigDxe/TcgConfigDriver.c
> b/SecurityPkg/Tcg/TcgConfigDxe/TcgConfigDriver.c
> index fb06624fdb..8210548077 100644
> --- a/SecurityPkg/Tcg/TcgConfigDxe/TcgConfigDriver.c
> +++ b/SecurityPkg/Tcg/TcgConfigDxe/TcgConfigDriver.c
> @@ -17,7 +17,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
> 
>    @retval EFI_ALREADY_STARTED    The driver already exists in system.
>    @retval EFI_OUT_OF_RESOURCES   Fail to execute entry point due to lack of
> resources.
> -  @retval EFI_SUCCES             All the related protocols are installed on the driver.
> +  @retval EFI_SUCCESS            All the related protocols are installed on the
> driver.
>    @retval Others                 Fail to install protocols as indicated.
> 
>  **/
> diff --git a/SecurityPkg/Tcg/TcgDxe/TcgDxe.c
> b/SecurityPkg/Tcg/TcgDxe/TcgDxe.c
> index 6feb886308..3990d1e937 100644
> --- a/SecurityPkg/Tcg/TcgDxe/TcgDxe.c
> +++ b/SecurityPkg/Tcg/TcgDxe/TcgDxe.c
> @@ -79,7 +79,7 @@ EFI_TCG_CLIENT_ACPI_TABLE
> mTcgClientAcpiTemplate = {
> 
>  //
>  // The following EFI_TCG_SERVER_ACPI_TABLE default setting is just one
> example,
> -// the TPM device connectes to LPC, and also defined the ACPI _UID as 0xFF,
> +// the TPM device connects to LPC, and also defined the ACPI _UID as 0xFF,
>  // this _UID can be changed and should match with the _UID setting of the TPM
>  // ACPI device object
>  //
> @@ -1224,7 +1224,7 @@ InstallAcpiTable (
>      mTcgClientAcpiTemplate.Header.CreatorId        = PcdGet32
> (PcdAcpiDefaultCreatorId);
>      mTcgClientAcpiTemplate.Header.CreatorRevision  = PcdGet32
> (PcdAcpiDefaultCreatorRevision);
>      //
> -    // The ACPI table must be checksumed before calling the InstallAcpiTable()
> +    // The ACPI table must be checksummed before calling the InstallAcpiTable()
>      // service of the ACPI table protocol to install it.
>      //
>      Checksum = CalculateCheckSum8 ((UINT8 *)&mTcgClientAcpiTemplate, sizeof
> (mTcgClientAcpiTemplate));
> @@ -1244,7 +1244,7 @@ InstallAcpiTable (
>      mTcgServerAcpiTemplate.Header.CreatorId        = PcdGet32
> (PcdAcpiDefaultCreatorId);
>      mTcgServerAcpiTemplate.Header.CreatorRevision  = PcdGet32
> (PcdAcpiDefaultCreatorRevision);
>      //
> -    // The ACPI table must be checksumed before calling the InstallAcpiTable()
> +    // The ACPI table must be checksummed before calling the InstallAcpiTable()
>      // service of the ACPI table protocol to install it.
>      //
>      Checksum = CalculateCheckSum8 ((UINT8 *)&mTcgServerAcpiTemplate,
> sizeof (mTcgServerAcpiTemplate));
> diff --git a/SecurityPkg/Tcg/TcgPei/TcgPei.c b/SecurityPkg/Tcg/TcgPei/TcgPei.c
> index 0adfcc50c5..284fc394f0 100644
> --- a/SecurityPkg/Tcg/TcgPei/TcgPei.c
> +++ b/SecurityPkg/Tcg/TcgPei/TcgPei.c
> @@ -85,7 +85,7 @@ PhysicalPresencePpiNotifyCallback (
>    );
> 
>  /**
> -  Measure and record the Firmware Volum Information once FvInfoPPI install.
> +  Measure and record the Firmware Volume Information once FvInfoPPI install.
> 
>    @param[in] PeiServices       An indirect pointer to the EFI_PEI_SERVICES table
> published by the PEI Foundation.
>    @param[in] NotifyDescriptor  Address of the notification descriptor data
> structure.
> @@ -97,14 +97,14 @@ PhysicalPresencePpiNotifyCallback (
>  **/
>  EFI_STATUS
>  EFIAPI
> -FirmwareVolmeInfoPpiNotifyCallback (
> +FirmwareVolumeInfoPpiNotifyCallback (
>    IN EFI_PEI_SERVICES              **PeiServices,
>    IN EFI_PEI_NOTIFY_DESCRIPTOR     *NotifyDescriptor,
>    IN VOID                          *Ppi
>    );
> 
>  /**
> -  Record all measured Firmware Volum Information into a Guid Hob
> +  Record all measured Firmware Volume Information into a Guid Hob
> 
>    @param[in] PeiServices       An indirect pointer to the EFI_PEI_SERVICES table
> published by the PEI Foundation.
>    @param[in] NotifyDescriptor  Address of the notification descriptor data
> structure.
> @@ -131,12 +131,12 @@ EFI_PEI_NOTIFY_DESCRIPTOR           mNotifyList[] = {
>    {
>      EFI_PEI_PPI_DESCRIPTOR_NOTIFY_CALLBACK,
>      &gEfiPeiFirmwareVolumeInfoPpiGuid,
> -    FirmwareVolmeInfoPpiNotifyCallback
> +    FirmwareVolumeInfoPpiNotifyCallback
>    },
>    {
>      EFI_PEI_PPI_DESCRIPTOR_NOTIFY_CALLBACK,
>      &gEfiPeiFirmwareVolumeInfo2PpiGuid,
> -    FirmwareVolmeInfoPpiNotifyCallback
> +    FirmwareVolumeInfoPpiNotifyCallback
>    },
>    {
>      (EFI_PEI_PPI_DESCRIPTOR_NOTIFY_CALLBACK |
> EFI_PEI_PPI_DESCRIPTOR_TERMINATE_LIST),
> @@ -146,7 +146,7 @@ EFI_PEI_NOTIFY_DESCRIPTOR           mNotifyList[] = {
>  };
> 
>  /**
> -  Record all measured Firmware Volum Information into a Guid Hob
> +  Record all measured Firmware Voluem Information into a Guid Hob
>    Guid Hob payload layout is
> 
>       UINT32 *************************** FIRMWARE_BLOB number
> @@ -501,7 +501,7 @@ MeasureMainBios (
>  }
> 
>  /**
> -  Measure and record the Firmware Volum Information once FvInfoPPI install.
> +  Measure and record the Firmware Voluem Information once FvInfoPPI install.
> 
>    @param[in] PeiServices       An indirect pointer to the EFI_PEI_SERVICES table
> published by the PEI Foundation.
>    @param[in] NotifyDescriptor  Address of the notification descriptor data
> structure.
> @@ -513,7 +513,7 @@ MeasureMainBios (
>  **/
>  EFI_STATUS
>  EFIAPI
> -FirmwareVolmeInfoPpiNotifyCallback (
> +FirmwareVolumeInfoPpiNotifyCallback (
>    IN EFI_PEI_SERVICES               **PeiServices,
>    IN EFI_PEI_NOTIFY_DESCRIPTOR      *NotifyDescriptor,
>    IN VOID                           *Ppi
> @@ -672,7 +672,7 @@ PhysicalPresencePpiNotifyCallback (
>  }
> 
>  /**
> -  Check if TPM chip is activeated or not.
> +  Check if TPM chip is activated or not.
> 
>    @param[in]      PeiServices   Describes the list of possible PEI Services.
> 
> diff --git a/SecurityPkg/Tcg/TcgSmm/Tpm.asl
> b/SecurityPkg/Tcg/TcgSmm/Tpm.asl
> index 6ddc47ae91..739067dbec 100644
> --- a/SecurityPkg/Tcg/TcgSmm/Tpm.asl
> +++ b/SecurityPkg/Tcg/TcgSmm/Tpm.asl
> @@ -64,16 +64,16 @@ DefinitionBlock (
>        Field (TNVS, AnyAcc, NoLock, Preserve)
>        {
>          PPIN,   8,  //   Software SMI for Physical Presence Interface
> -        PPIP,   32, //   Used for save physical presence paramter
> +        PPIP,   32, //   Used for save physical presence parameter
>          PPRP,   32, //   Physical Presence request operation response
>          PPRQ,   32, //   Physical Presence request operation
>          LPPR,   32, //   Last Physical Presence request operation
>          FRET,   32, //   Physical Presence function return code
>          MCIN,   8,  //   Software SMI for Memory Clear Interface
> -        MCIP,   32, //   Used for save the Mor paramter
> +        MCIP,   32, //   Used for save the Mor parameter
>          MORD,   32, //   Memory Overwrite Request Data
>          MRET,   32, //   Memory Overwrite function return code
> -        UCRQ,   32  //   Phyical Presence request operation to Get User
> Confirmation Status
> +        UCRQ,   32  //   Physical Presence request operation to Get User
> Confirmation Status
>        }
> 
>        Method (PTS, 1, Serialized)
> diff --git
> a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfig
> Driver.c
> b/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfig
> Driver.c
> index 6c0294151e..8936a0a634 100644
> ---
> a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfig
> Driver.c
> +++
> b/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfig
> Driver.c
> @@ -16,7 +16,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
> 
>    @retval EFI_ALREADY_STARTED    The driver already exists in system.
>    @retval EFI_OUT_OF_RESOURCES   Fail to execute entry point due to lack of
> resources.
> -  @retval EFI_SUCCES             All the related protocols are installed on the driver.
> +  @retval EFI_SUCCESS            All the related protocols are installed on the
> driver.
>    @retval Others                 Fail to get the SecureBootEnable variable.
> 
>  **/
> diff --git
> a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfig
> Dxe.inf
> b/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfig
> Dxe.inf
> index 55a4239252..573efa6379 100644
> ---
> a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfig
> Dxe.inf
> +++
> b/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfig
> Dxe.inf
> @@ -1,5 +1,5 @@
>  ## @file
> -#  Provides the capbility to configure secure boot in a setup browser
> +#  Provides the capability to configure secure boot in a setup browser
>  #  By this module, user may change the content of DB, DBX, PK and KEK.
>  #
>  # Copyright (c) 2011 - 2018, Intel Corporation. All rights reserved.<BR>
> diff --git
> a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfig
> Dxe.uni
> b/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfig
> Dxe.uni
> index 56097deee5..4e9c6f3ff7 100644
> ---
> a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfig
> Dxe.uni
> +++
> b/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfig
> Dxe.uni
> @@ -1,5 +1,5 @@
>  // /** @file
> -// Provides the capbility to configure secure boot in a setup browser
> +// Provides the capability to configure secure boot in a setup browser
>  //
>  // By this module, user may change the content of DB, DBX, PK and KEK.
>  //
> diff --git
> a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigI
> mpl.c
> b/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigI
> mpl.c
> index a15038da12..83497a23f5 100644
> ---
> a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigI
> mpl.c
> +++
> b/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigI
> mpl.c
> @@ -234,7 +234,7 @@ SaveSecureBootVariable (
>                                     it's caller's responsibility to free the memory when finish
> using it.
> 
>    @retval EFI_SUCCESS              Create time based payload successfully.
> -  @retval EFI_OUT_OF_RESOURCES     There are not enough memory resourses
> to create time based payload.
> +  @retval EFI_OUT_OF_RESOURCES     There are not enough memory resources
> to create time based payload.
>    @retval EFI_INVALID_PARAMETER    The parameter is invalid.
>    @retval Others                   Unexpected error happens.
> 
> @@ -390,7 +390,7 @@ SetSecureBootMode (
>    @param[out]  PkCert                Point to the data buffer to store the signature
> list.
> 
>    @return EFI_UNSUPPORTED            Unsupported Key Length.
> -  @return EFI_OUT_OF_RESOURCES       There are not enough memory
> resourses to form the signature list.
> +  @return EFI_OUT_OF_RESOURCES       There are not enough memory
> resources to form the signature list.
> 
>  **/
>  EFI_STATUS
> @@ -507,7 +507,7 @@ EnrollPlatformKey (
>    DEBUG ((EFI_D_INFO, "FilePostFix = %s\n", FilePostFix));
> 
>    //
> -  // Prase the selected PK file and generature PK certificate list.
> +  // Prase the selected PK file and generate PK certificate list.
>    //
>    Status = CreatePkX509SignatureList (
>              Private->FileContext->FHandle,
> @@ -1088,7 +1088,7 @@ IsSignatureFoundInDatabase (
>    }
> 
>    //
> -  // Enumerate all signature data in SigDB to check if executable's signature
> exists.
> +  // Enumerate all signature data in SigDB to check if signature exists for
> executable.
>    //
>    CertList = (EFI_SIGNATURE_LIST *) Data;
>    while ((DataSize > 0) && (DataSize >= CertList->SignatureListSize)) {
> @@ -1312,7 +1312,7 @@ Done:
>  /**
>    Check whether the signature list exists in given variable data.
> 
> -  It searches the signature list for the ceritificate hash by CertType.
> +  It searches the signature list for the certificate hash by CertType.
>    If the signature list is found, get the offset of Database for the
>    next hash of a certificate.
> 
> @@ -2107,7 +2107,7 @@ HashPeImageByType (
>  }
> 
>  /**
> -  Enroll a new executable's signature into Signature Database.
> +  Enroll a new signature of executable into Signature Database.
> 
>    @param[in] PrivateData     The module's private data.
>    @param[in] VariableName    Variable name of signature database, must be
> @@ -2177,7 +2177,7 @@ EnrollAuthentication2Descriptor (
>    }
> 
>    //
> -  // Diretly set AUTHENTICATION_2 data to SetVariable
> +  // Directly set AUTHENTICATION_2 data to SetVariable
>    //
>    Status = gRT->SetVariable(
>                    VariableName,
> @@ -2208,7 +2208,7 @@ ON_EXIT:
> 
> 
>  /**
> -  Enroll a new executable's signature into Signature Database.
> +  Enroll a new signature of executable into Signature Database.
> 
>    @param[in] PrivateData     The module's private data.
>    @param[in] VariableName    Variable name of signature database, must be
> @@ -2247,7 +2247,7 @@ EnrollImageSignatureToSigDB (
>    // Form the SigDB certificate list.
>    // Format the data item into EFI_SIGNATURE_LIST type.
>    //
> -  // We need to parse executable's signature data from specified signed
> executable file.
> +  // We need to parse signature data of executable from specified signed
> executable file.
>    // In current implementation, we simply trust the pass-in signed executable file.
>    // In reality, it's OS's responsibility to verify the signed executable file.
>    //
> @@ -3269,7 +3269,7 @@ SecureBootExtractConfigFromVariable (
>    SecureBootMode   = NULL;
> 
>    //
> -  // Initilize the Date and Time using system time.
> +  // Initialize the Date and Time using system time.
>    //
>    ConfigData->CertificateFormat = HASHALG_RAW;
>    ConfigData->AlwaysRevocation = TRUE;
> @@ -3306,7 +3306,7 @@ SecureBootExtractConfigFromVariable (
>    }
> 
>    //
> -  // Check SecureBootEnable & Pk status, fix the inconsistence.
> +  // Check SecureBootEnable & Pk status, fix the inconsistency.
>    // If the SecureBootEnable Variable doesn't exist, hide the SecureBoot
> Enable/Disable
>    // Checkbox.
>    //
> @@ -3314,7 +3314,7 @@ SecureBootExtractConfigFromVariable (
>    GetVariable2 (EFI_SECURE_BOOT_ENABLE_NAME,
> &gEfiSecureBootEnableDisableGuid, (VOID**)&SecureBootEnable, NULL);
> 
>    //
> -  // Fix Pk, SecureBootEnable inconsistence
> +  // Fix Pk and SecureBootEnable inconsistency
>    //
>    if ((SetupMode != NULL) && (*SetupMode) == USER_MODE) {
>      ConfigData->HideSecureBoot = FALSE;
> diff --git
> a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigI
> mpl.h
> b/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigI
> mpl.h
> index 32e9619e91..1fafae07ac 100644
> ---
> a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigI
> mpl.h
> +++
> b/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigI
> mpl.h
> @@ -172,7 +172,7 @@ typedef struct {
>    CURRENT_VARIABLE_NAME             VariableName;     // The variable name we
> are processing.
>    UINT32                            ListCount;        // Record current variable has how many
> signature list.
>    UINTN                             ListIndex;        // Record which signature list is
> processing.
> -  BOOLEAN                           *CheckArray;      // Record whcih siganture data
> checked.
> +  BOOLEAN                           *CheckArray;      // Record which signature data
> checked.
>  } SECUREBOOT_CONFIG_PRIVATE_DATA;
> 
>  extern SECUREBOOT_CONFIG_PRIVATE_DATA
> mSecureBootConfigPrivateDateTemplate;
> @@ -182,7 +182,7 @@ extern SECUREBOOT_CONFIG_PRIVATE_DATA
> *gSecureBootPrivateData;
>  #define SECUREBOOT_CONFIG_PRIVATE_FROM_THIS(a)  CR (a,
> SECUREBOOT_CONFIG_PRIVATE_DATA, ConfigAccess,
> SECUREBOOT_CONFIG_PRIVATE_DATA_SIGNATURE)
> 
>  //
> -// Cryptograhpic Key Information
> +// Cryptographic Key Information
>  //
>  #pragma pack(1)
>  typedef struct _CPL_KEY_INFO {
> @@ -448,12 +448,12 @@ CleanUpPage (
> 
>  /**
>    Read file content into BufferPtr, the size of the allocate buffer
> -  is *FileSize plus AddtionAllocateSize.
> +  is *FileSize plus AdditionAllocateSize.
> 
>    @param[in]       FileHandle            The file to be read.
>    @param[in, out]  BufferPtr             Pointers to the pointer of allocated buffer.
>    @param[out]      FileSize              Size of input file
> -  @param[in]       AddtionAllocateSize   Addtion size the buffer need to be
> allocated.
> +  @param[in]       AdditionAllocateSize   Addition size the buffer need to be
> allocated.
>                                           In case the buffer need to contain others besides the file
> content.
> 
>    @retval   EFI_SUCCESS                  The file was read into the buffer.
> @@ -467,7 +467,7 @@ ReadFileContent (
>    IN      EFI_FILE_HANDLE           FileHandle,
>    IN OUT  VOID                      **BufferPtr,
>       OUT  UINTN                     *FileSize,
> -  IN      UINTN                     AddtionAllocateSize
> +  IN      UINTN                     AdditionAllocateSize
>    );
> 
> 
> diff --git
> a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfig
> Misc.c
> b/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfig
> Misc.c
> index f5ce94e06d..3730fbe646 100644
> ---
> a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfig
> Misc.c
> +++
> b/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfig
> Misc.c
> @@ -10,12 +10,12 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
> 
>  /**
>    Read file content into BufferPtr, the size of the allocate buffer
> -  is *FileSize plus AddtionAllocateSize.
> +  is *FileSize plus AdditionAllocateSize.
> 
>    @param[in]       FileHandle            The file to be read.
>    @param[in, out]  BufferPtr             Pointers to the pointer of allocated buffer.
>    @param[out]      FileSize              Size of input file
> -  @param[in]       AddtionAllocateSize   Addtion size the buffer need to be
> allocated.
> +  @param[in]       AdditionAllocateSize   Addition size the buffer need to be
> allocated.
>                                           In case the buffer need to contain others besides the file
> content.
> 
>    @retval   EFI_SUCCESS                  The file was read into the buffer.
> @@ -29,7 +29,7 @@ ReadFileContent (
>    IN      EFI_FILE_HANDLE           FileHandle,
>    IN OUT  VOID                      **BufferPtr,
>       OUT  UINTN                     *FileSize,
> -  IN      UINTN                     AddtionAllocateSize
> +  IN      UINTN                     AdditionAllocateSize
>    )
> 
>  {
> @@ -62,7 +62,7 @@ ReadFileContent (
>      goto ON_EXIT;
>    }
> 
> -  BufferSize = (UINTN) SourceFileSize + AddtionAllocateSize;
> +  BufferSize = (UINTN) SourceFileSize + AdditionAllocateSize;
>    Buffer =  AllocateZeroPool(BufferSize);
>    if (Buffer == NULL) {
>      return EFI_OUT_OF_RESOURCES;
> diff --git
> a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfig
> NvData.h
> b/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfig
> NvData.h
> index 8ea551d88e..6e54a4b0f2 100644
> ---
> a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfig
> NvData.h
> +++
> b/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfig
> NvData.h
> @@ -122,7 +122,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
>  //
>  typedef struct {
>    BOOLEAN AttemptSecureBoot;   // Attempt to enable/disable Secure Boot
> -  BOOLEAN HideSecureBoot;      // Hiden Attempt Secure Boot
> +  BOOLEAN HideSecureBoot;      // Hidden Attempt Secure Boot
>    CHAR16  SignatureGuid[SECURE_BOOT_GUID_STORAGE_SIZE];
>    BOOLEAN PhysicalPresent;     // If a Physical Present User
>    UINT8   SecureBootMode;      // Secure Boot Mode: Standard Or Custom
> @@ -132,7 +132,7 @@ typedef struct {
>    UINT8   CertificateFormat;   // The type of the certificate
>    EFI_HII_DATE RevocationDate; // The revocation date of the certificate
>    EFI_HII_TIME RevocationTime; // The revocation time of the certificate
> -  UINT8   FileEnrollType;      // File type of sigunature enroll
> +  UINT8   FileEnrollType;      // File type of signature enroll
>    UINT32  ListCount;           // The count of signature list.
>    UINT32  CheckedDataCount;    // The count of checked signature data.
>  } SECUREBOOT_CONFIGURATION;
> diff --git
> a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigS
> trings.uni
> b/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigS
> trings.uni
> index a412d5bfe9..ac783453cc 100644
> ---
> a/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigS
> trings.uni
> +++
> b/SecurityPkg/VariableAuthenticated/SecureBootConfigDxe/SecureBootConfigS
> trings.uni
> @@ -45,8 +45,8 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
>  #string STR_DBX_PE_IMAGE_FORMAT_HELP       #language en-US "PE image
> enrolled. Use SHA256 hash to enroll it into DBX"
>  #string STR_DBX_PE_FORMAT_SHA256           #language en-US "PE Image
> SHA256"
> 
> -#string STR_DBX_AUTH_2_FORMAT_HELP         #language en-US
> "VARIABLE_AUTHENICATION_2 binary enrolled. Use raw binary to enroll it into
> DBX"
> -#string STR_DBX_AUTH_2_FORMAT              #language en-US
> "VARIABLE_AUTHENICATION_2"
> +#string STR_DBX_AUTH_2_FORMAT_HELP         #language en-US
> "VARIABLE_AUTHENTICATION_2 binary enrolled. Use raw binary to enroll it into
> DBX"
> +#string STR_DBX_AUTH_2_FORMAT              #language en-US
> "VARIABLE_AUTHENTICATION_2"
> 
>  #string STR_CERTIFICATE_REVOCATION_TIME_PROMPT #language en-US "
> Revocation Time"
>  #string STR_CERTIFICATE_REVOCATION_TIME_HELP   #language en-US "Input
> the revocation time of the certificate"
> --
> 2.21.0.windows.1


  reply	other threads:[~2019-10-22  4:39 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-18 19:01 [Patch] SecurityPkg: Fix spelling errors Michael D Kinney
2019-10-22  4:39 ` Wang, Jian J [this message]
2019-10-22  4:50   ` Yao, Jiewen
2019-10-22 12:41 ` [edk2-devel] " Philippe Mathieu-Daudé
2019-10-22 16:08   ` Michael D Kinney
2019-10-22 16:20     ` Philippe Mathieu-Daudé

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=D827630B58408649ACB04F44C510003625996C40@SHSMSX107.ccr.corp.intel.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox