From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp.nue.novell.com (smtp.nue.novell.com [195.135.221.5]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id E42061A1E6B for ; Wed, 19 Oct 2016 19:08:58 -0700 (PDT) Received: from nwb-ext-pat.microfocus.com ([10.120.13.103]) by smtp.nue.novell.com with ESMTP (TLS encrypted); Thu, 20 Oct 2016 04:08:56 +0200 Received: from GaryWorkstation (nwb-a10-snat.microfocus.com [10.120.13.202]) by nwb-ext-pat.microfocus.com with ESMTP (TLS encrypted); Thu, 20 Oct 2016 03:08:52 +0100 Date: Thu, 20 Oct 2016 10:08:46 +0800 From: Gary Lin To: "Fan, Jeff" Cc: "edk2-devel@lists.01.org" Message-ID: <20161020020846.xwaap72vfxk5hwnb@GaryWorkstation> References: <20161019070138.16424-1-glin@suse.com> <20161019070138.16424-33-glin@suse.com> <542CF652F8836A4AB8DBFAAD40ED192A4A2B0113@shsmsx102.ccr.corp.intel.com> MIME-Version: 1.0 In-Reply-To: <542CF652F8836A4AB8DBFAAD40ED192A4A2B0113@shsmsx102.ccr.corp.intel.com> User-Agent: Mutt/1.6.2 (2016-07-01) Subject: Re: [PATCH 32/33] UefiCpuPkg: Fix typos in comments X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 20 Oct 2016 02:08:59 -0000 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Thu, Oct 20, 2016 at 01:58:51AM +0000, Fan, Jeff wrote: > Reviewed-by: Jeff Fan > > 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 > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Gary Lin > --- > 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.
@@ -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 >