Reviewed-by: Jiewen Yao <jiewen....@intel.com>

> -----Original Message-----
> From: pierre.gond...@arm.com <pierre.gond...@arm.com>
> Sent: Wednesday, August 24, 2022 12:00 AM
> To: devel@edk2.groups.io
> Cc: Pierre Gondois <pierre.gond...@arm.com>; Ard Biesheuvel
> <ardb+tianoc...@kernel.org>; Yao, Jiewen <jiewen....@intel.com>
> Subject: [PATCH 15/24] OvmfPkg: Remove duplicated words
> 
> From: Pierre Gondois <pierre.gond...@arm.com>
> 
> In an effort to clean the documentation of the above
> package, remove duplicated words.
> 
> Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org>
> Cc: Jiewen Yao <jiewen....@intel.com>
> Signed-off-by: Pierre Gondois <pierre.gond...@arm.com>
> ---
>  OvmfPkg/Library/PeilessStartupLib/X64/VirtualMemory.c     | 6 +++---
>  OvmfPkg/Library/PlatformDebugLibIoPort/DebugLib.c         | 2 +-
>  OvmfPkg/Library/XenRealTimeClockLib/XenRealTimeClockLib.c | 2 +-
>  OvmfPkg/QemuFlashFvbServicesRuntimeDxe/QemuFlashDxe.c     | 2 +-
>  OvmfPkg/SataControllerDxe/ComponentName.c                 | 4 ++--
>  OvmfPkg/SataControllerDxe/SataController.h                | 4 ++--
>  OvmfPkg/XenBusDxe/ComponentName.c                         | 2 +-
>  OvmfPkg/XenBusDxe/ComponentName.h                         | 2 +-
>  OvmfPkg/XenPvBlkDxe/BlockFront.c                          | 2 +-
>  OvmfPkg/XenPvBlkDxe/ComponentName.c                       | 2 +-
>  OvmfPkg/XenPvBlkDxe/ComponentName.h                       | 2 +-
>  11 files changed, 15 insertions(+), 15 deletions(-)
> 
> diff --git a/OvmfPkg/Library/PeilessStartupLib/X64/VirtualMemory.c
> b/OvmfPkg/Library/PeilessStartupLib/X64/VirtualMemory.c
> index 8da06da74ef5..6877e521e485 100644
> --- a/OvmfPkg/Library/PeilessStartupLib/X64/VirtualMemory.c
> +++ b/OvmfPkg/Library/PeilessStartupLib/X64/VirtualMemory.c
> @@ -813,7 +813,7 @@ CreateIdentityMappingPageTables (
>    {
>      //
>      // Each PML5 entry points to a page of PML4 entires.
> -    // So lets allocate space for them and fill them in in the 
> IndexOfPml4Entries
> loop.
> +    // So lets allocate space for them and fill them in the 
> IndexOfPml4Entries
> loop.
>      // When 5-Level Paging is disabled, below allocation happens only once.
>      //
>      PageMapLevel4Entry = (VOID *)BigPageAddress;
> @@ -835,7 +835,7 @@ CreateIdentityMappingPageTables (
>      {
>        //
>        // Each PML4 entry points to a page of Page Directory Pointer entires.
> -      // So lets allocate space for them and fill them in in the 
> IndexOfPdpEntries
> loop.
> +      // So lets allocate space for them and fill them in the 
> IndexOfPdpEntries
> loop.
>        //
>        PageDirectoryPointerEntry = (VOID *)BigPageAddress;
>        BigPageAddress           += SIZE_4KB;
> @@ -875,7 +875,7 @@ CreateIdentityMappingPageTables (
>          {
>            //
>            // Each Directory Pointer entries points to a page of Page 
> Directory entires.
> -          // So allocate space for them and fill them in in the
> IndexOfPageDirectoryEntries loop.
> +          // So allocate space for them and fill them in the
> IndexOfPageDirectoryEntries loop.
>            //
>            PageDirectoryEntry = (VOID *)BigPageAddress;
>            BigPageAddress    += SIZE_4KB;
> diff --git a/OvmfPkg/Library/PlatformDebugLibIoPort/DebugLib.c
> b/OvmfPkg/Library/PlatformDebugLibIoPort/DebugLib.c
> index 4e25f198aa76..a4c12b9057e5 100644
> --- a/OvmfPkg/Library/PlatformDebugLibIoPort/DebugLib.c
> +++ b/OvmfPkg/Library/PlatformDebugLibIoPort/DebugLib.c
> @@ -259,7 +259,7 @@ DebugClearMemory (
>    ASSERT (Buffer != NULL);
> 
>    //
> -  // SetMem() checks for the the ASSERT() condition on Length and returns
> Buffer
> +  // SetMem() checks for the ASSERT() condition on Length and returns Buffer
>    //
>    return SetMem (Buffer, Length, PcdGet8 (PcdDebugClearMemoryValue));
>  }
> diff --git a/OvmfPkg/Library/XenRealTimeClockLib/XenRealTimeClockLib.c
> b/OvmfPkg/Library/XenRealTimeClockLib/XenRealTimeClockLib.c
> index 7f37710b964f..30c258313135 100644
> --- a/OvmfPkg/Library/XenRealTimeClockLib/XenRealTimeClockLib.c
> +++ b/OvmfPkg/Library/XenRealTimeClockLib/XenRealTimeClockLib.c
> @@ -108,7 +108,7 @@ LibGetTime (
> 
>    @retval EFI_SUCCESS           The operation completed successfully.
>    @retval EFI_INVALID_PARAMETER A time field is out of range.
> -  @retval EFI_DEVICE_ERROR      The time could not be set due due to hardware
> error.
> +  @retval EFI_DEVICE_ERROR      The time could not be set due to hardware
> error.
> 
>  **/
>  EFI_STATUS
> diff --git a/OvmfPkg/QemuFlashFvbServicesRuntimeDxe/QemuFlashDxe.c
> b/OvmfPkg/QemuFlashFvbServicesRuntimeDxe/QemuFlashDxe.c
> index 9e074c29bb2b..172d6a4267f8 100644
> --- a/OvmfPkg/QemuFlashFvbServicesRuntimeDxe/QemuFlashDxe.c
> +++ b/OvmfPkg/QemuFlashFvbServicesRuntimeDxe/QemuFlashDxe.c
> @@ -79,7 +79,7 @@ QemuFlashPtrWrite (
>      // Writing to flash is emulated by the hypervisor through the use of 
> write
>      // protection. This won't work for an SEV-ES guest because the write 
> won't
>      // be recognized as a true MMIO write, which would result in the required
> -    // #VC exception. Instead, use the the VMGEXIT MMIO write support 
> directly
> +    // #VC exception. Instead, use the VMGEXIT MMIO write support directly
>      // to perform the update.
>      //
>      VmgInit (Ghcb, &InterruptState);
> diff --git a/OvmfPkg/SataControllerDxe/ComponentName.c
> b/OvmfPkg/SataControllerDxe/ComponentName.c
> index d45803f95f6e..7b5912bab822 100644
> --- a/OvmfPkg/SataControllerDxe/ComponentName.c
> +++ b/OvmfPkg/SataControllerDxe/ComponentName.c
> @@ -59,7 +59,7 @@ GLOBAL_REMOVE_IF_UNREFERENCED
> EFI_UNICODE_STRING_TABLE  mSataControllerControlle
> 
>    @param This           A pointer to the EFI_COMPONENT_NAME_PROTOCOL
> instance.
>    @param Language       A pointer to a three character ISO 639-2 language
> identifier.
> -                        This is the language of the driver name that that 
> the caller
> +                        This is the language of the driver name that the 
> caller
>                          is requesting, and it must match one of the 
> languages specified
>                          in SupportedLanguages.  The number of languages 
> supported by a
>                          driver is up to the driver writer.
> @@ -108,7 +108,7 @@ SataControllerComponentNameGetDriverName (
>                                  that wishes to retrieve the name of a child 
> controller.
>    @param Language               A pointer to a three character ISO 639-2 
> language
>                                  identifier.  This is the language of the 
> controller name
> -                                that that the caller is requesting, and it 
> must match one
> +                                that the caller is requesting, and it must 
> match one
>                                  of the languages specified in 
> SupportedLanguages.  The
>                                  number of languages supported by a driver is 
> up to the
>                                  driver writer.
> diff --git a/OvmfPkg/SataControllerDxe/SataController.h
> b/OvmfPkg/SataControllerDxe/SataController.h
> index ad93ab1d98c3..cb1abacfdc0f 100644
> --- a/OvmfPkg/SataControllerDxe/SataController.h
> +++ b/OvmfPkg/SataControllerDxe/SataController.h
> @@ -465,7 +465,7 @@ IdeInitSetTiming (
> 
>    @param This           A pointer to the EFI_COMPONENT_NAME_PROTOCOL
> instance.
>    @param Language       A pointer to a three character ISO 639-2 language
> identifier.
> -                        This is the language of the driver name that that 
> the caller
> +                        This is the language of the driver name that the 
> caller
>                          is requesting, and it must match one of the 
> languages specified
>                          in SupportedLanguages.  The number of languages 
> supported by a
>                          driver is up to the driver writer.
> @@ -506,7 +506,7 @@ SataControllerComponentNameGetDriverName (
>                                  that wishes to retrieve the name of a child 
> controller.
>    @param Language               A pointer to a three character ISO 639-2 
> language
>                                  identifier.  This is the language of the 
> controller name
> -                                that that the caller is requesting, and it 
> must match one
> +                                that the caller is requesting, and it must 
> match one
>                                  of the languages specified in 
> SupportedLanguages.  The
>                                  number of languages supported by a driver is 
> up to the
>                                  driver writer.
> diff --git a/OvmfPkg/XenBusDxe/ComponentName.c
> b/OvmfPkg/XenBusDxe/ComponentName.c
> index 9141e9264fc9..a5622bfef93b 100644
> --- a/OvmfPkg/XenBusDxe/ComponentName.c
> +++ b/OvmfPkg/XenBusDxe/ComponentName.c
> @@ -52,7 +52,7 @@ EFI_UNICODE_STRING_TABLE
> mXenBusDxeControllerNameTable[] = {
> 
>    @param  This       A pointer to the EFI_COMPONENT_NAME_PROTOCOL
> instance.
>    @param  Language   A pointer to a three-character ISO 639-2 language
> identifier.
> -                     This is the language of the driver name that that the 
> caller
> +                     This is the language of the driver name that the caller
>                       is requesting, and it must match one of the languages 
> specified
>                       in SupportedLanguages.  The number of languages 
> supported by a
>                       driver is up to the driver writer.
> diff --git a/OvmfPkg/XenBusDxe/ComponentName.h
> b/OvmfPkg/XenBusDxe/ComponentName.h
> index 592c5426dd77..88f3fe5b017b 100644
> --- a/OvmfPkg/XenBusDxe/ComponentName.h
> +++ b/OvmfPkg/XenBusDxe/ComponentName.h
> @@ -12,7 +12,7 @@
> 
>    @param  This       A pointer to the EFI_COMPONENT_NAME_PROTOCOL
> instance.
>    @param  Language   A pointer to a three-character ISO 639-2 language
> identifier.
> -                     This is the language of the driver name that that the 
> caller
> +                     This is the language of the driver name that the caller
>                       is requesting, and it must match one of the languages 
> specified
>                       in SupportedLanguages.  The number of languages 
> supported by a
>                       driver is up to the driver writer.
> diff --git a/OvmfPkg/XenPvBlkDxe/BlockFront.c
> b/OvmfPkg/XenPvBlkDxe/BlockFront.c
> index 7aa5ef044be0..1827cc58e390 100644
> --- a/OvmfPkg/XenPvBlkDxe/BlockFront.c
> +++ b/OvmfPkg/XenPvBlkDxe/BlockFront.c
> @@ -88,7 +88,7 @@ XenPvBlockFree (
>  }
> 
>  /**
> -  Wait until until the backend has reached the ExpectedState.
> +  Wait until the backend has reached the ExpectedState.
> 
>    @param Dev            A XEN_BLOCK_FRONT_DEVICE instance.
>    @param ExpectedState  The backend state expected.
> diff --git a/OvmfPkg/XenPvBlkDxe/ComponentName.c
> b/OvmfPkg/XenPvBlkDxe/ComponentName.c
> index 6a27822749aa..f1fd7830fb1d 100644
> --- a/OvmfPkg/XenPvBlkDxe/ComponentName.c
> +++ b/OvmfPkg/XenPvBlkDxe/ComponentName.c
> @@ -52,7 +52,7 @@ EFI_UNICODE_STRING_TABLE
> mXenPvBlkDxeControllerNameTable[] = {
> 
>    @param  This       A pointer to the EFI_COMPONENT_NAME_PROTOCOL
> instance.
>    @param  Language   A pointer to a three-character ISO 639-2 language
> identifier.
> -                     This is the language of the driver name that that the 
> caller
> +                     This is the language of the driver name that the caller
>                       is requesting, and it must match one of the languages 
> specified
>                       in SupportedLanguages.  The number of languages 
> supported by a
>                       driver is up to the driver writer.
> diff --git a/OvmfPkg/XenPvBlkDxe/ComponentName.h
> b/OvmfPkg/XenPvBlkDxe/ComponentName.h
> index 12ea12601d7f..cd897966931f 100644
> --- a/OvmfPkg/XenPvBlkDxe/ComponentName.h
> +++ b/OvmfPkg/XenPvBlkDxe/ComponentName.h
> @@ -12,7 +12,7 @@
> 
>    @param  This       A pointer to the EFI_COMPONENT_NAME_PROTOCOL
> instance.
>    @param  Language   A pointer to a three-character ISO 639-2 language
> identifier.
> -                     This is the language of the driver name that that the 
> caller
> +                     This is the language of the driver name that the caller
>                       is requesting, and it must match one of the languages 
> specified
>                       in SupportedLanguages.  The number of languages 
> supported by a
>                       driver is up to the driver writer.
> --
> 2.25.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#92731): https://edk2.groups.io/g/devel/message/92731
Mute This Topic: https://groups.io/mt/93207401/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to