From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-1.mimecast.com (us-smtp-delivery-1.mimecast.com [205.139.110.61]) by mx.groups.io with SMTP id smtpd.web10.1937.1575406236010154555 for ; Tue, 03 Dec 2019 12:50:36 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=EiAKXroV; spf=pass (domain: redhat.com, ip: 205.139.110.61, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1575406235; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=90LjmUOrx2fr7tU+DhAGXsC3E48UP8Mx/yHPWWzh9uk=; b=EiAKXroV18f8eeTU75g/NWJ3WInHYPz30IrhFIUhEwcMhh+NPNIej/UMLT44083flh0g5a PpqqS+kpusqyynVmo46INEr01IcNGh5H/1bI5WzGf/1MlPkr2tHsGAFTOrXpgQEvtp7Dpp mFurQmJhowDIuzQF7lmzWhGrA1CxsMk= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-374-jHrEuyKlNLeiKHLNV5o_vQ-1; Tue, 03 Dec 2019 15:50:31 -0500 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 782CD64A7D; Tue, 3 Dec 2019 20:50:30 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-117-183.ams2.redhat.com [10.36.117.183]) by smtp.corp.redhat.com (Postfix) with ESMTP id 327625D6A7; Tue, 3 Dec 2019 20:50:26 +0000 (UTC) Subject: Re: [PATCH 57/79] OvmfPkg/Csm: Fix various typos To: Philippe Mathieu-Daude , devel@edk2.groups.io Cc: Antoine Coeur , Jordan Justen , Ard Biesheuvel , David Woodhouse References: <20191203161602.15969-1-philmd@redhat.com> <20191203161602.15969-58-philmd@redhat.com> From: "Laszlo Ersek" Message-ID: Date: Tue, 3 Dec 2019 21:50:25 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: <20191203161602.15969-58-philmd@redhat.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.15 X-MC-Unique: jHrEuyKlNLeiKHLNV5o_vQ-1 X-Mimecast-Spam-Score: 0 Content-Language: en-US Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit On 12/03/19 17:15, Philippe Mathieu-Daude wrote: > From: Antoine Coeur > > Fix various typos in documentation, comments and strings. > > Cc: Jordan Justen > Cc: Laszlo Ersek > Cc: Ard Biesheuvel > Reviewed-by: Philippe Mathieu-Daude > Signed-off-by: Philippe Mathieu-Daude > --- > 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 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.
> # 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.
> 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.
> 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.
> 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.
> (C) Copyright 2018 Hewlett Packard Enterprise Development LP
> @@ -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.
> // > @@ -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." > > >