public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: Gary Lin <glin@suse.com>
To: "Fan, Jeff" <jeff.fan@intel.com>
Cc: "edk2-devel@lists.01.org" <edk2-devel@lists.01.org>
Subject: Re: [PATCH 32/33] UefiCpuPkg: Fix typos in comments
Date: Thu, 20 Oct 2016 10:08:46 +0800	[thread overview]
Message-ID: <20161020020846.xwaap72vfxk5hwnb@GaryWorkstation> (raw)
In-Reply-To: <542CF652F8836A4AB8DBFAAD40ED192A4A2B0113@shsmsx102.ccr.corp.intel.com>

On Thu, Oct 20, 2016 at 01:58:51AM +0000, Fan, Jeff wrote:
> Reviewed-by: Jeff Fan <jeff.fan@intel.com>
> 
> Could you please help to fix typo *ruturn* when you commit the patch.
> -  @return                       The ruturn value of this paramter indicates the number of
> +  @return                       The ruturn value of this parameter indicates the number of
> 
I don't have the write access to the edk2 git. Should I submit a v2 to
fix this one?

Thanks,

Gary Lin

> Thanks!
> 
> -----Original Message-----
> From: Gary Lin [mailto:glin@suse.com] 
> Sent: Wednesday, October 19, 2016 3:02 PM
> To: edk2-devel@lists.01.org
> Cc: Fan, Jeff
> Subject: [PATCH 32/33] UefiCpuPkg: Fix typos in comments
> 
> - excute -> execute
> - Retrive -> Retrieve
> - possilbe -> possible
> - CONTINOUS -> CONTINUOUS
> - storgage -> storage
> - allcated -> allocated
> - triggerred -> triggered
> - paramter -> parameter
> - perodically -> periodically
> - retore -> restore
> 
> Cc: Jeff Fan <jeff.fan@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Gary Lin <glin@suse.com>
> ---
>  UefiCpuPkg/Include/Library/MtrrLib.h                              |  2 +-
>  UefiCpuPkg/Library/CpuExceptionHandlerLib/PeiDxeSmmCpuException.c |  2 +-
>  UefiCpuPkg/Library/MpInitLib/DxeMpLib.c                           |  2 +-
>  UefiCpuPkg/Library/MtrrLib/MtrrLib.c                              |  2 +-
>  UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume.c                 | 10 +++++-----
>  5 files changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/UefiCpuPkg/Include/Library/MtrrLib.h b/UefiCpuPkg/Include/Library/MtrrLib.h
> index 36cd2cd..5d62adb 100644
> --- a/UefiCpuPkg/Include/Library/MtrrLib.h
> +++ b/UefiCpuPkg/Include/Library/MtrrLib.h
> @@ -306,7 +306,7 @@ MtrrSetAllMtrrs (
>                                      MtrrValidBitsMask & 0xfffffffffffff000ULL
>    @param[out]  VariableMtrr         The array to shadow variable MTRRs content
>  
> -  @return                       The ruturn value of this paramter indicates the number of
> +  @return                       The ruturn value of this parameter indicates the number of
>                                  MTRRs which has been used.
>  **/
>  UINT32
> diff --git a/UefiCpuPkg/Library/CpuExceptionHandlerLib/PeiDxeSmmCpuException.c b/UefiCpuPkg/Library/CpuExceptionHandlerLib/PeiDxeSmmCpuException.c
> index 02d5622..e55709c 100644
> --- a/UefiCpuPkg/Library/CpuExceptionHandlerLib/PeiDxeSmmCpuException.c
> +++ b/UefiCpuPkg/Library/CpuExceptionHandlerLib/PeiDxeSmmCpuException.c
> @@ -65,7 +65,7 @@ CommonExceptionHandlerWorker (
>        //
>        if (ReservedVectors[ExceptionType].ApicId == GetApicId ()) {
>          //
> -        // Old IDT handler has been executed, then retore CPU exception content to
> +        // Old IDT handler has been executed, then restore CPU 
> + exception content to
>          // run new exception handler.
>          //
>          ArchRestoreExceptionContext (ExceptionType, SystemContext); diff --git a/UefiCpuPkg/Library/MpInitLib/DxeMpLib.c b/UefiCpuPkg/Library/MpInitLib/DxeMpLib.c
> index 50b5b27..b399f1c 100644
> --- a/UefiCpuPkg/Library/MpInitLib/DxeMpLib.c
> +++ b/UefiCpuPkg/Library/MpInitLib/DxeMpLib.c
> @@ -172,7 +172,7 @@ CheckAndUpdateApsStatus (
>  /**
>    Checks APs' status periodically.
>  
> -  This function is triggerred by timer perodically to check the
> +  This function is triggered by timer periodically to check the
>    state of APs for StartupAllAPs() and StartupThisAP() executed
>    in non-blocking mode.
>  
> diff --git a/UefiCpuPkg/Library/MtrrLib/MtrrLib.c b/UefiCpuPkg/Library/MtrrLib/MtrrLib.c
> index 5aef772..7783b63 100644
> --- a/UefiCpuPkg/Library/MtrrLib/MtrrLib.c
> +++ b/UefiCpuPkg/Library/MtrrLib/MtrrLib.c
> @@ -606,7 +606,7 @@ MtrrGetMemoryAttributeInVariableMtrrWorker (
>    @param[in]   MtrrValidAddressMask  The valid address mask for MTRR
>    @param[out]  VariableMtrr          The array to shadow variable MTRRs content
>  
> -  @return                       The return value of this paramter indicates the
> +  @return                       The return value of this parameter indicates the
>                                  number of MTRRs which has been used.
>  
>  **/
> diff --git a/UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume.c b/UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume.c
> index 0ccf3a4..f907b30 100644
> --- a/UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume.c
> +++ b/UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume.c
> @@ -1,7 +1,7 @@
>  /** @file
>    This module produces the EFI_PEI_S3_RESUME2_PPI.
>    This module works with StandAloneBootScriptExecutor to S3 resume to OS.
> -  This module will excute the boot script saved during last boot and after that,
> +  This module will execute the boot script saved during last boot and 
> + after that,
>    control is passed to OS waking up handler.
>  
>    Copyright (c) 2006 - 2016, Intel Corporation. All rights reserved.<BR> @@ -294,7 +294,7 @@ WriteToOsS3PerformanceData (
>    UINT64                                        Freq;
>  
>    //
> -  // Retrive time stamp count as early as possilbe
> +  // Retrieve time stamp count as early as possible
>    //
>    Ticker = GetPerformanceCounter ();
>  
> @@ -612,12 +612,12 @@ RestoreS3PageTables (
>      // NOTE: We have to ASSUME the page table generation format, because we do not know whole page table information.
>      // The whole page table is too large to be saved in SMRAM.
>      //
> -    // The assumption is : whole page table is allocated in CONTINOUS memory and CR3 points to TOP page.
> +    // The assumption is : whole page table is allocated in CONTINUOUS memory and CR3 points to TOP page.
>      //
>      DEBUG ((EFI_D_ERROR, "S3NvsPageTableAddress - %x (%x)\n", (UINTN)S3NvsPageTableAddress, (UINTN)Build4GPageTableOnly));
>  
>      //
> -    // By architecture only one PageMapLevel4 exists - so lets allocate storgage for it.
> +    // By architecture only one PageMapLevel4 exists - so lets allocate storage for it.
>      //
>      PageMap = (PAGE_MAP_AND_DIRECTORY_POINTER *)S3NvsPageTableAddress;
>      S3NvsPageTableAddress += SIZE_4KB;
> @@ -819,7 +819,7 @@ S3ResumeExecuteBootScript (
>      //
>      IdtDescriptor = (IA32_DESCRIPTOR *) (UINTN) (AcpiS3Context->IdtrProfile);
>      //
> -    // Make sure the newly allcated IDT align with 16-bytes
> +    // Make sure the newly allocated IDT align with 16-bytes
>      // 
>      IdtBuffer = AllocatePages (EFI_SIZE_TO_PAGES((IdtDescriptor->Limit + 1) + 16));
>      if (IdtBuffer == NULL) {
> --
> 2.10.1
> 
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
> 


  reply	other threads:[~2016-10-20  2:08 UTC|newest]

Thread overview: 79+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-10-19  7:01 [PATCH 00/33] Fix typos in comments and variables Gary Lin
2016-10-19  7:01 ` [PATCH 01/33] ArmPkg: Fix typos in comments Gary Lin
2016-10-19 15:09   ` Ard Biesheuvel
2016-10-19 15:15     ` Laszlo Ersek
2016-10-19 15:33       ` Leif Lindholm
2016-10-19  7:01 ` [PATCH 02/33] BaseTools: Fix typos in comments and variables Gary Lin
2016-10-19  7:30   ` Gao, Liming
2016-10-19  7:36   ` Zhu, Yonghong
2016-10-19  7:01 ` [PATCH 03/33] CorebootPayloadPkg: Fix typos in comments Gary Lin
2016-10-26 18:50   ` Ma, Maurice
2016-10-19  7:01 ` [PATCH 04/33] CorebootModulePkg: " Gary Lin
2016-10-26 18:41   ` Ma, Maurice
2016-10-19  7:01 ` [PATCH 05/33] CryptoPkg: " Gary Lin
2016-10-26  3:00   ` Long, Qin
2016-10-19  7:01 ` [PATCH 06/33] DuetPkg: " Gary Lin
2016-10-19  7:46   ` Ni, Ruiyu
2016-10-19  7:01 ` [PATCH 07/33] EdkCompatibilityPkg: " Gary Lin
2016-10-19  8:19   ` Gao, Liming
2016-10-19  7:01 ` [PATCH 08/33] EmulatorPkg: Fix typos in comments and variables Gary Lin
2016-10-19 20:37   ` Jordan Justen
2016-10-20  8:16     ` Laszlo Ersek
2016-10-19  7:01 ` [PATCH 09/33] EmbeddedPkg: Fix typos in comments Gary Lin
2016-10-19 15:09   ` Ard Biesheuvel
2016-10-19  7:01 ` [PATCH 10/33] IntelFrameworkModulePkg: " Gary Lin
2016-10-20  1:08   ` Fan, Jeff
2016-10-19  7:01 ` [PATCH 11/33] IntelFrameworkPkg: " Gary Lin
2016-10-20  1:08   ` Fan, Jeff
2016-10-19  7:01 ` [PATCH 12/33] IntelFspPkg: " Gary Lin
2016-10-19  7:06   ` Yao, Jiewen
2016-10-19  7:01 ` [PATCH 13/33] IntelFspWrapperPkg: " Gary Lin
2016-10-19  7:08   ` Yao, Jiewen
2016-10-19  7:01 ` [PATCH 14/33] IntelFsp2Pkg: " Gary Lin
2016-10-19  7:06   ` Yao, Jiewen
2016-10-19  7:12   ` Mudusuru, Giri P
2016-10-19  7:01 ` [PATCH 15/33] IntelFsp2WrapperPkg: Fix a typo in the comment Gary Lin
2016-10-19  7:07   ` Yao, Jiewen
2016-10-19  7:11   ` Mudusuru, Giri P
2016-10-19  7:01 ` [PATCH 16/33] MdeModulePkg: BootManagerMenuApp: Fix typos in comments Gary Lin
2016-10-19  8:20   ` Tian, Feng
2016-10-19  7:01 ` [PATCH 17/33] MdeModulePkg/Bus: " Gary Lin
2016-10-19  7:01 ` [PATCH 18/33] MdeModulePkg/Core: " Gary Lin
2016-10-19  8:20   ` Tian, Feng
2016-10-19  7:01 ` [PATCH 19/33] MdeModulePkg/Include: " Gary Lin
2016-10-19  7:01 ` [PATCH 20/33] MdeModulePkg/Library: Fix typos in comments and variables Gary Lin
2016-10-19  7:01 ` [PATCH 21/33] MdeModulePkg/Universal: Fix typos in comments Gary Lin
2016-10-19  7:01 ` [PATCH 22/33] MdePkg: " Gary Lin
2016-10-19  8:21   ` Gao, Liming
2016-10-19  7:01 ` [PATCH 23/33] NetworkPkg: " Gary Lin
2016-10-19  7:21   ` Fu, Siyuan
2016-10-26  0:53   ` Fu, Siyuan
2016-10-19  7:01 ` [PATCH 24/33] Nt32Pkg: Fix typos in comments and variables Gary Lin
2016-10-19  7:50   ` Ni, Ruiyu
2016-10-19  7:01 ` [PATCH 25/33] OptionRomPkg: Fix typos in comments Gary Lin
2016-10-19  7:51   ` Ni, Ruiyu
2016-10-19  7:01 ` [PATCH 26/33] OvmfPkg: " Gary Lin
2016-10-19  7:01 ` [PATCH 27/33] QuarkPlatformPkg: " Gary Lin
2016-10-25 20:53   ` Kinney, Michael D
2016-10-19  7:01 ` [PATCH 28/33] QuarkSocPkg: " Gary Lin
2016-10-25 21:00   ` Kinney, Michael D
2016-10-19  7:01 ` [PATCH 29/33] SecurityPkg: " Gary Lin
2016-10-27  1:29   ` Zhang, Chao B
2016-11-14  3:12   ` Zhang, Chao B
2016-11-14  4:03     ` Gary Lin
2016-11-14  6:24       ` Gao, Liming
2016-11-14  8:30         ` Gary Lin
2016-10-19  7:01 ` [PATCH 30/33] ShellPkg: Fix typos in comments and variables Gary Lin
2016-10-19  7:51   ` Ni, Ruiyu
2016-10-19 20:29     ` Carsey, Jaben
2016-10-20  1:55       ` Gary Lin
2016-10-19  7:01 ` [PATCH 31/33] SourceLevelDebugPkg: Fix typos in comments Gary Lin
2016-10-19  7:46   ` Wu, Hao A
2016-10-19  7:01 ` [PATCH 32/33] UefiCpuPkg: " Gary Lin
2016-10-20  1:58   ` Fan, Jeff
2016-10-20  2:08     ` Gary Lin [this message]
2016-10-20  2:45       ` Fan, Jeff
2016-10-19  7:01 ` [PATCH 33/33] Vlv2TbltDevicePkg: " Gary Lin
2016-10-24  1:33   ` Wei, David
2016-10-19  7:28 ` [PATCH 00/33] Fix typos in comments and variables Zeng, Star
2016-10-19 14:50 ` Leif Lindholm

Reply instructions:

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

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

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

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

  git send-email \
    --in-reply-to=20161020020846.xwaap72vfxk5hwnb@GaryWorkstation \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

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

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