Refine the comments.
authorklu2 <klu2@6f19259b-4bc3-4df7-8a09-765794883524>
Fri, 19 Jun 2009 06:52:51 +0000 (06:52 +0000)
committerklu2 <klu2@6f19259b-4bc3-4df7-8a09-765794883524>
Fri, 19 Jun 2009 06:52:51 +0000 (06:52 +0000)
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@8605 6f19259b-4bc3-4df7-8a09-765794883524

IntelFrameworkModulePkg/Bus/Isa/Ps2KeyboardDxe/ComponentName.c
IntelFrameworkModulePkg/Bus/Isa/Ps2KeyboardDxe/Ps2KbdCtrller.c
IntelFrameworkModulePkg/Bus/Isa/Ps2KeyboardDxe/Ps2KbdTextIn.c
IntelFrameworkModulePkg/Bus/Isa/Ps2KeyboardDxe/Ps2Keyboard.c
IntelFrameworkModulePkg/Bus/Isa/Ps2KeyboardDxe/Ps2Keyboard.h
IntelFrameworkModulePkg/Bus/Isa/Ps2KeyboardDxe/Ps2keyboardDxe.inf

index c20a4350a0b8702ec05b89196dc408e122509400..89662a7ab68c9b792a39a6f52386ea0be3ab2708 100644 (file)
@@ -1,6 +1,7 @@
-/**@file\r
-\r
-Copyright (c) 2006 - 2007, Intel Corporation\r
+/** @file\r
+  Routines related Component Name protocol.\r
+  \r
+Copyright (c) 2006 - 2009, Intel Corporation\r
 All rights reserved. This program and the accompanying materials\r
 are licensed and made available under the terms and conditions of the BSD License\r
 which accompanies this distribution.  The full text of the license may be found at\r
index eb03c0c836614f90940c8660683f8acb639d0e18..8a1fa777f5e117f2cd78d1ca7bb755e46248ac2b 100644 (file)
@@ -1,4 +1,4 @@
-/**@file\r
+/** @file\r
   Routines that access 8042 keyboard controller\r
 \r
 Copyright (c) 2006 - 2007, Intel Corporation\r
@@ -567,7 +567,7 @@ UINTN  mWaitForValueTimeOut = KEYBOARD_WAITFORVALUE_TIMEOUT;
 BOOLEAN          mEnableMouseInterface;\r
 \r
 /**\r
-  Read data register \r
+  Read data register .\r
 \r
   @param ConsoleIn Pointer to instance of KEYBOARD_CONSOLE_IN_DEV\r
 \r
@@ -600,7 +600,7 @@ KeyReadDataRegister (
 }\r
 \r
 /**\r
-  Write data register\r
+  Write data register.\r
 \r
   @param ConsoleIn Pointer to instance of KEYBOARD_CONSOLE_IN_DEV\r
   @param Data      value wanted to be written\r
@@ -631,7 +631,7 @@ KeyWriteDataRegister (
 }\r
 \r
 /**\r
-  Read status register\r
+  Read status register.\r
 \r
   @param ConsoleIn  Pointer to instance of KEYBOARD_CONSOLE_IN_DEV\r
 \r
@@ -664,13 +664,12 @@ KeyReadStatusRegister (
 }\r
 \r
 /**\r
-  Write command register \r
+  Write command register .\r
 \r
   @param ConsoleIn Pointer to instance of KEYBOARD_CONSOLE_IN_DEV\r
   @param Data      The value wanted to be written\r
 \r
 **/\r
-\r
 VOID\r
 KeyWriteCommandRegister (\r
   IN KEYBOARD_CONSOLE_IN_DEV *ConsoleIn,\r
@@ -695,7 +694,7 @@ KeyWriteCommandRegister (
 }\r
 \r
 /**\r
-  Display error message\r
+  Display error message.\r
 \r
   @param ConsoleIn Pointer to instance of KEYBOARD_CONSOLE_IN_DEV\r
   @param ErrMsg    Unicode string of error message\r
@@ -826,8 +825,9 @@ KeyboardTimerHandler (
   This function is called to see if there are enough bytes of scancode\r
   representing a single key.\r
 \r
-  @param Count - Number of bytes to be read\r
-  @param Buf - Store the results\r
+  @param ConsoleIn  Pointer to instance of KEYBOARD_CONSOLE_IN_DEV\r
+  @param Count      Number of bytes to be read\r
+  @param Buf        Store the results\r
 \r
   @retval EFI_SUCCESS success to scan the keyboard code\r
   @retval EFI_NOT_READY invalid parameter\r
@@ -878,8 +878,9 @@ GetScancodeBufHead (
   Read & remove several bytes from the scancode buffer.\r
   This function is usually called after GetScancodeBufHead()\r
 \r
-  @param Count - Number of bytes to be read\r
-  @param Buf - Store the results\r
+  @param ConsoleIn  Pointer to instance of KEYBOARD_CONSOLE_IN_DEV\r
+  @param Count      Number of bytes to be read\r
+  @param Buf        Store the results\r
 \r
   @retval EFI_SUCCESS success to scan the keyboard code\r
   @retval EFI_NOT_READY invalid parameter\r
@@ -927,7 +928,7 @@ PopScancodeBufHead (
 }\r
 \r
 /**\r
-  Read key value \r
+  Read key value .\r
 \r
   @param ConsoleIn - Pointer to instance of KEYBOARD_CONSOLE_IN_DEV\r
   @param Data      - Pointer to outof buffer for keeping key value\r
@@ -962,7 +963,7 @@ KeyboardRead (
     MicroSecondDelay (30);\r
   }\r
 \r
-  if (!RegFilled) {\r
+  if (RegFilled == 0) {\r
     return EFI_TIMEOUT;\r
   }\r
 \r
@@ -995,7 +996,7 @@ KeyboardWrite (
   // wait for input buffer empty\r
   //\r
   for (TimeOut = 0; TimeOut < KEYBOARD_TIMEOUT; TimeOut += 30) {\r
-    if (!(KeyReadStatusRegister (ConsoleIn) & 0x02)) {\r
+    if ((KeyReadStatusRegister (ConsoleIn) & 0x02) == 0) {\r
       RegEmptied = 1;\r
       break;\r
     }\r
@@ -1003,7 +1004,7 @@ KeyboardWrite (
     MicroSecondDelay (30);\r
   }\r
 \r
-  if (!RegEmptied) {\r
+  if (RegEmptied == 0) {\r
     return EFI_TIMEOUT;\r
   }\r
   //\r
@@ -1015,7 +1016,7 @@ KeyboardWrite (
 }\r
 \r
 /**\r
-  Issue keyboard command\r
+  Issue keyboard command.\r
 \r
   @param ConsoleIn Pointer to instance of KEYBOARD_CONSOLE_IN_DEV\r
   @param Data      The buff holding the command \r
@@ -1040,7 +1041,7 @@ KeyboardCommand (
   // Wait For Input Buffer Empty\r
   //\r
   for (TimeOut = 0; TimeOut < KEYBOARD_TIMEOUT; TimeOut += 30) {\r
-    if (!(KeyReadStatusRegister (ConsoleIn) & 0x02)) {\r
+    if ((KeyReadStatusRegister (ConsoleIn) & 0x02) == 0) {\r
       RegEmptied = 1;\r
       break;\r
     }\r
@@ -1048,7 +1049,7 @@ KeyboardCommand (
     MicroSecondDelay (30);\r
   }\r
 \r
-  if (!RegEmptied) {\r
+  if (RegEmptied == 0) {\r
     return EFI_TIMEOUT;\r
   }\r
   //\r
@@ -1061,7 +1062,7 @@ KeyboardCommand (
   //\r
   RegEmptied = 0;\r
   for (TimeOut = 0; TimeOut < KEYBOARD_TIMEOUT; TimeOut += 30) {\r
-    if (!(KeyReadStatusRegister (ConsoleIn) & 0x02)) {\r
+    if ((KeyReadStatusRegister (ConsoleIn) & 0x02) == 0) {\r
       RegEmptied = 1;\r
       break;\r
     }\r
@@ -1069,7 +1070,7 @@ KeyboardCommand (
     MicroSecondDelay (30);\r
   }\r
 \r
-  if (!RegEmptied) {\r
+  if (RegEmptied == 0) {\r
     return EFI_TIMEOUT;\r
   }\r
 \r
@@ -1519,10 +1520,10 @@ KeyGetchar (
   // Save the Shift/Toggle state\r
   //\r
   if (ConsoleIn->Ctrl) {\r
-    ConsoleIn->KeyState.KeyShiftState  |= (Extended == TRUE) ? EFI_RIGHT_CONTROL_PRESSED : EFI_LEFT_CONTROL_PRESSED;\r
+    ConsoleIn->KeyState.KeyShiftState  |= (Extended) ? EFI_RIGHT_CONTROL_PRESSED : EFI_LEFT_CONTROL_PRESSED;\r
   }                                    \r
   if (ConsoleIn->Alt) {                \r
-    ConsoleIn->KeyState.KeyShiftState  |= (Extended == TRUE) ? EFI_RIGHT_ALT_PRESSED : EFI_LEFT_ALT_PRESSED;\r
+    ConsoleIn->KeyState.KeyShiftState  |= (Extended) ? EFI_RIGHT_ALT_PRESSED : EFI_LEFT_ALT_PRESSED;\r
   }                                    \r
   if (ConsoleIn->LeftShift) {          \r
     ConsoleIn->KeyState.KeyShiftState  |= EFI_LEFT_SHIFT_PRESSED;\r
@@ -1556,7 +1557,7 @@ KeyGetchar (
 }\r
 \r
 /**\r
-  Perform 8042 controller and keyboard Initialization\r
+  Perform 8042 controller and keyboard Initialization.  \r
   If ExtendedVerification is TRUE, do additional test for\r
   the keyboard interface\r
 \r
@@ -1624,7 +1625,7 @@ InitKeyboard (
   // Test the system flag in to determine whether this is the first\r
   // time initialization\r
   //\r
-  if ((KeyReadStatusRegister (ConsoleIn) & KEYBOARD_STATUS_REGISTER_SYSTEM_FLAG)) {\r
+  if ((KeyReadStatusRegister (ConsoleIn) & KEYBOARD_STATUS_REGISTER_SYSTEM_FLAG) != 0) {\r
     //\r
     // 8042 controller is already setup (by myself or by mouse driver):\r
     //   See whether mouse interface is already enabled\r
@@ -1647,7 +1648,7 @@ InitKeyboard (
     //\r
     // Test the mouse enabling bit\r
     //\r
-    if (CommandByte & 0x20) {\r
+    if ((CommandByte & 0x20) != 0) {\r
       mEnableMouseInterface = FALSE;\r
     } else {\r
       mEnableMouseInterface = TRUE;\r
@@ -1901,9 +1902,9 @@ Done:
 }\r
 \r
 /**\r
-  Disable the keyboard interface of the 8042 controller\r
+  Disable the keyboard interface of the 8042 controller.\r
 \r
-  @param ConsoleIn   - the device instance\r
+  @param ConsoleIn   The device instance\r
 \r
   @return status of issuing disable command\r
 \r
index 49245c6496834a94f35615f954e4e41292df78c4..ee615338a58b9212c9846ae77a01d87d1b3d4ad9 100644 (file)
@@ -1,8 +1,8 @@
-/**@file\r
+/** @file\r
   Routines implements SIMPLE_TEXT_IN protocol's interfaces based on 8042 interfaces\r
   provided by Ps2KbdCtrller.c.\r
 \r
-Copyright (c) 2006 - 2007, Intel Corporation\r
+Copyright (c) 2006 - 2009, Intel Corporation\r
 All rights reserved. This program and the accompanying materials\r
 are licensed and made available under the terms and conditions of the BSD License\r
 which accompanies this distribution.  The full text of the license may be found at\r
@@ -17,7 +17,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #include "Ps2Keyboard.h"\r
 \r
 /**\r
-  Check keyboard for given key value\r
+  Check keyboard for given key value.\r
   \r
   @param  This  Point to instance of EFI_SIMPLE_TEXT_INPUT_PROTOCOL\r
   \r
@@ -248,7 +248,7 @@ KeyboardEfiReset (
   //\r
   // Report the status If keyboard is locked\r
   //\r
-  if (!(KeyReadStatusRegister (ConsoleIn) & 0x10)) {\r
+  if ((KeyReadStatusRegister (ConsoleIn) & 0x10) == 0) {\r
     REPORT_STATUS_CODE_WITH_DEVICE_PATH (\r
       EFI_ERROR_CODE | EFI_ERROR_MINOR,\r
       EFI_PERIPHERAL_KEYBOARD | EFI_P_KEYBOARD_EC_LOCKED,\r
index 060a172ddf9c359bfd0add1f57bd52574c29d03b..25aadfe481c38c5a3c78c89d3bfd51f786e32539 100644 (file)
@@ -1,9 +1,9 @@
-/**@file\r
+/** @file\r
 \r
   PS/2 Keyboard driver. Routines that interacts with callers,\r
   conforming to EFI driver model\r
 \r
-Copyright (c) 2006 - 2007, Intel Corporation\r
+Copyright (c) 2006 - 2009, Intel Corporation\r
 All rights reserved. This program and the accompanying materials\r
 are licensed and made available under the terms and conditions of the BSD License\r
 which accompanies this distribution.  The full text of the license may be found at\r
@@ -19,6 +19,16 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 //\r
 // Function prototypes\r
 //\r
+/**\r
+  Test controller is a keyboard Controller.\r
+  \r
+  @param This                 Pointer of EFI_DRIVER_BINDING_PROTOCOL\r
+  @param Controller           driver's controller\r
+  @param RemainingDevicePath  children device path\r
+  \r
+  @retval EFI_UNSUPPORTED controller is not floppy disk\r
+  @retval EFI_SUCCESS     controller is floppy disk\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 KbdControllerDriverSupported (\r
@@ -27,6 +37,15 @@ KbdControllerDriverSupported (
   IN EFI_DEVICE_PATH_PROTOCOL       *RemainingDevicePath\r
   );\r
 \r
+/**\r
+  Create KEYBOARD_CONSOLE_IN_DEV instance on controller.\r
+  \r
+  @param This         Pointer of EFI_DRIVER_BINDING_PROTOCOL\r
+  @param Controller   driver controller handle\r
+  @param RemainingDevicePath Children's device path\r
+  \r
+  @retval whether success to create floppy control instance.\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 KbdControllerDriverStart (\r
@@ -35,6 +54,20 @@ KbdControllerDriverStart (
   IN EFI_DEVICE_PATH_PROTOCOL       *RemainingDevicePath\r
   );\r
 \r
+/**\r
+  Stop this driver on ControllerHandle. Support stoping any child handles\r
+  created by this driver.\r
+\r
+  @param  This              Protocol instance pointer.\r
+  @param  ControllerHandle  Handle of device to stop driver on\r
+  @param  NumberOfChildren  Number of Handles in ChildHandleBuffer. If number of\r
+                            children is zero stop the entire bus driver.\r
+  @param  ChildHandleBuffer List of Child Handles to Stop.\r
+\r
+  @retval EFI_SUCCESS       This driver is removed ControllerHandle\r
+  @retval other             This driver was not removed from this device\r
+\r
+**/\r
 EFI_STATUS\r
 EFIAPI\r
 KbdControllerDriverStop (\r
@@ -44,6 +77,14 @@ KbdControllerDriverStop (
   IN  EFI_HANDLE                     *ChildHandleBuffer\r
   );\r
 \r
+/**\r
+  Free the waiting key notify list.\r
+  \r
+  @param ListHead  Pointer to list head\r
+  \r
+  @retval EFI_INVALID_PARAMETER  ListHead is NULL\r
+  @retval EFI_SUCCESS            Sucess to free NotifyList\r
+**/\r
 EFI_STATUS\r
 KbdFreeNotifyList (\r
   IN OUT LIST_ENTRY           *ListHead\r
@@ -62,7 +103,7 @@ EFI_DRIVER_BINDING_PROTOCOL gKeyboardControllerDriver = {
 };\r
 \r
 /**\r
-  Test controller is a keyboard Controller\r
+  Test controller is a keyboard Controller.\r
   \r
   @param This                 Pointer of EFI_DRIVER_BINDING_PROTOCOL\r
   @param Controller           driver's controller\r
@@ -461,7 +502,7 @@ KbdControllerDriverStop (
     ConsoleIn->DevicePath\r
     );\r
 \r
-  if (ConsoleIn->TimerEvent) {\r
+  if (ConsoleIn->TimerEvent != NULL) {\r
     gBS->CloseEvent (ConsoleIn->TimerEvent);\r
     ConsoleIn->TimerEvent = NULL;\r
   }\r
@@ -529,6 +570,9 @@ KbdControllerDriverStop (
   Free the waiting key notify list.\r
   \r
   @param ListHead  Pointer to list head\r
+  \r
+  @retval EFI_INVALID_PARAMETER  ListHead is NULL\r
+  @retval EFI_SUCCESS            Sucess to free NotifyList\r
 **/\r
 EFI_STATUS\r
 KbdFreeNotifyList (\r
index 5bf3a6eefc36a19653ec1ada209af6a760ed01f0..55b56a10451a2f6df826a1bcc457073f12460e31 100644 (file)
@@ -1,7 +1,7 @@
-/**@file\r
+/** @file\r
   PS/2 keyboard driver header file\r
 \r
-Copyright (c) 2006 - 2007, Intel Corporation\r
+Copyright (c) 2006 - 2009, Intel Corporation\r
 All rights reserved. This program and the accompanying materials\r
 are licensed and made available under the terms and conditions of the BSD License\r
 which accompanies this distribution.  The full text of the license may be found at\r
@@ -12,8 +12,8 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 \r
 **/\r
 \r
-#ifndef _PS2KEYBOARD_H\r
-#define _PS2KEYBOARD_H\r
+#ifndef _PS2KEYBOARD_H_\r
+#define _PS2KEYBOARD_H_\r
 \r
 #include <PiDxe.h>\r
 #include <Framework/StatusCode.h>\r
@@ -231,7 +231,7 @@ UpdateStatusLights (
   );\r
 \r
 /**\r
-  write key to keyboard\r
+  write key to keyboard.\r
 \r
   @param ConsoleIn Pointer to instance of KEYBOARD_CONSOLE_IN_DEV\r
   @param Data      value wanted to be written\r
@@ -263,7 +263,7 @@ KeyGetchar (
   );\r
 \r
 /**\r
-  Perform 8042 controller and keyboard Initialization\r
+  Perform 8042 controller and keyboard Initialization.\r
   If ExtendedVerification is TRUE, do additional test for\r
   the keyboard interface\r
 \r
@@ -280,7 +280,7 @@ InitKeyboard (
   );\r
 \r
 /**\r
-  Disable the keyboard interface of the 8042 controller\r
+  Disable the keyboard interface of the 8042 controller.\r
 \r
   @param ConsoleIn   - the device instance\r
 \r
@@ -360,7 +360,7 @@ KeyboardWaitForKey (
   );\r
 \r
 /**\r
-  Read status register\r
+  Read status register.\r
 \r
   @param ConsoleIn  Pointer to instance of KEYBOARD_CONSOLE_IN_DEV\r
 \r
@@ -377,7 +377,7 @@ KeyReadStatusRegister (
   If Keyboard receives 0xF4, it will respond with 'ACK'. If it doesn't respond, the device\r
   should not be in system.\r
 \r
-  @param[in]  BiosKeyboardPrivate   Keyboard Private Data Structure\r
+  @param[in]  ConsoleIn   Pointer to instance of KEYBOARD_CONSOLE_IN_DEV\r
 \r
   @retval     TRUE                  Keyboard in System.\r
   @retval     FALSE                 Keyboard not in System.\r
index 2f0b46b6f6fc8ca4702f2076b191f52e637b94f2..38c769428d114966e9caa1e6827ef32bcdaa7b1c 100644 (file)
@@ -1,5 +1,7 @@
 #/** @file\r
-# Ps2 Keyboard Driver\r
+# Ps2 Keyboard UEFI Driver which produce gEfiSimpleTextInProtocolGuid and gEfiSimpleTextInputExProtocolGuid\r
+# protocol. \r
+# The keyboard type implemented follows IBM compatible PS2 protocol, with Scan Codes Set 1.\r
 #\r
 # This dirver directly uses IsaIo protocol service to support KeyBoard work.\r
 # Copyright (c) 2006 - 2009, Intel Corporation.\r