Re: [edk2] [PATCH] PcAtChipsetPkg: Fix typing errors

2017-01-19 Thread Zeng, Star 
Pushed at 
https://github.com/tianocore/edk2/commit/619ad10f3af2774880867dd638a8016d3181b516,
 thanks for the contribution.

Star
-Original Message-
From: Zeng, Star 
Sent: Thursday, January 19, 2017 9:43 AM
To: Thomas Huth <th...@redhat.com>; Ni, Ruiyu <ruiyu...@intel.com>; 
edk2-de...@ml01.01.org
Cc: Zeng, Star <star.z...@intel.com>
Subject: RE: [edk2] [PATCH] PcAtChipsetPkg: Fix typing errors

Reviewed-by: Star Zeng <star.z...@intel.com>

I can help push the patch at tomorrow if no other comments are received.

Thanks,
Star
-Original Message-
From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Thomas 
Huth
Sent: Wednesday, January 18, 2017 9:40 PM
To: Ni, Ruiyu <ruiyu...@intel.com>; edk2-de...@ml01.01.org
Subject: [edk2] [PATCH] PcAtChipsetPkg: Fix typing errors

Correct the reported by the codespell utility in some files of PcAtChipsetPkg.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Thomas Huth <th...@redhat.com>
---
 PcAtChipsetPkg/8254TimerDxe/Timer.c |  6 +++---
 PcAtChipsetPkg/8254TimerDxe/Timer.h |  2 +-
 PcAtChipsetPkg/8259InterruptControllerDxe/8259.c|  4 ++--
 PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/IdeController.c |  4 ++--  
PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/IdeController.h |  4 ++--
 PcAtChipsetPkg/HpetTimerDxe/HpetTimer.c |  6 +++---
 PcAtChipsetPkg/IsaAcpiDxe/IsaAcpi.c | 14 +++---
 PcAtChipsetPkg/IsaAcpiDxe/PcatIsaAcpi.c |  2 +-
 PcAtChipsetPkg/IsaAcpiDxe/PcatIsaAcpi.h | 12 ++--
 PcAtChipsetPkg/Library/SerialIoLib/SerialPortLib.c  |  2 +-
 10 files changed, 28 insertions(+), 28 deletions(-)

diff --git a/PcAtChipsetPkg/8254TimerDxe/Timer.c 
b/PcAtChipsetPkg/8254TimerDxe/Timer.c
index fd8b2ec..2cd888a 100644
--- a/PcAtChipsetPkg/8254TimerDxe/Timer.c
+++ b/PcAtChipsetPkg/8254TimerDxe/Timer.c
@@ -71,8 +71,8 @@ SetPitCount (
 /**
   8254 Timer #0 Interrupt Handler.
 
-  @param InterruptTypeThe type of interrupt that occured
-  @param SystemContextA pointer to the system context when the interrupt 
occured
+  @param InterruptTypeThe type of interrupt that occurred
+  @param SystemContextA pointer to the system context when the interrupt 
occurred
 **/
 VOID
 EFIAPI
@@ -332,7 +332,7 @@ TimerDriverGenerateSoftInterrupt (
 
   @retval EFI_SUCCESSTimer Architectural Protocol created
   @retval EFI_OUT_OF_RESOURCES   Not enough resources available to initialize 
driver.
-  @retval EFI_DEVICE_ERROR   A device error occured attempting to 
initialize the driver.
+  @retval EFI_DEVICE_ERROR   A device error occurred attempting to 
initialize the driver.
 
 **/
 EFI_STATUS
diff --git a/PcAtChipsetPkg/8254TimerDxe/Timer.h 
b/PcAtChipsetPkg/8254TimerDxe/Timer.h
index c2c29bb..b2c8afb 100644
--- a/PcAtChipsetPkg/8254TimerDxe/Timer.h
+++ b/PcAtChipsetPkg/8254TimerDxe/Timer.h
@@ -57,7 +57,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER 
EXPRESS OR IMPLIED.
 
   @retval EFI_SUCCESSTimer Architectural Protocol created
   @retval EFI_OUT_OF_RESOURCES   Not enough resources available to initialize 
driver.
-  @retval EFI_DEVICE_ERROR   A device error occured attempting to 
initialize the driver.
+  @retval EFI_DEVICE_ERROR   A device error occurred attempting to 
initialize the driver.
 
 **/
 EFI_STATUS
diff --git a/PcAtChipsetPkg/8259InterruptControllerDxe/8259.c 
b/PcAtChipsetPkg/8259InterruptControllerDxe/8259.c
index e73a100..7d8265d 100644
--- a/PcAtChipsetPkg/8259InterruptControllerDxe/8259.c
+++ b/PcAtChipsetPkg/8259InterruptControllerDxe/8259.c
@@ -139,7 +139,7 @@ Interrupt8259SetVectorBase (
 
 //
 // Preserve interrtup mask register before initialization sequence
-// because it will be cleared during intialization
+// because it will be cleared during initialization
 //
 Mask = IoRead8 (LEGACY_8259_MASK_REGISTER_SLAVE);
 
@@ -181,7 +181,7 @@ Interrupt8259SetVectorBase (
 
 //
 // Preserve interrtup mask register before initialization sequence
-// because it will be cleared during intialization
+// because it will be cleared during initialization
 //
 Mask = IoRead8 (LEGACY_8259_MASK_REGISTER_MASTER);
 
diff --git a/PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/IdeController.c 
b/PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/IdeController.c
index 21b8e26..ebe28e9 100644
--- a/PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/IdeController.c
+++ b/PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/IdeController.c
@@ -110,7 +110,7 @@ InitializeIdeControllerDriver (
 driver but used by bus driver
 
   @retval EFI_SUCCESS   Driver loaded.
-  @retval !EFI_SUCESS   Driver not loaded.
+  @retval !EFI_SUCCESS  Driver not loaded.
 **/
 EFI_STATUS
 EFIAPI
@@ -249,7 +249,7 @@ IdeControllerStart (
   @param NumberOfChildren   Not used

Re: [edk2] [PATCH] PcAtChipsetPkg: Fix typing errors

2017-01-18 Thread Zeng, Star 
Reviewed-by: Star Zeng 

I can help push the patch at tomorrow if no other comments are received.

Thanks,
Star
-Original Message-
From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Thomas 
Huth
Sent: Wednesday, January 18, 2017 9:40 PM
To: Ni, Ruiyu ; edk2-de...@ml01.01.org
Subject: [edk2] [PATCH] PcAtChipsetPkg: Fix typing errors

Correct the reported by the codespell utility in some files of PcAtChipsetPkg.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Thomas Huth 
---
 PcAtChipsetPkg/8254TimerDxe/Timer.c |  6 +++---
 PcAtChipsetPkg/8254TimerDxe/Timer.h |  2 +-
 PcAtChipsetPkg/8259InterruptControllerDxe/8259.c|  4 ++--
 PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/IdeController.c |  4 ++--  
PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/IdeController.h |  4 ++--
 PcAtChipsetPkg/HpetTimerDxe/HpetTimer.c |  6 +++---
 PcAtChipsetPkg/IsaAcpiDxe/IsaAcpi.c | 14 +++---
 PcAtChipsetPkg/IsaAcpiDxe/PcatIsaAcpi.c |  2 +-
 PcAtChipsetPkg/IsaAcpiDxe/PcatIsaAcpi.h | 12 ++--
 PcAtChipsetPkg/Library/SerialIoLib/SerialPortLib.c  |  2 +-
 10 files changed, 28 insertions(+), 28 deletions(-)

diff --git a/PcAtChipsetPkg/8254TimerDxe/Timer.c 
b/PcAtChipsetPkg/8254TimerDxe/Timer.c
index fd8b2ec..2cd888a 100644
--- a/PcAtChipsetPkg/8254TimerDxe/Timer.c
+++ b/PcAtChipsetPkg/8254TimerDxe/Timer.c
@@ -71,8 +71,8 @@ SetPitCount (
 /**
   8254 Timer #0 Interrupt Handler.
 
-  @param InterruptTypeThe type of interrupt that occured
-  @param SystemContextA pointer to the system context when the interrupt 
occured
+  @param InterruptTypeThe type of interrupt that occurred
+  @param SystemContextA pointer to the system context when the interrupt 
occurred
 **/
 VOID
 EFIAPI
@@ -332,7 +332,7 @@ TimerDriverGenerateSoftInterrupt (
 
   @retval EFI_SUCCESSTimer Architectural Protocol created
   @retval EFI_OUT_OF_RESOURCES   Not enough resources available to initialize 
driver.
-  @retval EFI_DEVICE_ERROR   A device error occured attempting to 
initialize the driver.
+  @retval EFI_DEVICE_ERROR   A device error occurred attempting to 
initialize the driver.
 
 **/
 EFI_STATUS
diff --git a/PcAtChipsetPkg/8254TimerDxe/Timer.h 
b/PcAtChipsetPkg/8254TimerDxe/Timer.h
index c2c29bb..b2c8afb 100644
--- a/PcAtChipsetPkg/8254TimerDxe/Timer.h
+++ b/PcAtChipsetPkg/8254TimerDxe/Timer.h
@@ -57,7 +57,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER 
EXPRESS OR IMPLIED.
 
   @retval EFI_SUCCESSTimer Architectural Protocol created
   @retval EFI_OUT_OF_RESOURCES   Not enough resources available to initialize 
driver.
-  @retval EFI_DEVICE_ERROR   A device error occured attempting to 
initialize the driver.
+  @retval EFI_DEVICE_ERROR   A device error occurred attempting to 
initialize the driver.
 
 **/
 EFI_STATUS
diff --git a/PcAtChipsetPkg/8259InterruptControllerDxe/8259.c 
b/PcAtChipsetPkg/8259InterruptControllerDxe/8259.c
index e73a100..7d8265d 100644
--- a/PcAtChipsetPkg/8259InterruptControllerDxe/8259.c
+++ b/PcAtChipsetPkg/8259InterruptControllerDxe/8259.c
@@ -139,7 +139,7 @@ Interrupt8259SetVectorBase (
 
 //
 // Preserve interrtup mask register before initialization sequence
-// because it will be cleared during intialization
+// because it will be cleared during initialization
 //
 Mask = IoRead8 (LEGACY_8259_MASK_REGISTER_SLAVE);
 
@@ -181,7 +181,7 @@ Interrupt8259SetVectorBase (
 
 //
 // Preserve interrtup mask register before initialization sequence
-// because it will be cleared during intialization
+// because it will be cleared during initialization
 //
 Mask = IoRead8 (LEGACY_8259_MASK_REGISTER_MASTER);
 
diff --git a/PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/IdeController.c 
b/PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/IdeController.c
index 21b8e26..ebe28e9 100644
--- a/PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/IdeController.c
+++ b/PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/IdeController.c
@@ -110,7 +110,7 @@ InitializeIdeControllerDriver (
 driver but used by bus driver
 
   @retval EFI_SUCCESS   Driver loaded.
-  @retval !EFI_SUCESS   Driver not loaded.
+  @retval !EFI_SUCCESS  Driver not loaded.
 **/
 EFI_STATUS
 EFIAPI
@@ -249,7 +249,7 @@ IdeControllerStart (
   @param NumberOfChildren   Not used
   @param ChildHandleBuffer  Not used
 
-  @retval EFI_SUCESSThis driver is removed DeviceHandle
+  @retval EFI_SUCCESS   This driver is removed DeviceHandle
   @retval !EFI_SUCCESS  This driver was not removed from this device
 **/
 EFI_STATUS
diff --git a/PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/IdeController.h 
b/PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/IdeController.h
index 3ecde08..a994b9d 100644
---