1 files changed, 15 insertions, 32 deletions
diff --git a/IntelFrameworkModulePkg/Bus/Isa/Ps2MouseAbsolutePointerDxe/CommPs2.c b/IntelFrameworkModulePkg/Bus/Isa/Ps2MouseAbsolutePointerDxe/CommPs2.c
index 60614fb9e1..b5d16792c4 100644
--- a/IntelFrameworkModulePkg/Bus/Isa/Ps2MouseAbsolutePointerDxe/CommPs2.c
+++ b/IntelFrameworkModulePkg/Bus/Isa/Ps2MouseAbsolutePointerDxe/CommPs2.c
@@ -1,5 +1,5 @@
-/**@file
-  PS2 Mouse Communication Interface 
+/** @file
+  PS2 Mouse Communication Interface.
   
 Copyright (c) 2006 - 2007, Intel Corporation
 All rights reserved. This program and the accompanying materials
@@ -341,7 +341,7 @@ PS2MouseEnable (
 /**
   Get mouse packet . Only care first 3 bytes
 
-  @param MouseDev  Pointer of PS2 Mouse Private Data Structure 
+  @param MouseAbsolutePointerDev  Pointer to PS2 Absolute Pointer Simulation Device Private Data Structure 
 
   @retval EFI_NOT_READY  Mouse Device not ready to input data packet, or some error happened during getting the packet
   @retval EFI_SUCCESS    The data packet is gotten successfully.
@@ -589,7 +589,7 @@ Out8042Data (
   )
 {
   EFI_STATUS  Status;
-  UINT8       temp;
+  UINT8       Temp;
   //
   // Wait keyboard controller input buffer empty
   //
@@ -598,8 +598,8 @@ Out8042Data (
     return Status;
   }
 
-  temp = Data;
-  IsaIo->Io.Write (IsaIo, EfiIsaIoWidthUint8, KBC_DATA_PORT, 1, &temp);
+  Temp = Data;
+  IsaIo->Io.Write (IsaIo, EfiIsaIoWidthUint8, KBC_DATA_PORT, 1, &Temp);
 
   Status = WaitInputEmpty (IsaIo, TIMEOUT);
   if (EFI_ERROR (Status)) {
@@ -625,23 +625,23 @@ In8042Data (
   )
 {
   UINTN Delay;
-  UINT8 temp;
+  UINT8 Temp;
 
   Delay = TIMEOUT / 50;
 
   do {
-    IsaIo->Io.Read (IsaIo, EfiIsaIoWidthUint8, KBC_CMD_STS_PORT, 1, &temp);
+    IsaIo->Io.Read (IsaIo, EfiIsaIoWidthUint8, KBC_CMD_STS_PORT, 1, &Temp);
 
     //
     // Check keyboard controller status bit 0(output buffer status)
     //
-    if ((temp & KBC_OUTB) == KBC_OUTB) {
+    if ((Temp & KBC_OUTB) == KBC_OUTB) {
       break;
     }
 
     gBS->Stall (50);
     Delay--;
-  } while (Delay);
+  } while (Delay != 0);
 
   if (Delay == 0) {
     return EFI_TIMEOUT;
@@ -738,9 +738,9 @@ Out8042AuxCommand (
   I/O work flow of outing 8042 Aux data.
   
   @param IsaIo   Pointer to instance of EFI_ISA_IO_PROTOCOL
-  @param Command Aux I/O command
+  @param Data    Buffer holding return value  
   
-  @retval EFI_SUCCESS Success to excute I/O work flow
+  @retval EFI_SUCCESS Success to excute I/O work flow.
   @retval EFI_TIMEOUT Keyboard controller time out.
 **/
 EFI_STATUS
@@ -812,7 +812,7 @@ In8042AuxData (
 
 
 /**
-  Check keyboard controller status, if it is output buffer full and for auxiliary device
+  Check keyboard controller status, if it is output buffer full and for auxiliary device.
   
   @param IsaIo   Pointer to instance of EFI_ISA_IO_PROTOCOL
   
@@ -870,7 +870,7 @@ WaitInputEmpty (
 
     gBS->Stall (50);
     Delay--;
-  } while (Delay);
+  } while (Delay != 0);
 
   if (Delay == 0) {
     return EFI_TIMEOUT;
@@ -893,23 +893,6 @@ WaitOutputFull (
   IN EFI_ISA_IO_PROTOCOL                  *IsaIo,
   IN UINTN                                Timeout
   )
-/**
-
-Routine Description:
-
-  GC_TODO: Add function description
-
-Arguments:
-
-  IsaIo   - GC_TODO: add argument description
-  Timeout - GC_TODO: add argument description
-
-Returns:
-
-  EFI_TIMEOUT - GC_TODO: Add description for return value
-  EFI_SUCCESS - GC_TODO: Add description for return value
-
-**/
 {
   UINTN Delay;
   UINT8 Data;
@@ -929,7 +912,7 @@ Returns:
 
     gBS->Stall (50);
     Delay--;
-  } while (Delay);
+  } while (Delay != 0);
 
   if (Delay == 0) {
     return EFI_TIMEOUT;