Re: [PATCH 57/79] OvmfPkg/Csm: Fix various typos

["Laszlo Ersek" <lersek@redhat.com>]

On 12/03/19 17:15, Philippe Mathieu-Daude wrote:
> From: Antoine Coeur <coeur@gmx.fr>
> 
> Fix various typos in documentation, comments and strings.
> 
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Cc: Laszlo Ersek <lersek@redhat.com>
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Reviewed-by: Philippe Mathieu-Daude <philmd@redhat.com>
> Signed-off-by: Philippe Mathieu-Daude <philmd@redhat.com>
> ---
>  OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiLib.inf           |  2 +-
>  OvmfPkg/Csm/BiosThunk/VideoDxe/BiosVideo.h                          | 10 +++++-----
>  OvmfPkg/Csm/BiosThunk/VideoDxe/VesaBiosExtensions.h                 |  8 ++++----
>  OvmfPkg/Csm/Include/Framework/BootScript.h                          |  2 +-
>  OvmfPkg/Csm/Include/Framework/FrameworkInternalFormRepresentation.h |  2 +-
>  OvmfPkg/Csm/Include/Protocol/IsaIo.h                                |  2 +-
>  OvmfPkg/Csm/Include/Protocol/LegacyBios.h                           |  6 +++---
>  OvmfPkg/Csm/Include/Protocol/LegacyBiosPlatform.h                   |  4 ++--
>  OvmfPkg/Csm/Include/Protocol/VgaMiniPort.h                          |  2 +-
>  OvmfPkg/Csm/LegacyBiosDxe/LegacyBiosInterface.h                     |  8 ++++----
>  OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUi.h                |  4 ++--
>  OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiVfr.h             |  2 +-
>  OvmfPkg/Csm/BiosThunk/VideoDxe/BiosVideo.c                          |  4 ++--
>  OvmfPkg/Csm/CsmSupportLib/LegacyPlatform.c                          |  6 +++---
>  OvmfPkg/Csm/LegacyBiosDxe/LegacyBios.c                              |  6 +++---
>  OvmfPkg/Csm/LegacyBiosDxe/LegacyBootSupport.c                       |  4 ++--
>  OvmfPkg/Csm/LegacyBiosDxe/LegacyPci.c                               |  8 ++++----
>  OvmfPkg/Csm/LegacyBiosDxe/Thunk.c                                   |  4 ++--
>  OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUi.c                | 14 +++++++-------
>  OvmfPkg/Csm/LegacyBootManagerLib/LegacyBm.c                         | 18 +++++++++---------
>  OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiLib.uni           |  8 ++++----
>  21 files changed, 62 insertions(+), 62 deletions(-)

CC'ing David -- David is the designated Reviewer for OvmfPkg/Csm/, based
on "Maintainers.txt".

Conditional on David's R-b, you can add:

Acked-by: Laszlo Ersek <lersek@redhat.com>

Thanks
Laszlo

> diff --git a/OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiLib.inf b/OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiLib.inf
> index 5cc64a1b2df2..9a790d34a0e4 100644
> --- a/OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiLib.inf
> +++ b/OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiLib.inf
> @@ -1,5 +1,5 @@
>  ## @file
> -#  Legacy Boot Maintainence UI module is library for BDS phase.
> +#  Legacy Boot Maintenance UI module is library for BDS phase.
>  #
>  #  Copyright (c) 2015 - 2019, Intel Corporation. All rights reserved.<BR>
>  #  SPDX-License-Identifier: BSD-2-Clause-Patent
> diff --git a/OvmfPkg/Csm/BiosThunk/VideoDxe/BiosVideo.h b/OvmfPkg/Csm/BiosThunk/VideoDxe/BiosVideo.h
> index 951f4e6f768f..522e8ab15bb9 100644
> --- a/OvmfPkg/Csm/BiosThunk/VideoDxe/BiosVideo.h
> +++ b/OvmfPkg/Csm/BiosThunk/VideoDxe/BiosVideo.h
> @@ -241,7 +241,7 @@ BiosVideoCheckForVga (
>  
>  
>  /**
> -  Release resource for biso video instance.
> +  Release resource for bios video instance.
>  
>    @param  BiosVideoPrivate       Video child device private data structure
>  
> @@ -347,9 +347,9 @@ BiosVideoGraphicsOutputVbeBlt (
>  
>  
>  /**
> -  Grahpics Output protocol instance to block transfer for VGA device.
> +  Graphics Output protocol instance to block transfer for VGA device.
>  
> -  @param  This                   Pointer to Grahpics Output protocol instance
> +  @param  This                   Pointer to Graphics Output protocol instance
>    @param  BltBuffer              The data to transfer to screen
>    @param  BltOperation           The operation to perform
>    @param  SourceX                The X coordinate of the source for BltOperation
> @@ -412,7 +412,7 @@ BiosVideoVgaMiniPortSetMode (
>  /**
>    Event handler for Exit Boot Service.
>  
> -  @param  Event       The event that be siganlled when exiting boot service.
> +  @param  Event       The event that be signalled when exiting boot service.
>    @param  Context     Pointer to instance of BIOS_VIDEO_DEV.
>  
>  **/
> @@ -506,7 +506,7 @@ BiosVideoChildHandleUninstall (
>    );
>  
>  /**
> -  Release resource for biso video instance.
> +  Release resource for bios video instance.
>  
>    @param  BiosVideoPrivate       Video child device private data structure
>  
> diff --git a/OvmfPkg/Csm/BiosThunk/VideoDxe/VesaBiosExtensions.h b/OvmfPkg/Csm/BiosThunk/VideoDxe/VesaBiosExtensions.h
> index dbf706179ff9..88065f15c645 100644
> --- a/OvmfPkg/Csm/BiosThunk/VideoDxe/VesaBiosExtensions.h
> +++ b/OvmfPkg/Csm/BiosThunk/VideoDxe/VesaBiosExtensions.h
> @@ -250,7 +250,7 @@ typedef struct {
>  #define VESA_BIOS_EXTENSIONS_VERSION_3_0  0x0300
>  
>  //
> -// Super VGA Information Block Capabilities field bit defintions
> +// Super VGA Information Block Capabilities field bit definitions
>  //
>  #define VESA_BIOS_EXTENSIONS_CAPABILITY_8_BIT_DAC 0x01  // 0: DAC width is fixed at 6 bits/color
>  // 1: DAC width switchable to 8 bits/color
> @@ -358,7 +358,7 @@ typedef struct {
>  } VESA_BIOS_EXTENSIONS_MODE_INFORMATION_BLOCK;
>  
>  //
> -// Super VGA Mode Information Block ModeAttributes field bit defintions
> +// Super VGA Mode Information Block ModeAttributes field bit definitions
>  //
>  #define VESA_BIOS_EXTENSIONS_MODE_ATTRIBUTE_HARDWARE  0x0001  // 0: Mode not supported in handware
>  // 1: Mode supported in handware
> @@ -396,7 +396,7 @@ typedef struct {
>  #define VESA_BIOS_EXTENSIONS_MODE_ATTRIBUTE_DUAL_DISPLAY  0x1000  // 0: No dual display start address support
>  // 1: Dual display start address support
>  //
> -// Super VGA Mode Information Block WinAAttribite/WinBAttributes field bit defintions
> +// Super VGA Mode Information Block WinAAttribite/WinBAttributes field bit definitions
>  //
>  #define VESA_BIOS_EXTENSIONS_WINX_ATTRIBUTE_RELOCATABLE 0x01  // 0: Single non-relocatable window only
>  // 1: Relocatable window(s) are supported
> @@ -407,7 +407,7 @@ typedef struct {
>  #define VESA_BIOS_EXTENSIONS_WINX_ATTRIBUTE_WRITABLE  0x04  // 0: Window is not writable
>  // 1: Window is writable
>  //
> -// Super VGA Mode Information Block DirectColorMode field bit defintions
> +// Super VGA Mode Information Block DirectColorMode field bit definitions
>  //
>  #define VESA_BIOS_EXTENSIONS_DIRECT_COLOR_MODE_PROG_COLOR_RAMP  0x01  // 0: Color ram is fixed
>  // 1: Color ramp is programmable
> diff --git a/OvmfPkg/Csm/Include/Framework/BootScript.h b/OvmfPkg/Csm/Include/Framework/BootScript.h
> index cb7220c1a7bd..a3518ea837cd 100644
> --- a/OvmfPkg/Csm/Include/Framework/BootScript.h
> +++ b/OvmfPkg/Csm/Include/Framework/BootScript.h
> @@ -1,5 +1,5 @@
>  /** @file
> -  This file contains the boot script defintions that are shared between the
> +  This file contains the boot script definitions that are shared between the
>    Boot Script Executor PPI and the Boot Script Save Protocol.
>  
>  Copyright (c) 2009 - 2018, Intel Corporation. All rights reserved.<BR>
> diff --git a/OvmfPkg/Csm/Include/Framework/FrameworkInternalFormRepresentation.h b/OvmfPkg/Csm/Include/Framework/FrameworkInternalFormRepresentation.h
> index 04cbae1ef5ae..f10fc5231289 100644
> --- a/OvmfPkg/Csm/Include/Framework/FrameworkInternalFormRepresentation.h
> +++ b/OvmfPkg/Csm/Include/Framework/FrameworkInternalFormRepresentation.h
> @@ -271,7 +271,7 @@ typedef struct {
>  
>  ///
>  /// Inconsistent with specification here:
> -/// The following defintion may not comply with Framework Specification HII 0.92. To
> +/// The following definition may not comply with Framework Specification HII 0.92. To
>  /// keep the inconsistant is for implementation needed.
>  ///@{
>  typedef struct {
> diff --git a/OvmfPkg/Csm/Include/Protocol/IsaIo.h b/OvmfPkg/Csm/Include/Protocol/IsaIo.h
> index 30000305fb7d..8807e421feec 100644
> --- a/OvmfPkg/Csm/Include/Protocol/IsaIo.h
> +++ b/OvmfPkg/Csm/Include/Protocol/IsaIo.h
> @@ -145,7 +145,7 @@ typedef struct {
>    @param[in] SrcOffset    The offset of the source in ISA MMIO space.
>    @param[in] Count        The number tranfers to perform for this copy operation.
>  
> -  @retval EFI_SUCCESS             The data was copied sucessfully.
> +  @retval EFI_SUCCESS             The data was copied successfully.
>    @retval EFI_UNSUPPORTED         The DestOffset or SrcOffset is not valid for this device.
>    @retval EFI_INVALID_PARAMETER   Width or Count, or both, were invalid.
>    @retval EFI_OUT_OF_RESOURCES    The request could not be completed due to a lack of resources.
> diff --git a/OvmfPkg/Csm/Include/Protocol/LegacyBios.h b/OvmfPkg/Csm/Include/Protocol/LegacyBios.h
> index 36761da39740..cd585a8d58f3 100644
> --- a/OvmfPkg/Csm/Include/Protocol/LegacyBios.h
> +++ b/OvmfPkg/Csm/Include/Protocol/LegacyBios.h
> @@ -1,7 +1,7 @@
>  /** @file
>    The EFI Legacy BIOS Protocol is used to abstract legacy Option ROM usage
>    under EFI and Legacy OS boot.  This file also includes all the related
> -  COMPATIBILIY16 structures and defintions.
> +  COMPATIBILIY16 structures and definitions.
>  
>    Note: The names for EFI_IA32_REGISTER_SET elements were picked to follow
>    well known naming conventions.
> @@ -213,7 +213,7 @@ typedef struct {
>    /// value of the start of the PCI Express memory-mapped configuration registers and
>    /// must be filled in prior to EfiCompatibility code issuing the Compatibility16 function
>    /// Compatibility16InitializeYourself().
> -  /// Compatibility16InitializeYourself() is defined in Compatability16
> +  /// Compatibility16InitializeYourself() is defined in Compatibility16
>    /// Functions.
>    ///
>    UINT32                            PciExpressBase;
> @@ -251,7 +251,7 @@ typedef struct {
>  
>  ///
>  /// Functions provided by the CSM binary which communicate between the EfiCompatibility
> -/// and Compatability16 code.
> +/// and Compatibility16 code.
>  ///
>  /// Inconsistent with the specification here:
>  /// The member's name started with "Compatibility16" [defined in Intel Framework
> diff --git a/OvmfPkg/Csm/Include/Protocol/LegacyBiosPlatform.h b/OvmfPkg/Csm/Include/Protocol/LegacyBiosPlatform.h
> index 0a164dad3bde..607ef358f0b0 100644
> --- a/OvmfPkg/Csm/Include/Protocol/LegacyBiosPlatform.h
> +++ b/OvmfPkg/Csm/Include/Protocol/LegacyBiosPlatform.h
> @@ -1,5 +1,5 @@
>  /** @file
> -  The EFI Legacy BIOS Patform Protocol is used to mate a Legacy16
> +  The EFI Legacy BIOS Platform Protocol is used to mate a Legacy16
>    implementation with this EFI code. The EFI driver that produces
>    the Legacy BIOS protocol is generic and consumes this protocol.
>    A driver that matches the Legacy16 produces this protocol
> @@ -383,7 +383,7 @@ typedef enum {
>    ///
>    ///     ShadowAddress First free OpROM area, after other OpROMs have been dispatched..
>    ///
> -  ///     Compatibility16Table Pointer to the Compatability16 Table.
> +  ///     Compatibility16Table Pointer to the Compatibility16 Table.
>    ///
>    ///       AdditionalData NULL.
>    ///
> diff --git a/OvmfPkg/Csm/Include/Protocol/VgaMiniPort.h b/OvmfPkg/Csm/Include/Protocol/VgaMiniPort.h
> index 41ff58e14cfa..5071c712ff9c 100644
> --- a/OvmfPkg/Csm/Include/Protocol/VgaMiniPort.h
> +++ b/OvmfPkg/Csm/Include/Protocol/VgaMiniPort.h
> @@ -30,7 +30,7 @@ typedef struct _EFI_VGA_MINI_PORT_PROTOCOL  EFI_VGA_MINI_PORT_PROTOCOL;
>    ModeNumber of 1 is a request for an 80x50 text mode.  If ModeNumber is greater
>    than MaxModeNumber, then EFI_UNSUPPORTED is returned.  If the VGA controller
>    is not functioning properly, then EFI_DEVICE_ERROR is returned.  If the VGA
> -  controller is sucessfully set to the mode number specified by ModeNumber, then
> +  controller is successfully set to the mode number specified by ModeNumber, then
>    EFI_SUCCESS is returned.
>  
>    @param[in] This         A pointer to the EFI_VGA_MINI_PORT_PROTOCOL instance.
> diff --git a/OvmfPkg/Csm/LegacyBiosDxe/LegacyBiosInterface.h b/OvmfPkg/Csm/LegacyBiosDxe/LegacyBiosInterface.h
> index a72c8470f6e4..88ec93d428bf 100644
> --- a/OvmfPkg/Csm/LegacyBiosDxe/LegacyBiosInterface.h
> +++ b/OvmfPkg/Csm/LegacyBiosDxe/LegacyBiosInterface.h
> @@ -501,7 +501,7 @@ typedef struct {
>    EFI_GENERIC_MEMORY_TEST_PROTOCOL  *GenericMemoryTest;
>  
>    //
> -  // TRUE if PCI Interupt Line registers have been programmed.
> +  // TRUE if PCI Interrupt Line registers have been programmed.
>    //
>    BOOLEAN                           PciInterruptLine;
>  
> @@ -706,7 +706,7 @@ LegacyBiosInt86 (
>    the Stack argument
>  
>    @param  This                   Protocol instance pointer.
> -  @param  Segment                Segemnt of 16-bit mode call
> +  @param  Segment                Segment of 16-bit mode call
>    @param  Offset                 Offset of 16-bit mdoe call
>    @param  Regs                   Register contexted passed into (and returned) from
>                                   thunk to  16-bit mode
> @@ -1004,7 +1004,7 @@ InitLegacyIdeController (
>  
>  /**
>    Program the interrupt routing register in all the PCI devices. On a PC AT system
> -  this register contains the 8259 IRQ vector that matches it's PCI interrupt.
> +  this register contains the 8259 IRQ vector that matches its PCI interrupt.
>  
>    @param  Private                Legacy  BIOS Instance data
>  
> @@ -1418,7 +1418,7 @@ RelocateImageUnder4GIfNeeded (
>    the Stack argument
>  
>    @param  This       Protocol instance pointer.
> -  @param  Segment    Segemnt of 16-bit mode call
> +  @param  Segment    Segment of 16-bit mode call
>    @param  Offset     Offset of 16-bit mdoe call
>    @param  Regs       Register contexted passed into (and returned) from thunk to
>                       16-bit mode
> diff --git a/OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUi.h b/OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUi.h
> index 0c491318c272..87290294df26 100644
> --- a/OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUi.h
> +++ b/OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUi.h
> @@ -1,5 +1,5 @@
>  /** @file
> -  Legacy boot maintainence Ui definition.
> +  Legacy boot maintenance Ui definition.
>  
>  Copyright (c) 2004 - 2015, Intel Corporation. All rights reserved.<BR>
>  SPDX-License-Identifier: BSD-2-Clause-Patent
> @@ -61,7 +61,7 @@ SPDX-License-Identifier: BSD-2-Clause-Patent
>  
>  
>  //
> -// String Contant
> +// String Constant
>  //
>  #define STR_FLOPPY          L"Floppy Drive #%02x"
>  #define STR_HARDDISK        L"HardDisk Drive #%02x"
> diff --git a/OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiVfr.h b/OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiVfr.h
> index e16fc3941515..13354865745d 100644
> --- a/OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiVfr.h
> +++ b/OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiVfr.h
> @@ -1,5 +1,5 @@
>  /** @file
> -  Legacy Boot Maintainence UI definition.
> +  Legacy Boot Maintenance UI definition.
>  
>  Copyright (c) 2004 - 2015, Intel Corporation. All rights reserved.<BR>
>  SPDX-License-Identifier: BSD-2-Clause-Patent
> diff --git a/OvmfPkg/Csm/BiosThunk/VideoDxe/BiosVideo.c b/OvmfPkg/Csm/BiosThunk/VideoDxe/BiosVideo.c
> index 0640656dba14..0a189bc59205 100644
> --- a/OvmfPkg/Csm/BiosThunk/VideoDxe/BiosVideo.c
> +++ b/OvmfPkg/Csm/BiosThunk/VideoDxe/BiosVideo.c
> @@ -381,7 +381,7 @@ BiosVideoDriverBindingStart (
>          (FeaturePcdGet (PcdBiosVideoCheckVbeEnable) || FeaturePcdGet (PcdBiosVideoCheckVgaEnable))) {
>        //
>        // If RemainingDevicePath is the End of Device Path Node,
> -      // don't create any child device and return EFI_SUCESS
> +      // don't create any child device and return EFI_SUCCESS
>        Status = EFI_SUCCESS;
>        goto Done;
>      }
> @@ -3211,7 +3211,7 @@ BiosVideoVgaMiniPortSetMode (
>  /**
>    Event handler for Exit Boot Service.
>  
> -  @param  Event       The event that be siganlled when exiting boot service.
> +  @param  Event       The event that be signalled when exiting boot service.
>    @param  Context     Pointer to instance of BIOS_VIDEO_DEV.
>  
>  **/
> diff --git a/OvmfPkg/Csm/CsmSupportLib/LegacyPlatform.c b/OvmfPkg/Csm/CsmSupportLib/LegacyPlatform.c
> index a07bb82da7f6..9c60c4a4cfc4 100644
> --- a/OvmfPkg/Csm/CsmSupportLib/LegacyPlatform.c
> +++ b/OvmfPkg/Csm/CsmSupportLib/LegacyPlatform.c
> @@ -282,7 +282,7 @@ GetSelectedVgaDeviceInfo (
>      Status = gBS->HandleProtocol (HandleBuffer[Index], &gEfiPciIoProtocolGuid, (VOID**)&PciIo);
>      if (!EFI_ERROR (Status)) {
>        //
> -      // Detemine if this is in the correct bus range.
> +      // Determine if this is in the correct bus range.
>        //
>        Status = PciIo->GetLocation (PciIo, &Segment, &Bus, &Device, &Function);
>        if (EFI_ERROR(Status) || (Bus < MinBus || Bus > MaxBus)) {
> @@ -501,7 +501,7 @@ GetPlatformHandle (
>  
>              //
>              // Be sure to only fill out correct information based on platform
> -            // configureation.
> +            // configuration.
>              //
>              HddInfo[HddIndex].Status        |= HDD_PRIMARY;
>              HddInfo[HddIndex].Bus           = (UINT32)Bus;
> @@ -882,7 +882,7 @@ TranslatePirq (
>          if (PirqData == 0) {
>  
>            //
> -          // No unused interrpts, so start reusing them.
> +          // No unused interrupts, so start reusing them.
>            //
>            MatchData = (UINT8) (~MatchData);
>          }
> diff --git a/OvmfPkg/Csm/LegacyBiosDxe/LegacyBios.c b/OvmfPkg/Csm/LegacyBiosDxe/LegacyBios.c
> index 5e795bfe6570..dd3608a64ece 100644
> --- a/OvmfPkg/Csm/LegacyBiosDxe/LegacyBios.c
> +++ b/OvmfPkg/Csm/LegacyBiosDxe/LegacyBios.c
> @@ -107,7 +107,7 @@ AllocateLegacyMemory (
>    64 KB blocks.
>  
>    Note: inconsistency with the Framework CSM spec. Per the spec, this function may be
> -  invoked only once. This limitation is relaxed to allow multiple calls in this implemenation.
> +  invoked only once. This limitation is relaxed to allow multiple calls in this implementation.
>  
>    @param  This                       Protocol instance pointer.
>    @param  LegacyMemorySize           Size of required region
> @@ -319,7 +319,7 @@ ShadowAndStartLegacy16 (
>    Private->LegacyBiosImageSize  = (UINT32) LegacyBiosImageSize;
>  
>    //
> -  // Can only shadow into memory allocated for legacy useage.
> +  // Can only shadow into memory allocated for legacy usage.
>    //
>    ASSERT (Private->BiosStart > Private->OptionRom);
>  
> @@ -983,7 +983,7 @@ LegacyBiosInstall (
>           );
>  
>    //
> -  // Allocate 0 - 4K for real mode interupt vectors and BDA.
> +  // Allocate 0 - 4K for real mode interrupt vectors and BDA.
>    //
>    AllocateLegacyMemory (
>      AllocateAddress,
> diff --git a/OvmfPkg/Csm/LegacyBiosDxe/LegacyBootSupport.c b/OvmfPkg/Csm/LegacyBiosDxe/LegacyBootSupport.c
> index cd4cd24f4246..20784504d501 100644
> --- a/OvmfPkg/Csm/LegacyBiosDxe/LegacyBootSupport.c
> +++ b/OvmfPkg/Csm/LegacyBiosDxe/LegacyBootSupport.c
> @@ -962,7 +962,7 @@ GenericLegacyBoot (
>    }
>  
>    //
> -  // We do not ASSERT if SmbiosTable not found. It is possbile that a platform does not produce SmbiosTable.
> +  // We do not ASSERT if SmbiosTable not found. It is possible that a platform does not produce SmbiosTable.
>    //
>    if (mReserveSmbiosEntryPoint == 0) {
>      DEBUG ((EFI_D_INFO, "Smbios table is not found!\n"));
> @@ -982,7 +982,7 @@ GenericLegacyBoot (
>                 );
>    }
>    //
> -  // We do not ASSERT if AcpiTable not found. It is possbile that a platform does not produce AcpiTable.
> +  // We do not ASSERT if AcpiTable not found. It is possible that a platform does not produce AcpiTable.
>    //
>    if (AcpiTable == NULL) {
>      DEBUG ((EFI_D_INFO, "ACPI table is not found!\n"));
> diff --git a/OvmfPkg/Csm/LegacyBiosDxe/LegacyPci.c b/OvmfPkg/Csm/LegacyBiosDxe/LegacyPci.c
> index dc1f760876a6..2656dafde943 100644
> --- a/OvmfPkg/Csm/LegacyBiosDxe/LegacyPci.c
> +++ b/OvmfPkg/Csm/LegacyBiosDxe/LegacyPci.c
> @@ -678,7 +678,7 @@ TranslateBusPirq (
>  
>  
>  Search busses starting from slot bus for final bus >= Secondary bus and
> -final bus <= Suborninate bus. Assumption is bus entries increase in bus
> +final bus <= Subordinate bus. Assumption is bus entries increase in bus
>  number.
>  Starting PIRQ is A,B,C,D.
>  Bus 2, Device 7 satisfies search criteria. Rotate (A,B,C,D) left by device
> @@ -914,7 +914,7 @@ InstallLegacyIrqHandler (
>    Legacy8259      = Private->Legacy8259;
>    //
>    // Disable interrupt in PIC, in case shared, to prevent an
> -  // interrupt from occuring.
> +  // interrupt from occurring.
>    //
>    Legacy8259->GetMask (
>                  Legacy8259,
> @@ -1049,7 +1049,7 @@ InstallLegacyIrqHandler (
>  
>  /**
>    Program the interrupt routing register in all the PCI devices. On a PC AT system
> -  this register contains the 8259 IRQ vector that matches it's PCI interrupt.
> +  this register contains the 8259 IRQ vector that matches its PCI interrupt.
>  
>    @param  Private                Legacy  BIOS Instance data
>  
> @@ -2582,7 +2582,7 @@ LegacyBiosInstallRom (
>    // do not follow the standard of setting AX = 0 on success.
>    //
>    //
> -  // The ROM could have updated it's size so we need to read again.
> +  // The ROM could have updated its size so we need to read again.
>    //
>    if (((EFI_LEGACY_EXPANSION_ROM_HEADER *) RuntimeAddress)->Signature != PCI_EXPANSION_ROM_HEADER_SIGNATURE) {
>      //
> diff --git a/OvmfPkg/Csm/LegacyBiosDxe/Thunk.c b/OvmfPkg/Csm/LegacyBiosDxe/Thunk.c
> index a4985ede77a5..6432030a23ac 100644
> --- a/OvmfPkg/Csm/LegacyBiosDxe/Thunk.c
> +++ b/OvmfPkg/Csm/LegacyBiosDxe/Thunk.c
> @@ -127,8 +127,8 @@ LegacyBiosFarCall86 (
>    Provide NULL interrupt handler which is used to check
>    if there is more than one HW interrupt registers with the CPU AP.
>  
> -  @param  InterruptType - The type of interrupt that occured
> -  @param  SystemContext - A pointer to the system context when the interrupt occured
> +  @param  InterruptType - The type of interrupt that occurred
> +  @param  SystemContext - A pointer to the system context when the interrupt occurred
>  
>  **/
>  VOID
> diff --git a/OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUi.c b/OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUi.c
> index af7d80eb5f04..35470abf914a 100644
> --- a/OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUi.c
> +++ b/OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUi.c
> @@ -1,5 +1,5 @@
>  /** @file
> -  Legacy Boot Maintainence UI implementation.
> +  Legacy Boot Maintenance UI implementation.
>  
>  Copyright (c) 2004 - 2018, Intel Corporation. All rights reserved.<BR>
>  (C) Copyright 2018 Hewlett Packard Enterprise Development LP<BR>
> @@ -237,10 +237,10 @@ OrderLegacyBootOption4SameType (
>  
>  /**
>    Update the legacy BBS boot option. L"LegacyDevOrder" and gEfiLegacyDevOrderVariableGuid EFI Variable
> -  is udpated with the new Legacy Boot order. The EFI Variable of "Boot####" and gEfiGlobalVariableGuid
> +  is updated with the new Legacy Boot order. The EFI Variable of "Boot####" and gEfiGlobalVariableGuid
>    is also updated.
>  
> -  @param NVMapData   The data for egacy BBS boot.
> +  @param NVMapData   The data for legacy BBS boot.
>  
>    @return EFI_SUCCESS           The function completed successfully.
>    @retval EFI_NOT_FOUND         If L"LegacyDevOrder" and gEfiLegacyDevOrderVariableGuid EFI Variable can not be found.
> @@ -725,7 +725,7 @@ CreateLegacyMenuStringToken (
>    Create a dynamic page so that Legacy Device boot order
>    can be set for specified device type.
>  
> -  @param UpdatePageId    The form ID. It also spefies the legacy device type.
> +  @param UpdatePageId    The form ID. It also specifies the legacy device type.
>  
>  
>  **/
> @@ -1127,8 +1127,8 @@ LegacyBootOptionCallback (
>      if (QuestionId == FORM_FLOPPY_BOOT_ID) {
>        if (!mFirstEnterLegacyForm) {
>          //
> -        // The leagcyBootMaintUiLib depends on the LegacyBootManagerLib to realize its functionality.
> -        // We need to do the leagcy boot options related actions after the LegacyBootManagerLib has been initialized.
> +        // The legacyBootMaintUiLib depends on the LegacyBootManagerLib to realize its functionality.
> +        // We need to do the legacy boot options related actions after the LegacyBootManagerLib has been initialized.
>          // Opening the legacy menus is the appropriate time that the LegacyBootManagerLib has already been initialized.
>          //
>          mFirstEnterLegacyForm = TRUE;
> @@ -1403,7 +1403,7 @@ GetLegacyOptions (
>    @param ImageHandle     The image handle.
>    @param SystemTable     The system table.
>  
> -  @retval  EFI_SUCEESS  Install Boot manager menu success.
> +  @retval  EFI_SUCCESS  Install Boot manager menu success.
>    @retval  Other        Return error status.
>  
>  **/
> diff --git a/OvmfPkg/Csm/LegacyBootManagerLib/LegacyBm.c b/OvmfPkg/Csm/LegacyBootManagerLib/LegacyBm.c
> index 6138a32ad78e..6ed45785be72 100644
> --- a/OvmfPkg/Csm/LegacyBootManagerLib/LegacyBm.c
> +++ b/OvmfPkg/Csm/LegacyBootManagerLib/LegacyBm.c
> @@ -173,7 +173,7 @@ LegacyBmBuildLegacyDevNameString (
>    StringDesc = (CHAR8 *) (((UINTN) CurBBSEntry->DescStringSegment << 4) + CurBBSEntry->DescStringOffset);
>    if (NULL != StringDesc) {
>      //
> -    // Only get fisrt 32 characters, this is suggested by BBS spec
> +    // Only get first 32 characters, this is suggested by BBS spec
>      //
>      CopyMem (StringBufferA, StringDesc, LEGACY_BM_BOOT_DESCRIPTION_LENGTH);
>      StringBufferA[LEGACY_BM_BOOT_DESCRIPTION_LENGTH] = 0;
> @@ -378,9 +378,9 @@ LegacyBmDeleteAllBootOptions (
>  /**
>    Delete all the invalid legacy boot options.
>  
> -  @retval EFI_SUCCESS             All invalide legacy boot options are deleted.
> +  @retval EFI_SUCCESS             All invalid legacy boot options are deleted.
>    @retval EFI_OUT_OF_RESOURCES    Fail to allocate necessary memory.
> -  @retval EFI_NOT_FOUND           Fail to retrive variable of boot order.
> +  @retval EFI_NOT_FOUND           Fail to retrieve variable of boot order.
>  **/
>  EFI_STATUS
>  LegacyBmDeleteAllInvalidBootOptions (
> @@ -499,7 +499,7 @@ LegacyBmDeleteAllInvalidBootOptions (
>  /**
>    Create legacy boot option.
>  
> -  @param BootOption        Ponter to the boot option which will be crated.
> +  @param BootOption        Pointer to the boot option which will be crated.
>    @param BbsEntry          The input bbs entry info.
>    @param BbsIndex          The BBS index.
>  
> @@ -615,10 +615,10 @@ LegacyBmFillDevOrderBuf (
>    @param BbsTable        The BBS table.
>    @param BbsCount        The BBS Count.
>  
> -  @retval EFI_SUCCES             The buffer is created and the EFI variable named
> +  @retval EFI_SUCCESS            The buffer is created and the EFI variable named
>                                   VAR_LEGACY_DEV_ORDER and EfiLegacyDevOrderGuid is
>                                   set correctly.
> -  @retval EFI_OUT_OF_RESOURCES   Memmory or storage is not enough.
> +  @retval EFI_OUT_OF_RESOURCES   Memory or storage is not enough.
>    @retval EFI_DEVICE_ERROR       Fail to add the device order into EFI variable fail
>                                   because of hardware error.
>  **/
> @@ -742,7 +742,7 @@ LegacyBmCreateDevOrder (
>  
>    @retval EFI_SUCCESS           The boot devices are added successfully.
>    @retval EFI_NOT_FOUND         The legacy boot devices are not found.
> -  @retval EFI_OUT_OF_RESOURCES  Memmory or storage is not enough.
> +  @retval EFI_OUT_OF_RESOURCES  Memory or storage is not enough.
>    @retval EFI_DEVICE_ERROR      Fail to add the legacy device boot order into EFI variable
>                                  because of hardware error.
>  **/
> @@ -1042,7 +1042,7 @@ LegacyBmUpdateDevOrder (
>    @param DeviceType      The device type.
>    @param BbsIndex        The BBS index to set the highest priority. Ignore when -1.
>    @param LocalBbsTable   The BBS table.
> -  @param Priority        The prority table.
> +  @param Priority        The priority table.
>  
>    @retval EFI_SUCCESS           The function completes successfully.
>    @retval EFI_NOT_FOUND         Failed to find device.
> @@ -1494,7 +1494,7 @@ LegacyBmRefreshAllBootOption (
>    //
>    // Same algorithm pattern as the EfiBootManagerRefreshAllBootOption
>    // Firstly delete the invalid legacy boot options,
> -  // then enumreate and save the newly appeared legacy boot options
> +  // then enumerate and save the newly appeared legacy boot options
>    // the last step is legacy boot option special action to refresh the LegacyDevOrder variable
>    //
>    LegacyBmDeleteAllInvalidBootOptions ();
> diff --git a/OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiLib.uni b/OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiLib.uni
> index c2c100d378b3..f29e1449a749 100644
> --- a/OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiLib.uni
> +++ b/OvmfPkg/Csm/LegacyBootMaintUiLib/LegacyBootMaintUiLib.uni
> @@ -1,7 +1,7 @@
>  // /** @file
> -// Legacy Boot Maintainence UI module is library for BDS phase.
> +// Legacy Boot Maintenance UI module is library for BDS phase.
>  //
> -// Legacy Boot Maintainence UI module is library for BDS phase.
> +// Legacy Boot Maintenance UI module is library for BDS phase.
>  //
>  // Copyright (c) 2015, Intel Corporation. All rights reserved.<BR>
>  //
> @@ -11,10 +11,10 @@
>  
>  #string STR_MODULE_ABSTRACT
>  #language en-US
> -"Legacy Boot Maintainence UI module is library for BDS phase."
> +"Legacy Boot Maintenance UI module is library for BDS phase."
>  
>  #string STR_MODULE_DESCRIPTION
>  #language en-US
> -"Legacy Boot Maintainence UI module is library for BDS phase."
> +"Legacy Boot Maintenance UI module is library for BDS phase."
>  
>  
>