On 12/03/19 17:15, Philippe Mathieu-Daude wrote: > From: Antoine Coeur <co...@gmx.fr> > > Fix various typos in documentation, comments and strings. > > Cc: Jordan Justen <jordan.l.jus...@intel.com> > Cc: Laszlo Ersek <ler...@redhat.com> > Cc: Ard Biesheuvel <ard.biesheu...@linaro.org> > Reviewed-by: Philippe Mathieu-Daude <phi...@redhat.com> > Signed-off-by: Philippe Mathieu-Daude <phi...@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 <ler...@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." > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#51663): https://edk2.groups.io/g/devel/message/51663 Mute This Topic: https://groups.io/mt/65928418/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-