public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [PATCH] MdePkg: Fix some typing errors in the header files
@ 2016-09-12  8:35 Thomas Huth
  2016-09-12  9:09 ` Gao, Liming
  0 siblings, 1 reply; 4+ messages in thread
From: Thomas Huth @ 2016-09-12  8:35 UTC (permalink / raw)
  To: edk2-devel; +Cc: Michael D Kinney, Liming Gao

Correct the typos in some header files of MdePkg.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 MdePkg/Include/AArch64/ProcessorBind.h                            | 2 +-
 MdePkg/Include/Arm/ProcessorBind.h                                | 2 +-
 MdePkg/Include/Base.h                                             | 2 +-
 MdePkg/Include/Ia32/ProcessorBind.h                               | 4 ++--
 MdePkg/Include/Ipf/ProcessorBind.h                                | 2 +-
 MdePkg/Include/Library/BaseLib.h                                  | 2 +-
 MdePkg/Include/Library/DebugPrintErrorLevelLib.h                  | 2 +-
 MdePkg/Include/Library/FileHandleLib.h                            | 8 ++++----
 MdePkg/Include/Library/PeCoffLib.h                                | 2 +-
 MdePkg/Include/Pi/PiSmmCis.h                                      | 2 +-
 MdePkg/Include/Protocol/BlockIo2.h                                | 2 +-
 MdePkg/Include/Protocol/BlockIoCrypto.h                           | 2 +-
 MdePkg/Include/Protocol/Dns4.h                                    | 2 +-
 MdePkg/Include/Protocol/Dns6.h                                    | 2 +-
 MdePkg/Include/Protocol/FirmwareManagement.h                      | 2 +-
 MdePkg/Include/Protocol/ImageDecoder.h                            | 2 +-
 MdePkg/Include/Protocol/Mtftp4.h                                  | 4 ++--
 MdePkg/Include/Protocol/Mtftp6.h                                  | 8 ++++----
 MdePkg/Include/Protocol/NvmExpressPassthru.h                      | 2 +-
 MdePkg/Include/Protocol/SimpleTextOut.h                           | 2 +-
 MdePkg/Include/Protocol/Timer.h                                   | 2 +-
 MdePkg/Include/Uefi/UefiPxe.h                                     | 2 +-
 MdePkg/Include/X64/ProcessorBind.h                                | 4 ++--
 MdePkg/Library/BaseMemoryLibMmx/MemLibInternals.h                 | 2 +-
 .../BaseUefiDecompressLib/BaseUefiDecompressLibInternals.h        | 2 +-
 MdePkg/Library/DxeIoLibCpuIo2/DxeCpuIo2LibInternal.h              | 2 +-
 MdePkg/Library/SmmIoLibSmmCpuIo2/SmmCpuIoLibInternal.h            | 2 +-
 27 files changed, 36 insertions(+), 36 deletions(-)

diff --git a/MdePkg/Include/AArch64/ProcessorBind.h b/MdePkg/Include/AArch64/ProcessorBind.h
index c5fe874..fa311e2 100644
--- a/MdePkg/Include/AArch64/ProcessorBind.h
+++ b/MdePkg/Include/AArch64/ProcessorBind.h
@@ -32,7 +32,7 @@
 
 #if _MSC_EXTENSIONS
   //
-  // use Microsoft* C complier dependent integer width types 
+  // use Microsoft* C compiler dependent integer width types 
   //
   typedef unsigned __int64    UINT64;
   typedef __int64             INT64;
diff --git a/MdePkg/Include/Arm/ProcessorBind.h b/MdePkg/Include/Arm/ProcessorBind.h
index 5ee7465..dc45b94 100644
--- a/MdePkg/Include/Arm/ProcessorBind.h
+++ b/MdePkg/Include/Arm/ProcessorBind.h
@@ -37,7 +37,7 @@
 
 #if _MSC_EXTENSIONS 
   //
-  // use Microsoft* C complier dependent integer width types 
+  // use Microsoft* C compiler dependent integer width types 
   //
   typedef unsigned __int64    UINT64;
   typedef __int64             INT64;
diff --git a/MdePkg/Include/Base.h b/MdePkg/Include/Base.h
index 93f6686..c666148 100644
--- a/MdePkg/Include/Base.h
+++ b/MdePkg/Include/Base.h
@@ -302,7 +302,7 @@ struct _LIST_ENTRY {
 
 //
 //  UEFI specification claims 1 and 0. We are concerned about the
-//  complier portability so we did it this way.
+//  compiler portability so we did it this way.
 //
 
 ///
diff --git a/MdePkg/Include/Ia32/ProcessorBind.h b/MdePkg/Include/Ia32/ProcessorBind.h
index 458f7dd..da1810b 100644
--- a/MdePkg/Include/Ia32/ProcessorBind.h
+++ b/MdePkg/Include/Ia32/ProcessorBind.h
@@ -79,7 +79,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #pragma warning ( disable : 4057 )
 
 //
-// ASSERT(FALSE) or while (TRUE) are legal constructes so supress this warning
+// ASSERT(FALSE) or while (TRUE) are legal constructs so suppress this warning
 //
 #pragma warning ( disable : 4127 )
 
@@ -119,7 +119,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #if defined(_MSC_EXTENSIONS)
 
   //
-  // use Microsoft C complier dependent integer width types 
+  // use Microsoft C compiler dependent integer width types 
   //
 
   ///
diff --git a/MdePkg/Include/Ipf/ProcessorBind.h b/MdePkg/Include/Ipf/ProcessorBind.h
index cead575..c19e47d 100644
--- a/MdePkg/Include/Ipf/ProcessorBind.h
+++ b/MdePkg/Include/Ipf/ProcessorBind.h
@@ -84,7 +84,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #pragma warning ( disable : 4054 )
 
 //
-// ASSERT(FALSE) or while (TRUE) are legal constructes so supress this warning
+// ASSERT(FALSE) or while (TRUE) are legal constructs so suppress this warning
 //
 #pragma warning ( disable : 4127 )
 
diff --git a/MdePkg/Include/Library/BaseLib.h b/MdePkg/Include/Library/BaseLib.h
index 7e085b9..6268e6f 100644
--- a/MdePkg/Include/Library/BaseLib.h
+++ b/MdePkg/Include/Library/BaseLib.h
@@ -1772,7 +1772,7 @@ PathRemoveLastItem(
 
   @param[in] Path       The pointer to the string containing the path.
 
-  @return       Returns Path, otherwise returns NULL to indicate that an error has occured.
+  @return       Returns Path, otherwise returns NULL to indicate that an error has occurred.
 **/
 CHAR16*
 EFIAPI
diff --git a/MdePkg/Include/Library/DebugPrintErrorLevelLib.h b/MdePkg/Include/Library/DebugPrintErrorLevelLib.h
index 2b051f4..5a866d5 100644
--- a/MdePkg/Include/Library/DebugPrintErrorLevelLib.h
+++ b/MdePkg/Include/Library/DebugPrintErrorLevelLib.h
@@ -31,7 +31,7 @@ GetDebugPrintErrorLevel (
   
   @param   ErrorLevel     Global debug print error level
   
-  @retval  TRUE           The debug print error level mask was sucessfully set.
+  @retval  TRUE           The debug print error level mask was successfully set.
   @retval  FALSE          The debug print error level mask could not be set.
 
 **/
diff --git a/MdePkg/Include/Library/FileHandleLib.h b/MdePkg/Include/Library/FileHandleLib.h
index 00706f9..e94f83e 100644
--- a/MdePkg/Include/Library/FileHandleLib.h
+++ b/MdePkg/Include/Library/FileHandleLib.h
@@ -184,9 +184,9 @@ FileHandleDelete (
   has the effect of starting the read process of the directory entries over again.
 
   @param[in] FileHandle         The file handle on which the position is being set.
-  @param[in] Position           The byte position from the begining of the file.
+  @param[in] Position           The byte position from the beginning of the file.
 
-  @retval EFI_SUCCESS           The operation completed sucessfully.
+  @retval EFI_SUCCESS           The operation completed successfully.
   @retval EFI_UNSUPPORTED       The request for non-zero is not valid on
                                 directories.
   @retval INVALID_PARAMETER     One of the parameters has an invalid value.
@@ -207,7 +207,7 @@ FileHandleSetPosition (
   if FileHandle is a directory.
 
   @param[in] FileHandle         The open file handle on which to get the position.
-  @param[out] Position          The byte position from begining of file.
+  @param[out] Position          The byte position from beginning of file.
 
   @retval EFI_SUCCESS           The operation completed successfully.
   @retval INVALID_PARAMETER     One of the parameters has an invalid value.
@@ -322,7 +322,7 @@ FileHandleFindNextFile(
   @param[in] FileHandle         The file handle from which size is retrieved.
   @param[out] Size              The pointer to size.
 
-  @retval EFI_SUCCESS           Operation was completed sucessfully.
+  @retval EFI_SUCCESS           Operation was completed successfully.
   @retval EFI_DEVICE_ERROR      Cannot access the file.
   @retval EFI_INVALID_PARAMETER FileHandle is NULL.
                                 Size is NULL.
diff --git a/MdePkg/Include/Library/PeCoffLib.h b/MdePkg/Include/Library/PeCoffLib.h
index 9ed6d61..5a3f988 100644
--- a/MdePkg/Include/Library/PeCoffLib.h
+++ b/MdePkg/Include/Library/PeCoffLib.h
@@ -337,7 +337,7 @@ PeCoffLoaderImageReadFromMemory (
 
 
 /**
-  Reapply fixups on a fixed up PE32/PE32+ image to allow virutal calling at EFI
+  Reapply fixups on a fixed up PE32/PE32+ image to allow virtual calling at EFI
   runtime. 
   
   This function reapplies relocation fixups to the PE/COFF image specified by ImageBase 
diff --git a/MdePkg/Include/Pi/PiSmmCis.h b/MdePkg/Include/Pi/PiSmmCis.h
index 6d3ddab..6cb28b7 100644
--- a/MdePkg/Include/Pi/PiSmmCis.h
+++ b/MdePkg/Include/Pi/PiSmmCis.h
@@ -178,7 +178,7 @@ EFI_STATUS
 /**
   Registers a handler to execute within SMM.
 
-  @param[in]  Handler            Handler service funtion pointer.
+  @param[in]  Handler            Handler service function pointer.
   @param[in]  HandlerType        Points to the handler type or NULL for root SMI handlers.
   @param[out] DispatchHandle     On return, contains a unique handle which can be used to later
                                  unregister the handler function.
diff --git a/MdePkg/Include/Protocol/BlockIo2.h b/MdePkg/Include/Protocol/BlockIo2.h
index 6f0c280..5fad107 100644
--- a/MdePkg/Include/Protocol/BlockIo2.h
+++ b/MdePkg/Include/Protocol/BlockIo2.h
@@ -52,7 +52,7 @@ typedef struct {
 
   @param[in]  This                 Indicates a pointer to the calling context.
   @param[in]  ExtendedVerification Indicates that the driver may perform a more
-                                   exhausive verfication operation of the device
+                                   exhausive verification operation of the device
                                    during reset.
 
   @retval EFI_SUCCESS          The device was reset.
diff --git a/MdePkg/Include/Protocol/BlockIoCrypto.h b/MdePkg/Include/Protocol/BlockIoCrypto.h
index 77931e7..821b846 100644
--- a/MdePkg/Include/Protocol/BlockIoCrypto.h
+++ b/MdePkg/Include/Protocol/BlockIoCrypto.h
@@ -172,7 +172,7 @@ typedef struct {
 
   @param[in]  This                 Pointer to the EFI_BLOCK_IO_CRYPTO_PROTOCOL instance.
   @param[in]  ExtendedVerification Indicates that the driver may perform a more exhausive
-                                   verfication operation of the device during reset.
+                                   verification operation of the device during reset.
 
   @retval EFI_SUCCESS              The block device was reset.
   @retval EFI_DEVICE_ERROR         The block device is not functioning correctly and could
diff --git a/MdePkg/Include/Protocol/Dns4.h b/MdePkg/Include/Protocol/Dns4.h
index 2489268..f96c50f 100644
--- a/MdePkg/Include/Protocol/Dns4.h
+++ b/MdePkg/Include/Protocol/Dns4.h
@@ -310,7 +310,7 @@ EFI_STATUS
 
   @retval EFI_SUCCESS             The operation completed successfully.
   @retval EFI_UNSUPPORTED         The designated protocol is not supported.
-  @retval EFI_INVALID_PARAMTER    Thisis NULL.
+  @retval EFI_INVALID_PARAMETER   This is NULL.
                                   The StationIp address provided in DnsConfigData is not a 
                                   valid unicast.
                                   DnsServerList is NULL while DnsServerListCount
diff --git a/MdePkg/Include/Protocol/Dns6.h b/MdePkg/Include/Protocol/Dns6.h
index 1ea85bc..813893a 100644
--- a/MdePkg/Include/Protocol/Dns6.h
+++ b/MdePkg/Include/Protocol/Dns6.h
@@ -300,7 +300,7 @@ EFI_STATUS
                                   storage to be allocated and released by caller.
 
   @retval EFI_SUCCESS             The operation completed successfully.
-  @retval EFI_INVALID_PARAMTER    This is NULL.
+  @retval EFI_INVALID_PARAMETER   This is NULL.
                                   The StationIp address provided in DnsConfigData is not zero and not a valid unicast.
                                   DnsServerList is NULL while DnsServerList Count is not ZERO.
                                   DnsServerList Count is ZERO while DnsServerList is not NULL.
diff --git a/MdePkg/Include/Protocol/FirmwareManagement.h b/MdePkg/Include/Protocol/FirmwareManagement.h
index e615573..b199888 100644
--- a/MdePkg/Include/Protocol/FirmwareManagement.h
+++ b/MdePkg/Include/Protocol/FirmwareManagement.h
@@ -241,7 +241,7 @@ typedef struct {
 #define  PACKAGE_ATTRIBUTE_AUTHENTICATION_REQUIRED 0x0000000000000004
 
 /**
-  Callback funtion to report the process of the firmware updating.
+  Callback function to report the process of the firmware updating.
 
   @param[in]  Completion    A value between 1 and 100 indicating the current completion
                             progress of the firmware update. Completion progress is
diff --git a/MdePkg/Include/Protocol/ImageDecoder.h b/MdePkg/Include/Protocol/ImageDecoder.h
index aebb813..3a7845b 100644
--- a/MdePkg/Include/Protocol/ImageDecoder.h
+++ b/MdePkg/Include/Protocol/ImageDecoder.h
@@ -130,7 +130,7 @@ EFI_STATUS
   @param This                    EFI_HII_IMAGE_DECODER_PROTOCOL instance.
   @param Image                   Pointer to the image raw data.
   @param SizeOfImage             Size of the entire image raw data.
-  @param ImageInfo               Pointer to recieve EFI_HII_IMAGE_DECODER_IMAGE_INFO_HEADER.
+  @param ImageInfo               Pointer to receive EFI_HII_IMAGE_DECODER_IMAGE_INFO_HEADER.
 
   @retval EFI_SUCCESS            Get image info success.
   @retval EFI_UNSUPPORTED        Unsupported format of image.
diff --git a/MdePkg/Include/Protocol/Mtftp4.h b/MdePkg/Include/Protocol/Mtftp4.h
index 1afac6c..ae2dc27 100644
--- a/MdePkg/Include/Protocol/Mtftp4.h
+++ b/MdePkg/Include/Protocol/Mtftp4.h
@@ -1,5 +1,5 @@
 /** @file
-  EFI Multicast Trivial File Tranfer Protocol Definition
+  EFI Multicast Trivial File Transfer Protocol Definition
 
 Copyright (c) 2006 - 2011, Intel Corporation. All rights reserved.<BR>
 This program and the accompanying materials are licensed and made available under 
@@ -212,7 +212,7 @@ EFI_STATUS
   );
 
 /**
-  Timeout callback funtion.
+  Timeout callback function.
 
   @param  This           The pointer to the EFI_MTFTP4_PROTOCOL instance.
   @param  Token          The token that is provided in the
diff --git a/MdePkg/Include/Protocol/Mtftp6.h b/MdePkg/Include/Protocol/Mtftp6.h
index b39575d..4667add 100644
--- a/MdePkg/Include/Protocol/Mtftp6.h
+++ b/MdePkg/Include/Protocol/Mtftp6.h
@@ -1,5 +1,5 @@
 /** @file
-  UEFI Multicast Trivial File Tranfer Protocol v6 Definition, which is built upon
+  UEFI Multicast Trivial File Transfer Protocol v6 Definition, which is built upon
   the EFI UDPv6 Protocol and provides basic services for client-side unicast and/or
   multicast TFTP operations.
 
@@ -325,7 +325,7 @@ typedef struct {
   @param[in] PacketLen     Indicates the length of the packet.
   @param[in] Packet        Pointer to an MTFTPv6 packet.
 
-  @retval EFI_SUCCESS      Operation sucess.
+  @retval EFI_SUCCESS      Operation success.
   @retval Others           Aborts session.
 
 **/
@@ -352,7 +352,7 @@ EFI_STATUS
                            EFI_MTFTP6_PROTOCOL.WriteFile() or EFI_MTFTP6_PROTOCOL.ReadDirectory()
                            functions by the caller.
 
-  @retval EFI_SUCCESS      Operation sucess.
+  @retval EFI_SUCCESS      Operation success.
   @retval Others           Aborts session.
 
 **/
@@ -379,7 +379,7 @@ EFI_STATUS
                            length the data available on output.
   @param[out]     Buffer   Pointer to the buffer where the data is stored.
 
-  @retval EFI_SUCCESS      Operation sucess.
+  @retval EFI_SUCCESS      Operation success.
   @retval Others           Aborts session.
 
 **/
diff --git a/MdePkg/Include/Protocol/NvmExpressPassthru.h b/MdePkg/Include/Protocol/NvmExpressPassthru.h
index ebd7a8b..7a90c63 100644
--- a/MdePkg/Include/Protocol/NvmExpressPassthru.h
+++ b/MdePkg/Include/Protocol/NvmExpressPassthru.h
@@ -115,7 +115,7 @@ typedef struct {
 } EFI_NVM_EXPRESS_PASS_THRU_COMMAND_PACKET;
 
 //
-// Protocol funtion prototypes
+// Protocol function prototypes
 //
 /**
   Sends an NVM Express Command Packet to an NVM Express controller or namespace. This function supports
diff --git a/MdePkg/Include/Protocol/SimpleTextOut.h b/MdePkg/Include/Protocol/SimpleTextOut.h
index 5f64b64..c1a23fa 100644
--- a/MdePkg/Include/Protocol/SimpleTextOut.h
+++ b/MdePkg/Include/Protocol/SimpleTextOut.h
@@ -160,7 +160,7 @@ typedef EFI_SIMPLE_TEXT_OUTPUT_PROTOCOL   SIMPLE_TEXT_OUTPUT_INTERFACE;
   Reset the text output device hardware and optionaly run diagnostics
 
   @param  This                 The protocol instance pointer.
-  @param  ExtendedVerification Driver may perform more exhaustive verfication
+  @param  ExtendedVerification Driver may perform more exhaustive verification
                                operation of the device during reset.
 
   @retval EFI_SUCCESS          The text output device was reset.
diff --git a/MdePkg/Include/Protocol/Timer.h b/MdePkg/Include/Protocol/Timer.h
index c9a998e..ffdb0a1 100644
--- a/MdePkg/Include/Protocol/Timer.h
+++ b/MdePkg/Include/Protocol/Timer.h
@@ -30,7 +30,7 @@ typedef struct _EFI_TIMER_ARCH_PROTOCOL   EFI_TIMER_ARCH_PROTOCOL;
 
 /**
   This function of this type is called when a timer interrupt fires.  This 
-  function executes at TPL_HIGH_LEVEL.  The DXE Core will register a funtion
+  function executes at TPL_HIGH_LEVEL.  The DXE Core will register a function
   of this type to be called for the timer interrupt, so it can know how much 
   time has passed.  This information is used to signal timer based events.  
 
diff --git a/MdePkg/Include/Uefi/UefiPxe.h b/MdePkg/Include/Uefi/UefiPxe.h
index 0af9b90..249a8c2 100644
--- a/MdePkg/Include/Uefi/UefiPxe.h
+++ b/MdePkg/Include/Uefi/UefiPxe.h
@@ -1079,7 +1079,7 @@ typedef struct s_pxe_cpb_start_31 {
 
   ///
   /// protocol driver can provide anything for this Unique_ID, UNDI remembers
-  /// that as just a 64bit value assocaited to the interface specified by
+  /// that as just a 64bit value associated to the interface specified by
   /// the ifnum and gives it back as a parameter to all the call-back routines
   /// when calling for that interface!
   ///
diff --git a/MdePkg/Include/X64/ProcessorBind.h b/MdePkg/Include/X64/ProcessorBind.h
index ffcf2b0..fcbe28a 100644
--- a/MdePkg/Include/X64/ProcessorBind.h
+++ b/MdePkg/Include/X64/ProcessorBind.h
@@ -93,7 +93,7 @@
 #pragma warning ( disable : 4057 )
 
 //
-// ASSERT(FALSE) or while (TRUE) are legal constructes so supress this warning
+// ASSERT(FALSE) or while (TRUE) are legal constructs so suppress this warning
 //
 #pragma warning ( disable : 4127 )
 
@@ -132,7 +132,7 @@
 
 #if defined(_MSC_EXTENSIONS)
   //
-  // use Microsoft C complier dependent integer width types 
+  // use Microsoft C compiler dependent integer width types 
   //
 
   ///
diff --git a/MdePkg/Library/BaseMemoryLibMmx/MemLibInternals.h b/MdePkg/Library/BaseMemoryLibMmx/MemLibInternals.h
index cc89979..9505265 100644
--- a/MdePkg/Library/BaseMemoryLibMmx/MemLibInternals.h
+++ b/MdePkg/Library/BaseMemoryLibMmx/MemLibInternals.h
@@ -103,7 +103,7 @@ InternalMemSetMem32 (
 /**
   Fills a target buffer with a 64-bit value, and returns the target buffer.
 
-  @param  Buffer  Teh pointer to the target buffer to fill.
+  @param  Buffer  The pointer to the target buffer to fill.
   @param  Length  The count of 64-bit value to fill.
   @param  Value   The value with which to fill Length bytes of Buffer.
 
diff --git a/MdePkg/Library/BaseUefiDecompressLib/BaseUefiDecompressLibInternals.h b/MdePkg/Library/BaseUefiDecompressLib/BaseUefiDecompressLibInternals.h
index 358c598..aba510c 100644
--- a/MdePkg/Library/BaseUefiDecompressLib/BaseUefiDecompressLibInternals.h
+++ b/MdePkg/Library/BaseUefiDecompressLib/BaseUefiDecompressLibInternals.h
@@ -1,5 +1,5 @@
 /** @file
-  Internal data structure defintions for Base UEFI Decompress Libary.
+  Internal data structure defintions for Base UEFI Decompress Library.
 
   Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.<BR>
   This program and the accompanying materials
diff --git a/MdePkg/Library/DxeIoLibCpuIo2/DxeCpuIo2LibInternal.h b/MdePkg/Library/DxeIoLibCpuIo2/DxeCpuIo2LibInternal.h
index b459214..160bedf 100644
--- a/MdePkg/Library/DxeIoLibCpuIo2/DxeCpuIo2LibInternal.h
+++ b/MdePkg/Library/DxeIoLibCpuIo2/DxeCpuIo2LibInternal.h
@@ -58,7 +58,7 @@ IoReadWorker (
   @param  Width         The width of the I/O operation.
   @param  Data          The value to write to the I/O port.
 
-  @return The paramter of Data.
+  @return The parameter of Data.
 
 **/
 UINT64
diff --git a/MdePkg/Library/SmmIoLibSmmCpuIo2/SmmCpuIoLibInternal.h b/MdePkg/Library/SmmIoLibSmmCpuIo2/SmmCpuIoLibInternal.h
index 0e94b20..ced6358 100644
--- a/MdePkg/Library/SmmIoLibSmmCpuIo2/SmmCpuIoLibInternal.h
+++ b/MdePkg/Library/SmmIoLibSmmCpuIo2/SmmCpuIoLibInternal.h
@@ -60,7 +60,7 @@ IoReadWorker (
   @param  Width         The width of the I/O operation.
   @param  Data          The value to write to the I/O port.
 
-  @return The paramter of Data.
+  @return The parameter of Data.
 
 **/
 UINT64
-- 
1.8.3.1



^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] MdePkg: Fix some typing errors in the header files
  2016-09-12  8:35 [PATCH] MdePkg: Fix some typing errors in the header files Thomas Huth
@ 2016-09-12  9:09 ` Gao, Liming
  2016-10-08  1:16   ` Kinney, Michael D
       [not found]   ` <60e78d8e-c7dc-ee87-df2c-69ddfd4ef202@redhat.com>
  0 siblings, 2 replies; 4+ messages in thread
From: Gao, Liming @ 2016-09-12  9:09 UTC (permalink / raw)
  To: Thomas Huth, edk2-devel@ml01.01.org; +Cc: Kinney, Michael D

Reviewed-by: Liming Gao <liming.gao@intel.com>

> -----Original Message-----
> From: Thomas Huth [mailto:thuth@redhat.com]
> Sent: Monday, September 12, 2016 4:36 PM
> To: edk2-devel@ml01.01.org
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
> <liming.gao@intel.com>
> Subject: [PATCH] MdePkg: Fix some typing errors in the header files
> 
> Correct the typos in some header files of MdePkg.
> 
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  MdePkg/Include/AArch64/ProcessorBind.h                            | 2 +-
>  MdePkg/Include/Arm/ProcessorBind.h                                | 2 +-
>  MdePkg/Include/Base.h                                             | 2 +-
>  MdePkg/Include/Ia32/ProcessorBind.h                               | 4 ++--
>  MdePkg/Include/Ipf/ProcessorBind.h                                | 2 +-
>  MdePkg/Include/Library/BaseLib.h                                  | 2 +-
>  MdePkg/Include/Library/DebugPrintErrorLevelLib.h                  | 2 +-
>  MdePkg/Include/Library/FileHandleLib.h                            | 8 ++++----
>  MdePkg/Include/Library/PeCoffLib.h                                | 2 +-
>  MdePkg/Include/Pi/PiSmmCis.h                                      | 2 +-
>  MdePkg/Include/Protocol/BlockIo2.h                                | 2 +-
>  MdePkg/Include/Protocol/BlockIoCrypto.h                           | 2 +-
>  MdePkg/Include/Protocol/Dns4.h                                    | 2 +-
>  MdePkg/Include/Protocol/Dns6.h                                    | 2 +-
>  MdePkg/Include/Protocol/FirmwareManagement.h                      | 2 +-
>  MdePkg/Include/Protocol/ImageDecoder.h                            | 2 +-
>  MdePkg/Include/Protocol/Mtftp4.h                                  | 4 ++--
>  MdePkg/Include/Protocol/Mtftp6.h                                  | 8 ++++----
>  MdePkg/Include/Protocol/NvmExpressPassthru.h                      | 2 +-
>  MdePkg/Include/Protocol/SimpleTextOut.h                           | 2 +-
>  MdePkg/Include/Protocol/Timer.h                                   | 2 +-
>  MdePkg/Include/Uefi/UefiPxe.h                                     | 2 +-
>  MdePkg/Include/X64/ProcessorBind.h                                | 4 ++--
>  MdePkg/Library/BaseMemoryLibMmx/MemLibInternals.h                 | 2 +-
>  .../BaseUefiDecompressLib/BaseUefiDecompressLibInternals.h        | 2 +-
>  MdePkg/Library/DxeIoLibCpuIo2/DxeCpuIo2LibInternal.h              | 2 +-
>  MdePkg/Library/SmmIoLibSmmCpuIo2/SmmCpuIoLibInternal.h            | 2 +-
>  27 files changed, 36 insertions(+), 36 deletions(-)
> 
> diff --git a/MdePkg/Include/AArch64/ProcessorBind.h
> b/MdePkg/Include/AArch64/ProcessorBind.h
> index c5fe874..fa311e2 100644
> --- a/MdePkg/Include/AArch64/ProcessorBind.h
> +++ b/MdePkg/Include/AArch64/ProcessorBind.h
> @@ -32,7 +32,7 @@
> 
>  #if _MSC_EXTENSIONS
>    //
> -  // use Microsoft* C complier dependent integer width types
> +  // use Microsoft* C compiler dependent integer width types
>    //
>    typedef unsigned __int64    UINT64;
>    typedef __int64             INT64;
> diff --git a/MdePkg/Include/Arm/ProcessorBind.h
> b/MdePkg/Include/Arm/ProcessorBind.h
> index 5ee7465..dc45b94 100644
> --- a/MdePkg/Include/Arm/ProcessorBind.h
> +++ b/MdePkg/Include/Arm/ProcessorBind.h
> @@ -37,7 +37,7 @@
> 
>  #if _MSC_EXTENSIONS
>    //
> -  // use Microsoft* C complier dependent integer width types
> +  // use Microsoft* C compiler dependent integer width types
>    //
>    typedef unsigned __int64    UINT64;
>    typedef __int64             INT64;
> diff --git a/MdePkg/Include/Base.h b/MdePkg/Include/Base.h
> index 93f6686..c666148 100644
> --- a/MdePkg/Include/Base.h
> +++ b/MdePkg/Include/Base.h
> @@ -302,7 +302,7 @@ struct _LIST_ENTRY {
> 
>  //
>  //  UEFI specification claims 1 and 0. We are concerned about the
> -//  complier portability so we did it this way.
> +//  compiler portability so we did it this way.
>  //
> 
>  ///
> diff --git a/MdePkg/Include/Ia32/ProcessorBind.h
> b/MdePkg/Include/Ia32/ProcessorBind.h
> index 458f7dd..da1810b 100644
> --- a/MdePkg/Include/Ia32/ProcessorBind.h
> +++ b/MdePkg/Include/Ia32/ProcessorBind.h
> @@ -79,7 +79,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY
> KIND, EITHER EXPRESS OR IMPLIED.
>  #pragma warning ( disable : 4057 )
> 
>  //
> -// ASSERT(FALSE) or while (TRUE) are legal constructes so supress this
> warning
> +// ASSERT(FALSE) or while (TRUE) are legal constructs so suppress this
> warning
>  //
>  #pragma warning ( disable : 4127 )
> 
> @@ -119,7 +119,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF
> ANY KIND, EITHER EXPRESS OR IMPLIED.
>  #if defined(_MSC_EXTENSIONS)
> 
>    //
> -  // use Microsoft C complier dependent integer width types
> +  // use Microsoft C compiler dependent integer width types
>    //
> 
>    ///
> diff --git a/MdePkg/Include/Ipf/ProcessorBind.h
> b/MdePkg/Include/Ipf/ProcessorBind.h
> index cead575..c19e47d 100644
> --- a/MdePkg/Include/Ipf/ProcessorBind.h
> +++ b/MdePkg/Include/Ipf/ProcessorBind.h
> @@ -84,7 +84,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY
> KIND, EITHER EXPRESS OR IMPLIED.
>  #pragma warning ( disable : 4054 )
> 
>  //
> -// ASSERT(FALSE) or while (TRUE) are legal constructes so supress this
> warning
> +// ASSERT(FALSE) or while (TRUE) are legal constructs so suppress this
> warning
>  //
>  #pragma warning ( disable : 4127 )
> 
> diff --git a/MdePkg/Include/Library/BaseLib.h
> b/MdePkg/Include/Library/BaseLib.h
> index 7e085b9..6268e6f 100644
> --- a/MdePkg/Include/Library/BaseLib.h
> +++ b/MdePkg/Include/Library/BaseLib.h
> @@ -1772,7 +1772,7 @@ PathRemoveLastItem(
> 
>    @param[in] Path       The pointer to the string containing the path.
> 
> -  @return       Returns Path, otherwise returns NULL to indicate that an error
> has occured.
> +  @return       Returns Path, otherwise returns NULL to indicate that an error
> has occurred.
>  **/
>  CHAR16*
>  EFIAPI
> diff --git a/MdePkg/Include/Library/DebugPrintErrorLevelLib.h
> b/MdePkg/Include/Library/DebugPrintErrorLevelLib.h
> index 2b051f4..5a866d5 100644
> --- a/MdePkg/Include/Library/DebugPrintErrorLevelLib.h
> +++ b/MdePkg/Include/Library/DebugPrintErrorLevelLib.h
> @@ -31,7 +31,7 @@ GetDebugPrintErrorLevel (
> 
>    @param   ErrorLevel     Global debug print error level
> 
> -  @retval  TRUE           The debug print error level mask was sucessfully set.
> +  @retval  TRUE           The debug print error level mask was successfully set.
>    @retval  FALSE          The debug print error level mask could not be set.
> 
>  **/
> diff --git a/MdePkg/Include/Library/FileHandleLib.h
> b/MdePkg/Include/Library/FileHandleLib.h
> index 00706f9..e94f83e 100644
> --- a/MdePkg/Include/Library/FileHandleLib.h
> +++ b/MdePkg/Include/Library/FileHandleLib.h
> @@ -184,9 +184,9 @@ FileHandleDelete (
>    has the effect of starting the read process of the directory entries over
> again.
> 
>    @param[in] FileHandle         The file handle on which the position is being
> set.
> -  @param[in] Position           The byte position from the begining of the file.
> +  @param[in] Position           The byte position from the beginning of the file.
> 
> -  @retval EFI_SUCCESS           The operation completed sucessfully.
> +  @retval EFI_SUCCESS           The operation completed successfully.
>    @retval EFI_UNSUPPORTED       The request for non-zero is not valid on
>                                  directories.
>    @retval INVALID_PARAMETER     One of the parameters has an invalid value.
> @@ -207,7 +207,7 @@ FileHandleSetPosition (
>    if FileHandle is a directory.
> 
>    @param[in] FileHandle         The open file handle on which to get the
> position.
> -  @param[out] Position          The byte position from begining of file.
> +  @param[out] Position          The byte position from beginning of file.
> 
>    @retval EFI_SUCCESS           The operation completed successfully.
>    @retval INVALID_PARAMETER     One of the parameters has an invalid value.
> @@ -322,7 +322,7 @@ FileHandleFindNextFile(
>    @param[in] FileHandle         The file handle from which size is retrieved.
>    @param[out] Size              The pointer to size.
> 
> -  @retval EFI_SUCCESS           Operation was completed sucessfully.
> +  @retval EFI_SUCCESS           Operation was completed successfully.
>    @retval EFI_DEVICE_ERROR      Cannot access the file.
>    @retval EFI_INVALID_PARAMETER FileHandle is NULL.
>                                  Size is NULL.
> diff --git a/MdePkg/Include/Library/PeCoffLib.h
> b/MdePkg/Include/Library/PeCoffLib.h
> index 9ed6d61..5a3f988 100644
> --- a/MdePkg/Include/Library/PeCoffLib.h
> +++ b/MdePkg/Include/Library/PeCoffLib.h
> @@ -337,7 +337,7 @@ PeCoffLoaderImageReadFromMemory (
> 
> 
>  /**
> -  Reapply fixups on a fixed up PE32/PE32+ image to allow virutal calling at EFI
> +  Reapply fixups on a fixed up PE32/PE32+ image to allow virtual calling at EFI
>    runtime.
> 
>    This function reapplies relocation fixups to the PE/COFF image specified by
> ImageBase
> diff --git a/MdePkg/Include/Pi/PiSmmCis.h
> b/MdePkg/Include/Pi/PiSmmCis.h
> index 6d3ddab..6cb28b7 100644
> --- a/MdePkg/Include/Pi/PiSmmCis.h
> +++ b/MdePkg/Include/Pi/PiSmmCis.h
> @@ -178,7 +178,7 @@ EFI_STATUS
>  /**
>    Registers a handler to execute within SMM.
> 
> -  @param[in]  Handler            Handler service funtion pointer.
> +  @param[in]  Handler            Handler service function pointer.
>    @param[in]  HandlerType        Points to the handler type or NULL for root
> SMI handlers.
>    @param[out] DispatchHandle     On return, contains a unique handle which
> can be used to later
>                                   unregister the handler function.
> diff --git a/MdePkg/Include/Protocol/BlockIo2.h
> b/MdePkg/Include/Protocol/BlockIo2.h
> index 6f0c280..5fad107 100644
> --- a/MdePkg/Include/Protocol/BlockIo2.h
> +++ b/MdePkg/Include/Protocol/BlockIo2.h
> @@ -52,7 +52,7 @@ typedef struct {
> 
>    @param[in]  This                 Indicates a pointer to the calling context.
>    @param[in]  ExtendedVerification Indicates that the driver may perform a
> more
> -                                   exhausive verfication operation of the device
> +                                   exhausive verification operation of the device
>                                     during reset.
> 
>    @retval EFI_SUCCESS          The device was reset.
> diff --git a/MdePkg/Include/Protocol/BlockIoCrypto.h
> b/MdePkg/Include/Protocol/BlockIoCrypto.h
> index 77931e7..821b846 100644
> --- a/MdePkg/Include/Protocol/BlockIoCrypto.h
> +++ b/MdePkg/Include/Protocol/BlockIoCrypto.h
> @@ -172,7 +172,7 @@ typedef struct {
> 
>    @param[in]  This                 Pointer to the EFI_BLOCK_IO_CRYPTO_PROTOCOL
> instance.
>    @param[in]  ExtendedVerification Indicates that the driver may perform a
> more exhausive
> -                                   verfication operation of the device during reset.
> +                                   verification operation of the device during reset.
> 
>    @retval EFI_SUCCESS              The block device was reset.
>    @retval EFI_DEVICE_ERROR         The block device is not functioning
> correctly and could
> diff --git a/MdePkg/Include/Protocol/Dns4.h
> b/MdePkg/Include/Protocol/Dns4.h
> index 2489268..f96c50f 100644
> --- a/MdePkg/Include/Protocol/Dns4.h
> +++ b/MdePkg/Include/Protocol/Dns4.h
> @@ -310,7 +310,7 @@ EFI_STATUS
> 
>    @retval EFI_SUCCESS             The operation completed successfully.
>    @retval EFI_UNSUPPORTED         The designated protocol is not supported.
> -  @retval EFI_INVALID_PARAMTER    Thisis NULL.
> +  @retval EFI_INVALID_PARAMETER   This is NULL.
>                                    The StationIp address provided in DnsConfigData is not a
>                                    valid unicast.
>                                    DnsServerList is NULL while DnsServerListCount
> diff --git a/MdePkg/Include/Protocol/Dns6.h
> b/MdePkg/Include/Protocol/Dns6.h
> index 1ea85bc..813893a 100644
> --- a/MdePkg/Include/Protocol/Dns6.h
> +++ b/MdePkg/Include/Protocol/Dns6.h
> @@ -300,7 +300,7 @@ EFI_STATUS
>                                    storage to be allocated and released by caller.
> 
>    @retval EFI_SUCCESS             The operation completed successfully.
> -  @retval EFI_INVALID_PARAMTER    This is NULL.
> +  @retval EFI_INVALID_PARAMETER   This is NULL.
>                                    The StationIp address provided in DnsConfigData is not zero
> and not a valid unicast.
>                                    DnsServerList is NULL while DnsServerList Count is not ZERO.
>                                    DnsServerList Count is ZERO while DnsServerList is not NULL.
> diff --git a/MdePkg/Include/Protocol/FirmwareManagement.h
> b/MdePkg/Include/Protocol/FirmwareManagement.h
> index e615573..b199888 100644
> --- a/MdePkg/Include/Protocol/FirmwareManagement.h
> +++ b/MdePkg/Include/Protocol/FirmwareManagement.h
> @@ -241,7 +241,7 @@ typedef struct {
>  #define  PACKAGE_ATTRIBUTE_AUTHENTICATION_REQUIRED
> 0x0000000000000004
> 
>  /**
> -  Callback funtion to report the process of the firmware updating.
> +  Callback function to report the process of the firmware updating.
> 
>    @param[in]  Completion    A value between 1 and 100 indicating the current
> completion
>                              progress of the firmware update. Completion progress is
> diff --git a/MdePkg/Include/Protocol/ImageDecoder.h
> b/MdePkg/Include/Protocol/ImageDecoder.h
> index aebb813..3a7845b 100644
> --- a/MdePkg/Include/Protocol/ImageDecoder.h
> +++ b/MdePkg/Include/Protocol/ImageDecoder.h
> @@ -130,7 +130,7 @@ EFI_STATUS
>    @param This                    EFI_HII_IMAGE_DECODER_PROTOCOL instance.
>    @param Image                   Pointer to the image raw data.
>    @param SizeOfImage             Size of the entire image raw data.
> -  @param ImageInfo               Pointer to recieve
> EFI_HII_IMAGE_DECODER_IMAGE_INFO_HEADER.
> +  @param ImageInfo               Pointer to receive
> EFI_HII_IMAGE_DECODER_IMAGE_INFO_HEADER.
> 
>    @retval EFI_SUCCESS            Get image info success.
>    @retval EFI_UNSUPPORTED        Unsupported format of image.
> diff --git a/MdePkg/Include/Protocol/Mtftp4.h
> b/MdePkg/Include/Protocol/Mtftp4.h
> index 1afac6c..ae2dc27 100644
> --- a/MdePkg/Include/Protocol/Mtftp4.h
> +++ b/MdePkg/Include/Protocol/Mtftp4.h
> @@ -1,5 +1,5 @@
>  /** @file
> -  EFI Multicast Trivial File Tranfer Protocol Definition
> +  EFI Multicast Trivial File Transfer Protocol Definition
> 
>  Copyright (c) 2006 - 2011, Intel Corporation. All rights reserved.<BR>
>  This program and the accompanying materials are licensed and made
> available under
> @@ -212,7 +212,7 @@ EFI_STATUS
>    );
> 
>  /**
> -  Timeout callback funtion.
> +  Timeout callback function.
> 
>    @param  This           The pointer to the EFI_MTFTP4_PROTOCOL instance.
>    @param  Token          The token that is provided in the
> diff --git a/MdePkg/Include/Protocol/Mtftp6.h
> b/MdePkg/Include/Protocol/Mtftp6.h
> index b39575d..4667add 100644
> --- a/MdePkg/Include/Protocol/Mtftp6.h
> +++ b/MdePkg/Include/Protocol/Mtftp6.h
> @@ -1,5 +1,5 @@
>  /** @file
> -  UEFI Multicast Trivial File Tranfer Protocol v6 Definition, which is built upon
> +  UEFI Multicast Trivial File Transfer Protocol v6 Definition, which is built
> upon
>    the EFI UDPv6 Protocol and provides basic services for client-side unicast
> and/or
>    multicast TFTP operations.
> 
> @@ -325,7 +325,7 @@ typedef struct {
>    @param[in] PacketLen     Indicates the length of the packet.
>    @param[in] Packet        Pointer to an MTFTPv6 packet.
> 
> -  @retval EFI_SUCCESS      Operation sucess.
> +  @retval EFI_SUCCESS      Operation success.
>    @retval Others           Aborts session.
> 
>  **/
> @@ -352,7 +352,7 @@ EFI_STATUS
>                             EFI_MTFTP6_PROTOCOL.WriteFile() or
> EFI_MTFTP6_PROTOCOL.ReadDirectory()
>                             functions by the caller.
> 
> -  @retval EFI_SUCCESS      Operation sucess.
> +  @retval EFI_SUCCESS      Operation success.
>    @retval Others           Aborts session.
> 
>  **/
> @@ -379,7 +379,7 @@ EFI_STATUS
>                             length the data available on output.
>    @param[out]     Buffer   Pointer to the buffer where the data is stored.
> 
> -  @retval EFI_SUCCESS      Operation sucess.
> +  @retval EFI_SUCCESS      Operation success.
>    @retval Others           Aborts session.
> 
>  **/
> diff --git a/MdePkg/Include/Protocol/NvmExpressPassthru.h
> b/MdePkg/Include/Protocol/NvmExpressPassthru.h
> index ebd7a8b..7a90c63 100644
> --- a/MdePkg/Include/Protocol/NvmExpressPassthru.h
> +++ b/MdePkg/Include/Protocol/NvmExpressPassthru.h
> @@ -115,7 +115,7 @@ typedef struct {
>  } EFI_NVM_EXPRESS_PASS_THRU_COMMAND_PACKET;
> 
>  //
> -// Protocol funtion prototypes
> +// Protocol function prototypes
>  //
>  /**
>    Sends an NVM Express Command Packet to an NVM Express controller or
> namespace. This function supports
> diff --git a/MdePkg/Include/Protocol/SimpleTextOut.h
> b/MdePkg/Include/Protocol/SimpleTextOut.h
> index 5f64b64..c1a23fa 100644
> --- a/MdePkg/Include/Protocol/SimpleTextOut.h
> +++ b/MdePkg/Include/Protocol/SimpleTextOut.h
> @@ -160,7 +160,7 @@ typedef EFI_SIMPLE_TEXT_OUTPUT_PROTOCOL
> SIMPLE_TEXT_OUTPUT_INTERFACE;
>    Reset the text output device hardware and optionaly run diagnostics
> 
>    @param  This                 The protocol instance pointer.
> -  @param  ExtendedVerification Driver may perform more exhaustive
> verfication
> +  @param  ExtendedVerification Driver may perform more exhaustive
> verification
>                                 operation of the device during reset.
> 
>    @retval EFI_SUCCESS          The text output device was reset.
> diff --git a/MdePkg/Include/Protocol/Timer.h
> b/MdePkg/Include/Protocol/Timer.h
> index c9a998e..ffdb0a1 100644
> --- a/MdePkg/Include/Protocol/Timer.h
> +++ b/MdePkg/Include/Protocol/Timer.h
> @@ -30,7 +30,7 @@ typedef struct _EFI_TIMER_ARCH_PROTOCOL
> EFI_TIMER_ARCH_PROTOCOL;
> 
>  /**
>    This function of this type is called when a timer interrupt fires.  This
> -  function executes at TPL_HIGH_LEVEL.  The DXE Core will register a funtion
> +  function executes at TPL_HIGH_LEVEL.  The DXE Core will register a
> function
>    of this type to be called for the timer interrupt, so it can know how much
>    time has passed.  This information is used to signal timer based events.
> 
> diff --git a/MdePkg/Include/Uefi/UefiPxe.h
> b/MdePkg/Include/Uefi/UefiPxe.h
> index 0af9b90..249a8c2 100644
> --- a/MdePkg/Include/Uefi/UefiPxe.h
> +++ b/MdePkg/Include/Uefi/UefiPxe.h
> @@ -1079,7 +1079,7 @@ typedef struct s_pxe_cpb_start_31 {
> 
>    ///
>    /// protocol driver can provide anything for this Unique_ID, UNDI
> remembers
> -  /// that as just a 64bit value assocaited to the interface specified by
> +  /// that as just a 64bit value associated to the interface specified by
>    /// the ifnum and gives it back as a parameter to all the call-back routines
>    /// when calling for that interface!
>    ///
> diff --git a/MdePkg/Include/X64/ProcessorBind.h
> b/MdePkg/Include/X64/ProcessorBind.h
> index ffcf2b0..fcbe28a 100644
> --- a/MdePkg/Include/X64/ProcessorBind.h
> +++ b/MdePkg/Include/X64/ProcessorBind.h
> @@ -93,7 +93,7 @@
>  #pragma warning ( disable : 4057 )
> 
>  //
> -// ASSERT(FALSE) or while (TRUE) are legal constructes so supress this
> warning
> +// ASSERT(FALSE) or while (TRUE) are legal constructs so suppress this
> warning
>  //
>  #pragma warning ( disable : 4127 )
> 
> @@ -132,7 +132,7 @@
> 
>  #if defined(_MSC_EXTENSIONS)
>    //
> -  // use Microsoft C complier dependent integer width types
> +  // use Microsoft C compiler dependent integer width types
>    //
> 
>    ///
> diff --git a/MdePkg/Library/BaseMemoryLibMmx/MemLibInternals.h
> b/MdePkg/Library/BaseMemoryLibMmx/MemLibInternals.h
> index cc89979..9505265 100644
> --- a/MdePkg/Library/BaseMemoryLibMmx/MemLibInternals.h
> +++ b/MdePkg/Library/BaseMemoryLibMmx/MemLibInternals.h
> @@ -103,7 +103,7 @@ InternalMemSetMem32 (
>  /**
>    Fills a target buffer with a 64-bit value, and returns the target buffer.
> 
> -  @param  Buffer  Teh pointer to the target buffer to fill.
> +  @param  Buffer  The pointer to the target buffer to fill.
>    @param  Length  The count of 64-bit value to fill.
>    @param  Value   The value with which to fill Length bytes of Buffer.
> 
> diff --git
> a/MdePkg/Library/BaseUefiDecompressLib/BaseUefiDecompressLibInternal
> s.h
> b/MdePkg/Library/BaseUefiDecompressLib/BaseUefiDecompressLibInternal
> s.h
> index 358c598..aba510c 100644
> ---
> a/MdePkg/Library/BaseUefiDecompressLib/BaseUefiDecompressLibInternal
> s.h
> +++
> b/MdePkg/Library/BaseUefiDecompressLib/BaseUefiDecompressLibInternal
> s.h
> @@ -1,5 +1,5 @@
>  /** @file
> -  Internal data structure defintions for Base UEFI Decompress Libary.
> +  Internal data structure defintions for Base UEFI Decompress Library.
> 
>    Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.<BR>
>    This program and the accompanying materials
> diff --git a/MdePkg/Library/DxeIoLibCpuIo2/DxeCpuIo2LibInternal.h
> b/MdePkg/Library/DxeIoLibCpuIo2/DxeCpuIo2LibInternal.h
> index b459214..160bedf 100644
> --- a/MdePkg/Library/DxeIoLibCpuIo2/DxeCpuIo2LibInternal.h
> +++ b/MdePkg/Library/DxeIoLibCpuIo2/DxeCpuIo2LibInternal.h
> @@ -58,7 +58,7 @@ IoReadWorker (
>    @param  Width         The width of the I/O operation.
>    @param  Data          The value to write to the I/O port.
> 
> -  @return The paramter of Data.
> +  @return The parameter of Data.
> 
>  **/
>  UINT64
> diff --git a/MdePkg/Library/SmmIoLibSmmCpuIo2/SmmCpuIoLibInternal.h
> b/MdePkg/Library/SmmIoLibSmmCpuIo2/SmmCpuIoLibInternal.h
> index 0e94b20..ced6358 100644
> --- a/MdePkg/Library/SmmIoLibSmmCpuIo2/SmmCpuIoLibInternal.h
> +++ b/MdePkg/Library/SmmIoLibSmmCpuIo2/SmmCpuIoLibInternal.h
> @@ -60,7 +60,7 @@ IoReadWorker (
>    @param  Width         The width of the I/O operation.
>    @param  Data          The value to write to the I/O port.
> 
> -  @return The paramter of Data.
> +  @return The parameter of Data.
> 
>  **/
>  UINT64
> --
> 1.8.3.1



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] MdePkg: Fix some typing errors in the header files
  2016-09-12  9:09 ` Gao, Liming
@ 2016-10-08  1:16   ` Kinney, Michael D
       [not found]   ` <60e78d8e-c7dc-ee87-df2c-69ddfd4ef202@redhat.com>
  1 sibling, 0 replies; 4+ messages in thread
From: Kinney, Michael D @ 2016-10-08  1:16 UTC (permalink / raw)
  To: Gao, Liming, Thomas Huth, edk2-devel@ml01.01.org,
	Kinney, Michael D

Reviewed-by: Michael Kinney <michael.d.kinney@intel.com>

> -----Original Message-----
> From: Gao, Liming
> Sent: Monday, September 12, 2016 2:10 AM
> To: Thomas Huth <thuth@redhat.com>; edk2-devel@ml01.01.org
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>
> Subject: RE: [PATCH] MdePkg: Fix some typing errors in the header files
> 
> Reviewed-by: Liming Gao <liming.gao@intel.com>
> 
> > -----Original Message-----
> > From: Thomas Huth [mailto:thuth@redhat.com]
> > Sent: Monday, September 12, 2016 4:36 PM
> > To: edk2-devel@ml01.01.org
> > Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
> > <liming.gao@intel.com>
> > Subject: [PATCH] MdePkg: Fix some typing errors in the header files
> >
> > Correct the typos in some header files of MdePkg.
> >
> > Contributed-under: TianoCore Contribution Agreement 1.0
> > Signed-off-by: Thomas Huth <thuth@redhat.com>
> > ---
> >  MdePkg/Include/AArch64/ProcessorBind.h                            | 2 +-
> >  MdePkg/Include/Arm/ProcessorBind.h                                | 2 +-
> >  MdePkg/Include/Base.h                                             | 2 +-
> >  MdePkg/Include/Ia32/ProcessorBind.h                               | 4 ++--
> >  MdePkg/Include/Ipf/ProcessorBind.h                                | 2 +-
> >  MdePkg/Include/Library/BaseLib.h                                  | 2 +-
> >  MdePkg/Include/Library/DebugPrintErrorLevelLib.h                  | 2 +-
> >  MdePkg/Include/Library/FileHandleLib.h                            | 8 ++++----
> >  MdePkg/Include/Library/PeCoffLib.h                                | 2 +-
> >  MdePkg/Include/Pi/PiSmmCis.h                                      | 2 +-
> >  MdePkg/Include/Protocol/BlockIo2.h                                | 2 +-
> >  MdePkg/Include/Protocol/BlockIoCrypto.h                           | 2 +-
> >  MdePkg/Include/Protocol/Dns4.h                                    | 2 +-
> >  MdePkg/Include/Protocol/Dns6.h                                    | 2 +-
> >  MdePkg/Include/Protocol/FirmwareManagement.h                      | 2 +-
> >  MdePkg/Include/Protocol/ImageDecoder.h                            | 2 +-
> >  MdePkg/Include/Protocol/Mtftp4.h                                  | 4 ++--
> >  MdePkg/Include/Protocol/Mtftp6.h                                  | 8 ++++----
> >  MdePkg/Include/Protocol/NvmExpressPassthru.h                      | 2 +-
> >  MdePkg/Include/Protocol/SimpleTextOut.h                           | 2 +-
> >  MdePkg/Include/Protocol/Timer.h                                   | 2 +-
> >  MdePkg/Include/Uefi/UefiPxe.h                                     | 2 +-
> >  MdePkg/Include/X64/ProcessorBind.h                                | 4 ++--
> >  MdePkg/Library/BaseMemoryLibMmx/MemLibInternals.h                 | 2 +-
> >  .../BaseUefiDecompressLib/BaseUefiDecompressLibInternals.h        | 2 +-
> >  MdePkg/Library/DxeIoLibCpuIo2/DxeCpuIo2LibInternal.h              | 2 +-
> >  MdePkg/Library/SmmIoLibSmmCpuIo2/SmmCpuIoLibInternal.h            | 2 +-
> >  27 files changed, 36 insertions(+), 36 deletions(-)
> >
> > diff --git a/MdePkg/Include/AArch64/ProcessorBind.h
> > b/MdePkg/Include/AArch64/ProcessorBind.h
> > index c5fe874..fa311e2 100644
> > --- a/MdePkg/Include/AArch64/ProcessorBind.h
> > +++ b/MdePkg/Include/AArch64/ProcessorBind.h
> > @@ -32,7 +32,7 @@
> >
> >  #if _MSC_EXTENSIONS
> >    //
> > -  // use Microsoft* C complier dependent integer width types
> > +  // use Microsoft* C compiler dependent integer width types
> >    //
> >    typedef unsigned __int64    UINT64;
> >    typedef __int64             INT64;
> > diff --git a/MdePkg/Include/Arm/ProcessorBind.h
> > b/MdePkg/Include/Arm/ProcessorBind.h
> > index 5ee7465..dc45b94 100644
> > --- a/MdePkg/Include/Arm/ProcessorBind.h
> > +++ b/MdePkg/Include/Arm/ProcessorBind.h
> > @@ -37,7 +37,7 @@
> >
> >  #if _MSC_EXTENSIONS
> >    //
> > -  // use Microsoft* C complier dependent integer width types
> > +  // use Microsoft* C compiler dependent integer width types
> >    //
> >    typedef unsigned __int64    UINT64;
> >    typedef __int64             INT64;
> > diff --git a/MdePkg/Include/Base.h b/MdePkg/Include/Base.h
> > index 93f6686..c666148 100644
> > --- a/MdePkg/Include/Base.h
> > +++ b/MdePkg/Include/Base.h
> > @@ -302,7 +302,7 @@ struct _LIST_ENTRY {
> >
> >  //
> >  //  UEFI specification claims 1 and 0. We are concerned about the
> > -//  complier portability so we did it this way.
> > +//  compiler portability so we did it this way.
> >  //
> >
> >  ///
> > diff --git a/MdePkg/Include/Ia32/ProcessorBind.h
> > b/MdePkg/Include/Ia32/ProcessorBind.h
> > index 458f7dd..da1810b 100644
> > --- a/MdePkg/Include/Ia32/ProcessorBind.h
> > +++ b/MdePkg/Include/Ia32/ProcessorBind.h
> > @@ -79,7 +79,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY
> > KIND, EITHER EXPRESS OR IMPLIED.
> >  #pragma warning ( disable : 4057 )
> >
> >  //
> > -// ASSERT(FALSE) or while (TRUE) are legal constructes so supress this
> > warning
> > +// ASSERT(FALSE) or while (TRUE) are legal constructs so suppress this
> > warning
> >  //
> >  #pragma warning ( disable : 4127 )
> >
> > @@ -119,7 +119,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF
> > ANY KIND, EITHER EXPRESS OR IMPLIED.
> >  #if defined(_MSC_EXTENSIONS)
> >
> >    //
> > -  // use Microsoft C complier dependent integer width types
> > +  // use Microsoft C compiler dependent integer width types
> >    //
> >
> >    ///
> > diff --git a/MdePkg/Include/Ipf/ProcessorBind.h
> > b/MdePkg/Include/Ipf/ProcessorBind.h
> > index cead575..c19e47d 100644
> > --- a/MdePkg/Include/Ipf/ProcessorBind.h
> > +++ b/MdePkg/Include/Ipf/ProcessorBind.h
> > @@ -84,7 +84,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY
> > KIND, EITHER EXPRESS OR IMPLIED.
> >  #pragma warning ( disable : 4054 )
> >
> >  //
> > -// ASSERT(FALSE) or while (TRUE) are legal constructes so supress this
> > warning
> > +// ASSERT(FALSE) or while (TRUE) are legal constructs so suppress this
> > warning
> >  //
> >  #pragma warning ( disable : 4127 )
> >
> > diff --git a/MdePkg/Include/Library/BaseLib.h
> > b/MdePkg/Include/Library/BaseLib.h
> > index 7e085b9..6268e6f 100644
> > --- a/MdePkg/Include/Library/BaseLib.h
> > +++ b/MdePkg/Include/Library/BaseLib.h
> > @@ -1772,7 +1772,7 @@ PathRemoveLastItem(
> >
> >    @param[in] Path       The pointer to the string containing the path.
> >
> > -  @return       Returns Path, otherwise returns NULL to indicate that an error
> > has occured.
> > +  @return       Returns Path, otherwise returns NULL to indicate that an error
> > has occurred.
> >  **/
> >  CHAR16*
> >  EFIAPI
> > diff --git a/MdePkg/Include/Library/DebugPrintErrorLevelLib.h
> > b/MdePkg/Include/Library/DebugPrintErrorLevelLib.h
> > index 2b051f4..5a866d5 100644
> > --- a/MdePkg/Include/Library/DebugPrintErrorLevelLib.h
> > +++ b/MdePkg/Include/Library/DebugPrintErrorLevelLib.h
> > @@ -31,7 +31,7 @@ GetDebugPrintErrorLevel (
> >
> >    @param   ErrorLevel     Global debug print error level
> >
> > -  @retval  TRUE           The debug print error level mask was sucessfully set.
> > +  @retval  TRUE           The debug print error level mask was successfully set.
> >    @retval  FALSE          The debug print error level mask could not be set.
> >
> >  **/
> > diff --git a/MdePkg/Include/Library/FileHandleLib.h
> > b/MdePkg/Include/Library/FileHandleLib.h
> > index 00706f9..e94f83e 100644
> > --- a/MdePkg/Include/Library/FileHandleLib.h
> > +++ b/MdePkg/Include/Library/FileHandleLib.h
> > @@ -184,9 +184,9 @@ FileHandleDelete (
> >    has the effect of starting the read process of the directory entries over
> > again.
> >
> >    @param[in] FileHandle         The file handle on which the position is being
> > set.
> > -  @param[in] Position           The byte position from the begining of the file.
> > +  @param[in] Position           The byte position from the beginning of the file.
> >
> > -  @retval EFI_SUCCESS           The operation completed sucessfully.
> > +  @retval EFI_SUCCESS           The operation completed successfully.
> >    @retval EFI_UNSUPPORTED       The request for non-zero is not valid on
> >                                  directories.
> >    @retval INVALID_PARAMETER     One of the parameters has an invalid value.
> > @@ -207,7 +207,7 @@ FileHandleSetPosition (
> >    if FileHandle is a directory.
> >
> >    @param[in] FileHandle         The open file handle on which to get the
> > position.
> > -  @param[out] Position          The byte position from begining of file.
> > +  @param[out] Position          The byte position from beginning of file.
> >
> >    @retval EFI_SUCCESS           The operation completed successfully.
> >    @retval INVALID_PARAMETER     One of the parameters has an invalid value.
> > @@ -322,7 +322,7 @@ FileHandleFindNextFile(
> >    @param[in] FileHandle         The file handle from which size is retrieved.
> >    @param[out] Size              The pointer to size.
> >
> > -  @retval EFI_SUCCESS           Operation was completed sucessfully.
> > +  @retval EFI_SUCCESS           Operation was completed successfully.
> >    @retval EFI_DEVICE_ERROR      Cannot access the file.
> >    @retval EFI_INVALID_PARAMETER FileHandle is NULL.
> >                                  Size is NULL.
> > diff --git a/MdePkg/Include/Library/PeCoffLib.h
> > b/MdePkg/Include/Library/PeCoffLib.h
> > index 9ed6d61..5a3f988 100644
> > --- a/MdePkg/Include/Library/PeCoffLib.h
> > +++ b/MdePkg/Include/Library/PeCoffLib.h
> > @@ -337,7 +337,7 @@ PeCoffLoaderImageReadFromMemory (
> >
> >
> >  /**
> > -  Reapply fixups on a fixed up PE32/PE32+ image to allow virutal calling at EFI
> > +  Reapply fixups on a fixed up PE32/PE32+ image to allow virtual calling at EFI
> >    runtime.
> >
> >    This function reapplies relocation fixups to the PE/COFF image specified by
> > ImageBase
> > diff --git a/MdePkg/Include/Pi/PiSmmCis.h
> > b/MdePkg/Include/Pi/PiSmmCis.h
> > index 6d3ddab..6cb28b7 100644
> > --- a/MdePkg/Include/Pi/PiSmmCis.h
> > +++ b/MdePkg/Include/Pi/PiSmmCis.h
> > @@ -178,7 +178,7 @@ EFI_STATUS
> >  /**
> >    Registers a handler to execute within SMM.
> >
> > -  @param[in]  Handler            Handler service funtion pointer.
> > +  @param[in]  Handler            Handler service function pointer.
> >    @param[in]  HandlerType        Points to the handler type or NULL for root
> > SMI handlers.
> >    @param[out] DispatchHandle     On return, contains a unique handle which
> > can be used to later
> >                                   unregister the handler function.
> > diff --git a/MdePkg/Include/Protocol/BlockIo2.h
> > b/MdePkg/Include/Protocol/BlockIo2.h
> > index 6f0c280..5fad107 100644
> > --- a/MdePkg/Include/Protocol/BlockIo2.h
> > +++ b/MdePkg/Include/Protocol/BlockIo2.h
> > @@ -52,7 +52,7 @@ typedef struct {
> >
> >    @param[in]  This                 Indicates a pointer to the calling context.
> >    @param[in]  ExtendedVerification Indicates that the driver may perform a
> > more
> > -                                   exhausive verfication operation of the device
> > +                                   exhausive verification operation of the device
> >                                     during reset.
> >
> >    @retval EFI_SUCCESS          The device was reset.
> > diff --git a/MdePkg/Include/Protocol/BlockIoCrypto.h
> > b/MdePkg/Include/Protocol/BlockIoCrypto.h
> > index 77931e7..821b846 100644
> > --- a/MdePkg/Include/Protocol/BlockIoCrypto.h
> > +++ b/MdePkg/Include/Protocol/BlockIoCrypto.h
> > @@ -172,7 +172,7 @@ typedef struct {
> >
> >    @param[in]  This                 Pointer to the EFI_BLOCK_IO_CRYPTO_PROTOCOL
> > instance.
> >    @param[in]  ExtendedVerification Indicates that the driver may perform a
> > more exhausive
> > -                                   verfication operation of the device during
> reset.
> > +                                   verification operation of the device during
> reset.
> >
> >    @retval EFI_SUCCESS              The block device was reset.
> >    @retval EFI_DEVICE_ERROR         The block device is not functioning
> > correctly and could
> > diff --git a/MdePkg/Include/Protocol/Dns4.h
> > b/MdePkg/Include/Protocol/Dns4.h
> > index 2489268..f96c50f 100644
> > --- a/MdePkg/Include/Protocol/Dns4.h
> > +++ b/MdePkg/Include/Protocol/Dns4.h
> > @@ -310,7 +310,7 @@ EFI_STATUS
> >
> >    @retval EFI_SUCCESS             The operation completed successfully.
> >    @retval EFI_UNSUPPORTED         The designated protocol is not supported.
> > -  @retval EFI_INVALID_PARAMTER    Thisis NULL.
> > +  @retval EFI_INVALID_PARAMETER   This is NULL.
> >                                    The StationIp address provided in DnsConfigData
> is not a
> >                                    valid unicast.
> >                                    DnsServerList is NULL while DnsServerListCount
> > diff --git a/MdePkg/Include/Protocol/Dns6.h
> > b/MdePkg/Include/Protocol/Dns6.h
> > index 1ea85bc..813893a 100644
> > --- a/MdePkg/Include/Protocol/Dns6.h
> > +++ b/MdePkg/Include/Protocol/Dns6.h
> > @@ -300,7 +300,7 @@ EFI_STATUS
> >                                    storage to be allocated and released by caller.
> >
> >    @retval EFI_SUCCESS             The operation completed successfully.
> > -  @retval EFI_INVALID_PARAMTER    This is NULL.
> > +  @retval EFI_INVALID_PARAMETER   This is NULL.
> >                                    The StationIp address provided in DnsConfigData
> is not zero
> > and not a valid unicast.
> >                                    DnsServerList is NULL while DnsServerList Count
> is not ZERO.
> >                                    DnsServerList Count is ZERO while DnsServerList
> is not NULL.
> > diff --git a/MdePkg/Include/Protocol/FirmwareManagement.h
> > b/MdePkg/Include/Protocol/FirmwareManagement.h
> > index e615573..b199888 100644
> > --- a/MdePkg/Include/Protocol/FirmwareManagement.h
> > +++ b/MdePkg/Include/Protocol/FirmwareManagement.h
> > @@ -241,7 +241,7 @@ typedef struct {
> >  #define  PACKAGE_ATTRIBUTE_AUTHENTICATION_REQUIRED
> > 0x0000000000000004
> >
> >  /**
> > -  Callback funtion to report the process of the firmware updating.
> > +  Callback function to report the process of the firmware updating.
> >
> >    @param[in]  Completion    A value between 1 and 100 indicating the current
> > completion
> >                              progress of the firmware update. Completion progress
> is
> > diff --git a/MdePkg/Include/Protocol/ImageDecoder.h
> > b/MdePkg/Include/Protocol/ImageDecoder.h
> > index aebb813..3a7845b 100644
> > --- a/MdePkg/Include/Protocol/ImageDecoder.h
> > +++ b/MdePkg/Include/Protocol/ImageDecoder.h
> > @@ -130,7 +130,7 @@ EFI_STATUS
> >    @param This                    EFI_HII_IMAGE_DECODER_PROTOCOL instance.
> >    @param Image                   Pointer to the image raw data.
> >    @param SizeOfImage             Size of the entire image raw data.
> > -  @param ImageInfo               Pointer to recieve
> > EFI_HII_IMAGE_DECODER_IMAGE_INFO_HEADER.
> > +  @param ImageInfo               Pointer to receive
> > EFI_HII_IMAGE_DECODER_IMAGE_INFO_HEADER.
> >
> >    @retval EFI_SUCCESS            Get image info success.
> >    @retval EFI_UNSUPPORTED        Unsupported format of image.
> > diff --git a/MdePkg/Include/Protocol/Mtftp4.h
> > b/MdePkg/Include/Protocol/Mtftp4.h
> > index 1afac6c..ae2dc27 100644
> > --- a/MdePkg/Include/Protocol/Mtftp4.h
> > +++ b/MdePkg/Include/Protocol/Mtftp4.h
> > @@ -1,5 +1,5 @@
> >  /** @file
> > -  EFI Multicast Trivial File Tranfer Protocol Definition
> > +  EFI Multicast Trivial File Transfer Protocol Definition
> >
> >  Copyright (c) 2006 - 2011, Intel Corporation. All rights reserved.<BR>
> >  This program and the accompanying materials are licensed and made
> > available under
> > @@ -212,7 +212,7 @@ EFI_STATUS
> >    );
> >
> >  /**
> > -  Timeout callback funtion.
> > +  Timeout callback function.
> >
> >    @param  This           The pointer to the EFI_MTFTP4_PROTOCOL instance.
> >    @param  Token          The token that is provided in the
> > diff --git a/MdePkg/Include/Protocol/Mtftp6.h
> > b/MdePkg/Include/Protocol/Mtftp6.h
> > index b39575d..4667add 100644
> > --- a/MdePkg/Include/Protocol/Mtftp6.h
> > +++ b/MdePkg/Include/Protocol/Mtftp6.h
> > @@ -1,5 +1,5 @@
> >  /** @file
> > -  UEFI Multicast Trivial File Tranfer Protocol v6 Definition, which is built upon
> > +  UEFI Multicast Trivial File Transfer Protocol v6 Definition, which is built
> > upon
> >    the EFI UDPv6 Protocol and provides basic services for client-side unicast
> > and/or
> >    multicast TFTP operations.
> >
> > @@ -325,7 +325,7 @@ typedef struct {
> >    @param[in] PacketLen     Indicates the length of the packet.
> >    @param[in] Packet        Pointer to an MTFTPv6 packet.
> >
> > -  @retval EFI_SUCCESS      Operation sucess.
> > +  @retval EFI_SUCCESS      Operation success.
> >    @retval Others           Aborts session.
> >
> >  **/
> > @@ -352,7 +352,7 @@ EFI_STATUS
> >                             EFI_MTFTP6_PROTOCOL.WriteFile() or
> > EFI_MTFTP6_PROTOCOL.ReadDirectory()
> >                             functions by the caller.
> >
> > -  @retval EFI_SUCCESS      Operation sucess.
> > +  @retval EFI_SUCCESS      Operation success.
> >    @retval Others           Aborts session.
> >
> >  **/
> > @@ -379,7 +379,7 @@ EFI_STATUS
> >                             length the data available on output.
> >    @param[out]     Buffer   Pointer to the buffer where the data is stored.
> >
> > -  @retval EFI_SUCCESS      Operation sucess.
> > +  @retval EFI_SUCCESS      Operation success.
> >    @retval Others           Aborts session.
> >
> >  **/
> > diff --git a/MdePkg/Include/Protocol/NvmExpressPassthru.h
> > b/MdePkg/Include/Protocol/NvmExpressPassthru.h
> > index ebd7a8b..7a90c63 100644
> > --- a/MdePkg/Include/Protocol/NvmExpressPassthru.h
> > +++ b/MdePkg/Include/Protocol/NvmExpressPassthru.h
> > @@ -115,7 +115,7 @@ typedef struct {
> >  } EFI_NVM_EXPRESS_PASS_THRU_COMMAND_PACKET;
> >
> >  //
> > -// Protocol funtion prototypes
> > +// Protocol function prototypes
> >  //
> >  /**
> >    Sends an NVM Express Command Packet to an NVM Express controller or
> > namespace. This function supports
> > diff --git a/MdePkg/Include/Protocol/SimpleTextOut.h
> > b/MdePkg/Include/Protocol/SimpleTextOut.h
> > index 5f64b64..c1a23fa 100644
> > --- a/MdePkg/Include/Protocol/SimpleTextOut.h
> > +++ b/MdePkg/Include/Protocol/SimpleTextOut.h
> > @@ -160,7 +160,7 @@ typedef EFI_SIMPLE_TEXT_OUTPUT_PROTOCOL
> > SIMPLE_TEXT_OUTPUT_INTERFACE;
> >    Reset the text output device hardware and optionaly run diagnostics
> >
> >    @param  This                 The protocol instance pointer.
> > -  @param  ExtendedVerification Driver may perform more exhaustive
> > verfication
> > +  @param  ExtendedVerification Driver may perform more exhaustive
> > verification
> >                                 operation of the device during reset.
> >
> >    @retval EFI_SUCCESS          The text output device was reset.
> > diff --git a/MdePkg/Include/Protocol/Timer.h
> > b/MdePkg/Include/Protocol/Timer.h
> > index c9a998e..ffdb0a1 100644
> > --- a/MdePkg/Include/Protocol/Timer.h
> > +++ b/MdePkg/Include/Protocol/Timer.h
> > @@ -30,7 +30,7 @@ typedef struct _EFI_TIMER_ARCH_PROTOCOL
> > EFI_TIMER_ARCH_PROTOCOL;
> >
> >  /**
> >    This function of this type is called when a timer interrupt fires.  This
> > -  function executes at TPL_HIGH_LEVEL.  The DXE Core will register a funtion
> > +  function executes at TPL_HIGH_LEVEL.  The DXE Core will register a
> > function
> >    of this type to be called for the timer interrupt, so it can know how much
> >    time has passed.  This information is used to signal timer based events.
> >
> > diff --git a/MdePkg/Include/Uefi/UefiPxe.h
> > b/MdePkg/Include/Uefi/UefiPxe.h
> > index 0af9b90..249a8c2 100644
> > --- a/MdePkg/Include/Uefi/UefiPxe.h
> > +++ b/MdePkg/Include/Uefi/UefiPxe.h
> > @@ -1079,7 +1079,7 @@ typedef struct s_pxe_cpb_start_31 {
> >
> >    ///
> >    /// protocol driver can provide anything for this Unique_ID, UNDI
> > remembers
> > -  /// that as just a 64bit value assocaited to the interface specified by
> > +  /// that as just a 64bit value associated to the interface specified by
> >    /// the ifnum and gives it back as a parameter to all the call-back routines
> >    /// when calling for that interface!
> >    ///
> > diff --git a/MdePkg/Include/X64/ProcessorBind.h
> > b/MdePkg/Include/X64/ProcessorBind.h
> > index ffcf2b0..fcbe28a 100644
> > --- a/MdePkg/Include/X64/ProcessorBind.h
> > +++ b/MdePkg/Include/X64/ProcessorBind.h
> > @@ -93,7 +93,7 @@
> >  #pragma warning ( disable : 4057 )
> >
> >  //
> > -// ASSERT(FALSE) or while (TRUE) are legal constructes so supress this
> > warning
> > +// ASSERT(FALSE) or while (TRUE) are legal constructs so suppress this
> > warning
> >  //
> >  #pragma warning ( disable : 4127 )
> >
> > @@ -132,7 +132,7 @@
> >
> >  #if defined(_MSC_EXTENSIONS)
> >    //
> > -  // use Microsoft C complier dependent integer width types
> > +  // use Microsoft C compiler dependent integer width types
> >    //
> >
> >    ///
> > diff --git a/MdePkg/Library/BaseMemoryLibMmx/MemLibInternals.h
> > b/MdePkg/Library/BaseMemoryLibMmx/MemLibInternals.h
> > index cc89979..9505265 100644
> > --- a/MdePkg/Library/BaseMemoryLibMmx/MemLibInternals.h
> > +++ b/MdePkg/Library/BaseMemoryLibMmx/MemLibInternals.h
> > @@ -103,7 +103,7 @@ InternalMemSetMem32 (
> >  /**
> >    Fills a target buffer with a 64-bit value, and returns the target buffer.
> >
> > -  @param  Buffer  Teh pointer to the target buffer to fill.
> > +  @param  Buffer  The pointer to the target buffer to fill.
> >    @param  Length  The count of 64-bit value to fill.
> >    @param  Value   The value with which to fill Length bytes of Buffer.
> >
> > diff --git
> > a/MdePkg/Library/BaseUefiDecompressLib/BaseUefiDecompressLibInternal
> > s.h
> > b/MdePkg/Library/BaseUefiDecompressLib/BaseUefiDecompressLibInternal
> > s.h
> > index 358c598..aba510c 100644
> > ---
> > a/MdePkg/Library/BaseUefiDecompressLib/BaseUefiDecompressLibInternal
> > s.h
> > +++
> > b/MdePkg/Library/BaseUefiDecompressLib/BaseUefiDecompressLibInternal
> > s.h
> > @@ -1,5 +1,5 @@
> >  /** @file
> > -  Internal data structure defintions for Base UEFI Decompress Libary.
> > +  Internal data structure defintions for Base UEFI Decompress Library.
> >
> >    Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.<BR>
> >    This program and the accompanying materials
> > diff --git a/MdePkg/Library/DxeIoLibCpuIo2/DxeCpuIo2LibInternal.h
> > b/MdePkg/Library/DxeIoLibCpuIo2/DxeCpuIo2LibInternal.h
> > index b459214..160bedf 100644
> > --- a/MdePkg/Library/DxeIoLibCpuIo2/DxeCpuIo2LibInternal.h
> > +++ b/MdePkg/Library/DxeIoLibCpuIo2/DxeCpuIo2LibInternal.h
> > @@ -58,7 +58,7 @@ IoReadWorker (
> >    @param  Width         The width of the I/O operation.
> >    @param  Data          The value to write to the I/O port.
> >
> > -  @return The paramter of Data.
> > +  @return The parameter of Data.
> >
> >  **/
> >  UINT64
> > diff --git a/MdePkg/Library/SmmIoLibSmmCpuIo2/SmmCpuIoLibInternal.h
> > b/MdePkg/Library/SmmIoLibSmmCpuIo2/SmmCpuIoLibInternal.h
> > index 0e94b20..ced6358 100644
> > --- a/MdePkg/Library/SmmIoLibSmmCpuIo2/SmmCpuIoLibInternal.h
> > +++ b/MdePkg/Library/SmmIoLibSmmCpuIo2/SmmCpuIoLibInternal.h
> > @@ -60,7 +60,7 @@ IoReadWorker (
> >    @param  Width         The width of the I/O operation.
> >    @param  Data          The value to write to the I/O port.
> >
> > -  @return The paramter of Data.
> > +  @return The parameter of Data.
> >
> >  **/
> >  UINT64
> > --
> > 1.8.3.1



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] MdePkg: Fix some typing errors in the header files
       [not found]   ` <60e78d8e-c7dc-ee87-df2c-69ddfd4ef202@redhat.com>
@ 2016-10-08  1:16     ` Kinney, Michael D
  0 siblings, 0 replies; 4+ messages in thread
From: Kinney, Michael D @ 2016-10-08  1:16 UTC (permalink / raw)
  To: Thomas Huth, Gao, Liming, edk2-devel@ml01.01.org,
	Kinney, Michael D

Thomas,

Thanks for fixing these typos.

I have also reviewed both of your patches and have pushed them to edk2/master.

https://github.com/tianocore/edk2/commit/00b7cc0fe327fcd00d70a8d30f16245cca0c55f2
https://github.com/tianocore/edk2/commit/3868d06d8209d4907c0c6b6f5ec690aebb8b5971


Mike

> -----Original Message-----
> From: Thomas Huth [mailto:thuth@redhat.com]
> Sent: Tuesday, October 4, 2016 1:38 AM
> To: Gao, Liming <liming.gao@intel.com>; edk2-devel@ml01.01.org
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>
> Subject: Re: [PATCH] MdePkg: Fix some typing errors in the header files
> 
>  Hi,
> 
> thanks for the review! Could you please also push my patches to the
> repository? I do not have write access there. Or do you rather expect a
> pull request for the patches?
> 
>  Regards,
>   Thomas
> 
> On 12.09.2016 11:09, Gao, Liming wrote:
> > Reviewed-by: Liming Gao <liming.gao@intel.com>
> >
> >> -----Original Message-----
> >> From: Thomas Huth [mailto:thuth@redhat.com]
> >> Sent: Monday, September 12, 2016 4:36 PM
> >> To: edk2-devel@ml01.01.org
> >> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Gao, Liming
> >> <liming.gao@intel.com>
> >> Subject: [PATCH] MdePkg: Fix some typing errors in the header files
> >>
> >> Correct the typos in some header files of MdePkg.
> >>
> >> Contributed-under: TianoCore Contribution Agreement 1.0
> >> Signed-off-by: Thomas Huth <thuth@redhat.com>
> >> ---
> >>  MdePkg/Include/AArch64/ProcessorBind.h                            | 2 +-
> >>  MdePkg/Include/Arm/ProcessorBind.h                                | 2 +-
> >>  MdePkg/Include/Base.h                                             | 2 +-
> >>  MdePkg/Include/Ia32/ProcessorBind.h                               | 4 ++--
> >>  MdePkg/Include/Ipf/ProcessorBind.h                                | 2 +-
> >>  MdePkg/Include/Library/BaseLib.h                                  | 2 +-
> >>  MdePkg/Include/Library/DebugPrintErrorLevelLib.h                  | 2 +-
> >>  MdePkg/Include/Library/FileHandleLib.h                            | 8 ++++----
> >>  MdePkg/Include/Library/PeCoffLib.h                                | 2 +-
> >>  MdePkg/Include/Pi/PiSmmCis.h                                      | 2 +-
> >>  MdePkg/Include/Protocol/BlockIo2.h                                | 2 +-
> >>  MdePkg/Include/Protocol/BlockIoCrypto.h                           | 2 +-
> >>  MdePkg/Include/Protocol/Dns4.h                                    | 2 +-
> >>  MdePkg/Include/Protocol/Dns6.h                                    | 2 +-
> >>  MdePkg/Include/Protocol/FirmwareManagement.h                      | 2 +-
> >>  MdePkg/Include/Protocol/ImageDecoder.h                            | 2 +-
> >>  MdePkg/Include/Protocol/Mtftp4.h                                  | 4 ++--
> >>  MdePkg/Include/Protocol/Mtftp6.h                                  | 8 ++++----
> >>  MdePkg/Include/Protocol/NvmExpressPassthru.h                      | 2 +-
> >>  MdePkg/Include/Protocol/SimpleTextOut.h                           | 2 +-
> >>  MdePkg/Include/Protocol/Timer.h                                   | 2 +-
> >>  MdePkg/Include/Uefi/UefiPxe.h                                     | 2 +-
> >>  MdePkg/Include/X64/ProcessorBind.h                                | 4 ++--
> >>  MdePkg/Library/BaseMemoryLibMmx/MemLibInternals.h                 | 2 +-
> >>  .../BaseUefiDecompressLib/BaseUefiDecompressLibInternals.h        | 2 +-
> >>  MdePkg/Library/DxeIoLibCpuIo2/DxeCpuIo2LibInternal.h              | 2 +-
> >>  MdePkg/Library/SmmIoLibSmmCpuIo2/SmmCpuIoLibInternal.h            | 2 +-
> >>  27 files changed, 36 insertions(+), 36 deletions(-)
> >>
> >> diff --git a/MdePkg/Include/AArch64/ProcessorBind.h
> >> b/MdePkg/Include/AArch64/ProcessorBind.h
> >> index c5fe874..fa311e2 100644
> >> --- a/MdePkg/Include/AArch64/ProcessorBind.h
> >> +++ b/MdePkg/Include/AArch64/ProcessorBind.h
> >> @@ -32,7 +32,7 @@
> >>
> >>  #if _MSC_EXTENSIONS
> >>    //
> >> -  // use Microsoft* C complier dependent integer width types
> >> +  // use Microsoft* C compiler dependent integer width types
> >>    //
> >>    typedef unsigned __int64    UINT64;
> >>    typedef __int64             INT64;
> >> diff --git a/MdePkg/Include/Arm/ProcessorBind.h
> >> b/MdePkg/Include/Arm/ProcessorBind.h
> >> index 5ee7465..dc45b94 100644
> >> --- a/MdePkg/Include/Arm/ProcessorBind.h
> >> +++ b/MdePkg/Include/Arm/ProcessorBind.h
> >> @@ -37,7 +37,7 @@
> >>
> >>  #if _MSC_EXTENSIONS
> >>    //
> >> -  // use Microsoft* C complier dependent integer width types
> >> +  // use Microsoft* C compiler dependent integer width types
> >>    //
> >>    typedef unsigned __int64    UINT64;
> >>    typedef __int64             INT64;
> >> diff --git a/MdePkg/Include/Base.h b/MdePkg/Include/Base.h
> >> index 93f6686..c666148 100644
> >> --- a/MdePkg/Include/Base.h
> >> +++ b/MdePkg/Include/Base.h
> >> @@ -302,7 +302,7 @@ struct _LIST_ENTRY {
> >>
> >>  //
> >>  //  UEFI specification claims 1 and 0. We are concerned about the
> >> -//  complier portability so we did it this way.
> >> +//  compiler portability so we did it this way.
> >>  //
> >>
> >>  ///
> >> diff --git a/MdePkg/Include/Ia32/ProcessorBind.h
> >> b/MdePkg/Include/Ia32/ProcessorBind.h
> >> index 458f7dd..da1810b 100644
> >> --- a/MdePkg/Include/Ia32/ProcessorBind.h
> >> +++ b/MdePkg/Include/Ia32/ProcessorBind.h
> >> @@ -79,7 +79,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY
> >> KIND, EITHER EXPRESS OR IMPLIED.
> >>  #pragma warning ( disable : 4057 )
> >>
> >>  //
> >> -// ASSERT(FALSE) or while (TRUE) are legal constructes so supress this
> >> warning
> >> +// ASSERT(FALSE) or while (TRUE) are legal constructs so suppress this
> >> warning
> >>  //
> >>  #pragma warning ( disable : 4127 )
> >>
> >> @@ -119,7 +119,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF
> >> ANY KIND, EITHER EXPRESS OR IMPLIED.
> >>  #if defined(_MSC_EXTENSIONS)
> >>
> >>    //
> >> -  // use Microsoft C complier dependent integer width types
> >> +  // use Microsoft C compiler dependent integer width types
> >>    //
> >>
> >>    ///
> >> diff --git a/MdePkg/Include/Ipf/ProcessorBind.h
> >> b/MdePkg/Include/Ipf/ProcessorBind.h
> >> index cead575..c19e47d 100644
> >> --- a/MdePkg/Include/Ipf/ProcessorBind.h
> >> +++ b/MdePkg/Include/Ipf/ProcessorBind.h
> >> @@ -84,7 +84,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY
> >> KIND, EITHER EXPRESS OR IMPLIED.
> >>  #pragma warning ( disable : 4054 )
> >>
> >>  //
> >> -// ASSERT(FALSE) or while (TRUE) are legal constructes so supress this
> >> warning
> >> +// ASSERT(FALSE) or while (TRUE) are legal constructs so suppress this
> >> warning
> >>  //
> >>  #pragma warning ( disable : 4127 )
> >>
> >> diff --git a/MdePkg/Include/Library/BaseLib.h
> >> b/MdePkg/Include/Library/BaseLib.h
> >> index 7e085b9..6268e6f 100644
> >> --- a/MdePkg/Include/Library/BaseLib.h
> >> +++ b/MdePkg/Include/Library/BaseLib.h
> >> @@ -1772,7 +1772,7 @@ PathRemoveLastItem(
> >>
> >>    @param[in] Path       The pointer to the string containing the path.
> >>
> >> -  @return       Returns Path, otherwise returns NULL to indicate that an error
> >> has occured.
> >> +  @return       Returns Path, otherwise returns NULL to indicate that an error
> >> has occurred.
> >>  **/
> >>  CHAR16*
> >>  EFIAPI
> >> diff --git a/MdePkg/Include/Library/DebugPrintErrorLevelLib.h
> >> b/MdePkg/Include/Library/DebugPrintErrorLevelLib.h
> >> index 2b051f4..5a866d5 100644
> >> --- a/MdePkg/Include/Library/DebugPrintErrorLevelLib.h
> >> +++ b/MdePkg/Include/Library/DebugPrintErrorLevelLib.h
> >> @@ -31,7 +31,7 @@ GetDebugPrintErrorLevel (
> >>
> >>    @param   ErrorLevel     Global debug print error level
> >>
> >> -  @retval  TRUE           The debug print error level mask was sucessfully set.
> >> +  @retval  TRUE           The debug print error level mask was successfully set.
> >>    @retval  FALSE          The debug print error level mask could not be set.
> >>
> >>  **/
> >> diff --git a/MdePkg/Include/Library/FileHandleLib.h
> >> b/MdePkg/Include/Library/FileHandleLib.h
> >> index 00706f9..e94f83e 100644
> >> --- a/MdePkg/Include/Library/FileHandleLib.h
> >> +++ b/MdePkg/Include/Library/FileHandleLib.h
> >> @@ -184,9 +184,9 @@ FileHandleDelete (
> >>    has the effect of starting the read process of the directory entries over
> >> again.
> >>
> >>    @param[in] FileHandle         The file handle on which the position is being
> >> set.
> >> -  @param[in] Position           The byte position from the begining of the file.
> >> +  @param[in] Position           The byte position from the beginning of the file.
> >>
> >> -  @retval EFI_SUCCESS           The operation completed sucessfully.
> >> +  @retval EFI_SUCCESS           The operation completed successfully.
> >>    @retval EFI_UNSUPPORTED       The request for non-zero is not valid on
> >>                                  directories.
> >>    @retval INVALID_PARAMETER     One of the parameters has an invalid value.
> >> @@ -207,7 +207,7 @@ FileHandleSetPosition (
> >>    if FileHandle is a directory.
> >>
> >>    @param[in] FileHandle         The open file handle on which to get the
> >> position.
> >> -  @param[out] Position          The byte position from begining of file.
> >> +  @param[out] Position          The byte position from beginning of file.
> >>
> >>    @retval EFI_SUCCESS           The operation completed successfully.
> >>    @retval INVALID_PARAMETER     One of the parameters has an invalid value.
> >> @@ -322,7 +322,7 @@ FileHandleFindNextFile(
> >>    @param[in] FileHandle         The file handle from which size is retrieved.
> >>    @param[out] Size              The pointer to size.
> >>
> >> -  @retval EFI_SUCCESS           Operation was completed sucessfully.
> >> +  @retval EFI_SUCCESS           Operation was completed successfully.
> >>    @retval EFI_DEVICE_ERROR      Cannot access the file.
> >>    @retval EFI_INVALID_PARAMETER FileHandle is NULL.
> >>                                  Size is NULL.
> >> diff --git a/MdePkg/Include/Library/PeCoffLib.h
> >> b/MdePkg/Include/Library/PeCoffLib.h
> >> index 9ed6d61..5a3f988 100644
> >> --- a/MdePkg/Include/Library/PeCoffLib.h
> >> +++ b/MdePkg/Include/Library/PeCoffLib.h
> >> @@ -337,7 +337,7 @@ PeCoffLoaderImageReadFromMemory (
> >>
> >>
> >>  /**
> >> -  Reapply fixups on a fixed up PE32/PE32+ image to allow virutal calling at EFI
> >> +  Reapply fixups on a fixed up PE32/PE32+ image to allow virtual calling at EFI
> >>    runtime.
> >>
> >>    This function reapplies relocation fixups to the PE/COFF image specified by
> >> ImageBase
> >> diff --git a/MdePkg/Include/Pi/PiSmmCis.h
> >> b/MdePkg/Include/Pi/PiSmmCis.h
> >> index 6d3ddab..6cb28b7 100644
> >> --- a/MdePkg/Include/Pi/PiSmmCis.h
> >> +++ b/MdePkg/Include/Pi/PiSmmCis.h
> >> @@ -178,7 +178,7 @@ EFI_STATUS
> >>  /**
> >>    Registers a handler to execute within SMM.
> >>
> >> -  @param[in]  Handler            Handler service funtion pointer.
> >> +  @param[in]  Handler            Handler service function pointer.
> >>    @param[in]  HandlerType        Points to the handler type or NULL for root
> >> SMI handlers.
> >>    @param[out] DispatchHandle     On return, contains a unique handle which
> >> can be used to later
> >>                                   unregister the handler function.
> >> diff --git a/MdePkg/Include/Protocol/BlockIo2.h
> >> b/MdePkg/Include/Protocol/BlockIo2.h
> >> index 6f0c280..5fad107 100644
> >> --- a/MdePkg/Include/Protocol/BlockIo2.h
> >> +++ b/MdePkg/Include/Protocol/BlockIo2.h
> >> @@ -52,7 +52,7 @@ typedef struct {
> >>
> >>    @param[in]  This                 Indicates a pointer to the calling context.
> >>    @param[in]  ExtendedVerification Indicates that the driver may perform a
> >> more
> >> -                                   exhausive verfication operation of the device
> >> +                                   exhausive verification operation of the device
> >>                                     during reset.
> >>
> >>    @retval EFI_SUCCESS          The device was reset.
> >> diff --git a/MdePkg/Include/Protocol/BlockIoCrypto.h
> >> b/MdePkg/Include/Protocol/BlockIoCrypto.h
> >> index 77931e7..821b846 100644
> >> --- a/MdePkg/Include/Protocol/BlockIoCrypto.h
> >> +++ b/MdePkg/Include/Protocol/BlockIoCrypto.h
> >> @@ -172,7 +172,7 @@ typedef struct {
> >>
> >>    @param[in]  This                 Pointer to the EFI_BLOCK_IO_CRYPTO_PROTOCOL
> >> instance.
> >>    @param[in]  ExtendedVerification Indicates that the driver may perform a
> >> more exhausive
> >> -                                   verfication operation of the device during
> reset.
> >> +                                   verification operation of the device during
> reset.
> >>
> >>    @retval EFI_SUCCESS              The block device was reset.
> >>    @retval EFI_DEVICE_ERROR         The block device is not functioning
> >> correctly and could
> >> diff --git a/MdePkg/Include/Protocol/Dns4.h
> >> b/MdePkg/Include/Protocol/Dns4.h
> >> index 2489268..f96c50f 100644
> >> --- a/MdePkg/Include/Protocol/Dns4.h
> >> +++ b/MdePkg/Include/Protocol/Dns4.h
> >> @@ -310,7 +310,7 @@ EFI_STATUS
> >>
> >>    @retval EFI_SUCCESS             The operation completed successfully.
> >>    @retval EFI_UNSUPPORTED         The designated protocol is not supported.
> >> -  @retval EFI_INVALID_PARAMTER    Thisis NULL.
> >> +  @retval EFI_INVALID_PARAMETER   This is NULL.
> >>                                    The StationIp address provided in DnsConfigData
> is not a
> >>                                    valid unicast.
> >>                                    DnsServerList is NULL while DnsServerListCount
> >> diff --git a/MdePkg/Include/Protocol/Dns6.h
> >> b/MdePkg/Include/Protocol/Dns6.h
> >> index 1ea85bc..813893a 100644
> >> --- a/MdePkg/Include/Protocol/Dns6.h
> >> +++ b/MdePkg/Include/Protocol/Dns6.h
> >> @@ -300,7 +300,7 @@ EFI_STATUS
> >>                                    storage to be allocated and released by caller.
> >>
> >>    @retval EFI_SUCCESS             The operation completed successfully.
> >> -  @retval EFI_INVALID_PARAMTER    This is NULL.
> >> +  @retval EFI_INVALID_PARAMETER   This is NULL.
> >>                                    The StationIp address provided in DnsConfigData
> is not zero
> >> and not a valid unicast.
> >>                                    DnsServerList is NULL while DnsServerList Count
> is not ZERO.
> >>                                    DnsServerList Count is ZERO while DnsServerList
> is not NULL.
> >> diff --git a/MdePkg/Include/Protocol/FirmwareManagement.h
> >> b/MdePkg/Include/Protocol/FirmwareManagement.h
> >> index e615573..b199888 100644
> >> --- a/MdePkg/Include/Protocol/FirmwareManagement.h
> >> +++ b/MdePkg/Include/Protocol/FirmwareManagement.h
> >> @@ -241,7 +241,7 @@ typedef struct {
> >>  #define  PACKAGE_ATTRIBUTE_AUTHENTICATION_REQUIRED
> >> 0x0000000000000004
> >>
> >>  /**
> >> -  Callback funtion to report the process of the firmware updating.
> >> +  Callback function to report the process of the firmware updating.
> >>
> >>    @param[in]  Completion    A value between 1 and 100 indicating the current
> >> completion
> >>                              progress of the firmware update. Completion progress
> is
> >> diff --git a/MdePkg/Include/Protocol/ImageDecoder.h
> >> b/MdePkg/Include/Protocol/ImageDecoder.h
> >> index aebb813..3a7845b 100644
> >> --- a/MdePkg/Include/Protocol/ImageDecoder.h
> >> +++ b/MdePkg/Include/Protocol/ImageDecoder.h
> >> @@ -130,7 +130,7 @@ EFI_STATUS
> >>    @param This                    EFI_HII_IMAGE_DECODER_PROTOCOL instance.
> >>    @param Image                   Pointer to the image raw data.
> >>    @param SizeOfImage             Size of the entire image raw data.
> >> -  @param ImageInfo               Pointer to recieve
> >> EFI_HII_IMAGE_DECODER_IMAGE_INFO_HEADER.
> >> +  @param ImageInfo               Pointer to receive
> >> EFI_HII_IMAGE_DECODER_IMAGE_INFO_HEADER.
> >>
> >>    @retval EFI_SUCCESS            Get image info success.
> >>    @retval EFI_UNSUPPORTED        Unsupported format of image.
> >> diff --git a/MdePkg/Include/Protocol/Mtftp4.h
> >> b/MdePkg/Include/Protocol/Mtftp4.h
> >> index 1afac6c..ae2dc27 100644
> >> --- a/MdePkg/Include/Protocol/Mtftp4.h
> >> +++ b/MdePkg/Include/Protocol/Mtftp4.h
> >> @@ -1,5 +1,5 @@
> >>  /** @file
> >> -  EFI Multicast Trivial File Tranfer Protocol Definition
> >> +  EFI Multicast Trivial File Transfer Protocol Definition
> >>
> >>  Copyright (c) 2006 - 2011, Intel Corporation. All rights reserved.<BR>
> >>  This program and the accompanying materials are licensed and made
> >> available under
> >> @@ -212,7 +212,7 @@ EFI_STATUS
> >>    );
> >>
> >>  /**
> >> -  Timeout callback funtion.
> >> +  Timeout callback function.
> >>
> >>    @param  This           The pointer to the EFI_MTFTP4_PROTOCOL instance.
> >>    @param  Token          The token that is provided in the
> >> diff --git a/MdePkg/Include/Protocol/Mtftp6.h
> >> b/MdePkg/Include/Protocol/Mtftp6.h
> >> index b39575d..4667add 100644
> >> --- a/MdePkg/Include/Protocol/Mtftp6.h
> >> +++ b/MdePkg/Include/Protocol/Mtftp6.h
> >> @@ -1,5 +1,5 @@
> >>  /** @file
> >> -  UEFI Multicast Trivial File Tranfer Protocol v6 Definition, which is built upon
> >> +  UEFI Multicast Trivial File Transfer Protocol v6 Definition, which is built
> >> upon
> >>    the EFI UDPv6 Protocol and provides basic services for client-side unicast
> >> and/or
> >>    multicast TFTP operations.
> >>
> >> @@ -325,7 +325,7 @@ typedef struct {
> >>    @param[in] PacketLen     Indicates the length of the packet.
> >>    @param[in] Packet        Pointer to an MTFTPv6 packet.
> >>
> >> -  @retval EFI_SUCCESS      Operation sucess.
> >> +  @retval EFI_SUCCESS      Operation success.
> >>    @retval Others           Aborts session.
> >>
> >>  **/
> >> @@ -352,7 +352,7 @@ EFI_STATUS
> >>                             EFI_MTFTP6_PROTOCOL.WriteFile() or
> >> EFI_MTFTP6_PROTOCOL.ReadDirectory()
> >>                             functions by the caller.
> >>
> >> -  @retval EFI_SUCCESS      Operation sucess.
> >> +  @retval EFI_SUCCESS      Operation success.
> >>    @retval Others           Aborts session.
> >>
> >>  **/
> >> @@ -379,7 +379,7 @@ EFI_STATUS
> >>                             length the data available on output.
> >>    @param[out]     Buffer   Pointer to the buffer where the data is stored.
> >>
> >> -  @retval EFI_SUCCESS      Operation sucess.
> >> +  @retval EFI_SUCCESS      Operation success.
> >>    @retval Others           Aborts session.
> >>
> >>  **/
> >> diff --git a/MdePkg/Include/Protocol/NvmExpressPassthru.h
> >> b/MdePkg/Include/Protocol/NvmExpressPassthru.h
> >> index ebd7a8b..7a90c63 100644
> >> --- a/MdePkg/Include/Protocol/NvmExpressPassthru.h
> >> +++ b/MdePkg/Include/Protocol/NvmExpressPassthru.h
> >> @@ -115,7 +115,7 @@ typedef struct {
> >>  } EFI_NVM_EXPRESS_PASS_THRU_COMMAND_PACKET;
> >>
> >>  //
> >> -// Protocol funtion prototypes
> >> +// Protocol function prototypes
> >>  //
> >>  /**
> >>    Sends an NVM Express Command Packet to an NVM Express controller or
> >> namespace. This function supports
> >> diff --git a/MdePkg/Include/Protocol/SimpleTextOut.h
> >> b/MdePkg/Include/Protocol/SimpleTextOut.h
> >> index 5f64b64..c1a23fa 100644
> >> --- a/MdePkg/Include/Protocol/SimpleTextOut.h
> >> +++ b/MdePkg/Include/Protocol/SimpleTextOut.h
> >> @@ -160,7 +160,7 @@ typedef EFI_SIMPLE_TEXT_OUTPUT_PROTOCOL
> >> SIMPLE_TEXT_OUTPUT_INTERFACE;
> >>    Reset the text output device hardware and optionaly run diagnostics
> >>
> >>    @param  This                 The protocol instance pointer.
> >> -  @param  ExtendedVerification Driver may perform more exhaustive
> >> verfication
> >> +  @param  ExtendedVerification Driver may perform more exhaustive
> >> verification
> >>                                 operation of the device during reset.
> >>
> >>    @retval EFI_SUCCESS          The text output device was reset.
> >> diff --git a/MdePkg/Include/Protocol/Timer.h
> >> b/MdePkg/Include/Protocol/Timer.h
> >> index c9a998e..ffdb0a1 100644
> >> --- a/MdePkg/Include/Protocol/Timer.h
> >> +++ b/MdePkg/Include/Protocol/Timer.h
> >> @@ -30,7 +30,7 @@ typedef struct _EFI_TIMER_ARCH_PROTOCOL
> >> EFI_TIMER_ARCH_PROTOCOL;
> >>
> >>  /**
> >>    This function of this type is called when a timer interrupt fires.  This
> >> -  function executes at TPL_HIGH_LEVEL.  The DXE Core will register a funtion
> >> +  function executes at TPL_HIGH_LEVEL.  The DXE Core will register a
> >> function
> >>    of this type to be called for the timer interrupt, so it can know how much
> >>    time has passed.  This information is used to signal timer based events.
> >>
> >> diff --git a/MdePkg/Include/Uefi/UefiPxe.h
> >> b/MdePkg/Include/Uefi/UefiPxe.h
> >> index 0af9b90..249a8c2 100644
> >> --- a/MdePkg/Include/Uefi/UefiPxe.h
> >> +++ b/MdePkg/Include/Uefi/UefiPxe.h
> >> @@ -1079,7 +1079,7 @@ typedef struct s_pxe_cpb_start_31 {
> >>
> >>    ///
> >>    /// protocol driver can provide anything for this Unique_ID, UNDI
> >> remembers
> >> -  /// that as just a 64bit value assocaited to the interface specified by
> >> +  /// that as just a 64bit value associated to the interface specified by
> >>    /// the ifnum and gives it back as a parameter to all the call-back routines
> >>    /// when calling for that interface!
> >>    ///
> >> diff --git a/MdePkg/Include/X64/ProcessorBind.h
> >> b/MdePkg/Include/X64/ProcessorBind.h
> >> index ffcf2b0..fcbe28a 100644
> >> --- a/MdePkg/Include/X64/ProcessorBind.h
> >> +++ b/MdePkg/Include/X64/ProcessorBind.h
> >> @@ -93,7 +93,7 @@
> >>  #pragma warning ( disable : 4057 )
> >>
> >>  //
> >> -// ASSERT(FALSE) or while (TRUE) are legal constructes so supress this
> >> warning
> >> +// ASSERT(FALSE) or while (TRUE) are legal constructs so suppress this
> >> warning
> >>  //
> >>  #pragma warning ( disable : 4127 )
> >>
> >> @@ -132,7 +132,7 @@
> >>
> >>  #if defined(_MSC_EXTENSIONS)
> >>    //
> >> -  // use Microsoft C complier dependent integer width types
> >> +  // use Microsoft C compiler dependent integer width types
> >>    //
> >>
> >>    ///
> >> diff --git a/MdePkg/Library/BaseMemoryLibMmx/MemLibInternals.h
> >> b/MdePkg/Library/BaseMemoryLibMmx/MemLibInternals.h
> >> index cc89979..9505265 100644
> >> --- a/MdePkg/Library/BaseMemoryLibMmx/MemLibInternals.h
> >> +++ b/MdePkg/Library/BaseMemoryLibMmx/MemLibInternals.h
> >> @@ -103,7 +103,7 @@ InternalMemSetMem32 (
> >>  /**
> >>    Fills a target buffer with a 64-bit value, and returns the target buffer.
> >>
> >> -  @param  Buffer  Teh pointer to the target buffer to fill.
> >> +  @param  Buffer  The pointer to the target buffer to fill.
> >>    @param  Length  The count of 64-bit value to fill.
> >>    @param  Value   The value with which to fill Length bytes of Buffer.
> >>
> >> diff --git
> >> a/MdePkg/Library/BaseUefiDecompressLib/BaseUefiDecompressLibInternal
> >> s.h
> >> b/MdePkg/Library/BaseUefiDecompressLib/BaseUefiDecompressLibInternal
> >> s.h
> >> index 358c598..aba510c 100644
> >> ---
> >> a/MdePkg/Library/BaseUefiDecompressLib/BaseUefiDecompressLibInternal
> >> s.h
> >> +++
> >> b/MdePkg/Library/BaseUefiDecompressLib/BaseUefiDecompressLibInternal
> >> s.h
> >> @@ -1,5 +1,5 @@
> >>  /** @file
> >> -  Internal data structure defintions for Base UEFI Decompress Libary.
> >> +  Internal data structure defintions for Base UEFI Decompress Library.
> >>
> >>    Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.<BR>
> >>    This program and the accompanying materials
> >> diff --git a/MdePkg/Library/DxeIoLibCpuIo2/DxeCpuIo2LibInternal.h
> >> b/MdePkg/Library/DxeIoLibCpuIo2/DxeCpuIo2LibInternal.h
> >> index b459214..160bedf 100644
> >> --- a/MdePkg/Library/DxeIoLibCpuIo2/DxeCpuIo2LibInternal.h
> >> +++ b/MdePkg/Library/DxeIoLibCpuIo2/DxeCpuIo2LibInternal.h
> >> @@ -58,7 +58,7 @@ IoReadWorker (
> >>    @param  Width         The width of the I/O operation.
> >>    @param  Data          The value to write to the I/O port.
> >>
> >> -  @return The paramter of Data.
> >> +  @return The parameter of Data.
> >>
> >>  **/
> >>  UINT64
> >> diff --git a/MdePkg/Library/SmmIoLibSmmCpuIo2/SmmCpuIoLibInternal.h
> >> b/MdePkg/Library/SmmIoLibSmmCpuIo2/SmmCpuIoLibInternal.h
> >> index 0e94b20..ced6358 100644
> >> --- a/MdePkg/Library/SmmIoLibSmmCpuIo2/SmmCpuIoLibInternal.h
> >> +++ b/MdePkg/Library/SmmIoLibSmmCpuIo2/SmmCpuIoLibInternal.h
> >> @@ -60,7 +60,7 @@ IoReadWorker (
> >>    @param  Width         The width of the I/O operation.
> >>    @param  Data          The value to write to the I/O port.
> >>
> >> -  @return The paramter of Data.
> >> +  @return The parameter of Data.
> >>
> >>  **/
> >>  UINT64
> >> --
> >> 1.8.3.1
> >


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2016-10-08  1:16 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-12  8:35 [PATCH] MdePkg: Fix some typing errors in the header files Thomas Huth
2016-09-12  9:09 ` Gao, Liming
2016-10-08  1:16   ` Kinney, Michael D
     [not found]   ` <60e78d8e-c7dc-ee87-df2c-69ddfd4ef202@redhat.com>
2016-10-08  1:16     ` Kinney, Michael D

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox