1. Fixed tools_def.template to meet ICC build for IA32
authorvanjeff <vanjeff@6f19259b-4bc3-4df7-8a09-765794883524>
Tue, 17 Jul 2007 01:48:09 +0000 (01:48 +0000)
committervanjeff <vanjeff@6f19259b-4bc3-4df7-8a09-765794883524>
Tue, 17 Jul 2007 01:48:09 +0000 (01:48 +0000)
2. Modified some source files to meet ICC build for IA32 and IPF.

git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@3271 6f19259b-4bc3-4df7-8a09-765794883524

40 files changed:
BaseTools/Conf/tools_def.template
MdeModulePkg/Bus/Pci/EhciDxe/Ehci.c
MdeModulePkg/Bus/Pci/EhciDxe/Ehci.h
MdeModulePkg/Bus/Pci/EhciDxe/EhciDebug.h
MdeModulePkg/Bus/Pci/EhciDxe/EhciReg.h
MdeModulePkg/Bus/Pci/EhciDxe/EhciUrb.c
MdeModulePkg/Bus/Pci/EhciDxe/EhciUrb.h
MdeModulePkg/Bus/Pci/EhciDxe/UsbHcMem.c
MdeModulePkg/Bus/Pci/EhciDxe/UsbHcMem.h
MdeModulePkg/Bus/Pci/UhciDxe/Uhci.c
MdeModulePkg/Bus/Pci/UhciDxe/Uhci.h
MdeModulePkg/Bus/Pci/UhciDxe/UhciQueue.h
MdeModulePkg/Bus/Pci/UhciDxe/UhciReg.c
MdeModulePkg/Bus/Pci/UhciDxe/UhciReg.h
MdeModulePkg/Bus/Pci/UhciDxe/UhciSched.c
MdeModulePkg/Bus/Pci/UhciDxe/UhciSched.h
MdeModulePkg/Bus/Pci/UhciDxe/UsbHcMem.c
MdeModulePkg/Bus/Pci/UhciDxe/UsbHcMem.h
MdeModulePkg/Bus/Scsi/ScsiBusDxe/ScsiBus.c
MdeModulePkg/Bus/Usb/UsbBusDxe/UsbDesc.h
MdeModulePkg/Bus/Usb/UsbBusDxe/UsbEnumer.c
MdeModulePkg/Bus/Usb/UsbBusDxe/UsbEnumer.h
MdeModulePkg/Bus/Usb/UsbBusDxe/UsbHub.c
MdeModulePkg/Bus/Usb/UsbBusDxe/UsbHub.h
MdeModulePkg/Bus/Usb/UsbBusDxe/UsbUtility.c
MdeModulePkg/Bus/Usb/UsbBusDxe/usbbus.h
MdeModulePkg/Bus/Usb/UsbKbDxe/efikey.c
MdeModulePkg/Bus/Usb/UsbKbDxe/keyboard.c
MdeModulePkg/Bus/Usb/UsbMassStorageDxe/UsbMass.h
MdeModulePkg/Bus/Usb/UsbMassStorageDxe/UsbMassBoot.c
MdeModulePkg/Bus/Usb/UsbMassStorageDxe/UsbMassBoot.h
MdeModulePkg/Bus/Usb/UsbMassStorageDxe/UsbMassBot.c
MdeModulePkg/Bus/Usb/UsbMassStorageDxe/UsbMassBot.h
MdeModulePkg/Bus/Usb/UsbMassStorageDxe/UsbMassCbi.h
MdeModulePkg/Bus/Usb/UsbMassStorageDxe/UsbMassImpl.c
MdeModulePkg/Bus/Usb/UsbMassStorageDxe/UsbMassImpl.h
MdeModulePkg/Bus/Usb/UsbMouseDxe/usbmouse.c
MdeModulePkg/Universal/PCD/Dxe/Service.h
MdeModulePkg/Universal/PCD/Pei/Service.h
MdeModulePkg/Universal/RuntimeDxe/Runtime.c

index 55f2c25..dd7989c 100644 (file)
@@ -700,10 +700,10 @@ RELEASE_VS2005PRO_X64_DLINK_FLAGS = /NOLOGO /NODEFAULTLIB /IGNORE:4086 /OPT:REF
 \r
 *_ICC_IA32_APP_FLAGS         = /nologo /E /TC\r
 *_ICC_IA32_PP_FLAGS          = /nologo /E /TC /FI$(DEST_DIR_DEBUG)/AutoGen.h\r
-DEBUG_ICC_IA32_CC_FLAGS      = /nologo /W4 /WX /Gy /c /D UNICODE /D EFI32 /O1ib2 /GL /DEFI_FIRMWARE_VENDOR=L\"INTEL\" /DSTRING_ARRAY_NAME=$(BASE_NAME)Strings /FI$(DEST_DIR_DEBUG)/AutoGen.h /EHs-c- /GF /Gs8192 /Fp$(DEST_DIR_OUTPUT)/AutoGen.h.gch /Yu$(DEST_DIR_DEBUG)/AutoGen.h /Zi /Gm\r
-RELEASE_ICC_IA32_CC_FLAGS    = /nologo /W4 /WX /Gy /c /D UNICODE /D EFI32 /O1ib2 /GL /DEFI_FIRMWARE_VENDOR=L\"INTEL\" /DSTRING_ARRAY_NAME=$(BASE_NAME)Strings /FI$(DEST_DIR_DEBUG)/AutoGen.h /EHs-c- /GF /Gs8192 /Fp$(DEST_DIR_OUTPUT)/AutoGen.h.gch /Yu$(DEST_DIR_DEBUG)/AutoGen.h\r
-DEBUG_ICC_IA32_PCH_FLAGS     = /nologo /W4 /WX /Gy /c /D UNICODE /D EFI32 /O1ib2 /GL /DEFI_FIRMWARE_VENDOR=L\"INTEL\" /DSTRING_ARRAY_NAME=$(BASE_NAME)Strings /FI$(DEST_DIR_DEBUG)/AutoGen.h /EHs-c- /GF /Gs8192 /Fp$(DEST_DIR_OUTPUT)/AutoGen.h.gch /Yc /TC /Zi /Gm\r
-RELEASE_ICC_IA32_PCH_FLAGS   = /nologo /W4 /WX /Gy /c /D UNICODE /D EFI32 /O1ib2 /GL /DEFI_FIRMWARE_VENDOR=L\"INTEL\" /DSTRING_ARRAY_NAME=$(BASE_NAME)Strings /FI$(DEST_DIR_DEBUG)/AutoGen.h /EHs-c- /GF /Gs8192 /Fp$(DEST_DIR_OUTPUT)/AutoGen.h.gch /Yc /TC\r
+DEBUG_ICC_IA32_CC_FLAGS      = /nologo /W4 /WX /Gy /c /D UNICODE /D EFI32 /O1ib2 /GL /DEFI_FIRMWARE_VENDOR=L\"INTEL\" /DSTRING_ARRAY_NAME=$(BASE_NAME)Strings /FI$(DEST_DIR_DEBUG)/AutoGen.h /EHs-c- /GF /Gs8192 /Zi /Gm\r
+RELEASE_ICC_IA32_CC_FLAGS    = /nologo /W4 /WX /Gy /c /D UNICODE /D EFI32 /O1ib2 /GL /DEFI_FIRMWARE_VENDOR=L\"INTEL\" /DSTRING_ARRAY_NAME=$(BASE_NAME)Strings /FI$(DEST_DIR_DEBUG)/AutoGen.h /EHs-c- /GF /Gs8192\r
+DEBUG_ICC_IA32_PCH_FLAGS     = /nologo /W4 /WX /Gy /c /D UNICODE /D EFI32 /O1ib2 /GL /DEFI_FIRMWARE_VENDOR=L\"INTEL\" /DSTRING_ARRAY_NAME=$(BASE_NAME)Strings /FI$(DEST_DIR_DEBUG)/AutoGen.h /EHs-c- /GF /Gs8192 /Yc /TC /Zi /Gm\r
+RELEASE_ICC_IA32_PCH_FLAGS   = /nologo /W4 /WX /Gy /c /D UNICODE /D EFI32 /O1ib2 /GL /DEFI_FIRMWARE_VENDOR=L\"INTEL\" /DSTRING_ARRAY_NAME=$(BASE_NAME)Strings /FI$(DEST_DIR_DEBUG)/AutoGen.h /EHs-c- /GF /Gs8192 /Yc /TC\r
 DEBUG_ICC_IA32_ASM_FLAGS     = /nologo /W3 /WX /c /coff /Cx /Zd /W0 /Zi\r
 RELEASE_ICC_IA32_ASM_FLAGS   = /nologo /W3 /WX /c /coff /Cx /Zd /W0\r
 *_ICC_IA32_SLINK_FLAGS       = /nologo /LTCG\r
@@ -775,11 +775,11 @@ RELEASE_ICC_IPF_DLINK_FLAGS = /NOLOGO /NODEFAULTLIB /LTCG /DLL /OPT:REF,ICF /IGN
 \r
 *_MYTOOLS_*_LZMA_NAME = Lzma.bat\r
 *_MYTOOLS_*_LZMA_GUID = EE4E5898-3914-4259-9D6E-DC7B-D79403CF\r
-*_MYTOOLS_*_LAMA_PATH = \r
\r
+*_MYTOOLS_*_LAMA_PATH =\r
+\r
 *_MYTOOLS_*_TIANO_NAME = Tiano.bat\r
 *_MYTOOLS_*_TIANO_GUID = A31280AD-481E-41B6-95E8-127F-4C984779\r
-*_MYTOOLS_*_TIANO_PATH = \r
+*_MYTOOLS_*_TIANO_PATH =\r
 \r
 ##################\r
 # IA32 definitions\r
index 093388d..2e3edfd 100644 (file)
@@ -351,7 +351,7 @@ EhcGetRootHubPortStatus (
 \r
   for (Index = 0; Index < MapSize; Index++) {\r
     if (EHC_BIT_IS_SET (State, mUsbPortStateMap[Index].HwState)) {\r
-      PortStatus->PortStatus |= mUsbPortStateMap[Index].UefiState;\r
+      PortStatus->PortStatus = (UINT16) (PortStatus->PortStatus | mUsbPortStateMap[Index].UefiState);\r
     }\r
   }\r
 \r
@@ -359,7 +359,7 @@ EhcGetRootHubPortStatus (
 \r
   for (Index = 0; Index < MapSize; Index++) {\r
     if (EHC_BIT_IS_SET (State, mUsbPortChangeMap[Index].HwState)) {\r
-      PortStatus->PortChangeStatus |= mUsbPortChangeMap[Index].UefiState;\r
+      PortStatus->PortChangeStatus = (UINT16) (PortStatus->PortChangeStatus | mUsbPortChangeMap[Index].UefiState);\r
     }\r
   }\r
 \r
@@ -707,7 +707,7 @@ EhcControlTransfer (
   // endpoint is bidirectional. EhcCreateUrb expects this\r
   // combination of Ep addr and its direction.\r
   //\r
-  Endpoint = 0 | ((TransferDirection == EfiUsbDataIn) ? 0x80 : 0);\r
+  Endpoint = (UINT8) (0 | ((TransferDirection == EfiUsbDataIn) ? 0x80 : 0));\r
   Urb = EhcCreateUrb (\r
           Ehc,\r
           DeviceAddress,\r
@@ -1340,7 +1340,7 @@ EhcDriverBindingSupported (
   Status = gBS->OpenProtocol (\r
                   Controller,\r
                   &gEfiPciIoProtocolGuid,\r
-                  &PciIo,\r
+                  (VOID **) &PciIo,\r
                   This->DriverBindingHandle,\r
                   Controller,\r
                   EFI_OPEN_PROTOCOL_BY_DRIVER\r
@@ -1491,7 +1491,7 @@ EhcDriverBindingStart (
   Status = gBS->OpenProtocol (\r
                   Controller,\r
                   &gEfiPciIoProtocolGuid,\r
-                  &PciIo,\r
+                  (VOID **) &PciIo,\r
                   This->DriverBindingHandle,\r
                   Controller,\r
                   EFI_OPEN_PROTOCOL_BY_DRIVER\r
@@ -1636,7 +1636,7 @@ EhcDriverBindingStop (
   Status = gBS->OpenProtocol (\r
                   Controller,\r
                   &gEfiUsb2HcProtocolGuid,\r
-                  &Usb2Hc,\r
+                  (VOID **) &Usb2Hc,\r
                   This->DriverBindingHandle,\r
                   Controller,\r
                   EFI_OPEN_PROTOCOL_GET_PROTOCOL\r
index 57c0c00..2b0e4c9 100644 (file)
@@ -66,7 +66,7 @@ enum {
   EHC_SYNC_POLL_TIME        = 20 * EHC_STALL_1_MICROSECOND,\r
   EHC_ASYNC_POLL_TIME       = 50 * 10000UL,                 // The unit of time is 100us\r
 \r
-  EHC_TPL                   = TPL_NOTIFY,\r
+  EHC_TPL                   = TPL_NOTIFY\r
 };\r
 \r
 //\r
@@ -95,7 +95,7 @@ enum {
 \r
 #define EHC_FROM_THIS(a)   CR(a, USB2_HC_DEV, Usb2Hc, USB2_HC_DEV_SIGNATURE)\r
 \r
-typedef struct _USB2_HC_DEV {\r
+struct _USB2_HC_DEV {\r
   UINTN                     Signature;\r
   EFI_USB2_HC_PROTOCOL      Usb2Hc;\r
 \r
@@ -143,7 +143,7 @@ typedef struct _USB2_HC_DEV {
   // Misc\r
   //\r
   EFI_UNICODE_STRING_TABLE  *ControllerNameTable;\r
-} USB2_HC_DEV;\r
+};\r
 \r
 \r
 extern EFI_DRIVER_BINDING_PROTOCOL     gEhciDriverBinding;\r
index be4bcd6..247021b 100644 (file)
@@ -30,7 +30,7 @@ enum {
 \r
   EHC_DEBUG_QH            = (UINTN)(1 << 8),\r
   EHC_DEBUG_QTD           = (UINTN)(1 << 9),\r
-  EHC_DEBUG_BUF           = (UINTN)(1 << 10),\r
+  EHC_DEBUG_BUF           = (UINTN)(1 << 10)\r
 };\r
 \r
 \r
index 2ac27eb..929bd22 100644 (file)
@@ -93,7 +93,7 @@ enum {
   //\r
   EHC_PCI_CLASSC          = 0x09,\r
   EHC_PCI_CLASSC_PI       = 0x20,\r
-  EHC_BAR_INDEX           = 0, /* how many bytes away from USB_BASE to 0x10 */\r
+  EHC_BAR_INDEX           = 0 /* how many bytes away from USB_BASE to 0x10 */\r
 };\r
 \r
 #define EHC_LINK_TERMINATED(Link) (((Link) & 0x01) != 0)\r
index 7b8b7b5..005e059 100644 (file)
@@ -486,7 +486,7 @@ EhcCreateQtds (
     // Switch the Toggle bit if odd number of packets are included in the QTD.\r
     //\r
     if (((Qtd->DataLen + Ep->MaxPacket - 1) / Ep->MaxPacket) % 2) {\r
-      Toggle = 1 - Toggle;\r
+      Toggle = (UINT8) (1 - Toggle);\r
     }\r
 \r
     Len += Qtd->DataLen;\r
@@ -588,7 +588,7 @@ EhcCreateUrb (
 \r
   Ep              = &Urb->Ep;\r
   Ep->DevAddr     = DevAddr;\r
-  Ep->EpAddr      = EpAddr & 0x0F;\r
+  Ep->EpAddr      = (UINT8) (EpAddr & 0x0F);\r
   Ep->Direction   = ((EpAddr & 0x80) ? EfiUsbDataIn : EfiUsbDataOut);\r
   Ep->DevSpeed    = DevSpeed;\r
   Ep->MaxPacket   = MaxPacket;\r
index a2a5826..a1dc8e6 100644 (file)
@@ -83,7 +83,7 @@ enum {
   QH_MICROFRAME_6         = 0x40,\r
   QH_MICROFRAME_7         = 0x80,\r
 \r
-  USB_ERR_SHORT_PACKET    = 0x200,\r
+  USB_ERR_SHORT_PACKET    = 0x200\r
 };\r
 \r
 //\r
@@ -180,13 +180,13 @@ typedef struct _USB_ENDPOINT {
 // Software QTD strcture, this is used to manage all the\r
 // QTD generated from a URB. Don't add fields before QtdHw.\r
 //\r
-typedef struct _EHC_QTD {\r
+struct _EHC_QTD {\r
   QTD_HW                  QtdHw;\r
   UINT32                  Signature;\r
   LIST_ENTRY              QtdList;   // The list of QTDs to one end point\r
   UINT8                   *Data;     // Buffer of the original data\r
   UINTN                   DataLen;   // Original amount of data in this QTD\r
-} EHC_QTD;\r
+};\r
 \r
 //\r
 // Software QH structure. All three different transaction types\r
@@ -203,19 +203,19 @@ typedef struct _EHC_QTD {
 // transfer is supported. A dummy QH is linked to EHCI AsyncListAddr\r
 // as the reclamation header. New transfer is inserted after this QH.\r
 //\r
-typedef struct _EHC_QH {\r
+struct _EHC_QH {\r
   QH_HW                   QhHw;\r
   UINT32                  Signature;\r
   EHC_QH                  *NextQh;    // The queue head pointed to by horizontal link\r
   LIST_ENTRY              Qtds;       // The list of QTDs to this queue head\r
   UINTN                   Interval;\r
-} EHC_QH;\r
+};\r
 \r
 //\r
 // URB (Usb Request Block) contains information for all kinds of\r
 // usb requests.\r
 //\r
-typedef struct _URB {\r
+struct _URB {\r
   UINT32                          Signature;\r
   LIST_ENTRY                      UrbList;\r
 \r
@@ -244,7 +244,7 @@ typedef struct _URB {
   UINT32                          Result;\r
   UINTN                           Completed;    // completed data length\r
   UINT8                           DataToggle;\r
-} URB;\r
+};\r
 \r
 \r
 \r
index b7e7d66..22556e7 100644 (file)
@@ -520,7 +520,7 @@ UsbHcFreeMem (
       for (Count = 0; Count < (AllocSize / USBHC_MEM_UNIT); Count++) {\r
         ASSERT (USB_HC_BIT_IS_SET (Block->Bits[Byte], Bit));\r
 \r
-        Block->Bits[Byte] ^= (UINT8) USB_HC_BIT (Bit);\r
+        Block->Bits[Byte] = (UINT8) (Block->Bits[Byte] ^ USB_HC_BIT (Bit));\r
         NEXT_BIT (Byte, Bit);\r
       }\r
 \r
index 10d3545..09fd162 100644 (file)
@@ -37,7 +37,7 @@ Revision History
 \r
 typedef struct _USBHC_MEM_BLOCK  USBHC_MEM_BLOCK;\r
 \r
-typedef struct _USBHC_MEM_BLOCK {\r
+struct _USBHC_MEM_BLOCK {\r
   UINT8                   *Bits;    // Bit array to record which unit is allocated\r
   UINTN                   BitsLen;\r
   UINT8                   *Buf;\r
@@ -45,7 +45,7 @@ typedef struct _USBHC_MEM_BLOCK {
   UINTN                   BufLen;   // Memory size in bytes\r
   VOID                    *Mapping;\r
   USBHC_MEM_BLOCK         *Next;\r
-} USBHC_MEM_BLOCK;\r
+};\r
 \r
 //\r
 // USBHC_MEM_POOL is used to manage the memory used by USB\r
@@ -63,7 +63,7 @@ enum {
   USBHC_MEM_UNIT           = 64,     // Memory allocation unit, must be 2^n, n>4\r
 \r
   USBHC_MEM_UNIT_MASK      = USBHC_MEM_UNIT - 1,\r
-  USBHC_MEM_DEFAULT_PAGES  = 16,\r
+  USBHC_MEM_DEFAULT_PAGES  = 16\r
 };\r
 \r
 #define USBHC_MEM_ROUND(Len)  (((Len) + USBHC_MEM_UNIT_MASK) & (~USBHC_MEM_UNIT_MASK))\r
index 1c3e37a..8cda00f 100644 (file)
@@ -1538,7 +1538,7 @@ Uhci2ControlTransfer (
   BOOLEAN             IsSlow;\r
 \r
   Uhc     = UHC_FROM_USB2_HC_PROTO (This);\r
-  IsSlow  = (EFI_USB_SPEED_LOW == DeviceSpeed) ? TRUE : FALSE;\r
+  IsSlow  = (BOOLEAN) ((EFI_USB_SPEED_LOW == DeviceSpeed) ? TRUE : FALSE);\r
 \r
   return UhciControlTransfer (\r
            &Uhc->UsbHc,\r
@@ -1668,7 +1668,7 @@ Uhci2AsyncInterruptTransfer (
   BOOLEAN             IsSlow;\r
 \r
   Uhc     = UHC_FROM_USB2_HC_PROTO (This);\r
-  IsSlow  = (EFI_USB_SPEED_LOW == DeviceSpeed) ? TRUE : FALSE;\r
+  IsSlow  = (BOOLEAN) ((EFI_USB_SPEED_LOW == DeviceSpeed) ? TRUE : FALSE);\r
 \r
   return UhciAsyncInterruptTransfer (\r
            &Uhc->UsbHc,\r
@@ -1736,7 +1736,7 @@ Uhci2SyncInterruptTransfer (
   }\r
 \r
   Uhc     = UHC_FROM_USB2_HC_PROTO (This);\r
-  IsSlow  = (EFI_USB_SPEED_LOW == DeviceSpeed) ? TRUE : FALSE;\r
+  IsSlow  = (BOOLEAN) ((EFI_USB_SPEED_LOW == DeviceSpeed) ? TRUE : FALSE);\r
 \r
   return UhciSyncInterruptTransfer (\r
            &Uhc->UsbHc,\r
@@ -1897,7 +1897,7 @@ UhciDriverBindingSupported (
   OpenStatus = gBS->OpenProtocol (\r
                       Controller,\r
                       &gEfiPciIoProtocolGuid,\r
-                      &PciIo,\r
+                      (VOID **) &PciIo,\r
                       This->DriverBindingHandle,\r
                       Controller,\r
                       EFI_OPEN_PROTOCOL_BY_DRIVER\r
@@ -2148,7 +2148,7 @@ UhciDriverBindingStart (
   Status = gBS->OpenProtocol (\r
                   Controller,\r
                   &gEfiPciIoProtocolGuid,\r
-                  &PciIo,\r
+                  (VOID **) &PciIo,\r
                   This->DriverBindingHandle,\r
                   Controller,\r
                   EFI_OPEN_PROTOCOL_BY_DRIVER\r
@@ -2277,7 +2277,7 @@ UhciDriverBindingStop (
   Status = gBS->OpenProtocol (\r
                   Controller,\r
                   &gEfiUsbHcProtocolGuid,\r
-                  &UsbHc,\r
+                  (VOID **) &UsbHc,\r
                   This->DriverBindingHandle,\r
                   Controller,\r
                   EFI_OPEN_PROTOCOL_GET_PROTOCOL\r
@@ -2294,7 +2294,7 @@ UhciDriverBindingStop (
   Status = gBS->OpenProtocol (\r
                   Controller,\r
                   &gEfiUsb2HcProtocolGuid,\r
-                  &Usb2Hc,\r
+                  (VOID **) &Usb2Hc,\r
                   This->DriverBindingHandle,\r
                   Controller,\r
                   EFI_OPEN_PROTOCOL_GET_PROTOCOL\r
index c37894d..5a810a3 100644 (file)
@@ -75,7 +75,7 @@ enum {
   //\r
   UHCI_TPL                 = TPL_NOTIFY,\r
 \r
-  USB_HC_DEV_SIGNATURE     = EFI_SIGNATURE_32 ('u', 'h', 'c', 'i'),\r
+  USB_HC_DEV_SIGNATURE     = EFI_SIGNATURE_32 ('u', 'h', 'c', 'i')\r
 };\r
 \r
 #pragma pack(1)\r
@@ -100,7 +100,7 @@ typedef struct {
 // or bulk transfer can reclaim the unused bandwidth. Some USB\r
 // device requires this bandwidth reclamation capability.\r
 //\r
-typedef struct _USB_HC_DEV {\r
+struct _USB_HC_DEV {\r
   UINT32                    Signature;\r
   EFI_USB_HC_PROTOCOL       UsbHc;\r
   EFI_USB2_HC_PROTOCOL      Usb2Hc;\r
@@ -132,7 +132,7 @@ typedef struct _USB_HC_DEV {
   USBHC_MEM_POOL            *MemPool;\r
   EFI_UNICODE_STRING_TABLE  *CtrlNameTable;\r
   VOID                      *FrameMapping;\r
-} USB_HC_DEV;\r
+};\r
 \r
 extern EFI_DRIVER_BINDING_PROTOCOL  gUhciDriverBinding;\r
 extern EFI_COMPONENT_NAME_PROTOCOL  gUhciComponentName;\r
index 8b03ec6..1b60ad7 100644 (file)
@@ -88,19 +88,19 @@ typedef struct {
 typedef struct _UHCI_TD_SW  UHCI_TD_SW;\r
 typedef struct _UHCI_QH_SW  UHCI_QH_SW;\r
 \r
-typedef struct _UHCI_QH_SW {\r
+struct _UHCI_QH_SW {\r
   UHCI_QH_HW        QhHw;\r
   UHCI_QH_SW        *NextQh;\r
   UHCI_TD_SW        *TDs;\r
   UINTN             Interval;\r
-} UHCI_QH_SW;\r
+};\r
 \r
-typedef struct _UHCI_TD_SW {\r
+struct _UHCI_TD_SW {\r
   UHCI_TD_HW        TdHw;\r
   UHCI_TD_SW        *NextTd;\r
   UINT8             *Data;\r
   UINT16            DataLen;\r
-} UHCI_TD_SW;\r
+};\r
 \r
 \r
 /**\r
index 6332629..8bb0f2b 100644 (file)
@@ -116,7 +116,7 @@ UhciSetRegBit (
   UINT16  Data;\r
 \r
   Data = UhciReadReg (PciIo, Offset);\r
-  Data |= Bit;\r
+  Data = (UINT16) (Data |Bit);\r
   UhciWriteReg (PciIo, Offset, Data);\r
 }\r
 \r
@@ -141,7 +141,7 @@ UhciClearRegBit (
   UINT16  Data;\r
 \r
   Data = UhciReadReg (PciIo, Offset);\r
-  Data &= ~Bit;\r
+  Data = (UINT16) (Data & ~Bit);\r
   UhciWriteReg (PciIo, Offset, Data);\r
 }\r
 \r
index 95a98db..dd0103a 100644 (file)
@@ -98,7 +98,7 @@ enum {
   USBTD_BABBLE          = BIT(4),  // Babble condition\r
   USBTD_NAK             = BIT(3),  // NAK is received\r
   USBTD_CRC             = BIT(2),  // CRC/Time out error\r
-  USBTD_BITSTUFF        = BIT(1),  // Bit stuff error\r
+  USBTD_BITSTUFF        = BIT(1)  // Bit stuff error\r
 };\r
 \r
 \r
index 222322a..e1b602e 100644 (file)
@@ -498,7 +498,7 @@ UhciCheckTdStatus (
       // next to the last known-good TD's data toggle if\r
       // any TD is executed OK\r
       //\r
-      QhResult->NextToggle = 1 - (UINT8)TdHw->DataToggle;\r
+      QhResult->NextToggle = (UINT8) (1 - (UINT8)TdHw->DataToggle);\r
 \r
       //\r
       // This TD is finished OK or met short packet read. Update the\r
index cba65c6..61e459b 100644 (file)
@@ -38,7 +38,7 @@ enum {
   USB_ERR_FAIL_MASK = EFI_USB_ERR_STALL   | EFI_USB_ERR_BUFFER |\r
                       EFI_USB_ERR_BABBLE  | EFI_USB_ERR_CRC    |\r
                       EFI_USB_ERR_TIMEOUT | EFI_USB_ERR_BITSTUFF |\r
-                      EFI_USB_ERR_SYSTEM,\r
+                      EFI_USB_ERR_SYSTEM\r
 \r
 };\r
 \r
@@ -59,7 +59,7 @@ typedef struct _UHCI_ASYNC_REQUEST  UHCI_ASYNC_REQUEST;
 //\r
 // Structure used to manager the asynchronous interrupt transfers.\r
 //\r
-typedef struct _UHCI_ASYNC_REQUEST{\r
+struct _UHCI_ASYNC_REQUEST{\r
   UINTN                           Signature;\r
   LIST_ENTRY                      Link;\r
   UHCI_ASYNC_REQUEST              *Recycle;\r
@@ -86,7 +86,7 @@ typedef struct _UHCI_ASYNC_REQUEST{
   //\r
   EFI_ASYNC_USB_TRANSFER_CALLBACK Callback;\r
   VOID                            *Context;\r
-} UHCI_ASYNC_REQUEST;\r
+};\r
 \r
 #define UHCI_ASYNC_INT_FROM_LINK(a) \\r
           CR (a, UHCI_ASYNC_REQUEST, Link, UHCI_ASYNC_INT_SIGNATURE)\r
index 32082b1..7c598e2 100644 (file)
@@ -519,7 +519,7 @@ UsbHcFreeMem (
       for (Count = 0; Count < (AllocSize / USBHC_MEM_UNIT); Count++) {\r
         ASSERT (USB_HC_BIT_IS_SET (Block->Bits[Byte], Bit));\r
 \r
-        Block->Bits[Byte] ^= (UINT8) USB_HC_BIT (Bit);\r
+        Block->Bits[Byte] = (UINT8) (Block->Bits[Byte] ^ USB_HC_BIT (Bit));\r
         NEXT_BIT (Byte, Bit);\r
       }\r
 \r
index df9ab0e..4df144b 100644 (file)
@@ -36,7 +36,7 @@ Revision History
 \r
 typedef struct _USBHC_MEM_BLOCK   USBHC_MEM_BLOCK;\r
 \r
-typedef struct _USBHC_MEM_BLOCK {\r
+struct _USBHC_MEM_BLOCK {\r
   UINT8                   *Bits;    // Bit array to record which unit is allocated\r
   UINTN                   BitsLen;\r
   UINT8                   *Buf;\r
@@ -44,7 +44,7 @@ typedef struct _USBHC_MEM_BLOCK {
   UINTN                   BufLen;   // Memory size in bytes\r
   VOID                    *Mapping;\r
   USBHC_MEM_BLOCK         *Next;\r
-} USBHC_MEM_BLOCK;\r
+};\r
 \r
 //\r
 // USBHC_MEM_POOL is used to manage the memory used by USB\r
@@ -62,7 +62,7 @@ enum {
   USBHC_MEM_UNIT           = 64,     // Memory allocation unit, must be 2^n, n>4\r
 \r
   USBHC_MEM_UNIT_MASK      = USBHC_MEM_UNIT - 1,\r
-  USBHC_MEM_DEFAULT_PAGES  = 16,\r
+  USBHC_MEM_DEFAULT_PAGES  = 16\r
 };\r
 \r
 #define USBHC_MEM_ROUND(Len)  (((Len) + USBHC_MEM_UNIT_MASK) & (~USBHC_MEM_UNIT_MASK))\r
index 68f3683..7633fd4 100644 (file)
@@ -1,20 +1,20 @@
 /*++\r
 \r
-Copyright (c) 2006, 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
-http://opensource.org/licenses/bsd-license.php                                            \r
-                                                                                          \r
-THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,                     \r
-WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.             \r
+Copyright (c) 2006, 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
+http://opensource.org/licenses/bsd-license.php\r
+\r
+THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
+WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
 \r
 Module Name:\r
 \r
     scsibus.c\r
-    \r
-Abstract: \r
-    \r
+\r
+Abstract:\r
+\r
 \r
 Revision History\r
 --*/\r
@@ -96,9 +96,9 @@ NotifyFunction (
 /**\r
   The user Entry Point for module ScsiBus. The user code starts with this function.\r
 \r
-  @param[in] ImageHandle    The firmware allocated handle for the EFI image.  \r
+  @param[in] ImageHandle    The firmware allocated handle for the EFI image.\r
   @param[in] SystemTable    A pointer to the EFI System Table.\r
-  \r
+\r
   @retval EFI_SUCCESS       The entry point is executed successfully.\r
   @retval other             Some error occurs when executing this entry point.\r
 \r
@@ -172,11 +172,11 @@ Returns:
                   Controller,\r
                   EFI_OPEN_PROTOCOL_BY_DRIVER\r
                   );\r
-  \r
+\r
   if (Status == EFI_ALREADY_STARTED) {\r
     return EFI_SUCCESS;\r
   }\r
-  \r
+\r
   if (EFI_ERROR (Status)) {\r
     Status = gBS->OpenProtocol (\r
                     Controller,\r
@@ -186,11 +186,11 @@ Returns:
                     Controller,\r
                     EFI_OPEN_PROTOCOL_BY_DRIVER\r
                     );\r
-    \r
+\r
     if (Status == EFI_ALREADY_STARTED) {\r
       return EFI_SUCCESS;\r
     }\r
-    \r
+\r
     if (EFI_ERROR (Status)) {\r
       return Status;\r
     }\r
@@ -203,7 +203,7 @@ Returns:
       );\r
     return EFI_SUCCESS;\r
   }\r
-  \r
+\r
   gBS->CloseProtocol (\r
     Controller,\r
     &gEfiExtScsiPassThruProtocolGuid,\r
@@ -268,7 +268,7 @@ Returns:
   }\r
 \r
   TargetId = &ScsiTargetId->ScsiId.ExtScsi[0];\r
-  \r
+\r
   Status = gBS->OpenProtocol (\r
                   Controller,\r
                   &gEfiDevicePathProtocolGuid,\r
@@ -312,7 +312,7 @@ Returns:
              );\r
       gBS->FreePool (ScsiBusDev);\r
       return Status;\r
-    } \r
+    }\r
     DEBUG ((EFI_D_INFO, "Open Scsi Pass Thrugh Protocol\n"));\r
     ScsiBusDev->ExtScsiSupport  = FALSE;\r
   } else {\r
@@ -363,7 +363,7 @@ Returns:
     FromFirstTarget = TRUE;\r
   } else {\r
     if (ScsiBusDev->ExtScsiSupport) {\r
-      ScsiBusDev->ExtScsiInterface->GetTargetLun (ScsiBusDev->ExtScsiInterface, RemainingDevicePath, &TargetId, &Lun);  \r
+      ScsiBusDev->ExtScsiInterface->GetTargetLun (ScsiBusDev->ExtScsiInterface, RemainingDevicePath, &TargetId, &Lun);\r
     } else {\r
       ScsiBusDev->ScsiInterface->GetTargetLun (ScsiBusDev->ScsiInterface, RemainingDevicePath, &ScsiTargetId->ScsiId.Scsi, &Lun);\r
     }\r
@@ -419,13 +419,13 @@ SCSIBusDriverBindingStop (
   IN  EFI_HANDLE                      *ChildHandleBuffer\r
   )\r
 /*++\r
-  \r
+\r
   Routine Description:\r
-  \r
+\r
   Arguments:\r
-  \r
+\r
   Returns:\r
-  \r
+\r
 --*/\r
 // TODO:    This - add argument and description to function comment\r
 // TODO:    Controller - add argument and description to function comment\r
@@ -456,7 +456,7 @@ SCSIBusDriverBindingStop (
                     Controller,\r
                     EFI_OPEN_PROTOCOL_GET_PROTOCOL\r
                     );\r
-  \r
+\r
     if (EFI_ERROR (Status)) {\r
       return EFI_DEVICE_ERROR;\r
     }\r
@@ -471,7 +471,7 @@ SCSIBusDriverBindingStop (
            &mScsiBusProtocolGuid,\r
            &ScsiBusDev->BusIdentify\r
            );\r
-    \r
+\r
     //\r
     // Close the bus driver\r
     //\r
@@ -553,7 +553,7 @@ SCSIBusDriverBindingStop (
         gBS->OpenProtocol (\r
                Controller,\r
                &gEfiExtScsiPassThruProtocolGuid,\r
-               &(EFI_EXT_SCSI_PASS_THRU_PROTOCOL*)ScsiPassThru,\r
+               (VOID **) &(EFI_EXT_SCSI_PASS_THRU_PROTOCOL*)ScsiPassThru,\r
                This->DriverBindingHandle,\r
                ChildHandleBuffer[Index],\r
                EFI_OPEN_PROTOCOL_BY_CHILD_CONTROLLER\r
@@ -562,7 +562,7 @@ SCSIBusDriverBindingStop (
         gBS->OpenProtocol (\r
                Controller,\r
                &gEfiScsiPassThruProtocolGuid,\r
-               &(EFI_SCSI_PASS_THRU_PROTOCOL*)ScsiPassThru,\r
+               (VOID **) &(EFI_SCSI_PASS_THRU_PROTOCOL*)ScsiPassThru,\r
                This->DriverBindingHandle,\r
                ChildHandleBuffer[Index],\r
                EFI_OPEN_PROTOCOL_BY_CHILD_CONTROLLER\r
@@ -590,11 +590,11 @@ ScsiGetDeviceType (
 \r
   Routine Description:\r
     Retrieves the device type information of the SCSI Controller.\r
-    \r
+\r
   Arguments:\r
     This                  - Protocol instance pointer.\r
     DeviceType            - A pointer to the device type information\r
-                            retrieved from the SCSI Controller. \r
+                            retrieved from the SCSI Controller.\r
 \r
   Returns:\r
     EFI_SUCCESS           - Retrieves the device type information successfully.\r
@@ -622,12 +622,12 @@ ScsiGetDeviceLocation (
 /*++\r
   Routine Description:\r
     Retrieves the device location in the SCSI channel.\r
-    \r
+\r
   Arguments:\r
     This                  - Protocol instance pointer.\r
-    Target                - A pointer to the Target ID of a SCSI device \r
-                            on the SCSI channel. \r
-    Lun                   - A pointer to the LUN of the SCSI device on \r
+    Target                - A pointer to the Target ID of a SCSI device\r
+                            on the SCSI channel.\r
+    Lun                   - A pointer to the LUN of the SCSI device on\r
                             the SCSI channel.\r
 \r
   Returns:\r
@@ -659,7 +659,7 @@ ScsiResetBus (
 \r
   Routine Description:\r
     Resets the SCSI Bus that the SCSI Controller is attached to.\r
-    \r
+\r
   Arguments:\r
     This                  - Protocol instance pointer.\r
 \r
@@ -668,7 +668,7 @@ ScsiResetBus (
     EFI_DEVICE_ERROR      - Errors encountered when resetting the SCSI bus.\r
     EFI_UNSUPPORTED       - The bus reset operation is not supported by the\r
                             SCSI Host Controller.\r
-    EFI_TIMEOUT           - A timeout occurred while attempting to reset \r
+    EFI_TIMEOUT           - A timeout occurred while attempting to reset\r
                             the SCSI bus.\r
 --*/\r
 {\r
@@ -692,18 +692,18 @@ ScsiResetDevice (
 \r
   Routine Description:\r
     Resets the SCSI Controller that the device handle specifies.\r
-    \r
+\r
   Arguments:\r
     This                  - Protocol instance pointer.\r
-    \r
+\r
 \r
   Returns:\r
     EFI_SUCCESS           - Reset the SCSI controller successfully.\r
     EFI_DEVICE_ERROR      - Errors are encountered when resetting the\r
                             SCSI Controller.\r
-    EFI_UNSUPPORTED       - The SCSI bus does not support a device \r
+    EFI_UNSUPPORTED       - The SCSI bus does not support a device\r
                             reset operation.\r
-    EFI_TIMEOUT           - A timeout occurred while attempting to \r
+    EFI_TIMEOUT           - A timeout occurred while attempting to\r
                             reset the SCSI Controller.\r
 --*/\r
 {\r
@@ -740,50 +740,50 @@ ScsiExecuteSCSICommand (
 \r
   Routine Description:\r
     Sends a SCSI Request Packet to the SCSI Controller for execution.\r
-    \r
+\r
   Arguments:\r
     This                  - Protocol instance pointer.\r
-    Packet                - The SCSI request packet to send to the SCSI \r
+    Packet                - The SCSI request packet to send to the SCSI\r
                             Controller specified by the device handle.\r
     Event                 - If the SCSI bus where the SCSI device is attached\r
-                            does not support non-blocking I/O, then Event is \r
-                            ignored, and blocking I/O is performed.  \r
+                            does not support non-blocking I/O, then Event is\r
+                            ignored, and blocking I/O is performed.\r
                             If Event is NULL, then blocking I/O is performed.\r
-                            If Event is not NULL and non-blocking I/O is \r
+                            If Event is not NULL and non-blocking I/O is\r
                             supported, then non-blocking I/O is performed,\r
                             and Event will be signaled when the SCSI Request\r
                             Packet completes.\r
   Returns:\r
-    EFI_SUCCESS           - The SCSI Request Packet was sent by the host \r
-                            successfully, and TransferLength bytes were \r
-                            transferred to/from DataBuffer.See \r
-                            HostAdapterStatus, TargetStatus, \r
+    EFI_SUCCESS           - The SCSI Request Packet was sent by the host\r
+                            successfully, and TransferLength bytes were\r
+                            transferred to/from DataBuffer.See\r
+                            HostAdapterStatus, TargetStatus,\r
                             SenseDataLength, and SenseData in that order\r
                             for additional status information.\r
-    EFI_WARN_BUFFER_TOO_SMALL - The SCSI Request Packet was executed, \r
+    EFI_WARN_BUFFER_TOO_SMALL - The SCSI Request Packet was executed,\r
                             but the entire DataBuffer could not be transferred.\r
                             The actual number of bytes transferred is returned\r
-                            in TransferLength. See HostAdapterStatus, \r
-                            TargetStatus, SenseDataLength, and SenseData in \r
+                            in TransferLength. See HostAdapterStatus,\r
+                            TargetStatus, SenseDataLength, and SenseData in\r
                             that order for additional status information.\r
-    EFI_NOT_READY         - The SCSI Request Packet could not be sent because \r
-                            there are too many SCSI Command Packets already \r
+    EFI_NOT_READY         - The SCSI Request Packet could not be sent because\r
+                            there are too many SCSI Command Packets already\r
                             queued.The caller may retry again later.\r
-    EFI_DEVICE_ERROR      - A device error occurred while attempting to send \r
-                            the SCSI Request Packet. See HostAdapterStatus, \r
-                            TargetStatus, SenseDataLength, and SenseData in \r
+    EFI_DEVICE_ERROR      - A device error occurred while attempting to send\r
+                            the SCSI Request Packet. See HostAdapterStatus,\r
+                            TargetStatus, SenseDataLength, and SenseData in\r
                             that order for additional status information.\r
-    EFI_INVALID_PARAMETER - The contents of CommandPacket are invalid.  \r
-                            The SCSI Request Packet was not sent, so no \r
+    EFI_INVALID_PARAMETER - The contents of CommandPacket are invalid.\r
+                            The SCSI Request Packet was not sent, so no\r
                             additional status information is available.\r
     EFI_UNSUPPORTED       - The command described by the SCSI Request Packet\r
-                            is not supported by the SCSI initiator(i.e., SCSI \r
+                            is not supported by the SCSI initiator(i.e., SCSI\r
                             Host Controller). The SCSI Request Packet was not\r
-                            sent, so no additional status information is \r
+                            sent, so no additional status information is\r
                             available.\r
-    EFI_TIMEOUT           - A timeout occurred while waiting for the SCSI \r
+    EFI_TIMEOUT           - A timeout occurred while waiting for the SCSI\r
                             Request Packet to execute. See HostAdapterStatus,\r
-                            TargetStatus, SenseDataLength, and SenseData in \r
+                            TargetStatus, SenseDataLength, and SenseData in\r
                             that order for additional status information.\r
 --*/\r
 {\r
@@ -792,17 +792,17 @@ ScsiExecuteSCSICommand (
   UINT8                                       Target[TARGET_MAX_BYTES];\r
   EFI_EVENT                                   PacketEvent;\r
   EFI_EXT_SCSI_PASS_THRU_SCSI_REQUEST_PACKET  *ExtRequestPacket;\r
-  SCSI_EVENT_DATA                             EventData;                                     \r
+  SCSI_EVENT_DATA                             EventData;\r
 \r
   PacketEvent = NULL;\r
-  \r
+\r
   if (Packet == NULL) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
   ScsiIoDevice  = SCSI_IO_DEV_FROM_THIS (This);\r
   CopyMem (Target,&ScsiIoDevice->Pun, TARGET_MAX_BYTES);\r
-  \r
+\r
   if (ScsiIoDevice->ExtScsiSupport) {\r
     ExtRequestPacket = (EFI_EXT_SCSI_PASS_THRU_SCSI_REQUEST_PACKET *) Packet;\r
     Status = ScsiIoDevice->ExtScsiPassThru->PassThru (\r
@@ -825,11 +825,11 @@ ScsiExecuteSCSICommand (
     }\r
 \r
     //\r
-    // Convert package into EFI1.0, EFI_SCSI_PASS_THRU_SCSI_REQUEST_PACKET. \r
+    // Convert package into EFI1.0, EFI_SCSI_PASS_THRU_SCSI_REQUEST_PACKET.\r
     //\r
     Status = ScsiioToPassThruPacket(Packet, (EFI_SCSI_PASS_THRU_SCSI_REQUEST_PACKET*)WorkingBuffer);\r
     if (EFI_ERROR(Status)) {\r
-      gBS->FreePool(WorkingBuffer);  \r
+      gBS->FreePool(WorkingBuffer);\r
       return Status;\r
     }\r
 \r
@@ -850,7 +850,7 @@ ScsiExecuteSCSICommand (
         gBS->FreePool(WorkingBuffer);\r
         return Status;\r
       }\r
-    \r
+\r
       Status = ScsiIoDevice->ScsiPassThru->PassThru (\r
                                           ScsiIoDevice->ScsiPassThru,\r
                                           ScsiIoDevice->Pun.ScsiId.Scsi,\r
@@ -864,7 +864,7 @@ ScsiExecuteSCSICommand (
         gBS->CloseEvent(PacketEvent);\r
         return Status;\r
       }\r
-      \r
+\r
     } else {\r
       //\r
       // If there's no event or SCSI Device doesn't support NON-BLOCKING, just convert\r
@@ -894,7 +894,7 @@ ScsiExecuteSCSICommand (
 }\r
 \r
 EFI_STATUS\r
-EFIAPI    \r
+EFIAPI\r
 ScsiScanCreateDevice (\r
   EFI_DRIVER_BINDING_PROTOCOL   *This,\r
   EFI_HANDLE                    Controller,\r
@@ -991,7 +991,7 @@ Returns:
       return Status;\r
     }\r
   }\r
-  \r
+\r
   ScsiIoDevice->DevicePath = AppendDevicePathNode (\r
                               ScsiBusDev->DevicePath,\r
                               ScsiDevicePath\r
@@ -1007,7 +1007,7 @@ Returns:
     gBS->FreePool (ScsiIoDevice);\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
-  \r
+\r
   Status = gBS->InstallMultipleProtocolInterfaces (\r
                   &ScsiIoDevice->Handle,\r
                   &gEfiDevicePathProtocolGuid,\r
@@ -1044,7 +1044,7 @@ Returns:
 }\r
 \r
 BOOLEAN\r
-EFIAPI    \r
+EFIAPI\r
 DiscoverScsiDevice (\r
   SCSI_IO_DEV   *ScsiIoDevice\r
   )\r
@@ -1061,7 +1061,7 @@ Arguments:
 Returns:\r
 \r
   TRUE            - Find SCSI Device and verify it.\r
-  FALSE           - Unable to find SCSI Device.  \r
+  FALSE           - Unable to find SCSI Device.\r
 \r
 --*/\r
 {\r
@@ -1115,7 +1115,7 @@ Returns:
   if (0x1e >= InquiryData.Peripheral_Type >= 0xa) {\r
     return FALSE;\r
   }\r
-  \r
+\r
   //\r
   // valid device type and peripheral qualifier combination.\r
   //\r
@@ -1145,14 +1145,14 @@ ScsiioToPassThruPacket (
 \r
 Routine Description:\r
 \r
-  Convert EFI_SCSI_IO_SCSI_REQUEST_PACKET packet to \r
+  Convert EFI_SCSI_IO_SCSI_REQUEST_PACKET packet to\r
   EFI_SCSI_PASS_THRU_SCSI_REQUEST_PACKET packet\r
-  \r
+\r
 Arguments:\r
 \r
   Packet            - The pointer of EFI_SCSI_IO_SCSI_REQUEST_PACKET\r
   CommandPacket     - The pointer of EFI_SCSI_PASS_THRU_SCSI_REQUEST_PACKET\r
-   \r
+\r
 Returns:\r
 \r
   NONE\r
@@ -1165,7 +1165,7 @@ Returns:
   if (Packet->DataDirection == EFI_SCSI_IO_DATA_DIRECTION_BIDIRECTIONAL) {\r
     return EFI_UNSUPPORTED;\r
   }\r
-  \r
+\r
   ZeroMem (CommandPacket, sizeof (EFI_SCSI_PASS_THRU_SCSI_REQUEST_PACKET));\r
 \r
   CommandPacket->Timeout           = Packet->Timeout;\r
@@ -1199,14 +1199,14 @@ PassThruToScsiioPacket (
 \r
 Routine Description:\r
 \r
-  Convert EFI_SCSI_PASS_THRU_SCSI_REQUEST_PACKET packet to \r
+  Convert EFI_SCSI_PASS_THRU_SCSI_REQUEST_PACKET packet to\r
   EFI_SCSI_IO_SCSI_REQUEST_PACKET packet\r
-  \r
+\r
 Arguments:\r
 \r
   ScsiPacket        - The pointer of EFI_SCSI_PASS_THRU_SCSI_REQUEST_PACKET\r
   Packet            - The pointer of EFI_SCSI_IO_SCSI_REQUEST_PACKET\r
-   \r
+\r
 Returns:\r
 \r
   NONE\r
@@ -1229,7 +1229,7 @@ Returns:
     Packet->OutDataBuffer = ScsiPacket->DataBuffer;\r
     Packet->OutTransferLength = ScsiPacket->TransferLength;\r
   }\r
\r
+\r
   return EFI_SUCCESS;\r
 }\r
 \r
@@ -1246,24 +1246,24 @@ NotifyFunction (
 \r
 Routine Description:\r
 \r
-  Notify Function in which convert EFI1.0 PassThru Packet back to UEF2.0 \r
+  Notify Function in which convert EFI1.0 PassThru Packet back to UEF2.0\r
   SCSI IO Packet.\r
-  \r
+\r
 Arguments:\r
 \r
   Event          - The instance of EFI_EVENT.\r
   Context        - The parameter passed in.\r
-   \r
+\r
 Returns:\r
 \r
   NONE\r
 \r
---*/  \r
+--*/\r
 {\r
   EFI_SCSI_IO_SCSI_REQUEST_PACKET          *Packet;\r
   EFI_SCSI_PASS_THRU_SCSI_REQUEST_PACKET   *ScsiPacket;\r
   EFI_EVENT                                CallerEvent;\r
-  SCSI_EVENT_DATA                          *PassData;                                     \r
+  SCSI_EVENT_DATA                          *PassData;\r
 \r
   PassData = (SCSI_EVENT_DATA*)Context;\r
   Packet  = (EFI_SCSI_IO_SCSI_REQUEST_PACKET *)PassData->Data1;\r
@@ -1273,7 +1273,7 @@ Returns:
   // Convert EFI1.0 PassThru packet to UEFI2.0 SCSI IO Packet.\r
   //\r
   PassThruToScsiioPacket(ScsiPacket, Packet);\r
-  \r
+\r
   //\r
   // After converting EFI1.0 PassThru Packet back to UEFI2.0 SCSI IO Packet,\r
   // free WorkingBuffer.\r
index bdecfcf..14f3c7f 100644 (file)
@@ -26,7 +26,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #define _USB_DESCRIPTOR_H_\r
 \r
 enum {\r
-  USB_MAX_INTERFACE_SETTING  = 8,\r
+  USB_MAX_INTERFACE_SETTING  = 8\r
 };\r
 \r
 //\r
index 95ad870..9fbaf6b 100644 (file)
@@ -888,7 +888,7 @@ UsbEnumeratePort (
     // If overcurrent condition is cleared, enable the port again\r
     //\r
     if (!USB_BIT_IS_SET (PortState.PortStatus, USB_PORT_STAT_OVERCURRENT)) {\r
-      HubApi->SetPortFeature (HubIf, Port, USB_HUB_PORT_POWER);\r
+      HubApi->SetPortFeature (HubIf, Port, (EFI_USB_PORT_FEATURE) USB_HUB_PORT_POWER);\r
     }\r
 \r
   } else if (USB_BIT_IS_SET (PortState.PortChangeStatus, USB_PORT_STAT_C_CONNECTION)) {\r
index e14b2d7..3c22a0c 100644 (file)
@@ -100,7 +100,7 @@ EFI_STATUS
   IN USB_INTERFACE        *UsbIf\r
   );\r
 \r
-typedef struct _USB_HUB_API{\r
+struct _USB_HUB_API{\r
   USB_HUB_INIT                Init;\r
   USB_HUB_GET_PORT_STATUS     GetPortStatus;\r
   USB_HUB_CLEAR_PORT_CHANGE   ClearPortChange;\r
@@ -108,7 +108,7 @@ typedef struct _USB_HUB_API{
   USB_HUB_CLEAR_PORT_FEATURE  ClearPortFeature;\r
   USB_HUB_RESET_PORT          ResetPort;\r
   USB_HUB_RELEASE             Release;\r
-} USB_HUB_API;\r
+};\r
 \r
 USB_ENDPOINT_DESC*\r
 UsbGetEndpointDesc (\r
index 25332f4..cb54f03 100644 (file)
@@ -98,7 +98,7 @@ UsbHubCtrlClearPortFeature (
              USB_HUB_TARGET_PORT,\r
              USB_HUB_REQ_CLEAR_FEATURE,\r
              Feature,\r
-             Port + 1,\r
+             (UINT16) (Port + 1),\r
              NULL,\r
              0\r
              );\r
@@ -140,8 +140,8 @@ UsbHubCtrlClearTTBuffer (
   //\r
   // Check USB2.0 spec page 424 for wValue's encoding\r
   //\r
-  Value = (EpNum & 0x0F) | (DevAddr << 4) |\r
-          ((EpType & 0x03) << 11) | ((EpNum & 0x80) << 15);\r
+  Value = (UINT16) ((EpNum & 0x0F) | (DevAddr << 4) |\r
+          ((EpType & 0x03) << 11) | ((EpNum & 0x80) << 15));\r
 \r
   Status = UsbCtrlRequest (\r
              HubDev,\r
@@ -150,7 +150,7 @@ UsbHubCtrlClearTTBuffer (
              USB_HUB_TARGET_PORT,\r
              USB_HUB_REQ_CLEAR_TT,\r
              Value,\r
-             Port + 1,\r
+             (UINT16) (Port + 1),\r
              NULL,\r
              0\r
              );\r
@@ -265,7 +265,7 @@ UsbHubCtrlGetPortStatus (
              USB_HUB_TARGET_PORT,\r
              USB_HUB_REQ_GET_STATUS,\r
              0,\r
-             Port + 1,\r
+             (UINT16) (Port + 1),\r
              State,\r
              4\r
              );\r
@@ -300,7 +300,7 @@ UsbHubCtrlResetTT (
              USB_HUB_TARGET_HUB,\r
              USB_HUB_REQ_RESET_TT,\r
              0,\r
-             Port + 1,\r
+             (UINT16) (Port + 1),\r
              NULL,\r
              0\r
              );\r
@@ -376,7 +376,7 @@ UsbHubCtrlSetPortFeature (
              USB_HUB_TARGET_PORT,\r
              USB_HUB_REQ_SET_FEATURE,\r
              Feature,\r
-             Port + 1,\r
+             (UINT16) (Port + 1),\r
              NULL,\r
              0\r
              );\r
@@ -744,7 +744,7 @@ UsbHubInit (
   // for both gang/individual powered hubs.\r
   //\r
   for (Index = 0; Index < HubDesc.NumPorts; Index++) {\r
-    UsbHubCtrlSetPortFeature (HubIf->Device, Index, USB_HUB_PORT_POWER);\r
+    UsbHubCtrlSetPortFeature (HubIf->Device, Index, (EFI_USB_PORT_FEATURE) USB_HUB_PORT_POWER);\r
   }\r
 \r
   gBS->Stall (HubDesc.PwrOn2PwrGood * 2 * USB_STALL_1_MS);\r
@@ -852,7 +852,7 @@ UsbHubSetPortFeature (
 {\r
   EFI_STATUS              Status;\r
 \r
-  Status = UsbHubCtrlSetPortFeature (HubIf->Device, Port, Feature);\r
+  Status = UsbHubCtrlSetPortFeature (HubIf->Device, Port, (UINT8) Feature);\r
 \r
   return Status;\r
 }\r
@@ -879,7 +879,7 @@ UsbHubClearPortFeature (
 {\r
   EFI_STATUS              Status;\r
 \r
-  Status = UsbHubCtrlClearPortFeature (HubIf->Device, Port, Feature);\r
+  Status = UsbHubCtrlClearPortFeature (HubIf->Device, Port, (UINT8) Feature);\r
 \r
   return Status;\r
 }\r
@@ -907,7 +907,7 @@ UsbHubResetPort (
   UINTN                   Index;\r
   EFI_STATUS              Status;\r
 \r
-  Status  = UsbHubSetPortFeature (HubIf, Port, USB_HUB_PORT_RESET);\r
+  Status  = UsbHubSetPortFeature (HubIf, Port, (EFI_USB_PORT_FEATURE) USB_HUB_PORT_RESET);\r
 \r
   if (EFI_ERROR (Status)) {\r
     return Status;\r
@@ -1121,7 +1121,7 @@ UsbRootHubClearPortChange (
     Map = &mRootHubFeatureMap[Index];\r
 \r
     if (USB_BIT_IS_SET (PortState.PortChangeStatus, Map->ChangedBit)) {\r
-      UsbHcClearRootHubPortFeature (HubIf->Device->Bus, Port, Map->Feature);\r
+      UsbHcClearRootHubPortFeature (HubIf->Device->Bus, Port, (EFI_USB_PORT_FEATURE) Map->Feature);\r
     }\r
   }\r
 }\r
index 0238ea0..a5bcbb2 100644 (file)
@@ -25,6 +25,8 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 #ifndef _USB_HUB_H_\r
 #define _USB_HUB_H_\r
 \r
+#include <IndustryStandard/Usb.h>\r
+\r
 #define USB_ENDPOINT_ADDR(EpAddr) ((EpAddr) & 0x7F)\r
 #define USB_ENDPOINT_TYPE(Desc)   ((Desc)->Attributes & USB_ENDPOINT_TYPE_MASK)\r
 \r
@@ -89,7 +91,7 @@ enum {
   USB_HUB_SUBCLASS_CODE       = 0x00,\r
 \r
 \r
-  USB_HUB_LOOP                = 50,\r
+  USB_HUB_LOOP                = 50\r
 };\r
 \r
 #pragma pack(1)\r
@@ -110,7 +112,7 @@ typedef struct {
 \r
 typedef struct {\r
   UINT16                ChangedBit;\r
-  EFI_USB_PORT_FEATURE  Feature;\r
+  UINT8                 Feature;\r
 } USB_CHANGE_FEATURE_MAP;\r
 \r
 \r
index 216f948..28d73da 100644 (file)
@@ -518,7 +518,7 @@ UsbHcSyncInterruptTransfer (
                                UsbResult\r
                                );\r
   } else {\r
-    IsSlowDevice = (EFI_USB_SPEED_LOW == DevSpeed) ? TRUE : FALSE;\r
+    IsSlowDevice = (BOOLEAN) ((EFI_USB_SPEED_LOW == DevSpeed) ? TRUE : FALSE);\r
     Status = UsbBus->UsbHc->SyncInterruptTransfer (\r
                               UsbBus->UsbHc,\r
                               DevAddr,\r
@@ -634,7 +634,7 @@ UsbOpenHostProtoByChild (
     Status = gBS->OpenProtocol (\r
                     Bus->HostHandle,\r
                     &gEfiUsb2HcProtocolGuid,\r
-                    &Usb2Hc,\r
+                    (VOID **) &Usb2Hc,\r
                     mUsbBusDriverBinding.DriverBindingHandle,\r
                     Child,\r
                     EFI_OPEN_PROTOCOL_BY_CHILD_CONTROLLER\r
@@ -644,7 +644,7 @@ UsbOpenHostProtoByChild (
     Status = gBS->OpenProtocol (\r
                     Bus->HostHandle,\r
                     &gEfiUsbHcProtocolGuid,\r
-                    &UsbHc,\r
+                    (VOID **) &UsbHc,\r
                     mUsbBusDriverBinding.DriverBindingHandle,\r
                     Child,\r
                     EFI_OPEN_PROTOCOL_BY_CHILD_CONTROLLER\r
index 2ecc2c8..7407a36 100644 (file)
@@ -83,7 +83,7 @@ enum {
   USB_BUS_TPL               = TPL_NOTIFY,\r
 \r
   USB_INTERFACE_SIGNATURE   = EFI_SIGNATURE_32 ('U', 'S', 'B', 'I'),\r
-  USB_BUS_SIGNATURE         = EFI_SIGNATURE_32 ('U', 'S', 'B', 'B'),\r
+  USB_BUS_SIGNATURE         = EFI_SIGNATURE_32 ('U', 'S', 'B', 'B')\r
 };\r
 \r
 #define USB_BIT(a)                  ((UINTN)(1 << (a)))\r
@@ -110,7 +110,7 @@ typedef struct _EFI_USB_BUS_PROTOCOL {
 // Stands for the real USB device. Each device may\r
 // has several seperately working interfaces.\r
 //\r
-typedef struct _USB_DEVICE {\r
+struct _USB_DEVICE {\r
   USB_BUS                   *Bus;\r
 \r
   //\r
@@ -140,12 +140,12 @@ typedef struct _USB_DEVICE {
   UINT8                     ParentAddr;\r
   USB_INTERFACE             *ParentIf;\r
   UINT8                     ParentPort;       // Start at 0\r
-} USB_DEVICE;\r
+};\r
 \r
 //\r
 // Stands for different functions of USB device\r
 //\r
-typedef struct _USB_INTERFACE {\r
+struct _USB_INTERFACE {\r
   UINTN                     Signature;\r
   USB_DEVICE                *Device;\r
   USB_INTERFACE_DESC        *IfDesc;\r
@@ -179,12 +179,12 @@ typedef struct _USB_INTERFACE {
   // connected to EHCI.\r
   //\r
   UINT8                     MaxSpeed;\r
-} USB_INTERFACE;\r
+};\r
 \r
 //\r
 // Stands for the current USB Bus\r
 //\r
-typedef struct _USB_BUS {\r
+struct _USB_BUS {\r
   UINTN                     Signature;\r
   EFI_USB_BUS_PROTOCOL      BusId;\r
 \r
@@ -201,7 +201,7 @@ typedef struct _USB_BUS {
   // for root hub. Device with address i is at Devices[i].\r
   //\r
   USB_DEVICE                *Devices[USB_MAX_DEVICES];\r
-} USB_BUS;\r
+};\r
 \r
 extern EFI_USB_IO_PROTOCOL           mUsbIoProtocol;\r
 extern EFI_DRIVER_BINDING_PROTOCOL   mUsbBusDriverBinding;\r
index 3ea788c..a4a19e4 100644 (file)
@@ -174,7 +174,7 @@ USBKeyboardDriverBindingSupported (
   OpenStatus = gBS->OpenProtocol (\r
                       Controller,\r
                       &gEfiUsbIoProtocolGuid,\r
-                      &UsbIo,\r
+                      (VOID **) &UsbIo,\r
                       This->DriverBindingHandle,\r
                       Controller,\r
                       EFI_OPEN_PROTOCOL_BY_DRIVER\r
@@ -244,7 +244,7 @@ USBKeyboardDriverBindingStart (
   Status = gBS->OpenProtocol (\r
                   Controller,\r
                   &gEfiUsbIoProtocolGuid,\r
-                  &UsbIo,\r
+                  (VOID **) &UsbIo,\r
                   This->DriverBindingHandle,\r
                   Controller,\r
                   EFI_OPEN_PROTOCOL_BY_DRIVER\r
@@ -500,12 +500,11 @@ USBKeyboardDriverBindingStop (
   EFI_STATUS                  Status;\r
   EFI_SIMPLE_TEXT_INPUT_PROTOCOL *SimpleInput;\r
   USB_KB_DEV                  *UsbKeyboardDevice;\r
-  EFI_USB_IO_PROTOCOL         *UsbIo;\r
 \r
   Status = gBS->OpenProtocol (\r
                   Controller,\r
                   &gEfiSimpleTextInProtocolGuid,\r
-                  &SimpleInput,\r
+                  (VOID **) &SimpleInput,\r
                   This->DriverBindingHandle,\r
                   Controller,\r
                   EFI_OPEN_PROTOCOL_BY_DRIVER\r
@@ -526,7 +525,6 @@ USBKeyboardDriverBindingStop (
         Controller\r
         );\r
 \r
-  UsbIo = UsbKeyboardDevice->UsbIo;\r
   //\r
   // Uninstall the Asyn Interrupt Transfer from this device\r
   // will disable the key data input from this device\r
@@ -605,12 +603,9 @@ USBKeyboardReset (
 {\r
   EFI_STATUS          Status;\r
   USB_KB_DEV          *UsbKeyboardDevice;\r
-  EFI_USB_IO_PROTOCOL *UsbIo;\r
 \r
   UsbKeyboardDevice = USB_KB_DEV_FROM_THIS (This);\r
 \r
-  UsbIo             = UsbKeyboardDevice->UsbIo;\r
-\r
   KbdReportStatusCode (\r
     UsbKeyboardDevice->DevicePath,\r
     EFI_PROGRESS_CODE,\r
index 41286d6..44914af 100644 (file)
@@ -206,9 +206,6 @@ InitUSBKeyboard (
   UINT8               Duration;\r
   EFI_STATUS          Status;\r
   UINT32              TransferResult;\r
-  EFI_USB_IO_PROTOCOL *UsbIo;\r
-\r
-  UsbIo = UsbKeyboardDevice->UsbIo;\r
 \r
   KbdReportStatusCode (\r
     UsbKeyboardDevice->DevicePath,\r
@@ -361,19 +358,16 @@ KeyboardHandler (
   UINT8               Index;\r
   UINT8               Index2;\r
   BOOLEAN             Down;\r
-  EFI_STATUS          Status;\r
   BOOLEAN             KeyRelease;\r
   BOOLEAN             KeyPress;\r
   UINT8               SavedTail;\r
   USB_KEY             UsbKey;\r
   UINT8               NewRepeatKey;\r
   UINT32              UsbStatus;\r
-  UINT8               *DataPtr;\r
 \r
   ASSERT (Context);\r
 \r
   NewRepeatKey      = 0;\r
-  DataPtr           = (UINT8 *) Data;\r
   UsbKeyboardDevice = (USB_KB_DEV *) Context;\r
   UsbIo             = UsbKeyboardDevice->UsbIo;\r
 \r
@@ -413,7 +407,7 @@ KeyboardHandler (
     // Delete & Submit this interrupt again\r
     //\r
 \r
-    Status = UsbIo->UsbAsyncInterruptTransfer (\r
+    UsbIo->UsbAsyncInterruptTransfer (\r
                       UsbIo,\r
                       UsbKeyboardDevice->IntEndpointDescriptor.EndpointAddress,\r
                       FALSE,\r
index c48dee5..8fcbed7 100644 (file)
@@ -84,7 +84,7 @@ enum {
 \r
   USB_MASS_CMD_SUCCESS    = 0,\r
   USB_MASS_CMD_FAIL,\r
-  USB_MASS_CMD_PERSISTENT,\r
+  USB_MASS_CMD_PERSISTENT\r
 };\r
 \r
 typedef\r
index 80fdf47..e9b0a2a 100644 (file)
@@ -89,7 +89,7 @@ UsbBootPutUint16 (
   IN UINT16                  Data16\r
   )\r
 {\r
-  Data16 = USB_BOOT_SWAP16 (Data16);\r
+  Data16 = (UINT16) (USB_BOOT_SWAP16 (Data16));\r
   CopyMem (Buf, &Data16, sizeof (UINT16));\r
 }\r
 \r
@@ -125,7 +125,7 @@ UsbBootRequestSense (
   ZeroMem (&SenseData, sizeof (USB_BOOT_REQUEST_SENSE_DATA));\r
 \r
   SenseCmd.OpCode   = USB_BOOT_REQUEST_SENSE_OPCODE;\r
-  SenseCmd.Lun      = USB_BOOT_LUN (UsbMass->Lun);\r
+  SenseCmd.Lun      = (UINT8) (USB_BOOT_LUN (UsbMass->Lun));\r
   SenseCmd.AllocLen = sizeof (USB_BOOT_REQUEST_SENSE_DATA);\r
 \r
   Status = Transport->ExecCommand (\r
@@ -364,7 +364,7 @@ UsbBootIsUnitReady (
   ZeroMem (&TestCmd, sizeof (USB_BOOT_TEST_UNIT_READY_CMD));\r
 \r
   TestCmd.OpCode  = USB_BOOT_TEST_UNIT_READY_OPCODE;\r
-  TestCmd.Lun     = USB_BOOT_LUN (UsbMass->Lun);\r
+  TestCmd.Lun     = (UINT8) (USB_BOOT_LUN (UsbMass->Lun));\r
 \r
   return UsbBootExecCmdWithRetry (\r
            UsbMass,\r
@@ -407,7 +407,7 @@ UsbBootInquiry (
   ZeroMem (&InquiryData, sizeof (USB_BOOT_INQUIRY_DATA));\r
 \r
   InquiryCmd.OpCode   = USB_BOOT_INQUIRY_OPCODE;\r
-  InquiryCmd.Lun      = USB_BOOT_LUN (UsbMass->Lun);\r
+  InquiryCmd.Lun      = (UINT8) (USB_BOOT_LUN (UsbMass->Lun));\r
   InquiryCmd.AllocLen = sizeof (InquiryData);\r
 \r
   Status = UsbBootExecCmdWithRetry (\r
@@ -423,8 +423,8 @@ UsbBootInquiry (
     return Status;\r
   }\r
 \r
-  UsbMass->Pdt          = USB_BOOT_PDT (InquiryData.Pdt);\r
-  Media->RemovableMedia = USB_BOOT_REMOVABLE (InquiryData.Removable);\r
+  UsbMass->Pdt          = (UINT8) (USB_BOOT_PDT (InquiryData.Pdt));\r
+  Media->RemovableMedia = (BOOLEAN) (USB_BOOT_REMOVABLE (InquiryData.Removable));\r
   //\r
   // Default value 512 Bytes, in case no media present at first time\r
   //\r
@@ -466,7 +466,7 @@ UsbBootReadCapacity (
   ZeroMem (&CapacityData, sizeof (USB_BOOT_READ_CAPACITY_DATA));\r
 \r
   CapacityCmd.OpCode = USB_BOOT_READ_CAPACITY_OPCODE;\r
-  CapacityCmd.Lun    = USB_BOOT_LUN (UsbMass->Lun);\r
+  CapacityCmd.Lun    = (UINT8) (USB_BOOT_LUN (UsbMass->Lun));\r
 \r
   Status = UsbBootExecCmdWithRetry (\r
              UsbMass,\r
@@ -742,7 +742,7 @@ UsbBootReadBlocks (
     ZeroMem (&ReadCmd, sizeof (USB_BOOT_READ10_CMD));\r
 \r
     ReadCmd.OpCode  = USB_BOOT_READ10_OPCODE;\r
-    ReadCmd.Lun     = USB_BOOT_LUN (UsbMass->Lun);\r
+    ReadCmd.Lun     = (UINT8) (USB_BOOT_LUN (UsbMass->Lun));\r
     UsbBootPutUint32 (ReadCmd.Lba, Lba);\r
     UsbBootPutUint16 (ReadCmd.TransferLen, Count);\r
 \r
@@ -822,7 +822,7 @@ UsbBootWriteBlocks (
     ZeroMem (&WriteCmd, sizeof (USB_BOOT_WRITE10_CMD));\r
 \r
     WriteCmd.OpCode = USB_BOOT_WRITE10_OPCODE;\r
-    WriteCmd.Lun    = USB_BOOT_LUN (UsbMass->Lun);\r
+    WriteCmd.Lun    = (UINT8) (USB_BOOT_LUN (UsbMass->Lun));\r
     UsbBootPutUint32 (WriteCmd.Lba, Lba);\r
     UsbBootPutUint16 (WriteCmd.TransferLen, Count);\r
 \r
index cee70be..dc4b43f 100644 (file)
@@ -101,7 +101,7 @@ enum {
   USB_PDT_DIRECT_ACCESS           = 0x00,       // Direct access device\r
   USB_PDT_CDROM                   = 0x05,       // CDROM\r
   USB_PDT_OPTICAL                 = 0x07,       // Non-CD optical disks\r
-  USB_PDT_SIMPLE_DIRECT           = 0x0E,       // Simplified direct access device\r
+  USB_PDT_SIMPLE_DIRECT           = 0x0E        // Simplified direct access device\r
 };\r
 \r
 //\r
index 0d6cca1..939ba39 100644 (file)
@@ -182,7 +182,7 @@ UsbBotSendCommand (
   Cbw.Signature = USB_BOT_CBW_SIGNATURE;\r
   Cbw.Tag       = UsbBot->CbwTag;\r
   Cbw.DataLen   = TransLen;\r
-  Cbw.Flag      = ((DataDir == EfiUsbDataIn) ? 0x80 : 0);\r
+  Cbw.Flag      = (UINT8) ((DataDir == EfiUsbDataIn) ? 0x80 : 0);\r
   Cbw.Lun       = 0;\r
   Cbw.CmdLen    = CmdLen;\r
 \r
index e11e9c5..8ccc48b 100644 (file)
@@ -60,7 +60,7 @@ enum {
   //\r
   USB_BOT_CBW_TIMEOUT      = 1 * USB_MASS_STALL_1_S,\r
   USB_BOT_CSW_TIMEOUT      = 1 * USB_MASS_STALL_1_S,\r
-  USB_BOT_RESET_TIMEOUT    = 3 * USB_MASS_STALL_1_S,\r
+  USB_BOT_RESET_TIMEOUT    = 3 * USB_MASS_STALL_1_S\r
 };\r
 \r
 //\r
index d4a9f70..61954e6 100644 (file)
@@ -37,7 +37,7 @@ enum {
   //\r
   // Usb Cbi transfer timeout\r
   //\r
-  USB_CBI_RESET_TIMEOUT         = 1 * USB_MASS_STALL_1_S,\r
+  USB_CBI_RESET_TIMEOUT         = 1 * USB_MASS_STALL_1_S\r
 };\r
 \r
 //\r
index a7c3e82..33c3755 100644 (file)
@@ -344,7 +344,7 @@ USBMassDriverBindingSupported (
   Status = gBS->OpenProtocol (\r
                   Controller,\r
                   &gEfiUsbIoProtocolGuid,\r
-                  &UsbIo,\r
+                  (VOID **) &UsbIo,\r
                   This->DriverBindingHandle,\r
                   Controller,\r
                   EFI_OPEN_PROTOCOL_BY_DRIVER\r
@@ -421,7 +421,7 @@ USBMassDriverBindingStart (
   Status = gBS->OpenProtocol (\r
                   Controller,\r
                   &gEfiUsbIoProtocolGuid,\r
-                  &UsbIo,\r
+                  (VOID **) &UsbIo,\r
                   This->DriverBindingHandle,\r
                   Controller,\r
                   EFI_OPEN_PROTOCOL_BY_DRIVER\r
@@ -547,7 +547,7 @@ USBMassDriverBindingStop (
   Status = gBS->OpenProtocol (\r
                   Controller,\r
                   &gEfiBlockIoProtocolGuid,\r
-                  &BlockIo,\r
+                  (VOID **) &BlockIo,\r
                   This->DriverBindingHandle,\r
                   Controller,\r
                   EFI_OPEN_PROTOCOL_GET_PROTOCOL\r
index 39bc87c..140a990 100644 (file)
@@ -33,10 +33,10 @@ typedef struct _USB_MASS_DEVICE USB_MASS_DEVICE;
 #include "UsbMassBoot.h"\r
 \r
 enum {\r
-  USB_MASS_SIGNATURE= EFI_SIGNATURE_32 ('U', 's', 'b', 'K'),\r
+  USB_MASS_SIGNATURE= EFI_SIGNATURE_32 ('U', 's', 'b', 'K')\r
 };\r
 \r
-typedef struct _USB_MASS_DEVICE {\r
+struct _USB_MASS_DEVICE {\r
   UINT32                Signature;\r
   EFI_HANDLE            Controller;\r
   EFI_USB_IO_PROTOCOL   *UsbIo;\r
index a4c9e77..01c37c1 100644 (file)
@@ -208,7 +208,7 @@ USBMouseDriverBindingSupported (
   OpenStatus = gBS->OpenProtocol (\r
                       Controller,\r
                       &gEfiUsbIoProtocolGuid,\r
-                      &UsbIo,\r
+                      (VOID **) &UsbIo,\r
                       This->DriverBindingHandle,\r
                       Controller,\r
                       EFI_OPEN_PROTOCOL_BY_DRIVER\r
@@ -279,7 +279,7 @@ USBMouseDriverBindingStart (
   Status = gBS->OpenProtocol (\r
                   Controller,\r
                   &gEfiUsbIoProtocolGuid,\r
-                  &UsbIo,\r
+                  (VOID **) &UsbIo,\r
                   This->DriverBindingHandle,\r
                   Controller,\r
                   EFI_OPEN_PROTOCOL_BY_DRIVER\r
@@ -513,7 +513,7 @@ USBMouseDriverBindingStop (
   Status = gBS->OpenProtocol (\r
                   Controller,\r
                   &gEfiSimplePointerProtocolGuid,\r
-                  &SimplePointerProtocol,\r
+                  (VOID **) &SimplePointerProtocol,\r
                   This->DriverBindingHandle,\r
                   Controller,\r
                   EFI_OPEN_PROTOCOL_GET_PROTOCOL\r
@@ -944,12 +944,9 @@ UsbMouseReset (
   )\r
 {\r
   USB_MOUSE_DEV       *UsbMouseDevice;\r
-  EFI_USB_IO_PROTOCOL *UsbIo;\r
 \r
   UsbMouseDevice  = USB_MOUSE_DEV_FROM_MOUSE_PROTOCOL (This);\r
 \r
-  UsbIo           = UsbMouseDevice->UsbIo;\r
-\r
   MouseReportStatusCode (\r
     UsbMouseDevice->DevicePath,\r
     EFI_PROGRESS_CODE,\r
index 7ff2b99..3d6e3ed 100644 (file)
@@ -1,14 +1,14 @@
 /** @file\r
 Private functions used by PCD DXE driver.\r
 \r
-Copyright (c) 2006 - 2007, 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
-http://opensource.org/licenses/bsd-license.php                                            \r
-                                                                                          \r
-THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,                     \r
-WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.             \r
+Copyright (c) 2006 - 2007, 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
+http://opensource.org/licenses/bsd-license.php\r
+\r
+THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
+WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
 \r
 \r
 Module Name: Service.h\r
@@ -41,9 +41,9 @@ Module Name: Service.h
 // PCD_DXE_DATABASE_GENTOOL_VERSION is defined in Autogen.h\r
 // and generated by PCD Database generation tool.\r
 //\r
-#if (PCD_SERVICE_DXE_DRIVER_VERSION != PCD_DXE_SERVICE_DRIVER_AUTOGEN_VERSION)\r
+//#if (PCD_SERVICE_DXE_DRIVER_VERSION != PCD_DXE_SERVICE_DRIVER_AUTOGEN_VERSION)\r
 //  #error "Please make sure the version of PCD Service DXE Driver and PCD DXE Database Generation Tool matches"\r
-#endif\r
+//#endif\r
 \r
 //\r
 // Protocol Interface function declaration.\r
@@ -395,7 +395,7 @@ ExGetWorker (
   IN CONST EFI_GUID         *Guid,\r
   IN UINTN                  ExTokenNumber,\r
   IN UINTN                  GetSize\r
-  ) \r
+  )\r
 ;\r
 \r
 UINT32\r
@@ -403,7 +403,7 @@ GetSkuEnabledTokenNumber (
   UINT32 LocalTokenNumber,\r
   UINTN  Size,\r
   BOOLEAN IsPeiDb\r
-  ) \r
+  )\r
 ;\r
 \r
 EFI_STATUS\r
@@ -445,7 +445,7 @@ BuildPcdDxeDataBase (
 );\r
 \r
 \r
-UINTN           \r
+UINTN\r
 GetExPcdTokenNumber (\r
   IN CONST EFI_GUID             *Guid,\r
   IN UINT32                     ExTokenNumber\r
@@ -454,7 +454,7 @@ GetExPcdTokenNumber (
 \r
 \r
 \r
-EFI_STATUS           \r
+EFI_STATUS\r
 ExGetNextTokeNumber (\r
   IN      CONST EFI_GUID    *Guid,\r
   IN OUT  UINTN             *TokenNumber,\r
index fcfdda3..eb7a45d 100644 (file)
@@ -1,14 +1,14 @@
 /** @file\r
 Private functions used by PCD PEIM.\r
 \r
-Copyright (c) 2006, 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
-http://opensource.org/licenses/bsd-license.php                                            \r
-                                                                                          \r
-THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,                     \r
-WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.             \r
+Copyright (c) 2006, 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
+http://opensource.org/licenses/bsd-license.php\r
+\r
+THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
+WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
 \r
 \r
 Module Name: Service.h\r
@@ -41,9 +41,9 @@ Module Name: Service.h
 // PCD_PEI_DATABASE_GENTOOL_VERSION is defined in Autogen.h\r
 // and generated by PCD Database generation tool.\r
 //\r
-#if (PCD_SERVICE_PEIM_VERSION != PCD_PEI_SERVICE_DRIVER_AUTOGEN_VERSION )\r
+//#if (PCD_SERVICE_PEIM_VERSION != PCD_PEI_SERVICE_DRIVER_AUTOGEN_VERSION )\r
 //  #error "Please make sure the version of PCD Service PEIM and PCD PEI Database Generation Tool matches"\r
-#endif\r
+//#endif\r
 \r
 //\r
 // PPI Interface Implementation Declaration.\r
@@ -401,7 +401,7 @@ typedef struct {
 } EX_PCD_ENTRY_ATTRIBUTE;\r
 \r
 \r
-UINTN           \r
+UINTN\r
 GetExPcdTokenNumber (\r
   IN CONST EFI_GUID             *Guid,\r
   IN UINTN                      ExTokenNumber\r
index c2c0242..74d93f5 100644 (file)
@@ -150,7 +150,6 @@ Returns:
 --*/\r
 {\r
   UINTN                 Address;\r
-  VOID                  *PlabelConvertAddress;\r
   UINT64                VirtEndOfRange;\r
   EFI_MEMORY_DESCRIPTOR *VirtEntry;\r
   UINTN                 Index;\r
@@ -177,7 +176,6 @@ Returns:
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
-  PlabelConvertAddress  = NULL;\r
   VirtEntry             = mVirtualMap;\r
   for (Index = 0; Index < mVirtualMapMaxIndex; Index++) {\r
     //\r
@@ -368,9 +366,9 @@ Returns:
   RuntimeDriverCalculateEfiHdrCrc (&gRT->Hdr);\r
 \r
   //\r
-  // BugBug: PI requires System Configuration Tables Conversion. \r
+  // BugBug: PI requires System Configuration Tables Conversion.\r
   // Currently, we do not implement it.\r
-  // \r
+  //\r
 \r
   //\r
   // Convert the runtime fields of the EFI System Table and recompute the CRC-32\r