Partially make EdkModulePkg pass intel IPF compiler with /W4 /WX switched on.
authorxli24 <xli24@de2fecce-e211-0410-80a6-f3fac2684e05>
Thu, 25 Jan 2007 06:15:58 +0000 (06:15 +0000)
committerxli24 <xli24@de2fecce-e211-0410-80a6-f3fac2684e05>
Thu, 25 Jan 2007 06:15:58 +0000 (06:15 +0000)
git-svn-id: https://edk2.tianocore.org/svn/edk2/trunk@2313 de2fecce-e211-0410-80a6-f3fac2684e05

123 files changed:
edk2/EdkModulePkg/Application/HelloWorld/HelloWorld.c
edk2/EdkModulePkg/Bus/Pci/AtapiPassThru/Dxe/AtapiPassThru.c
edk2/EdkModulePkg/Bus/Pci/CirrusLogic/Dxe/CirrusLogic5430UgaDraw.c
edk2/EdkModulePkg/Bus/Pci/CirrusLogic/Dxe/ComponentName.c
edk2/EdkModulePkg/Bus/Pci/Ehci/Dxe/Ehci.c
edk2/EdkModulePkg/Bus/Pci/Ehci/Dxe/EhciMem.c
edk2/EdkModulePkg/Bus/Pci/Ehci/Dxe/EhciReg.c
edk2/EdkModulePkg/Bus/Pci/Ehci/Dxe/EhciSched.c
edk2/EdkModulePkg/Bus/Pci/IdeBus/Dxe/DriverConfiguration.c
edk2/EdkModulePkg/Bus/Pci/IdeBus/Dxe/ata.c
edk2/EdkModulePkg/Bus/Pci/IdeBus/Dxe/atapi.c
edk2/EdkModulePkg/Bus/Pci/IdeBus/Dxe/ide.c
edk2/EdkModulePkg/Bus/Pci/IdeBus/Dxe/idebus.c
edk2/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciCommand.c
edk2/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciEnumeratorSupport.c
edk2/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciIo.c
edk2/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciLib.c
edk2/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciOptionRomSupport.c
edk2/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciResourceSupport.c
edk2/EdkModulePkg/Bus/Pci/PciBus/Dxe/PciRomTable.c
edk2/EdkModulePkg/Bus/Pci/Uhci/Dxe/uhchlp.c
edk2/EdkModulePkg/Bus/Pci/Uhci/Dxe/uhci.c
edk2/EdkModulePkg/Bus/Scsi/ScsiBus/Dxe/ScsiBus.c
edk2/EdkModulePkg/Bus/Scsi/ScsiDisk/Dxe/ScsiDisk.c
edk2/EdkModulePkg/Bus/Usb/UsbCbi/Dxe/Cbi0/Cbi0.c
edk2/EdkModulePkg/Bus/Usb/UsbCbi/Dxe/Cbi1/cbi1.c
edk2/EdkModulePkg/Bus/Usb/UsbKb/Dxe/efikey.c
edk2/EdkModulePkg/Bus/Usb/UsbKb/Dxe/keyboard.c
edk2/EdkModulePkg/Bus/Usb/UsbMassStorage/Dxe/UsbMassStorage.c
edk2/EdkModulePkg/Bus/Usb/UsbMouse/Dxe/usbmouse.c
edk2/EdkModulePkg/Core/Dxe/Dispatcher/Dispatcher.c
edk2/EdkModulePkg/Core/Dxe/Dispatcher/dependency.c
edk2/EdkModulePkg/Core/Dxe/DxeMain/DxeProtocolNotify.c
edk2/EdkModulePkg/Core/Dxe/Event/event.c
edk2/EdkModulePkg/Core/Dxe/FwVol/Ffs.c
edk2/EdkModulePkg/Core/Dxe/Gcd/gcd.c
edk2/EdkModulePkg/Core/Dxe/Hand/DriverSupport.c
edk2/EdkModulePkg/Core/Dxe/Hand/handle.c
edk2/EdkModulePkg/Core/Dxe/Image/Image.c
edk2/EdkModulePkg/Core/Dxe/Mem/Page.c
edk2/EdkModulePkg/Core/Dxe/Mem/pool.c
edk2/EdkModulePkg/Core/DxeIplPeim/DxeLoad.c
edk2/EdkModulePkg/Core/Pei/BootMode/BootMode.c
edk2/EdkModulePkg/Core/Pei/Dependency/dependency.c
edk2/EdkModulePkg/Core/Pei/Ipf/SwitchToCacheMode.c
edk2/EdkModulePkg/Library/BaseUefiTianoDecompressLib/BaseUefiTianoDecompressLib.c
edk2/EdkModulePkg/Library/BaseUefiTianoDecompressLib/BaseUefiTianoDecompressLib.msa
edk2/EdkModulePkg/Library/BaseUefiTianoDecompressLib/BaseUefiTianoDecompressLibInternals.h [new file with mode: 0644]
edk2/EdkModulePkg/Library/EdkDxeRuntimeSalLib/Ipf/EsalServiceLib.c
edk2/EdkModulePkg/Library/EdkGraphicsLib/Graphics.c
edk2/EdkModulePkg/Library/EdkIfrSupportLib/IfrCommon.c
edk2/EdkModulePkg/Library/EdkIfrSupportLib/IfrVariable.c
edk2/EdkModulePkg/Library/EdkPeCoffLoaderLib/EdkPeCoffLoader.c
edk2/EdkModulePkg/Library/EdkPeCoffLoaderLib/EdkPeCoffLoaderLib.msa
edk2/EdkModulePkg/Library/EdkPeCoffLoaderLib/EdkPeCoffLoaderLibInternals.h [new file with mode: 0644]
edk2/EdkModulePkg/Library/EdkUefiRuntimeLib/Ipf/RuntimeLib.c
edk2/EdkModulePkg/Universal/Console/ConSplitter/Dxe/ConSplitter.c
edk2/EdkModulePkg/Universal/Console/ConSplitter/Dxe/ConSplitter.h
edk2/EdkModulePkg/Universal/Console/ConSplitter/Dxe/ConSplitterGraphics.c
edk2/EdkModulePkg/Universal/Console/GraphicsConsole/Dxe/GraphicsConsole.c
edk2/EdkModulePkg/Universal/Console/Terminal/Dxe/Terminal.c
edk2/EdkModulePkg/Universal/Console/Terminal/Dxe/TerminalConIn.c
edk2/EdkModulePkg/Universal/DataHub/DataHub/Dxe/DataHub.c
edk2/EdkModulePkg/Universal/DebugSupport/Dxe/ipf/plDebugSupport.h
edk2/EdkModulePkg/Universal/Debugger/Debugport/Dxe/DebugPort.c
edk2/EdkModulePkg/Universal/DevicePath/Dxe/DevicePathFromText.c
edk2/EdkModulePkg/Universal/DevicePath/Dxe/DevicePathToText.c
edk2/EdkModulePkg/Universal/Disk/Partition/Dxe/Mbr.c
edk2/EdkModulePkg/Universal/Disk/Partition/Dxe/Partition.c
edk2/EdkModulePkg/Universal/Ebc/Dxe/EbcExecute.c
edk2/EdkModulePkg/Universal/Ebc/Dxe/Ipf/EbcSupport.c
edk2/EdkModulePkg/Universal/EmuVariable/RuntimeDxe/EmuVariable.c
edk2/EdkModulePkg/Universal/EmuVariable/RuntimeDxe/Ipf/InitVariable.c
edk2/EdkModulePkg/Universal/FirmwareVolume/FaultTolerantWriteLite/Dxe/FtwWorkSpace.c
edk2/EdkModulePkg/Universal/GenericMemoryTest/Pei/BaseMemoryTest.c
edk2/EdkModulePkg/Universal/MonotonicCounter/RuntimeDxe/MonotonicCounter.c
edk2/EdkModulePkg/Universal/Network/PxeBc/Dxe/Bc.h
edk2/EdkModulePkg/Universal/Network/PxeBc/Dxe/bc.c
edk2/EdkModulePkg/Universal/Network/PxeBc/Dxe/dhcp.h
edk2/EdkModulePkg/Universal/Network/PxeBc/Dxe/pxe_bc_dhcp.c
edk2/EdkModulePkg/Universal/Network/PxeBc/Dxe/pxe_bc_mtftp.c
edk2/EdkModulePkg/Universal/Network/PxeBc/Dxe/pxe_loadfile.c
edk2/EdkModulePkg/Universal/Network/Snp32_64/Dxe/Snp.h
edk2/EdkModulePkg/Universal/Network/Snp32_64/Dxe/callback.c
edk2/EdkModulePkg/Universal/Network/Snp32_64/Dxe/get_status.c
edk2/EdkModulePkg/Universal/Network/Snp32_64/Dxe/initialize.c
edk2/EdkModulePkg/Universal/Network/Snp32_64/Dxe/mcast_ip_to_mac.c
edk2/EdkModulePkg/Universal/Network/Snp32_64/Dxe/nvdata.c
edk2/EdkModulePkg/Universal/Network/Snp32_64/Dxe/receive.c
edk2/EdkModulePkg/Universal/Network/Snp32_64/Dxe/receive_filters.c
edk2/EdkModulePkg/Universal/Network/Snp32_64/Dxe/reset.c
edk2/EdkModulePkg/Universal/Network/Snp32_64/Dxe/snp.c
edk2/EdkModulePkg/Universal/Network/Snp32_64/Dxe/station_address.c
edk2/EdkModulePkg/Universal/Network/Snp32_64/Dxe/transmit.c
edk2/EdkModulePkg/Universal/PCD/Dxe/Pcd.c
edk2/EdkModulePkg/Universal/PCD/Dxe/Service.c
edk2/EdkModulePkg/Universal/PCD/Pei/Pcd.c
edk2/EdkModulePkg/Universal/PCD/Pei/Service.c
edk2/EdkModulePkg/Universal/Runtime/RuntimeDxe/Crc32.c
edk2/EdkModulePkg/Universal/Runtime/RuntimeDxe/Runtime.c
edk2/EdkModulePkg/Universal/StatusCode/Dxe/DataHubStatusCodeWorker.c
edk2/EdkModulePkg/Universal/StatusCode/Dxe/Ipf/DxeStatusCodeIpf.c
edk2/EdkModulePkg/Universal/StatusCode/Dxe/SerialStatusCodeWorker.c
edk2/EdkModulePkg/Universal/StatusCode/Pei/MemoryStausCodeWorker.c
edk2/EdkModulePkg/Universal/StatusCode/Pei/SerialStatusCodeWorker.c
edk2/EdkModulePkg/Universal/UserInterface/DriverSample/DriverSample.c
edk2/EdkModulePkg/Universal/UserInterface/HiiDataBase/Dxe/Fonts.c
edk2/EdkModulePkg/Universal/UserInterface/HiiDataBase/Dxe/Forms.c
edk2/EdkModulePkg/Universal/UserInterface/HiiDataBase/Dxe/HiiDatabase.c
edk2/EdkModulePkg/Universal/UserInterface/HiiDataBase/Dxe/Package.c
edk2/EdkModulePkg/Universal/UserInterface/HiiDataBase/Dxe/Strings.c
edk2/EdkModulePkg/Universal/UserInterface/SetupBrowser/Dxe/Boolean.c
edk2/EdkModulePkg/Universal/UserInterface/SetupBrowser/Dxe/InputHandler.c
edk2/EdkModulePkg/Universal/UserInterface/SetupBrowser/Dxe/Presentation.c
edk2/EdkModulePkg/Universal/UserInterface/SetupBrowser/Dxe/Print.h
edk2/EdkModulePkg/Universal/UserInterface/SetupBrowser/Dxe/ProcessOptions.c
edk2/EdkModulePkg/Universal/UserInterface/SetupBrowser/Dxe/Setup.c
edk2/EdkModulePkg/Universal/UserInterface/SetupBrowser/Dxe/Ui.c
edk2/EdkModulePkg/Universal/Variable/Pei/Variable.c
edk2/EdkModulePkg/Universal/Variable/RuntimeDxe/Ipf/InitVariable.c
edk2/EdkModulePkg/Universal/Variable/RuntimeDxe/Variable.c
edk2/EdkModulePkg/Universal/Variable/RuntimeDxe/reclaim.c
edk2/EdkModulePkg/Universal/WatchdogTimer/Dxe/WatchDogTimer.c

index d3714eb..1870510 100644 (file)
@@ -12,6 +12,7 @@
 \r
 **/\r
 \r
+STATIC\r
 VOID\r
 Print (\r
   IN CONST CHAR16  *Format,\r
@@ -27,6 +28,13 @@ Print (
   return;\r
 }\r
 \r
+EFI_STATUS\r
+EFIAPI\r
+UefiMain (\r
+  IN EFI_HANDLE        ImageHandle,\r
+  IN EFI_SYSTEM_TABLE  *SystemTable\r
+  );\r
+\r
 EFI_STATUS\r
 EFIAPI\r
 UefiMain (\r
index f11158b..28be3f7 100644 (file)
@@ -45,7 +45,7 @@ static IDE_BASE_REGISTERS   gAtapiIoPortRegisters[2] = {
   { 0x170, { 0x171 }, 0x172, 0x173, 0x174, 0x175, 0x176, { 0x177 }, { 0x376 }, 0x377, 0 } \r
 };\r
 \r
-static SCSI_COMMAND_SET     gEndTable = { 0xff, 0xff };\r
+static SCSI_COMMAND_SET     gEndTable = { 0xff, (DATA_DIRECTION) 0xff };\r
 \r
 ///\r
 /// This table contains all the supported ATAPI commands.\r
@@ -83,7 +83,7 @@ static SCSI_COMMAND_SET     gSupportedATAPICommands[] = {
   { OP_WRITE_10,                    DataOut },\r
   { OP_WRITE_12,                    DataOut },\r
   { OP_WRITE_AND_VERIFY,            DataOut },\r
-  { 0xff,                           0xff    } \r
+  { 0xff,                           (DATA_DIRECTION) 0xff    } \r
 };\r
 \r
 static CHAR16               *gControllerNameString  = (CHAR16 *) L"ATAPI Controller";\r
@@ -595,14 +595,8 @@ AtapiScsiPassThruBuildDevicePath (
   IN OUT EFI_DEVICE_PATH_PROTOCOL       **DevicePath\r
   )\r
 {\r
-  ATAPI_SCSI_PASS_THRU_DEV  *AtapiScsiPrivate;\r
   EFI_DEV_PATH              *Node;\r
 \r
-  //\r
-  // Retrieve Device Private Data Structure.\r
-  //\r
-  AtapiScsiPrivate = ATAPI_SCSI_PASS_THRU_DEV_FROM_THIS (This);\r
-\r
   //\r
   // Validate parameters passed in.\r
   //\r
@@ -995,7 +989,7 @@ SubmitBlockingIoCommand (
                           PacketCommand,\r
                           Packet->DataBuffer,\r
                           &(Packet->TransferLength),\r
-                          Packet->DataDirection,\r
+                          (DATA_DIRECTION) Packet->DataDirection,\r
                           TimeoutInMicroSeconds\r
                           );\r
   if (!EFI_ERROR (PacketCommandStatus) || (Packet->SenseData == NULL)) {\r
@@ -1319,11 +1313,9 @@ AtapiPassThruPioReadWriteData (
     //\r
     // get current data transfer size from Cylinder Registers.\r
     //\r
-    WordCount =\r
-      (\r
-        (ReadPortB (AtapiScsiPrivate->PciIo, AtapiScsiPrivate->IoPort->CylinderMsb) << 8) |\r
-        ReadPortB (AtapiScsiPrivate->PciIo, AtapiScsiPrivate->IoPort->CylinderLsb)\r
-      ) & 0xffff;\r
+    WordCount = ReadPortB (AtapiScsiPrivate->PciIo, AtapiScsiPrivate->IoPort->CylinderMsb) << 8;\r
+    WordCount = WordCount | ReadPortB (AtapiScsiPrivate->PciIo, AtapiScsiPrivate->IoPort->CylinderLsb);\r
+    WordCount = WordCount & 0xffff;\r
     WordCount /= 2;\r
 \r
     //\r
index 60e0582..be05f87 100644 (file)
@@ -176,6 +176,7 @@ inw (
   @retval  EFI_SUCCESS TODO: Add description for return value\r
 \r
 **/\r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 CirrusLogic5430UgaDrawGetMode (\r
@@ -223,6 +224,7 @@ CirrusLogic5430UgaDrawGetMode (
   @retval  EFI_NOT_FOUND TODO: Add description for return value\r
 \r
 **/\r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 CirrusLogic5430UgaDrawSetMode (\r
@@ -301,6 +303,7 @@ CirrusLogic5430UgaDrawSetMode (
   @retval  EFI_SUCCESS TODO: Add description for return value\r
 \r
 **/\r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 CirrusLogic5430UgaDrawBlt (\r
index 933899f..38d86a5 100644 (file)
@@ -142,7 +142,6 @@ CirrusLogic5430ComponentNameGetControllerName (
 {\r
   EFI_UGA_DRAW_PROTOCOL           *UgaDraw;\r
   EFI_STATUS                      Status;\r
-  CIRRUS_LOGIC_5430_PRIVATE_DATA  *Private;\r
   EFI_PCI_IO_PROTOCOL             *PciIoProtocol;\r
 \r
   //\r
@@ -196,8 +195,6 @@ CirrusLogic5430ComponentNameGetControllerName (
   //\r
   // Get the Cirrus Logic 5430's Device structure\r
   //\r
-  Private = CIRRUS_LOGIC_5430_PRIVATE_DATA_FROM_UGA_DRAW_THIS (UgaDraw);\r
-\r
   return LookupUnicodeString (\r
           Language,\r
           gCirrusLogic5430ComponentName.SupportedLanguages,\r
index 2d6dc83..baea1f6 100644 (file)
@@ -1281,9 +1281,9 @@ EhciGetRootHubPortStatus (
     // Not Low Speed Device Attached\r
     //\r
     if ((PORTSC_CCS & PortStatusControlReg) && (PORTSC_CSC & PortStatusControlReg)) {\r
-      HcDev->DeviceSpeed[PortNumber] = IsHighSpeedDevice (This, PortNumber) ? USB_PORT_STAT_HIGH_SPEED : 0;\r
+      HcDev->DeviceSpeed[PortNumber] = (UINT16) (IsHighSpeedDevice (This, PortNumber) ? USB_PORT_STAT_HIGH_SPEED : 0);\r
     }\r
-    PortStatus->PortStatus |= HcDev->DeviceSpeed[PortNumber];\r
+    PortStatus->PortStatus = (UINT16) (PortStatus->PortStatus | HcDev->DeviceSpeed[PortNumber]);\r
   }\r
   //\r
   // Fill Port Status Change bits\r
index 0a992fe..aba27cb 100644 (file)
@@ -380,7 +380,7 @@ Returns:
       // reset associated bits in bit arry\r
       //\r
       for (Index = StartBytePos, Index2 = StartBitPos, Count = 0; Count < (RealAllocSize / 32); Count++) {\r
-        TempHeaderPtr->BitArrayPtr[Index] ^= (UINT8) (bit (Index2));\r
+        TempHeaderPtr->BitArrayPtr[Index] = (UINT8) (TempHeaderPtr->BitArrayPtr[Index] ^ (bit (Index2)));\r
         Index2++;\r
         if (Index2 == 8) {\r
           Index += 1;\r
@@ -595,7 +595,7 @@ Returns:
   //\r
   for (TempBytePos = FoundBytePos, Index = FoundBitPos, Count = 0; Count < NumberOfMemoryUnit; Count++) {\r
 \r
-    MemoryHeader->BitArrayPtr[TempBytePos] |= bit (Index);\r
+    MemoryHeader->BitArrayPtr[TempBytePos] = (UINT8) (MemoryHeader->BitArrayPtr[TempBytePos] | (bit (Index)));\r
     Index++;\r
     if (Index == 8) {\r
       TempBytePos += 1;\r
index 5767bbc..d5271c2 100644 (file)
@@ -1136,7 +1136,7 @@ Returns:
     &PortStatusControlReg\r
     );\r
 \r
-  return ((PortStatusControlReg & PORTSC_PED) ? TRUE : FALSE);\r
+  return ((BOOLEAN) ((PortStatusControlReg & PORTSC_PED) ? TRUE : FALSE));\r
 }\r
 \r
 BOOLEAN\r
@@ -1347,7 +1347,7 @@ Returns:
   //\r
   gBS->Stall (EHCI_CLEAR_PORT_RESET_RECOVERY_TIME);\r
 \r
-  return (IsEhcPortEnabled (HcDev, PortNum) ? TRUE : FALSE);\r
+  return ((BOOLEAN) (IsEhcPortEnabled (HcDev, PortNum) ? TRUE : FALSE));\r
 }\r
 \r
 EFI_STATUS\r
index 9f1a15d..48432c0 100644 (file)
@@ -2547,7 +2547,7 @@ Returns:
 \r
   MatchPtr        = NULL;\r
   QhHwPtr         = NULL;\r
-  EndPointNum     = EndPointAddress & 0x0f;\r
+  EndPointNum     = (UINT8) (EndPointAddress & 0x0f);\r
   AsyncRequestPtr = HcDev->AsyncRequestList;\r
 \r
   //\r
@@ -2916,13 +2916,11 @@ Returns:
   EFI_STATUS  Status;\r
   UINTN       ErrQtdPos;\r
   UINTN       Delay;\r
-  UINTN       RequireLen;\r
   BOOLEAN     Finished;\r
 \r
   Status          = EFI_SUCCESS;\r
   ErrQtdPos       = 0;\r
   *TransferResult = EFI_USB_NOERROR;\r
-  RequireLen      = *ActualLen;\r
   *ActualLen      = 0;\r
   Finished        = FALSE;\r
 \r
index 75794f8..6a05562 100644 (file)
@@ -64,6 +64,7 @@ EFI_DRIVER_CONFIGURATION_PROTOCOL gIDEBusDriverConfiguration = {
   @retval  EFI_NOT_FOUND TODO: Add description for return value\r
 \r
 **/\r
+STATIC\r
 EFI_STATUS\r
 GetResponse (\r
   VOID\r
@@ -190,7 +191,7 @@ IDEBusDriverConfigurationSetOptions (
     }\r
 \r
     if (!EFI_ERROR (Status)) {\r
-      NewValue |= (UINT8) (1 << Index);\r
+      NewValue = (UINT8) (NewValue | (1 << Index));\r
     }\r
   }\r
 \r
index 3273380..7ea54b9 100644 (file)
@@ -2073,7 +2073,7 @@ AtaCommandIssue (
   Lba1  = (UINT8) RShiftU64 (LbaAddress, 8);\r
   Lba2  = (UINT8) RShiftU64 (LbaAddress, 16);\r
   Lba3  = (UINT8) RShiftU64 (LbaAddress, 24);\r
-  Device |= Lba3;\r
+  Device = (UINT8) (Device | Lba3);\r
 \r
   //\r
   // Pass parameter into device register block\r
index a54acf6..93f7081 100644 (file)
@@ -713,11 +713,6 @@ PioReadWriteData (
   EFI_STATUS  Status;\r
   UINT16      *PtrBuffer;\r
 \r
-  //\r
-  // containing status byte read from Status Register.\r
-  //\r
-  UINT8       StatusRegister;\r
-\r
   //\r
   // No data transfer is premitted.\r
   //\r
@@ -750,16 +745,14 @@ PioReadWriteData (
     //\r
     // read Status Register will clear interrupt\r
     //\r
-    StatusRegister = IDEReadPortB (IdeDev->PciIo, IdeDev->IoPort->Reg.Status);\r
+    IDEReadPortB (IdeDev->PciIo, IdeDev->IoPort->Reg.Status);\r
 \r
     //\r
     // get current data transfer size from Cylinder Registers.\r
     //\r
-    WordCount =\r
-      (\r
-        (IDEReadPortB (IdeDev->PciIo, IdeDev->IoPort->CylinderMsb) << 8) |\r
-        IDEReadPortB (IdeDev->PciIo, IdeDev->IoPort->CylinderLsb)\r
-      ) & 0xffff;\r
+    WordCount = IDEReadPortB (IdeDev->PciIo, IdeDev->IoPort->CylinderMsb) << 8;\r
+    WordCount = WordCount | IDEReadPortB (IdeDev->PciIo, IdeDev->IoPort->CylinderLsb);\r
+    WordCount = WordCount & 0xffff;\r
     WordCount /= 2;\r
 \r
     WordCount = EFI_MIN (WordCount, (RequiredWordCount - ActualWordCount));\r
index 0b28c57..14cdb25 100644 (file)
@@ -236,6 +236,7 @@ IDEWritePortWMultiple (
   TODO: add return values\r
 \r
 **/\r
+STATIC\r
 BOOLEAN\r
 BadIdeDeviceCheck (\r
   IN IDE_BLK_IO_DEV *IdeDev\r
index c0cec60..fd04b67 100644 (file)
@@ -189,7 +189,6 @@ IDEBusDriverBindingStart (
   EFI_IDE_CONTROLLER_INIT_PROTOCOL  *IdeInit;\r
   BOOLEAN                           EnumAll;\r
   BOOLEAN                           ChannelEnabled;\r
-  UINT8                             ChannelCount;\r
   UINT8                             MaxDevices;\r
   EFI_IDENTIFY_DATA                 IdentifyData;\r
   EFI_ATA_COLLECTIVE_MODE           *SupportedModes;\r
@@ -239,10 +238,9 @@ IDEBusDriverBindingStart (
   }\r
 \r
   //\r
-  // Save Enumall and ChannelCount. Step7.2\r
+  // Save Enumall. Step7.2\r
   //\r
   EnumAll       = IdeInit->EnumAll;\r
-  ChannelCount  = IdeInit->ChannelCount;\r
 \r
   //\r
   // Consume PCI I/O protocol. Note that the OpenProtocol with _GET_PROTOCOL\r
@@ -453,8 +451,8 @@ IDEBusDriverBindingStart (
       ZeroMem (IdeBlkIoDevicePtr, sizeof (IDE_BLK_IO_DEV));\r
 \r
       IdeBlkIoDevicePtr->Signature  = IDE_BLK_IO_DEV_SIGNATURE;\r
-      IdeBlkIoDevicePtr->Channel    = IdeChannel;\r
-      IdeBlkIoDevicePtr->Device     = IdeDevice;\r
+      IdeBlkIoDevicePtr->Channel    = (EFI_IDE_CHANNEL) IdeChannel;\r
+      IdeBlkIoDevicePtr->Device     = (EFI_IDE_DEVICE) IdeDevice;\r
 \r
       //\r
       // initialize Block IO interface's Media pointer\r
@@ -666,7 +664,7 @@ IDEBusDriverBindingStart (
       //\r
       // Record PIO mode used in private data\r
       //\r
-      IdeBlkIoDevicePtr->PioMode = SupportedModes->PioMode.Mode;\r
+      IdeBlkIoDevicePtr->PioMode = (ATA_PIO_MODE) SupportedModes->PioMode.Mode;\r
 \r
       //\r
       // Set IDE controller Timing Blocks in the PCI Configuration Space\r
index f07c73f..6b47244 100644 (file)
@@ -71,9 +71,9 @@ Returns:
   }\r
 \r
   if (Operation == EFI_ENABLE_REGISTER) {\r
-    OldCommand |= Command;\r
+    OldCommand = (UINT16) (OldCommand | Command);\r
   } else if (Operation == EFI_DISABLE_REGISTER) {\r
-    OldCommand &= ~(Command);\r
+    OldCommand = (UINT16) (OldCommand & ~(Command));\r
   } else {\r
     OldCommand = Command;\r
   }\r
index ce605cc..3b796a6 100644 (file)
@@ -969,6 +969,7 @@ Returns:
 \r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 ProcessOptionRomLight (\r
   IN PCI_IO_DEVICE                      *PciIoDevice\r
@@ -1432,7 +1433,6 @@ Returns:
 // TODO:    BarIndex - add argument and description to function comment\r
 {\r
   UINT32      Value;\r
-  UINT64      BarValue64;\r
   UINT32      OriginalValue;\r
   UINT32      Mask;\r
   UINT32      Data;\r
@@ -1441,7 +1441,6 @@ Returns:
 \r
   OriginalValue = 0;\r
   Value         = 0;\r
-  BarValue64    = 0;\r
 \r
   Status = BarExisted (\r
             PciIoDevice,\r
@@ -1491,7 +1490,7 @@ Returns:
     // Need to treat it as no-bar\r
     //\r
     if (PciIoDevice->PciBar[BarIndex].Length == 0) {\r
-      PciIoDevice->PciBar[BarIndex].BarType = 0;\r
+      PciIoDevice->PciBar[BarIndex].BarType = (PCI_BAR_TYPE) 0;\r
     }\r
 \r
     PciIoDevice->PciBar[BarIndex].Prefetchable  = FALSE;\r
index 040c570..a636d4f 100644 (file)
@@ -184,7 +184,7 @@ Returns:
     Count = 1;\r
   }\r
 \r
-  Width &= 0x03;\r
+  Width = (EFI_PCI_IO_PROTOCOL_WIDTH) (Width & 0x03);\r
 \r
   if ((*Offset + Count * (UINTN)(1 << Width)) - 1 >= PciIoDevice->PciBar[BarIndex].Length) {\r
     return EFI_INVALID_PARAMETER;\r
@@ -233,7 +233,7 @@ Returns:
   //\r
   // If Width is EfiPciIoWidthFillUintX then convert to EfiPciIoWidthUintX\r
   //\r
-  Width &= 0x03;\r
+  Width = (EFI_PCI_IO_PROTOCOL_WIDTH) (Width & 0x03);\r
 \r
   if (PciIoDevice->IsPciExp) {\r
     if ((*Offset + Count * (UINTN)(1 << Width)) - 1 >= PCI_EXP_MAX_CONFIG_OFFSET) {\r
@@ -876,7 +876,7 @@ Returns:
   }\r
 \r
   if (PciIoDevice->Attributes & EFI_PCI_IO_ATTRIBUTE_DUAL_ADDRESS_CYCLE) {\r
-    Operation = Operation + EfiPciOperationBusMasterRead64;\r
+    Operation = (EFI_PCI_IO_PROTOCOL_OPERATION) (Operation + EfiPciOperationBusMasterRead64);\r
   }\r
 \r
   Status = PciIoDevice->PciRootBridgeIo->Map (\r
@@ -1966,10 +1966,15 @@ Returns:
 // TODO:    PciDevice1 - add argument and description to function comment\r
 // TODO:    PciDevice2 - add argument and description to function comment\r
 {\r
+  BOOLEAN   Existed1;\r
+  BOOLEAN   Existed2;\r
 \r
   if (PciDevice1->Parent == PciDevice2->Parent) {\r
     return TRUE;\r
   }\r
 \r
-  return (PciDeviceExisted (PciDevice1->Parent, PciDevice2)|| PciDeviceExisted (PciDevice2->Parent, PciDevice1));\r
+  Existed1 = PciDeviceExisted (PciDevice1->Parent, PciDevice2);\r
+  Existed2 = PciDeviceExisted (PciDevice2->Parent, PciDevice1);\r
+\r
+  return (BOOLEAN) (Existed1 || Existed2);\r
 }\r
index a4d3577..c9c46b3 100644 (file)
@@ -94,6 +94,7 @@ Returns:
                     EFI_NATIVE_INTERFACE,\r
                     NULL\r
                     );\r
+    ASSERT_EFI_ERROR (Status);\r
   }\r
 }\r
 \r
@@ -1446,7 +1447,7 @@ Returns:
   SecondBus       = 0;\r
   Register        = 0;\r
   State           = 0;\r
-  Attributes      = 0;\r
+  Attributes      = (EFI_HPC_PADDING_ATTRIBUTES) 0;\r
   BusRange        = 0;\r
 \r
   ResetAllPpbBusReg (Bridge, StartBusNumber);\r
index 449d0dc..2ced4ac 100644 (file)
@@ -151,7 +151,6 @@ Returns:
   UINT16                    OffsetPcir;\r
   UINT32                    RomBarOffset;\r
   UINT32                    RomBar;\r
-  UINT64                    Temp;\r
   EFI_STATUS                retStatus;\r
   BOOLEAN                   FirstCheck;\r
   UINT8                     *Image;\r
@@ -167,7 +166,6 @@ Returns:
   Indicator     = 0;\r
   RomImageSize  = 0;\r
   RomInMemory   = NULL;\r
-  Temp          = 0;\r
   CodeType      = 0xFF;\r
 \r
   //\r
index 58dbc9f..482ddc9 100644 (file)
@@ -647,10 +647,8 @@ Returns:
 // TODO:    ResType - add argument and description to function comment\r
 // TODO:    ResUsage - add argument and description to function comment\r
 {\r
-  EFI_STATUS        Status;\r
   PCI_RESOURCE_NODE *Node;\r
 \r
-  Status  = 0;\r
   Node    = NULL;\r
 \r
   Node    = AllocatePool (sizeof (PCI_RESOURCE_NODE));\r
@@ -704,7 +702,6 @@ Returns:
 // TODO:    EFI_SUCCESS - add return value to function comment\r
 {\r
   PCI_IO_DEVICE     *Temp;\r
-  EFI_STATUS        Status;\r
   PCI_RESOURCE_NODE *IoBridge;\r
   PCI_RESOURCE_NODE *Mem32Bridge;\r
   PCI_RESOURCE_NODE *PMem32Bridge;\r
@@ -789,14 +786,14 @@ Returns:
       //\r
       // Recursively create resouce map on this bridge\r
       //\r
-      Status = CreateResourceMap (\r
-                Temp,\r
-                IoBridge,\r
-                Mem32Bridge,\r
-                PMem32Bridge,\r
-                Mem64Bridge,\r
-                PMem64Bridge\r
-                );\r
+      CreateResourceMap (\r
+        Temp,\r
+        IoBridge,\r
+        Mem32Bridge,\r
+        PMem32Bridge,\r
+        Mem64Bridge,\r
+        PMem64Bridge\r
+        );\r
 \r
       if (ResourceRequestExisted (IoBridge)) {\r
         InsertResourceNode (\r
@@ -895,14 +892,14 @@ Returns:
   //\r
   // To do some platform specific resource padding ...\r
   //\r
-  Status = ResourcePaddingPolicy (\r
-            Bridge,\r
-            IoNode,\r
-            Mem32Node,\r
-            PMem32Node,\r
-            Mem64Node,\r
-            PMem64Node\r
-            );\r
+  ResourcePaddingPolicy (\r
+    Bridge,\r
+    IoNode,\r
+    Mem32Node,\r
+    PMem32Node,\r
+    Mem64Node,\r
+    PMem64Node\r
+    );\r
 \r
   //\r
   // Degrade resource if necessary\r
index a1e16a0..8f5a450 100644 (file)
@@ -101,6 +101,7 @@ Returns:
   mNumberOfPciRomImages++;\r
 }\r
 \r
+STATIC\r
 VOID\r
 HexToString (\r
   CHAR16  *String,\r
@@ -134,6 +135,8 @@ Returns:
     }\r
   }\r
 }\r
+\r
+STATIC\r
 EFI_STATUS\r
 PciRomLoadEfiDriversFromRomImage (\r
   IN EFI_DRIVER_BINDING_PROTOCOL    *This,\r
@@ -307,6 +310,7 @@ Returns:
   return retStatus;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 PciRomLoadEfiDriversFromOptionRomTable (\r
   IN EFI_DRIVER_BINDING_PROTOCOL      *This,\r
index 74ddc6b..8dbc0d9 100644 (file)
@@ -21,6 +21,7 @@ Revision History
 \r
 #include "uhci.h"\r
 \r
+STATIC\r
 EFI_STATUS\r
 USBReadPortW (\r
   IN       EFI_PCI_IO_PROTOCOL     *PciIo,\r
@@ -58,6 +59,7 @@ Returns:
                      );\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 USBReadPortDW (\r
   IN       EFI_PCI_IO_PROTOCOL     *PciIo,\r
@@ -95,6 +97,7 @@ Returns:
                      );\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 USBWritePortW (\r
   IN EFI_PCI_IO_PROTOCOL     *PciIo,\r
@@ -132,6 +135,7 @@ Returns:
                      );\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 USBWritePortDW (\r
   IN EFI_PCI_IO_PROTOCOL     *PciIo,\r
@@ -654,13 +658,12 @@ Returns:
 --*/\r
 {\r
   UINT16      CommandContent;\r
-  EFI_STATUS  Status;\r
 \r
-  Status = ReadUHCCommandReg (\r
-             HcDev->PciIo,\r
-             (UINT32) (USBCMD),\r
-             &CommandContent\r
-             );\r
+  ReadUHCCommandReg (\r
+    HcDev->PciIo,\r
+    (UINT32) (USBCMD),\r
+    &CommandContent\r
+    );\r
 \r
   if ((CommandContent & USBCMD_MAXP) != USBCMD_MAXP) {\r
     CommandContent |= USBCMD_MAXP;\r
@@ -835,6 +838,7 @@ Returns:
 //\r
 // functions for QH\r
 //\r
+STATIC\r
 EFI_STATUS\r
 AllocateQHStruct (\r
   IN  USB_HC_DEV     *HcDev,\r
@@ -1196,6 +1200,7 @@ Returns:
   return (BOOLEAN) (!(PtrQH->QH.QHVerticalTerminate));\r
 }\r
 \r
+STATIC\r
 BOOLEAN\r
 GetQHHorizontalValidorInvalid (\r
   IN QH_STRUCT     *PtrQH\r
@@ -1769,6 +1774,7 @@ Returns:
   return (VOID *) ((UINTN) (ptrTDStruct->TDData.TDLinkPtr << 4));\r
 }\r
 \r
+STATIC\r
 BOOLEAN\r
 IsTDLinkPtrQHOrTD (\r
   IN TD_STRUCT     *ptrTDStruct\r
@@ -2325,6 +2331,7 @@ SetCurFrameListQHorTD (
   pCurEntry->FrameListPtrQSelect = (IsQH ? 1 : 0);\r
 }\r
 \r
+STATIC\r
 BOOLEAN\r
 IsCurFrameListQHorTD (\r
   IN FRAMELIST_ENTRY     *pCurEntry\r
@@ -3854,7 +3861,7 @@ Returns:
       //\r
       for (Index = StartBytePos, Index2 = StartBitPos, Count = 0; Count < (RealAllocSize / 32); Count++) {\r
 \r
-        TempHeaderPtr->BitArrayPtr[Index] ^= (UINT8) (bit (Index2));\r
+        TempHeaderPtr->BitArrayPtr[Index] = (UINT8) (TempHeaderPtr->BitArrayPtr[Index] ^ bit (Index2));\r
         Index2++;\r
         if (Index2 == 8) {\r
           Index += 1;\r
@@ -4067,7 +4074,7 @@ Returns:
   for (TempBytePos = FoundBytePos, Index = FoundBitPos,Count = 0;\r
        Count < NumberOfMemoryUnit; Count ++) {\r
 \r
-    MemoryHeader->BitArrayPtr[TempBytePos] |= bit (Index);\r
+    MemoryHeader->BitArrayPtr[TempBytePos] = (UINT8) (MemoryHeader->BitArrayPtr[TempBytePos] | bit (Index));\r
     Index++;\r
     if (Index == 8) {\r
       TempBytePos += 1;\r
index 96c5be4..3d81d2c 100644 (file)
@@ -759,7 +759,7 @@ UHCIDriverBindingStart (
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
+STATIC\r
 EFI_STATUS\r
 UnInstallUHCInterface (\r
   IN  EFI_HANDLE              Controller,\r
@@ -3845,7 +3845,7 @@ UHCI2ControlTransfer (
 --*/\r
 {\r
   USB_HC_DEV *HcDev;\r
-  BOOLEAN IsSlowDevice = (EFI_USB_SPEED_LOW == DeviceSpeed) ? TRUE : FALSE;\r
+  BOOLEAN IsSlowDevice = (BOOLEAN) ((EFI_USB_SPEED_LOW == DeviceSpeed) ? TRUE : FALSE);\r
 \r
   HcDev = USB2_HC_DEV_FROM_THIS (This);\r
   \r
@@ -4056,7 +4056,7 @@ UHCI2AsyncInterruptTransfer (
 --*/  \r
 {\r
   USB_HC_DEV *HcDev;\r
-  BOOLEAN IsSlowDevice = (EFI_USB_SPEED_LOW == DeviceSpeed) ? TRUE : FALSE;\r
+  BOOLEAN IsSlowDevice = (BOOLEAN) ((EFI_USB_SPEED_LOW == DeviceSpeed) ? TRUE : FALSE);\r
    \r
   HcDev = USB2_HC_DEV_FROM_THIS (This);\r
   return UHCIAsyncInterruptTransfer(\r
@@ -4155,7 +4155,7 @@ UHCI2SyncInterruptTransfer (
   if(DeviceSpeed==EFI_USB_SPEED_HIGH)\r
        return EFI_INVALID_PARAMETER;\r
   \r
-  IsSlowDevice = (EFI_USB_SPEED_LOW == DeviceSpeed) ? TRUE : FALSE;  \r
+  IsSlowDevice = (BOOLEAN) ((EFI_USB_SPEED_LOW == DeviceSpeed) ? TRUE : FALSE);  \r
   HcDev = USB2_HC_DEV_FROM_THIS (This);\r
   \r
   return UHCISyncInterruptTransfer(\r
@@ -4437,11 +4437,9 @@ MonitorInterruptTrans (
   UINTN           DataLen;\r
   UINTN           ActualLen;\r
   UINTN           ErrTDPos;\r
-  UINT32          StatusAddr;\r
   LIST_ENTRY      *NextLink;\r
 \r
   HcDev       = (USB_HC_DEV *) Context;\r
-  StatusAddr  = (UINT32) (USBSTS);\r
 \r
   //\r
   // interrupt transfer list is empty, means that no interrupt transfer\r
index 2ef31fb..b7b5961 100644 (file)
@@ -362,6 +362,7 @@ ScsiGetDeviceType (
   return EFI_SUCCESS;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 ScsiGetDeviceLocation (\r
index 4be2c38..10134e8 100644 (file)
@@ -1614,11 +1614,9 @@ Returns:
   UINT8               MaxRetry;\r
   BOOLEAN             NeedRetry;\r
   EFI_SCSI_SENSE_DATA *SenseData;\r
-  UINT8               SenseDataLength;\r
   UINTN               NumberOfSenseKeys;\r
 \r
   SenseData         = NULL;\r
-  SenseDataLength   = 0;\r
   NumberOfSenseKeys = 0;\r
 \r
   Status            = EFI_SUCCESS;\r
@@ -1728,11 +1726,9 @@ Returns:
   UINT8               MaxRetry;\r
   BOOLEAN             NeedRetry;\r
   EFI_SCSI_SENSE_DATA *SenseData;\r
-  UINT8               SenseDataLength;\r
   UINTN               NumberOfSenseKeys;\r
 \r
   SenseData         = NULL;\r
-  SenseDataLength   = 0;\r
   NumberOfSenseKeys = 0;\r
 \r
   Status            = EFI_SUCCESS;\r
index d9b094d..91a447d 100644 (file)
@@ -457,7 +457,6 @@ Cbi0DriverBindingStop (
   EFI_STATUS              Status;\r
   EFI_USB_ATAPI_PROTOCOL  *Cbi0AtapiProtocol;\r
   USB_CBI_DEVICE          *UsbCbiDev;\r
-  EFI_USB_IO_PROTOCOL     *UsbIo;\r
 \r
   //\r
   // Get our context back.\r
@@ -476,8 +475,6 @@ Cbi0DriverBindingStop (
 \r
   UsbCbiDev = USB_CBI_DEVICE_FROM_THIS (Cbi0AtapiProtocol);\r
 \r
-  UsbIo     = UsbCbiDev->UsbIo;\r
-\r
   Cbi0ReportStatusCode (\r
     UsbCbiDev->DevicePath,\r
     EFI_PROGRESS_CODE,\r
@@ -850,11 +847,9 @@ Cbi0MassStorageReset (
 --*/\r
 {\r
   EFI_STATUS          Status;\r
-  EFI_USB_IO_PROTOCOL *UsbIo;\r
   USB_CBI_DEVICE      *UsbCbiDev;\r
 \r
   UsbCbiDev = USB_CBI_DEVICE_FROM_THIS (This);\r
-  UsbIo     = UsbCbiDev->UsbIo;\r
 \r
   if (ExtendedVerification) {\r
     //\r
index d778497..008627b 100644 (file)
@@ -400,7 +400,6 @@ CBI1DriverBindingStop (
   EFI_STATUS              Status;\r
   EFI_USB_ATAPI_PROTOCOL  *CBI1AtapiProtocol;\r
   USB_CBI_DEVICE          *UsbCbiDev;\r
-  EFI_USB_IO_PROTOCOL     *UsbIo;\r
 \r
   //\r
   // Get our context back.\r
@@ -419,8 +418,6 @@ CBI1DriverBindingStop (
 \r
   UsbCbiDev = USB_CBI_DEVICE_FROM_THIS (CBI1AtapiProtocol);\r
 \r
-  UsbIo     = UsbCbiDev->UsbIo;\r
-\r
   Cbi1ReportStatusCode (\r
     UsbCbiDev->DevicePath,\r
     EFI_PROGRESS_CODE,\r
@@ -644,7 +641,6 @@ CBI1MassStorageReset (
 --*/\r
 {\r
   UINT8               ResetCommand[12];\r
-  EFI_STATUS          Status;\r
   EFI_USB_IO_PROTOCOL *UsbIo;\r
   USB_CBI_DEVICE      *UsbCbiDev;\r
   UINT8               EndpointAddr;\r
@@ -669,12 +665,12 @@ CBI1MassStorageReset (
   ResetCommand[0] = 0x1d;\r
   ResetCommand[1] = 0x04;\r
 \r
-  Status = CBI1CommandPhase (\r
-            UsbCbiDev,\r
-            ResetCommand,\r
-            12,\r
-            &Result\r
-            );\r
+  CBI1CommandPhase (\r
+    UsbCbiDev,\r
+    ResetCommand,\r
+    12,\r
+    &Result\r
+    );\r
 \r
   //\r
   // clear bulk in endpoint stall feature\r
index ec496a6..1b8c961 100644 (file)
@@ -467,7 +467,6 @@ USBKeyboardDriverBindingStop (
   EFI_STATUS                  Status;\r
   EFI_SIMPLE_TEXT_IN_PROTOCOL *SimpleInput;\r
   USB_KB_DEV                  *UsbKeyboardDevice;\r
-  EFI_USB_IO_PROTOCOL         *UsbIo;\r
 \r
   Status = gBS->OpenProtocol (\r
                   Controller,\r
@@ -493,7 +492,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
@@ -574,12 +572,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 b6b97c3..79eded3 100644 (file)
@@ -210,9 +210,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
@@ -360,19 +357,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
@@ -412,15 +406,15 @@ KeyboardHandler (
     // Delete & Submit this interrupt again\r
     //\r
     \r
-    Status = UsbIo->UsbAsyncInterruptTransfer (\r
-                      UsbIo,\r
-                      UsbKeyboardDevice->IntEndpointDescriptor.EndpointAddress,\r
-                      FALSE,\r
-                      0,\r
-                      0,\r
-                      NULL,\r
-                      NULL\r
-                      );\r
+    UsbIo->UsbAsyncInterruptTransfer (\r
+             UsbIo,\r
+             UsbKeyboardDevice->IntEndpointDescriptor.EndpointAddress,\r
+             FALSE,\r
+             0,\r
+             0,\r
+             NULL,\r
+             NULL\r
+             );\r
 \r
     gBS->SetTimer (\r
           UsbKeyboardDevice->DelayedRecoveryEvent,\r
index 0d57f76..b86c06a 100644 (file)
@@ -443,12 +443,10 @@ USBFloppyReadBlocks (
   UINTN               NumberOfBlocks;\r
   BOOLEAN             MediaChange;\r
   EFI_TPL             OldTpl;\r
-  UINT32              Retry;\r
 \r
   OldTpl          = gBS->RaiseTPL (EFI_TPL_NOTIFY);\r
   Status          = EFI_SUCCESS;\r
   MediaChange     = FALSE;\r
-  Retry           = 0;\r
 \r
   UsbFloppyDevice = USB_FLOPPY_DEV_FROM_THIS (This);\r
 \r
@@ -594,12 +592,10 @@ USBFloppyWriteBlocks (
   UINTN               NumberOfBlocks;\r
   BOOLEAN             MediaChange;\r
   EFI_TPL             OldTpl;\r
-  UINT32              Retry;\r
 \r
   OldTpl          = gBS->RaiseTPL (EFI_TPL_NOTIFY);\r
   Status          = EFI_SUCCESS;\r
   MediaChange     = FALSE;\r
-  Retry           = 0;\r
 \r
   UsbFloppyDevice = USB_FLOPPY_DEV_FROM_THIS (This);\r
 \r
index 0632473..442b5a2 100644 (file)
@@ -906,12 +906,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 68bf63e..e7c23b5 100644 (file)
@@ -136,7 +136,7 @@ CoreProcessFvImageFile (
   IN  EFI_GUID                        *DriverName\r
   );\r
 \r
-\r
+STATIC\r
 VOID\r
 CoreAcquireDispatcherLock (\r
   VOID\r
@@ -161,6 +161,7 @@ Returns:
   CoreAcquireLock (&mDispatcherLock);\r
 }\r
 \r
+STATIC\r
 VOID\r
 CoreReleaseDispatcherLock (\r
   VOID\r
@@ -184,7 +185,7 @@ Returns:
   CoreReleaseLock (&mDispatcherLock);\r
 }\r
 \r
-\r
+STATIC\r
 EFI_STATUS\r
 CoreGetDepexSectionAndPreProccess (\r
   IN  EFI_CORE_DRIVER_ENTRY   *DriverEntry\r
@@ -593,7 +594,7 @@ Returns:
   }\r
 }\r
 \r
-\r
+STATIC\r
 BOOLEAN\r
 FvHasBeenProcessed (\r
   IN  EFI_HANDLE      FvHandle\r
@@ -628,7 +629,7 @@ Returns:
   return FALSE;\r
 }\r
 \r
-\r
+STATIC\r
 VOID\r
 FvIsBeingProcesssed (\r
   IN  EFI_HANDLE    FvHandle\r
index c6ab876..7c2093b 100644 (file)
@@ -432,7 +432,7 @@ Returns:
       break;\r
 \r
     default:      \r
-      return FALSE;\r
+      goto Done;\r
     }\r
     \r
     //\r
@@ -445,6 +445,8 @@ Returns:
     //\r
     Iterator++;\r
   }\r
+\r
+Done:\r
   return FALSE;\r
 }\r
 \r
index a4d653c..b485210 100644 (file)
@@ -96,7 +96,7 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
+STATIC\r
 VOID\r
 EFIAPI\r
 GenericArchProtocolNotify (\r
index 08cff42..3749657 100644 (file)
@@ -87,7 +87,7 @@ UINT32 mEventTable[] = {
   EFI_EVENT_TIMER | EFI_EVENT_NOTIFY_WAIT,\r
 };\r
 \r
-\r
+STATIC\r
 VOID\r
 CoreAcquireEventLock (\r
   VOID\r
@@ -111,7 +111,7 @@ Returns:
   CoreAcquireLock (&gEventQueueLock);\r
 }\r
 \r
-\r
+STATIC\r
 VOID\r
 CoreReleaseEventLock (\r
   VOID\r
@@ -232,8 +232,8 @@ Returns:
 }\r
 \r
 \r
-VOID\r
 STATIC\r
+VOID\r
 CoreNotifyEvent (\r
   IN  IEVENT      *Event\r
   )\r
index 6f7d353..23c84b3 100644 (file)
@@ -143,7 +143,7 @@ Returns:
   }\r
 }\r
 \r
-\r
+STATIC\r
 BOOLEAN\r
 VerifyHeaderChecksum (\r
   IN EFI_FFS_FILE_HEADER  *FfsHeader\r
@@ -172,7 +172,7 @@ Returns:
     HeaderChecksum = (UINT8)(HeaderChecksum + ptr[Index]);\r
   }\r
 \r
-  HeaderChecksum = HeaderChecksum - FfsHeader->State - FfsHeader->IntegrityCheck.Checksum.File;\r
+  HeaderChecksum = (UINT8) (HeaderChecksum - FfsHeader->State - FfsHeader->IntegrityCheck.Checksum.File);\r
 \r
   if (HeaderChecksum == 0) {\r
     return TRUE;\r
index 0735bf0..181cb1a 100644 (file)
@@ -61,7 +61,7 @@ EFI_GCD_MAP_ENTRY mGcdMemorySpaceMapEntryTemplate = {
   0,\r
   0,\r
   EfiGcdMemoryTypeNonExistent,\r
-  0,\r
+  (EFI_GCD_IO_TYPE) 0,\r
   NULL,\r
   NULL\r
 };\r
@@ -73,7 +73,7 @@ EFI_GCD_MAP_ENTRY mGcdIoSpaceMapEntryTemplate = {
   0,\r
   0,\r
   0,\r
-  0,\r
+  (EFI_GCD_MEMORY_TYPE) 0,\r
   EfiGcdIoTypeNonExistent,\r
   NULL,\r
   NULL\r
@@ -136,7 +136,7 @@ Returns:
 }\r
 \r
 \r
-\r
+STATIC\r
 VOID\r
 CoreAcquireGcdIoLock (\r
   VOID\r
@@ -157,7 +157,7 @@ Returns:
   CoreAcquireLock (&mGcdIoSpaceLock);\r
 }\r
 \r
-\r
+STATIC\r
 VOID\r
 CoreReleaseGcdIoLock (\r
   VOID\r
@@ -183,6 +183,7 @@ Returns:
 //\r
 // GCD Initialization Worker Functions\r
 //\r
+STATIC\r
 UINT64\r
 AlignValue (\r
   IN UINT64   Value,\r
@@ -217,6 +218,7 @@ Returns:
   return Value & (~AlignmentMask);\r
 }\r
 \r
+STATIC\r
 UINT64\r
 PageAlignAddress (\r
   IN UINT64 Value\r
@@ -240,6 +242,7 @@ Returns:
   return AlignValue (Value, EFI_PAGE_SHIFT, TRUE);\r
 }\r
 \r
+STATIC\r
 UINT64\r
 PageAlignLength (\r
   IN UINT64 Value\r
@@ -266,6 +269,7 @@ Returns:
 //\r
 // GCD Memory Space Worker Functions\r
 //\r
+STATIC\r
 EFI_STATUS\r
 CoreAllocateGcdMapEntry (\r
   IN OUT EFI_GCD_MAP_ENTRY  **TopEntry,\r
@@ -303,6 +307,7 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 CoreInsertGcdMapEntry (\r
   IN LIST_ENTRY           *Link,\r
@@ -359,6 +364,7 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 CoreMergeGcdMapEntry (\r
   IN LIST_ENTRY      *Link,\r
@@ -440,6 +446,7 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 CoreCleanupGcdMapEntry (\r
   IN EFI_GCD_MAP_ENTRY  *TopEntry,\r
@@ -491,6 +498,7 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 CoreSearchGcdMapEntry (\r
   IN  EFI_PHYSICAL_ADDRESS  BaseAddress,\r
@@ -551,6 +559,7 @@ Returns:
   return EFI_NOT_FOUND;\r
 }\r
 \r
+STATIC\r
 UINTN\r
 CoreCountGcdMapEntry (\r
   IN LIST_ENTRY  *Map\r
@@ -584,7 +593,7 @@ Returns:
 }\r
 \r
 \r
-\r
+STATIC\r
 UINT64\r
 ConverToCpuArchAttributes (\r
   UINT64 Attributes\r
@@ -629,7 +638,7 @@ Returns:
 \r
 }\r
 \r
-\r
+STATIC\r
 EFI_STATUS\r
 CoreConvertSpace (\r
   IN UINTN                 Operation,\r
@@ -897,6 +906,7 @@ Done:
   return Status;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 CoreAllocateSpaceCheckEntry (\r
   IN UINTN                Operation,\r
@@ -950,6 +960,7 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 CoreAllocateSpace (\r
   IN     UINTN                  Operation,\r
@@ -1218,7 +1229,7 @@ Done:
   return Status;\r
 }\r
 \r
-\r
+STATIC\r
 EFI_STATUS\r
 CoreInternalAddMemorySpace (\r
   IN EFI_GCD_MEMORY_TYPE   GcdMemoryType,\r
@@ -1257,7 +1268,7 @@ Returns:
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
-  return CoreConvertSpace (GCD_ADD_MEMORY_OPERATION, GcdMemoryType, 0, BaseAddress, Length, Capabilities, 0);\r
+  return CoreConvertSpace (GCD_ADD_MEMORY_OPERATION, GcdMemoryType, (EFI_GCD_IO_TYPE) 0, BaseAddress, Length, Capabilities, 0);\r
 }\r
 \r
 //\r
@@ -1310,7 +1321,7 @@ Returns:
            GCD_ALLOCATE_MEMORY_OPERATION, \r
            GcdAllocateType, \r
            GcdMemoryType, \r
-           0, \r
+           (EFI_GCD_IO_TYPE) 0, \r
            Alignment, \r
            Length, \r
            BaseAddress, \r
@@ -1427,7 +1438,7 @@ Returns:
 \r
 --*/\r
 {\r
-  return CoreConvertSpace (GCD_FREE_MEMORY_OPERATION, 0, 0, BaseAddress, Length, 0, 0);\r
+  return CoreConvertSpace (GCD_FREE_MEMORY_OPERATION, (EFI_GCD_MEMORY_TYPE) 0, (EFI_GCD_IO_TYPE) 0, BaseAddress, Length, 0, 0);\r
 }\r
 \r
 EFI_STATUS\r
@@ -1454,9 +1465,10 @@ Returns:
 \r
 --*/\r
 {\r
-  return CoreConvertSpace (GCD_REMOVE_MEMORY_OPERATION, 0, 0, BaseAddress, Length, 0, 0);\r
+  return CoreConvertSpace (GCD_REMOVE_MEMORY_OPERATION, (EFI_GCD_MEMORY_TYPE) 0, (EFI_GCD_IO_TYPE) 0, BaseAddress, Length, 0, 0);\r
 }\r
 \r
+STATIC\r
 VOID\r
 BuildMemoryDescriptor (\r
   IN OUT EFI_GCD_MEMORY_SPACE_DESCRIPTOR  *Descriptor,\r
@@ -1574,7 +1586,7 @@ Returns:
 \r
 --*/\r
 {\r
-  return CoreConvertSpace (GCD_SET_ATTRIBUTES_MEMORY_OPERATION, 0, 0, BaseAddress, Length, 0, Attributes);\r
+  return CoreConvertSpace (GCD_SET_ATTRIBUTES_MEMORY_OPERATION, (EFI_GCD_MEMORY_TYPE) 0, (EFI_GCD_IO_TYPE) 0, BaseAddress, Length, 0, Attributes);\r
 }\r
 \r
 EFI_STATUS\r
@@ -1687,7 +1699,7 @@ Returns:
   if (GcdIoType <= EfiGcdIoTypeNonExistent || GcdIoType >= EfiGcdIoTypeMaximum) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
-  return CoreConvertSpace (GCD_ADD_IO_OPERATION, 0, GcdIoType, BaseAddress, Length, 0, 0);\r
+  return CoreConvertSpace (GCD_ADD_IO_OPERATION, (EFI_GCD_MEMORY_TYPE) 0, GcdIoType, BaseAddress, Length, 0, 0);\r
 }\r
 \r
 EFI_STATUS\r
@@ -1736,7 +1748,7 @@ Returns:
   return CoreAllocateSpace (\r
            GCD_ALLOCATE_IO_OPERATION, \r
            GcdAllocateType, \r
-           0, \r
+           (EFI_GCD_MEMORY_TYPE) 0, \r
            GcdIoType, \r
            Alignment, \r
            Length, \r
@@ -1770,7 +1782,7 @@ Returns:
 \r
 --*/\r
 {\r
-  return CoreConvertSpace (GCD_FREE_IO_OPERATION, 0, 0, BaseAddress, Length, 0, 0);\r
+  return CoreConvertSpace (GCD_FREE_IO_OPERATION, (EFI_GCD_MEMORY_TYPE) 0, (EFI_GCD_IO_TYPE) 0, BaseAddress, Length, 0, 0);\r
 }\r
 \r
 EFI_STATUS\r
@@ -1797,9 +1809,10 @@ Returns:
 \r
 --*/\r
 {\r
-  return CoreConvertSpace (GCD_REMOVE_IO_OPERATION, 0, 0, BaseAddress, Length, 0, 0);\r
+  return CoreConvertSpace (GCD_REMOVE_IO_OPERATION, (EFI_GCD_MEMORY_TYPE) 0, (EFI_GCD_IO_TYPE) 0, BaseAddress, Length, 0, 0);\r
 }\r
 \r
+STATIC\r
 VOID\r
 BuildIoDescriptor (\r
   IN EFI_GCD_IO_SPACE_DESCRIPTOR  *Descriptor,\r
@@ -1964,6 +1977,7 @@ Done:
   return Status;\r
 }  \r
 \r
+STATIC\r
 UINT64\r
 CoreConvertResourceDescriptorHobAttributesToCapabilities (\r
   EFI_GCD_MEMORY_TYPE  GcdMemoryType,\r
index 3073857..e8a400c 100644 (file)
@@ -149,6 +149,7 @@ Returns:
   return ReturnStatus;\r
 }\r
 \r
+STATIC\r
 VOID\r
 AddSortedDriverBindingProtocol (\r
   IN      EFI_HANDLE                   DriverBindingHandle,\r
@@ -265,8 +266,6 @@ Returns:
   EFI_STATUS                                 Status;\r
   UINTN                                      Index;\r
   EFI_HANDLE                                 DriverImageHandle;\r
-  UINTN                                      PlatformDriverOverrideHandleCount;\r
-  EFI_HANDLE                                 *PlatformDriverOverrideHandleBuffer;\r
   EFI_PLATFORM_DRIVER_OVERRIDE_PROTOCOL      *PlatformDriverOverride;\r
   EFI_BUS_SPECIFIC_DRIVER_OVERRIDE_PROTOCOL  *BusSpecificDriverOverride;\r
   UINTN                                      DriverBindingHandleCount;\r
@@ -290,8 +289,6 @@ Returns:
   //\r
   DriverBindingHandleCount              = 0;\r
   DriverBindingHandleBuffer             = NULL;\r
-  PlatformDriverOverrideHandleCount     = 0;\r
-  PlatformDriverOverrideHandleBuffer    = NULL;\r
   NumberOfSortedDriverBindingProtocols  = 0;\r
   SortedDriverBindingProtocols          = NULL;\r
 \r
index f0dea71..2f43dc0 100644 (file)
@@ -978,6 +978,7 @@ Returns:
   return Status;\r
 }    \r
 \r
+STATIC\r
 PROTOCOL_INTERFACE  *\r
 CoreGetProtocolInterface (\r
   IN  EFI_HANDLE                UserHandle,\r
index 9a27237..e9541f7 100644 (file)
@@ -254,7 +254,7 @@ Returns:
     if (Image->ImageContext.ImageAddress >= 0x100000 || Image->ImageContext.RelocationsStripped) {\r
       Status = CoreAllocatePages (\r
                  AllocateAddress,\r
-                 Image->ImageContext.ImageCodeMemoryType,\r
+                 (EFI_MEMORY_TYPE) (Image->ImageContext.ImageCodeMemoryType),\r
                  Image->NumberOfPages,\r
                  &Image->ImageContext.ImageAddress\r
                  );\r
@@ -262,7 +262,7 @@ Returns:
     if (EFI_ERROR (Status) && !Image->ImageContext.RelocationsStripped) {\r
       Status = CoreAllocatePages (\r
                  AllocateAnyPages,\r
-                 Image->ImageContext.ImageCodeMemoryType,\r
+                 (EFI_MEMORY_TYPE) (Image->ImageContext.ImageCodeMemoryType),\r
                  Image->NumberOfPages,\r
                  &Image->ImageContext.ImageAddress\r
                  );\r
@@ -389,8 +389,8 @@ Returns:
   Image->Type               = Image->ImageContext.ImageType;\r
   Image->Info.ImageBase     = (VOID *)(UINTN)Image->ImageContext.ImageAddress;\r
   Image->Info.ImageSize     = Image->ImageContext.ImageSize;\r
-  Image->Info.ImageCodeType = Image->ImageContext.ImageCodeMemoryType;\r
-  Image->Info.ImageDataType = Image->ImageContext.ImageDataMemoryType;\r
+  Image->Info.ImageCodeType = (EFI_MEMORY_TYPE) (Image->ImageContext.ImageCodeMemoryType);\r
+  Image->Info.ImageDataType = (EFI_MEMORY_TYPE) (Image->ImageContext.ImageDataMemoryType);\r
   if (Attribute & EFI_LOAD_PE_IMAGE_ATTRIBUTE_RUNTIME_REGISTRATION) {\r
     if (Image->ImageContext.ImageType == EFI_IMAGE_SUBSYSTEM_EFI_RUNTIME_DRIVER) {\r
       //\r
@@ -514,7 +514,7 @@ Returns:
   return Image;\r
 }\r
 \r
-\r
+STATIC\r
 EFI_STATUS\r
 CoreLoadImageCommon (\r
   IN  BOOLEAN                          BootPolicy,\r
index d144689..44116ab 100644 (file)
@@ -308,7 +308,7 @@ Returns:
     //\r
     // Make sure the memory type in the gMemoryTypeInformation[] array is valid\r
     //\r
-    Type = gMemoryTypeInformation[Index].Type;\r
+    Type = (EFI_MEMORY_TYPE) (gMemoryTypeInformation[Index].Type);\r
     if (Type < 0 || Type > EfiMaxMemoryType) {\r
       continue;\r
     }\r
@@ -333,7 +333,7 @@ Returns:
           //\r
           // Make sure the memory type in the gMemoryTypeInformation[] array is valid\r
           //\r
-          Type = gMemoryTypeInformation[FreeIndex].Type;\r
+          Type = (EFI_MEMORY_TYPE) (gMemoryTypeInformation[FreeIndex].Type);\r
           if (Type < 0 || Type > EfiMaxMemoryType) {\r
             continue;\r
           }\r
@@ -376,7 +376,7 @@ Returns:
     //\r
     // Make sure the memory type in the gMemoryTypeInformation[] array is valid\r
     //\r
-    Type = gMemoryTypeInformation[Index].Type;\r
+    Type = (EFI_MEMORY_TYPE) (gMemoryTypeInformation[Index].Type);\r
     if (Type < 0 || Type > EfiMaxMemoryType) {\r
       continue;\r
     }\r
@@ -394,7 +394,7 @@ Returns:
   // If the number of pages reserved for a memory type is 0, then all allocations for that type\r
   // should be in the default range.\r
   //\r
-  for (Type = 0; Type < EfiMaxMemoryType; Type++) {\r
+  for (Type = (EFI_MEMORY_TYPE) 0; Type < EfiMaxMemoryType; Type++) {\r
     for (Index = 0; gMemoryTypeInformation[Index].Type != EfiMaxMemoryType; Index++) {\r
       if (Type == (EFI_MEMORY_TYPE)gMemoryTypeInformation[Index].Type) {\r
         mMemoryTypeStatistics[Type].InformationIndex = Index;\r
@@ -1457,7 +1457,6 @@ Returns:
 \r
 --*/\r
 {\r
-  EFI_STATUS        Status;\r
   UINT64            Start;\r
 \r
   //\r
@@ -1471,7 +1470,7 @@ Returns:
   if (Start == 0) {\r
     DEBUG ((EFI_D_ERROR | EFI_D_PAGE, "AllocatePoolPages: failed to allocate %d pages\n", NumberOfPages));\r
   } else {\r
-    Status = CoreConvertPages (Start, NumberOfPages, PoolType);\r
+    CoreConvertPages (Start, NumberOfPages, PoolType);\r
   }\r
 \r
   return (VOID *)(UINTN)Start;\r
index 3201a97..c024f03 100644 (file)
@@ -111,7 +111,7 @@ Returns:
   for (Type=0; Type < EfiMaxMemoryType; Type++) {\r
     PoolHead[Type].Signature  = 0;\r
     PoolHead[Type].Used       = 0;\r
-    PoolHead[Type].MemoryType = Type;\r
+    PoolHead[Type].MemoryType = (EFI_MEMORY_TYPE) Type;\r
     for (Index=0; Index < MAX_POOL_LIST; Index++) {\r
         InitializeListHead (&PoolHead[Type].FreeList[Index]);\r
     }\r
@@ -119,7 +119,7 @@ Returns:
   InitializeListHead (&PoolHeadList);\r
 }\r
 \r
-\r
+STATIC\r
 POOL *\r
 LookupPoolHead (\r
   IN EFI_MEMORY_TYPE  MemoryType\r
index b157b93..4d9ec52 100644 (file)
@@ -355,14 +355,12 @@ Returns:
 {\r
   EFI_FIRMWARE_VOLUME_HEADER  *FwVolHeader;\r
   EFI_FFS_FILE_HEADER         *FfsFileHeader;\r
-  VOID                        *SectionData;\r
   EFI_STATUS                  Status;\r
   EFI_PEI_HOB_POINTERS        Hob;\r
 \r
 \r
   FwVolHeader   = NULL;\r
   FfsFileHeader = NULL;\r
-  SectionData   = NULL;\r
   Status        = EFI_SUCCESS;\r
 \r
   //\r
@@ -654,7 +652,6 @@ Returns:
   UINTN                           SectionLength;\r
   UINTN                           OccupiedSectionLength;\r
   UINT64                          FileSize;\r
-  EFI_GUID_DEFINED_SECTION        *GuidedSectionHeader;\r
   UINT32                          AuthenticationStatus;\r
   EFI_PEI_SECTION_EXTRACTION_PPI  *SectionExtract;\r
   UINT32                          BufferSize;\r
@@ -697,10 +694,6 @@ Returns:
       // Was the DXE Core file encapsulated in a GUID'd section?\r
       //\r
       if (Section->Type == EFI_SECTION_GUID_DEFINED) {\r
-        //\r
-        // Locate the GUID'd Section Extractor\r
-        //\r
-        GuidedSectionHeader = (VOID *) (Section + 1);\r
 \r
         //\r
         // This following code constitutes the addition of the security model\r
index 049cfd1..bec0304 100644 (file)
@@ -65,7 +65,7 @@ Returns:
   \r
 \r
   return EFI_SUCCESS;   \r
-};\r
+}\r
 \r
 \r
 EFI_STATUS\r
@@ -103,4 +103,4 @@ Returns:
 \r
 \r
   return EFI_SUCCESS;   \r
-};\r
+}\r
index 0d9d797..c817a3d 100644 (file)
@@ -125,12 +125,10 @@ Returns:
                                   well-formed Grammar.\r
 --*/\r
 {\r
-  EFI_STATUS                     Status;\r
   DEPENDENCY_EXPRESSION_OPERAND  *Iterator;\r
   EVAL_STACK_ENTRY               *StackPtr;\r
   EVAL_STACK_ENTRY               EvalStack[MAX_GRAMMAR_SIZE];\r
 \r
-  Status    = EFI_SUCCESS;\r
   Iterator  = DependencyExpression;\r
   *Runnable = FALSE;\r
 \r
index bbf88f9..3d3fc4c 100644 (file)
@@ -19,59 +19,7 @@ Abstract:
 \r
 --*/\r
 \r
-//\r
-// Decompression algorithm begins here\r
-//\r
-#define BITBUFSIZ 32\r
-#define MAXMATCH  256\r
-#define THRESHOLD 3\r
-#define CODE_BIT  16\r
-#define BAD_TABLE - 1\r
-\r
-//\r
-// C: Char&Len Set; P: Position Set; T: exTra Set\r
-//\r
-#define NC      (0xff + MAXMATCH + 2 - THRESHOLD)\r
-#define CBIT    9\r
-#define MAXPBIT 5\r
-#define TBIT    5\r
-#define MAXNP   ((1U << MAXPBIT) - 1)\r
-#define NT      (CODE_BIT + 3)\r
-#if NT > MAXNP\r
-#define NPT NT\r
-#else\r
-#define NPT MAXNP\r
-#endif\r
-\r
-typedef struct {\r
-  UINT8   *mSrcBase;  // Starting address of compressed data\r
-  UINT8   *mDstBase;  // Starting address of decompressed data\r
-  UINT32  mOutBuf;\r
-  UINT32  mInBuf;\r
-\r
-  UINT16  mBitCount;\r
-  UINT32  mBitBuf;\r
-  UINT32  mSubBitBuf;\r
-  UINT16  mBlockSize;\r
-  UINT32  mCompSize;\r
-  UINT32  mOrigSize;\r
-\r
-  UINT16  mBadTableFlag;\r
-\r
-  UINT16  mLeft[2 * NC - 1];\r
-  UINT16  mRight[2 * NC - 1];\r
-  UINT8   mCLen[NC];\r
-  UINT8   mPTLen[NPT];\r
-  UINT16  mCTable[4096];\r
-  UINT16  mPTTable[256];\r
-\r
-  //\r
-  // The length of the field 'Position Set Code Length Array Size' in Block Header.\r
-  // For EFI 1.1 de/compression algorithm, mPBit = 4\r
-  // For Tiano de/compression algorithm, mPBit = 5\r
-  //\r
-  UINT8   mPBit;\r
-} SCRATCH_DATA;\r
+#include "BaseUefiTianoDecompressLibInternals.h"\r
 \r
 VOID\r
 FillBuf (\r
@@ -196,6 +144,8 @@ Returns:
   UINT16  Avail;\r
   UINT16  NextCode;\r
   UINT16  Mask;\r
+  UINT16  WordOfStart;\r
+  UINT16  WordOfCount;\r
 \r
   for (Index = 1; Index <= 16; Index++) {\r
     Count[Index] = 0;\r
@@ -208,7 +158,9 @@ Returns:
   Start[1] = 0;\r
 \r
   for (Index = 1; Index <= 16; Index++) {\r
-    Start[Index + 1] = (UINT16) (Start[Index] + (Count[Index] << (16 - Index)));\r
+    WordOfStart = Start[Index];\r
+    WordOfCount = Count[Index];\r
+    Start[Index + 1] = (UINT16) (WordOfStart + (WordOfCount << (16 - Index)));\r
   }\r
 \r
   if (Start[17] != 0) {\r
@@ -610,7 +562,7 @@ Returns: (VOID)
   for (;;) {\r
     CharC = DecodeC (Sd);\r
     if (Sd->mBadTableFlag != 0) {\r
-      return ;\r
+      goto Done ;\r
     }\r
 \r
     if (CharC < 256) {\r
@@ -618,7 +570,7 @@ Returns: (VOID)
       // Process an Original character\r
       //\r
       if (Sd->mOutBuf >= Sd->mOrigSize) {\r
-        return ;\r
+        goto Done ;\r
       } else {\r
         Sd->mDstBase[Sd->mOutBuf++] = (UINT8) CharC;\r
       }\r
@@ -637,7 +589,7 @@ Returns: (VOID)
       while ((INT16) (BytesRemain) >= 0) {\r
         Sd->mDstBase[Sd->mOutBuf++] = Sd->mDstBase[DataIdx++];\r
         if (Sd->mOutBuf >= Sd->mOrigSize) {\r
-          return ;\r
+          goto Done ;\r
         }\r
 \r
         BytesRemain--;\r
@@ -645,6 +597,7 @@ Returns: (VOID)
     }\r
   }\r
 \r
+Done:\r
   return ;\r
 }\r
 \r
index a484e10..eab5f54 100644 (file)
@@ -37,6 +37,7 @@
   </LibraryClassDefinitions>\r
   <SourceFiles>\r
     <Filename>BaseUefiTianoDecompressLib.c</Filename>\r
+    <Filename>BaseUefiTianoDecompressLibInternals.h</Filename>\r
   </SourceFiles>\r
   <PackageDependencies>\r
     <Package PackageGuid="5e0e9358-46b6-4ae2-8218-4ab8b9bbdcec"/>\r
diff --git a/edk2/EdkModulePkg/Library/BaseUefiTianoDecompressLib/BaseUefiTianoDecompressLibInternals.h b/edk2/EdkModulePkg/Library/BaseUefiTianoDecompressLib/BaseUefiTianoDecompressLibInternals.h
new file mode 100644 (file)
index 0000000..bb7aa5a
--- /dev/null
@@ -0,0 +1,224 @@
+/** @file\r
+  Internal include file for Base UEFI Decompress Libary.\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
+\r
+  Module Name:  BaseUefiDecompressLibInternals.h\r
+\r
+**/\r
+\r
+#ifndef __BASE_UEFI_TIANO_DECOMPRESS_LIB_INTERNALS_H__\r
+#define __BASE_UEFI_TIANO_DECOMPRESS_LIB_INTERNALS_H__\r
+\r
+//\r
+// Decompression algorithm begins here\r
+//\r
+#define BITBUFSIZ 32\r
+#define MAXMATCH  256\r
+#define THRESHOLD 3\r
+#define CODE_BIT  16\r
+#define BAD_TABLE - 1\r
+\r
+//\r
+// C: Char&Len Set; P: Position Set; T: exTra Set\r
+//\r
+#define NC      (0xff + MAXMATCH + 2 - THRESHOLD)\r
+#define CBIT    9\r
+#define MAXPBIT 5\r
+#define TBIT    5\r
+#define MAXNP   ((1U << MAXPBIT) - 1)\r
+#define NT      (CODE_BIT + 3)\r
+#if NT > MAXNP\r
+#define NPT NT\r
+#else\r
+#define NPT MAXNP\r
+#endif\r
+\r
+typedef struct {\r
+  UINT8   *mSrcBase;  // Starting address of compressed data\r
+  UINT8   *mDstBase;  // Starting address of decompressed data\r
+  UINT32  mOutBuf;\r
+  UINT32  mInBuf;\r
+\r
+  UINT16  mBitCount;\r
+  UINT32  mBitBuf;\r
+  UINT32  mSubBitBuf;\r
+  UINT16  mBlockSize;\r
+  UINT32  mCompSize;\r
+  UINT32  mOrigSize;\r
+\r
+  UINT16  mBadTableFlag;\r
+\r
+  UINT16  mLeft[2 * NC - 1];\r
+  UINT16  mRight[2 * NC - 1];\r
+  UINT8   mCLen[NC];\r
+  UINT8   mPTLen[NPT];\r
+  UINT16  mCTable[4096];\r
+  UINT16  mPTTable[256];\r
+\r
+  //\r
+  // The length of the field 'Position Set Code Length Array Size' in Block Header.\r
+  // For EFI 1.1 de/compression algorithm, mPBit = 4\r
+  // For Tiano de/compression algorithm, mPBit = 5\r
+  //\r
+  UINT8   mPBit;\r
+} SCRATCH_DATA;\r
+\r
+/**\r
+  Read NumOfBit of bits from source into mBitBuf\r
+\r
+  Shift mBitBuf NumOfBits left. Read in NumOfBits of bits from source.\r
+\r
+  @param  Sd        The global scratch data\r
+  @param  NumOfBits The number of bits to shift and read.\r
+\r
+**/\r
+VOID\r
+FillBuf (\r
+  IN  SCRATCH_DATA  *Sd,\r
+  IN  UINT16        NumOfBits\r
+  );\r
+\r
+/**\r
+  Get NumOfBits of bits out from mBitBuf\r
+\r
+  Get NumOfBits of bits out from mBitBuf. Fill mBitBuf with subsequent \r
+  NumOfBits of bits from source. Returns NumOfBits of bits that are \r
+  popped out.\r
+\r
+  @param  Sd        The global scratch data.\r
+  @param  NumOfBits The number of bits to pop and read.\r
+\r
+  @return The bits that are popped out.\r
+\r
+**/\r
+UINT32\r
+GetBits (\r
+  IN  SCRATCH_DATA  *Sd,\r
+  IN  UINT16        NumOfBits\r
+  );\r
+\r
+/**\r
+  Creates Huffman Code mapping table according to code length array.\r
+\r
+  Creates Huffman Code mapping table for Extra Set, Char&Len Set \r
+  and Position Set according to code length array.\r
+\r
+  @param  Sd        The global scratch data\r
+  @param  NumOfChar Number of symbols in the symbol set\r
+  @param  BitLen    Code length array\r
+  @param  TableBits The width of the mapping table\r
+  @param  Table     The table\r
+\r
+  @retval  0 OK.\r
+  @retval  BAD_TABLE The table is corrupted.\r
+\r
+**/\r
+UINT16\r
+MakeTable (\r
+  IN  SCRATCH_DATA  *Sd,\r
+  IN  UINT16        NumOfChar,\r
+  IN  UINT8         *BitLen,\r
+  IN  UINT16        TableBits,\r
+  OUT UINT16        *Table\r
+  );\r
+\r
+/**\r
+  Decodes a position value.\r
+\r
+  Get a position value according to Position Huffman Table.\r
+  \r
+  @param  Sd the global scratch data\r
+\r
+  @return The position value decoded.\r
+\r
+**/\r
+UINT32\r
+DecodeP (\r
+  IN  SCRATCH_DATA  *Sd\r
+  );\r
+\r
+/**\r
+  Reads code lengths for the Extra Set or the Position Set.\r
+\r
+  Read in the Extra Set or Pointion Set Length Arrary, then\r
+  generate the Huffman code mapping for them.\r
+\r
+  @param  Sd      The global scratch data.\r
+  @param  nn      Number of symbols.\r
+  @param  nbit    Number of bits needed to represent nn.\r
+  @param  Special The special symbol that needs to be taken care of.\r
+\r
+  @retval  0 OK.\r
+  @retval  BAD_TABLE Table is corrupted.\r
+\r
+**/\r
+UINT16\r
+ReadPTLen (\r
+  IN  SCRATCH_DATA  *Sd,\r
+  IN  UINT16        nn,\r
+  IN  UINT16        nbit,\r
+  IN  UINT16        Special\r
+  );\r
+\r
+/**\r
+  Reads code lengths for Char&Len Set.\r
+  \r
+  Read in and decode the Char&Len Set Code Length Array, then\r
+  generate the Huffman Code mapping table for the Char&Len Set.\r
+\r
+  @param  Sd the global scratch data\r
+\r
+**/\r
+VOID\r
+ReadCLen (\r
+  SCRATCH_DATA  *Sd\r
+  );\r
+\r
+/**\r
+  Decode a character/length value.\r
+  \r
+  Read one value from mBitBuf, Get one code from mBitBuf. If it is at block boundary, generates\r
+  Huffman code mapping table for Extra Set, Code&Len Set and\r
+  Position Set.\r
+\r
+  @param  Sd The global scratch data.\r
+\r
+  @return The value decoded.\r
+\r
+**/\r
+UINT16\r
+DecodeC (\r
+  SCRATCH_DATA  *Sd\r
+  );\r
+\r
+/**\r
+  Decode the source data and put the resulting data into the destination buffer.\r
+\r
+  Decode the source data and put the resulting data into the destination buffer.\r
+  \r
+  @param  Sd The global scratch data\r
+\r
+**/\r
+VOID\r
+Decode (\r
+  SCRATCH_DATA  *Sd\r
+  );\r
+\r
+RETURN_STATUS\r
+EFIAPI\r
+UefiTianoDecompress (\r
+  IN CONST VOID  *Source,\r
+  IN OUT VOID    *Destination,\r
+  IN OUT VOID    *Scratch,\r
+  IN UINT32      Version\r
+  );\r
+\r
+#endif\r
index 199b14b..55a25b4 100644 (file)
@@ -22,6 +22,7 @@ Abstract:
 EXTENDED_SAL_BOOT_SERVICE_PROTOCOL  *mEsalBootService = NULL;\r
 EFI_PLABEL                          mPlabel;\r
 \r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 DxeSalLibInitialize (\r
@@ -43,7 +44,7 @@ DxeSalLibInitialize (
   // virtual). So lets grap the physical PLABEL for the EsalEntryPoint and store it\r
   // away. We cache it in a module global, so we can register the vitrual version.\r
   //\r
-  Status = gBS->LocateProtocol (&gEfiExtendedSalBootServiceProtocolGuid, NULL, &mEsalBootService);\r
+  Status = gBS->LocateProtocol (&gEfiExtendedSalBootServiceProtocolGuid, NULL, (VOID **) &mEsalBootService);\r
   if (EFI_ERROR (Status)) {\r
     mEsalBootService = NULL;\r
     return EFI_SUCCESS;\r
index 4f0d378..c94234d 100644 (file)
@@ -105,7 +105,7 @@ Returns:
   return EFI_NOT_FOUND;\r
 }\r
 \r
-\r
+STATIC\r
 EFI_STATUS\r
 ConvertBmpToGopBlt (\r
   IN  VOID      *BmpImage,\r
@@ -664,6 +664,7 @@ Returns:
   UINT32            VerticalResolution;\r
   UINT32            ColorDepth;\r
   UINT32            RefreshRate;\r
+  UINTN             BufferGlyphWidth;\r
 \r
   GlyphStatus = 0;\r
 \r
@@ -743,6 +744,7 @@ Returns:
   //\r
   // Blt a character to the screen\r
   //\r
+  BufferGlyphWidth = GLYPH_WIDTH * StrLen (Buffer);\r
   if (GraphicsOutput != NULL) {\r
     Status = GraphicsOutput->Blt (\r
                         GraphicsOutput,\r
@@ -752,9 +754,9 @@ Returns:
                         0,\r
                         X,\r
                         Y,\r
-                        GLYPH_WIDTH * StrLen (Buffer),\r
+                        BufferGlyphWidth,\r
                         GLYPH_HEIGHT,\r
-                        GLYPH_WIDTH * StrLen (Buffer) * sizeof (EFI_GRAPHICS_OUTPUT_BLT_PIXEL)\r
+                        BufferGlyphWidth * sizeof (EFI_GRAPHICS_OUTPUT_BLT_PIXEL)\r
                         );\r
   } else {\r
     Status = UgaDraw->Blt (\r
@@ -765,9 +767,9 @@ Returns:
                         0,\r
                         X,\r
                         Y,\r
-                        GLYPH_WIDTH * StrLen (Buffer),\r
+                        BufferGlyphWidth,\r
                         GLYPH_HEIGHT,\r
-                        GLYPH_WIDTH * StrLen (Buffer) * sizeof (EFI_UGA_PIXEL)\r
+                        BufferGlyphWidth * sizeof (EFI_UGA_PIXEL)\r
                         );\r
   }\r
 \r
index cff559c..b2c9064 100644 (file)
@@ -128,6 +128,8 @@ Returns:
   UINT8               *Destination;\r
   UINTN               Index;\r
   BOOLEAN             Finished;\r
+  UINTN               SizeofLanguage;\r
+  UINTN               SizeofString;\r
 \r
   StringPack  = (EFI_HII_STRING_PACK *) StringBuffer;\r
   Finished    = FALSE;\r
@@ -248,14 +250,16 @@ Returns:
     //\r
     // Pointing to a new string pack location\r
     //\r
+    SizeofLanguage = StrSize (Language);\r
+    SizeofString   = StrSize (String);\r
     StringPackBuffer->Header.Length = (UINT32)\r
       (\r
         sizeof (EFI_HII_STRING_PACK) -\r
         sizeof (EFI_STRING) +\r
         sizeof (RELOFST) +\r
         sizeof (RELOFST) +\r
-        StrSize (Language) +\r
-        StrSize (String)\r
+        SizeofLanguage +\r
+        SizeofString\r
       );\r
     StringPackBuffer->Header.Type           = EFI_HII_STRING;\r
     StringPackBuffer->LanguageNameString    = (UINT16) ((UINTN) &PackDestination[3] - (UINTN) StringPackBuffer);\r
@@ -423,7 +427,7 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
-\r
+STATIC\r
 EFI_STATUS\r
 GetHiiInterface (\r
   OUT     EFI_HII_PROTOCOL    **Hii\r
index 3e37553..a4aab0f 100644 (file)
@@ -467,11 +467,15 @@ Returns:
 {\r
   EFI_STATUS    Status;\r
   CHAR16       *NameSuffixed;\r
+  UINTN         NameLength;\r
+  UINTN         SuffixLength;\r
 \r
   //\r
   // enough to concatenate both strings.\r
   //\r
-  NameSuffixed = AllocateZeroPool ((StrLen (Name) + StrLen (Suffix) + 1) * sizeof (CHAR16)); \r
+  NameLength   = StrLen (Name);\r
+  SuffixLength = StrLen (Suffix);\r
+  NameSuffixed = AllocateZeroPool ((NameLength + SuffixLength + 1) * sizeof (CHAR16)); \r
   \r
   StrCpy (NameSuffixed, Name);\r
   StrCat (NameSuffixed, Suffix);\r
index 09c1806..d17b937 100644 (file)
@@ -31,6 +31,7 @@
   </LibraryClassDefinitions>\r
   <SourceFiles>\r
     <Filename>EdkPeCoffLoader.c</Filename>\r
+    <Filename>EdkPeCoffLoaderLibInternals.h</Filename>\r
   </SourceFiles>\r
   <PackageDependencies>\r
     <Package PackageGuid="5e0e9358-46b6-4ae2-8218-4ab8b9bbdcec"/>\r
diff --git a/edk2/EdkModulePkg/Library/EdkPeCoffLoaderLib/EdkPeCoffLoaderLibInternals.h b/edk2/EdkModulePkg/Library/EdkPeCoffLoaderLib/EdkPeCoffLoaderLibInternals.h
new file mode 100644 (file)
index 0000000..b356724
--- /dev/null
@@ -0,0 +1,48 @@
+/** @file\r
+  Declaration of internal functions in PE/COFF Lib.\r
+\r
+  Copyright (c) 2006, Intel Corporation<BR>\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:  BasePeCoffLibInternals.h\r
+\r
+**/\r
+\r
+#ifndef __PECOFF_LOADER_LIB_INTERNALS__\r
+#define __PECOFF_LOADER_LIB_INTERNALS__\r
+\r
+EFI_STATUS\r
+EFIAPI\r
+TianoPeCoffLoaderLibGetImageInfo (\r
+  IN EFI_PEI_PE_COFF_LOADER_PROTOCOL          *This,\r
+  IN OUT PE_COFF_LOADER_IMAGE_CONTEXT         *ImageContext\r
+  );\r
+\r
+EFI_STATUS\r
+EFIAPI\r
+TianoPeCoffLoaderLibLoadImage (\r
+  IN EFI_PEI_PE_COFF_LOADER_PROTOCOL          *This,\r
+  IN OUT PE_COFF_LOADER_IMAGE_CONTEXT         *ImageContext\r
+  );\r
+\r
+EFI_STATUS\r
+EFIAPI\r
+TianoPeCoffLoaderLibRelocateImage (\r
+  IN EFI_PEI_PE_COFF_LOADER_PROTOCOL          *This,\r
+  IN OUT PE_COFF_LOADER_IMAGE_CONTEXT         *ImageContext\r
+  );\r
+\r
+EFI_STATUS\r
+EFIAPI\r
+TianoPeCoffLoaderLibUnloadimage (\r
+  IN EFI_PEI_PE_COFF_LOADER_PROTOCOL      *This,\r
+  IN PE_COFF_LOADER_IMAGE_CONTEXT         *ImageContext\r
+  );\r
+\r
+#endif\r
index 973f13d..34d6f26 100644 (file)
@@ -531,6 +531,7 @@ ConSplitterTextOutConstructor (
   return Status;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 ConSplitterSupported (\r
   IN  EFI_DRIVER_BINDING_PROTOCOL     *This,\r
@@ -717,6 +718,7 @@ Returns:
           );\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 ConSplitterStart (\r
@@ -1026,6 +1028,7 @@ Returns:
   return Status;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 ConSplitterStop (\r
@@ -1499,6 +1502,7 @@ Returns:
   return EFI_NOT_FOUND;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 ConSplitterGrowMapTable (\r
   IN  TEXT_OUT_SPLITTER_PRIVATE_DATA  *Private\r
@@ -1575,6 +1579,7 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 ConSplitterAddOutputMode (\r
   IN  TEXT_OUT_SPLITTER_PRIVATE_DATA  *Private,\r
@@ -1642,6 +1647,7 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
+STATIC\r
 VOID\r
 ConSplitterGetIntersection (\r
   IN  INT32                           *TextOutModeMap,\r
@@ -1694,6 +1700,7 @@ ConSplitterGetIntersection (
   return ;\r
 }\r
 \r
+STATIC\r
 VOID\r
 ConSplitterSyncOutputMode (\r
   IN  TEXT_OUT_SPLITTER_PRIVATE_DATA  *Private,\r
@@ -1712,7 +1719,6 @@ Returns:
 \r
 --*/\r
 {\r
-  INT32                         CurrentMode;\r
   INT32                         CurrentMaxMode;\r
   INT32                         Mode;\r
   INT32                         Index;\r
@@ -1726,7 +1732,6 @@ Returns:
   //\r
   // Must make sure that current mode won't change even if mode number changes\r
   //\r
-  CurrentMode       = Private->TextOutMode.Mode;\r
   CurrentMaxMode    = Private->TextOutMode.MaxMode;\r
   TextOutModeMap    = Private->TextOutModeMap;\r
   StepSize          = Private->TextOutListCount;\r
@@ -1771,6 +1776,7 @@ Returns:
   return ;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 ConSplitterGetIntersectionBetweenConOutAndStrErr (\r
   VOID\r
@@ -1796,8 +1802,6 @@ Returns:
   UINTN                         Indexj;\r
   UINTN                         Rows;\r
   UINTN                         Columns;\r
-  INT32                         ConOutCurrentMode;\r
-  INT32                         StdErrCurrentMode;\r
   INT32                         ConOutMaxMode;\r
   INT32                         StdErrMaxMode;\r
   INT32                         Mode;\r
@@ -1808,8 +1812,6 @@ Returns:
   INT32                         *StdErrMapTable;\r
   TEXT_OUT_SPLITTER_QUERY_DATA  *ConOutQueryData;\r
   TEXT_OUT_SPLITTER_QUERY_DATA  *StdErrQueryData;\r
-  UINTN                         ConOutStepSize;\r
-  UINTN                         StdErrStepSize;\r
   BOOLEAN                       FoundTheSameTextOut;\r
   UINTN                         ConOutMapTableSize;\r
   UINTN                         StdErrMapTableSize;\r
@@ -1843,16 +1845,12 @@ Returns:
   //\r
   // Must make sure that current mode won't change even if mode number changes\r
   //\r
-  ConOutCurrentMode = mConOut.TextOutMode.Mode;\r
   ConOutMaxMode     = mConOut.TextOutMode.MaxMode;\r
   ConOutModeMap     = mConOut.TextOutModeMap;\r
-  ConOutStepSize    = mConOut.TextOutListCount;\r
   ConOutQueryData   = mConOut.TextOutQueryData;\r
 \r
-  StdErrCurrentMode = mStdErr.TextOutMode.Mode;\r
   StdErrMaxMode     = mStdErr.TextOutMode.MaxMode;\r
   StdErrModeMap     = mStdErr.TextOutModeMap;\r
-  StdErrStepSize    = mStdErr.TextOutListCount;\r
   StdErrQueryData   = mStdErr.TextOutQueryData;\r
 \r
   //\r
@@ -1933,6 +1931,7 @@ Returns:
 }\r
 \r
 #if (EFI_SPECIFICATION_VERSION >= 0x00020000)\r
+STATIC\r
 EFI_STATUS\r
 ConSplitterAddGraphicsOutputMode (\r
   IN  TEXT_OUT_SPLITTER_PRIVATE_DATA  *Private,\r
@@ -2726,6 +2725,7 @@ ConSplitterSimplePointerReset (
   return ReturnStatus;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 ConSplitterSimplePointerPrivateGetState (\r
index 8ca743c..4509390 100644 (file)
@@ -578,7 +578,7 @@ DevNullGopSync (
   )\r
 ;\r
 \r
-#if (EFI_SPECIFICATION < 0x00020000)\r
+#if (EFI_SPECIFICATION_VERSION < 0x00020000)\r
 EFI_STATUS\r
 EFIAPI\r
 ConSpliterUgaDrawGetMode (\r
index 4bc15e6..cec8adb 100644 (file)
@@ -355,6 +355,7 @@ Routine Description:
   return ReturnStatus;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 DevNullGraphicsOutputBlt (\r
   IN  TEXT_OUT_SPLITTER_PRIVATE_DATA                *Private,\r
index 25b0429..866aaad 100644 (file)
@@ -1032,10 +1032,8 @@ GraphicsConsoleConOutTestString (
   UINT16                GlyphWidth;\r
   UINT32                GlyphStatus;\r
   UINT16                Count;\r
-  GRAPHICS_CONSOLE_DEV  *Private;\r
   GLYPH_UNION           *Glyph;\r
 \r
-  Private     = GRAPHICS_CONSOLE_CON_OUT_DEV_FROM_THIS (This);\r
   GlyphStatus = 0;\r
   Count       = 0;\r
 \r
index 6621b64..45eb632 100644 (file)
@@ -402,9 +402,9 @@ TerminalDriverBindingStart (
                                       Mode->BaudRate,\r
                                       Mode->ReceiveFifoDepth,\r
                                       (UINT32) SerialInTimeOut,\r
-                                      Mode->Parity,\r
+                                      (EFI_PARITY_TYPE) (Mode->Parity),\r
                                       (UINT8) Mode->DataBits,\r
-                                      Mode->StopBits\r
+                                      (EFI_STOP_BITS_TYPE) (Mode->StopBits)\r
                                       );\r
   if (EFI_ERROR (Status)) {\r
     //\r
@@ -897,6 +897,7 @@ TerminalUpdateConsoleDevVariable (
                   VariableSize,\r
                   Variable\r
                   );\r
+  ASSERT_EFI_ERROR (Status);\r
   gBS->FreePool (Variable);\r
 \r
   return ;\r
@@ -1015,6 +1016,7 @@ TerminalRemoveConsoleDevVariable (
                     VariableSize,\r
                     NewVariable\r
                     );\r
+    ASSERT_EFI_ERROR (Status);\r
   }\r
 \r
   if (NewVariable != NULL) {\r
index 3c62138..4dbb17e 100644 (file)
@@ -268,9 +268,9 @@ TerminalConInCheckForKey (
                         Mode->BaudRate,\r
                         Mode->ReceiveFifoDepth,\r
                         (UINT32) SerialInTimeOut,\r
-                        Mode->Parity,\r
+                        (EFI_PARITY_TYPE) (Mode->Parity),\r
                         (UINT8) Mode->DataBits,\r
-                        Mode->StopBits\r
+                        (EFI_STOP_BITS_TYPE) (Mode->StopBits)\r
                         );\r
 \r
     if (EFI_ERROR (Status)) {\r
@@ -681,6 +681,7 @@ UnicodeFiFoGetKeyCount (
   }\r
 }\r
 \r
+STATIC\r
 VOID\r
 UnicodeToEfiKeyFlushState (\r
   IN  TERMINAL_DEV    *TerminalDevice\r
@@ -1162,6 +1163,7 @@ ESC R ESC r ESC R = Reset System
                       TimerRelative,\r
                       (UINT64)20000000\r
                       );\r
+      ASSERT_EFI_ERROR (Status);\r
       continue;\r
     }\r
 \r
index 01e7a6e..0ff5b4f 100644 (file)
@@ -50,6 +50,7 @@ GetNextDataRecord (
   IN OUT  UINT64          *PtrCurrentMTC\r
   );\r
 \r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 DataHubLogData (\r
@@ -177,6 +178,7 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 DataHubGetNextRecord (\r
@@ -306,6 +308,7 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 DataHubRegisterFilterDriver (\r
@@ -404,6 +407,7 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 DataHubUnregisterFilterDriver (\r
index 10563ec..71c9578 100644 (file)
@@ -294,4 +294,26 @@ Returns:
 --*/\r
 ;\r
 \r
+VOID\r
+CommonHandler (\r
+  IN EFI_EXCEPTION_TYPE ExceptionType,\r
+  IN EFI_SYSTEM_CONTEXT Context\r
+  )\r
+/*++\r
+\r
+Routine Description:\r
+  C routine that is called for all registered exceptions.  This is the main\r
+  exception dispatcher.  Must be public because it's referenced from AsmFuncs.s.\r
+\r
+Arguments:\r
+  ExceptionType - Exception Type\r
+  Context       - System Context\r
+\r
+Returns:\r
+\r
+  Nothing\r
+  \r
+--*/\r
+;\r
+\r
 #endif\r
index 2152f75..88d19d4 100644 (file)
@@ -29,19 +29,91 @@ Revision History
 #include "DebugPort.h"\r
 \r
 //\r
-// Misc. functions local to this module\r
+// Misc. functions local to this module..\r
 //\r
 STATIC\r
 VOID\r
 GetDebugPortVariable (\r
-  DEBUGPORT_DEVICE  *DebugPortDevice\r
-  );\r
+  DEBUGPORT_DEVICE            *DebugPortDevice\r
+  )\r
+/*++\r
 \r
-EFI_STATUS\r
-EFIAPI\r
-ImageUnloadHandler (\r
-  EFI_HANDLE ImageHandle\r
-  );\r
+Routine Description:\r
+  Local worker function to obtain device path information from DebugPort variable.\r
+  Records requested settings in DebugPort device structure.\r
+  \r
+Arguments:\r
+  DEBUGPORT_DEVICE  *DebugPortDevice,\r
+\r
+Returns:\r
+\r
+  Nothing\r
+\r
+--*/\r
+{\r
+  UINTN                     DataSize;\r
+  EFI_DEVICE_PATH_PROTOCOL  *DevicePath;\r
+  EFI_STATUS                Status;\r
+\r
+  DataSize = 0;\r
+\r
+  Status = gRT->GetVariable (\r
+                  (CHAR16 *) EFI_DEBUGPORT_VARIABLE_NAME,\r
+                  &gEfiDebugPortVariableGuid,\r
+                  NULL,\r
+                  &DataSize,\r
+                  DebugPortDevice->DebugPortVariable\r
+                  );\r
+\r
+  if (Status == EFI_BUFFER_TOO_SMALL) {\r
+    if (gDebugPortDevice->DebugPortVariable != NULL) {\r
+      gBS->FreePool (gDebugPortDevice->DebugPortVariable);\r
+    }\r
+\r
+    DebugPortDevice->DebugPortVariable = AllocatePool (DataSize);\r
+    if (DebugPortDevice->DebugPortVariable != NULL) {\r
+      gRT->GetVariable (\r
+            (CHAR16 *) EFI_DEBUGPORT_VARIABLE_NAME,\r
+            &gEfiDebugPortVariableGuid,\r
+            NULL,\r
+            &DataSize,\r
+            DebugPortDevice->DebugPortVariable\r
+            );\r
+      DevicePath = (EFI_DEVICE_PATH_PROTOCOL *) DebugPortDevice->DebugPortVariable;\r
+      while (!EfiIsDevicePathEnd (DevicePath) && !EfiIsUartDevicePath (DevicePath)) {\r
+        DevicePath = EfiNextDevicePathNode (DevicePath);\r
+      }\r
+\r
+      if (EfiIsDevicePathEnd (DevicePath)) {\r
+        gBS->FreePool (gDebugPortDevice->DebugPortVariable);\r
+        DebugPortDevice->DebugPortVariable = NULL;\r
+      } else {\r
+        gBS->CopyMem (\r
+              &DebugPortDevice->BaudRate,\r
+              &((UART_DEVICE_PATH *) DevicePath)->BaudRate,\r
+              sizeof (((UART_DEVICE_PATH *) DevicePath)->BaudRate)\r
+              );\r
+        DebugPortDevice->ReceiveFifoDepth = DEBUGPORT_UART_DEFAULT_FIFO_DEPTH;\r
+        DebugPortDevice->Timeout          = DEBUGPORT_UART_DEFAULT_TIMEOUT;\r
+        gBS->CopyMem (\r
+              &DebugPortDevice->Parity,\r
+              &((UART_DEVICE_PATH *) DevicePath)->Parity,\r
+              sizeof (((UART_DEVICE_PATH *) DevicePath)->Parity)\r
+              );\r
+        gBS->CopyMem (\r
+              &DebugPortDevice->DataBits,\r
+              &((UART_DEVICE_PATH *) DevicePath)->DataBits,\r
+              sizeof (((UART_DEVICE_PATH *) DevicePath)->DataBits)\r
+              );\r
+        gBS->CopyMem (\r
+              &DebugPortDevice->StopBits,\r
+              &((UART_DEVICE_PATH *) DevicePath)->StopBits,\r
+              sizeof (((UART_DEVICE_PATH *) DevicePath)->StopBits)\r
+              );\r
+      }\r
+    }\r
+  }\r
+}\r
 \r
 //\r
 // Globals\r
@@ -57,8 +129,6 @@ EFI_DRIVER_BINDING_PROTOCOL gDebugPortDriverBinding = {
 };\r
 \r
 DEBUGPORT_DEVICE  *gDebugPortDevice;\r
-static UINT32     mHid16550;\r
-static UINT32     mHidStdPcComPort;\r
 \r
 //\r
 // implementation code\r
@@ -90,9 +160,6 @@ Returns:
 \r
 --*/\r
 {\r
-  mHid16550         = EFI_ACPI_16550UART_HID;\r
-  mHidStdPcComPort  = EFI_ACPI_PC_COMPORT_HID;\r
-\r
   //\r
   // Allocate and Initialize dev structure\r
   //\r
@@ -113,9 +180,9 @@ Returns:
   gDebugPortDevice->BaudRate = DEBUGPORT_UART_DEFAULT_BAUDRATE;\r
   gDebugPortDevice->ReceiveFifoDepth = DEBUGPORT_UART_DEFAULT_FIFO_DEPTH;\r
   gDebugPortDevice->Timeout = DEBUGPORT_UART_DEFAULT_TIMEOUT;\r
-  gDebugPortDevice->Parity = DEBUGPORT_UART_DEFAULT_PARITY;\r
+  gDebugPortDevice->Parity = (EFI_PARITY_TYPE) DEBUGPORT_UART_DEFAULT_PARITY;\r
   gDebugPortDevice->DataBits = DEBUGPORT_UART_DEFAULT_DATA_BITS;\r
-  gDebugPortDevice->StopBits = DEBUGPORT_UART_DEFAULT_STOP_BITS;\r
+  gDebugPortDevice->StopBits = (EFI_STOP_BITS_TYPE) DEBUGPORT_UART_DEFAULT_STOP_BITS;\r
 \r
   return EFI_SUCCESS;\r
 }\r
@@ -521,11 +588,9 @@ Returns:
 \r
 --*/\r
 {\r
-  DEBUGPORT_DEVICE  *DebugPortDevice;\r
   UINTN             BufferSize;\r
   UINTN             BitBucket;\r
 \r
-  DebugPortDevice = DEBUGPORT_DEVICE_FROM_THIS (This);\r
   while (This->Poll (This) == EFI_SUCCESS) {\r
     BufferSize = 1;\r
     This->Read (This, 0, &BufferSize, &BitBucket);\r
@@ -694,92 +759,6 @@ Returns:
 \r
   return Status;\r
 }\r
-//\r
-// Misc. functions local to this module..\r
-//\r
-STATIC\r
-VOID\r
-GetDebugPortVariable (\r
-  DEBUGPORT_DEVICE            *DebugPortDevice\r
-  )\r
-/*++\r
-\r
-Routine Description:\r
-  Local worker function to obtain device path information from DebugPort variable.\r
-  Records requested settings in DebugPort device structure.\r
-  \r
-Arguments:\r
-  DEBUGPORT_DEVICE  *DebugPortDevice,\r
-\r
-Returns:\r
-\r
-  Nothing\r
-\r
---*/\r
-{\r
-  UINTN                     DataSize;\r
-  EFI_DEVICE_PATH_PROTOCOL  *DevicePath;\r
-  EFI_STATUS                Status;\r
-\r
-  DataSize = 0;\r
-\r
-  Status = gRT->GetVariable (\r
-                  (CHAR16 *) EFI_DEBUGPORT_VARIABLE_NAME,\r
-                  &gEfiDebugPortVariableGuid,\r
-                  NULL,\r
-                  &DataSize,\r
-                  DebugPortDevice->DebugPortVariable\r
-                  );\r
-\r
-  if (Status == EFI_BUFFER_TOO_SMALL) {\r
-    if (gDebugPortDevice->DebugPortVariable != NULL) {\r
-      gBS->FreePool (gDebugPortDevice->DebugPortVariable);\r
-    }\r
-\r
-    DebugPortDevice->DebugPortVariable = AllocatePool (DataSize);\r
-    if (DebugPortDevice->DebugPortVariable != NULL) {\r
-      gRT->GetVariable (\r
-            (CHAR16 *) EFI_DEBUGPORT_VARIABLE_NAME,\r
-            &gEfiDebugPortVariableGuid,\r
-            NULL,\r
-            &DataSize,\r
-            DebugPortDevice->DebugPortVariable\r
-            );\r
-      DevicePath = (EFI_DEVICE_PATH_PROTOCOL *) DebugPortDevice->DebugPortVariable;\r
-      while (!EfiIsDevicePathEnd (DevicePath) && !EfiIsUartDevicePath (DevicePath)) {\r
-        DevicePath = EfiNextDevicePathNode (DevicePath);\r
-      }\r
-\r
-      if (EfiIsDevicePathEnd (DevicePath)) {\r
-        gBS->FreePool (gDebugPortDevice->DebugPortVariable);\r
-        DebugPortDevice->DebugPortVariable = NULL;\r
-      } else {\r
-        gBS->CopyMem (\r
-              &DebugPortDevice->BaudRate,\r
-              &((UART_DEVICE_PATH *) DevicePath)->BaudRate,\r
-              sizeof (((UART_DEVICE_PATH *) DevicePath)->BaudRate)\r
-              );\r
-        DebugPortDevice->ReceiveFifoDepth = DEBUGPORT_UART_DEFAULT_FIFO_DEPTH;\r
-        DebugPortDevice->Timeout          = DEBUGPORT_UART_DEFAULT_TIMEOUT;\r
-        gBS->CopyMem (\r
-              &DebugPortDevice->Parity,\r
-              &((UART_DEVICE_PATH *) DevicePath)->Parity,\r
-              sizeof (((UART_DEVICE_PATH *) DevicePath)->Parity)\r
-              );\r
-        gBS->CopyMem (\r
-              &DebugPortDevice->DataBits,\r
-              &((UART_DEVICE_PATH *) DevicePath)->DataBits,\r
-              sizeof (((UART_DEVICE_PATH *) DevicePath)->DataBits)\r
-              );\r
-        gBS->CopyMem (\r
-              &DebugPortDevice->StopBits,\r
-              &((UART_DEVICE_PATH *) DevicePath)->StopBits,\r
-              sizeof (((UART_DEVICE_PATH *) DevicePath)->StopBits)\r
-              );\r
-      }\r
-    }\r
-  }\r
-}\r
 \r
 EFI_STATUS\r
 EFIAPI\r
index aff628b..cce223f 100644 (file)
@@ -21,6 +21,7 @@ Abstract:
 \r
 #include "DevicePath.h"\r
 \r
+STATIC\r
 CHAR16 *\r
 StrDuplicate (\r
   IN CONST CHAR16  *Src\r
@@ -48,6 +49,7 @@ StrDuplicate (
   return ReturnStr;\r
 }\r
 \r
+STATIC\r
 CHAR16 *\r
 GetParamByNodeName (\r
   IN CHAR16 *Str,\r
@@ -118,6 +120,7 @@ GetParamByNodeName (
   return ParamStr;\r
 }\r
 \r
+STATIC\r
 CHAR16 *\r
 SplitStr (\r
   IN OUT CHAR16 **List,\r
@@ -176,6 +179,7 @@ SplitStr (
   return ReturnStr;\r
 }\r
 \r
+STATIC\r
 CHAR16 *\r
 GetNextParamStr (\r
   IN OUT CHAR16 **List\r
@@ -187,6 +191,7 @@ GetNextParamStr (
   return SplitStr (List, L',');\r
 }\r
 \r
+STATIC\r
 CHAR16 *\r
 GetNextDeviceNodeStr (\r
   IN OUT CHAR16   **DevicePath,\r
@@ -273,6 +278,7 @@ GetNextDeviceNodeStr (
   return ReturnStr;\r
 }\r
 \r
+STATIC\r
 BOOLEAN\r
 IsHexDigit (\r
   OUT UINT8      *Digit,\r
@@ -312,6 +318,7 @@ IsHexDigit (
   return FALSE;\r
 }\r
 \r
+STATIC\r
 CHAR16\r
 NibbleToHexChar (\r
   IN UINT8      Nibble\r
@@ -337,6 +344,7 @@ NibbleToHexChar (
   return (CHAR16)(Nibble - 0xA + L'A');\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 HexStringToBuf (\r
   IN OUT UINT8                     *Buf,   \r
@@ -410,7 +418,7 @@ HexStringToBuf (
     } else {\r
       Byte = Buf[Idx / 2];\r
       Byte &= 0x0F;\r
-      Byte |= Digit << 4;\r
+      Byte = (UINT8) (Byte | Digit << 4);\r
     }\r
 \r
     Buf[Idx / 2] = Byte;\r
@@ -423,6 +431,7 @@ HexStringToBuf (
   return EFI_SUCCESS;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 BufToHexString (\r
   IN OUT CHAR16                    *Str,\r
@@ -481,6 +490,7 @@ BufToHexString (
   return EFI_SUCCESS;\r
 }\r
 \r
+STATIC\r
 CHAR16 *\r
 TrimHexStr (\r
   IN CHAR16  *Str\r
@@ -519,6 +529,7 @@ TrimHexStr (
   return Str;\r
 }\r
 \r
+STATIC\r
 UINTN\r
 Xtoi (\r
   IN CHAR16  *Str\r
@@ -552,6 +563,7 @@ Returns:
   return Rvalue;\r
 }\r
 \r
+STATIC\r
 VOID\r
 Xtoi64 (\r
   IN CHAR16  *Str,\r
@@ -578,6 +590,7 @@ Returns:
   HexStringToBuf ((UINT8 *) Data, &Length, TrimHexStr (Str), NULL);\r
 }\r
 \r
+STATIC\r
 UINTN\r
 Atoi (\r
   IN CHAR16  *str\r
@@ -633,6 +646,7 @@ Returns:
   return Rvalue;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS \r
 StrToBuf (\r
   OUT UINT8    *Buf,\r
@@ -659,11 +673,11 @@ StrToBuf (
     // and for even characters, the lower nibble.\r
     //\r
     if ((Index & 1) == 0) {\r
-      Byte = Digit << 4;\r
+      Byte = (UINT8) (Digit << 4);\r
     } else {\r
       Byte = Buf[Index / 2];\r
       Byte &= 0xF0;\r
-      Byte |= Digit;\r
+      Byte = (UINT8) (Byte | Digit);\r
     }\r
 \r
     Buf[Index / 2] = Byte;\r
@@ -672,6 +686,7 @@ StrToBuf (
   return EFI_SUCCESS;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 StrToGuid (\r
   IN  CHAR16   *Str,\r
@@ -734,6 +749,7 @@ StrToGuid (
   return EFI_SUCCESS;\r
 }\r
 \r
+STATIC\r
 VOID\r
 StrToIPv4Addr (\r
   IN OUT CHAR16           **Str,\r
@@ -747,6 +763,7 @@ StrToIPv4Addr (
   }\r
 }\r
 \r
+STATIC\r
 VOID\r
 StrToIPv6Addr (\r
   IN OUT CHAR16           **Str,\r
@@ -763,6 +780,7 @@ StrToIPv6Addr (
   }\r
 }\r
 \r
+STATIC\r
 VOID\r
 StrToAscii (\r
   IN     CHAR16 *Str,\r
@@ -783,6 +801,7 @@ StrToAscii (
   *AsciiStr = Dest + 1;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextPci (\r
   IN CHAR16 *TextDeviceNode\r
@@ -806,6 +825,7 @@ DevPathFromTextPci (
   return (EFI_DEVICE_PATH_PROTOCOL *) Pci;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextPcCard (\r
   IN CHAR16 *TextDeviceNode\r
@@ -826,6 +846,7 @@ DevPathFromTextPcCard (
   return (EFI_DEVICE_PATH_PROTOCOL *) Pccard;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextMemoryMapped (\r
   IN CHAR16 *TextDeviceNode\r
@@ -851,6 +872,7 @@ DevPathFromTextMemoryMapped (
   return (EFI_DEVICE_PATH_PROTOCOL *) MemMap;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 ConvertFromTextVendor (\r
   IN CHAR16 *TextDeviceNode,\r
@@ -884,6 +906,7 @@ ConvertFromTextVendor (
   return (EFI_DEVICE_PATH_PROTOCOL *) Vendor;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextVenHw (\r
   IN CHAR16 *TextDeviceNode\r
@@ -896,6 +919,7 @@ DevPathFromTextVenHw (
            );\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextCtrl (\r
   IN CHAR16 *TextDeviceNode\r
@@ -915,6 +939,7 @@ DevPathFromTextCtrl (
   return (EFI_DEVICE_PATH_PROTOCOL *) Controller;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextAcpi (\r
   IN CHAR16 *TextDeviceNode\r
@@ -942,6 +967,7 @@ DevPathFromTextAcpi (
   return (EFI_DEVICE_PATH_PROTOCOL *) Acpi;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 ConvertFromTextAcpi (\r
   IN CHAR16 *TextDeviceNode,\r
@@ -964,6 +990,7 @@ ConvertFromTextAcpi (
   return (EFI_DEVICE_PATH_PROTOCOL *) Acpi;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextPciRoot (\r
   IN CHAR16 *TextDeviceNode\r
@@ -972,6 +999,7 @@ DevPathFromTextPciRoot (
   return ConvertFromTextAcpi (TextDeviceNode, 0x0a0341d0);\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextFloppy (\r
   IN CHAR16 *TextDeviceNode\r
@@ -980,6 +1008,7 @@ DevPathFromTextFloppy (
   return ConvertFromTextAcpi (TextDeviceNode, 0x060441d0);\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextKeyboard (\r
   IN CHAR16 *TextDeviceNode\r
@@ -988,6 +1017,7 @@ DevPathFromTextKeyboard (
   return ConvertFromTextAcpi (TextDeviceNode, 0x030141d0);\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextSerial (\r
   IN CHAR16 *TextDeviceNode\r
@@ -996,6 +1026,7 @@ DevPathFromTextSerial (
   return ConvertFromTextAcpi (TextDeviceNode, 0x050141d0);\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextParallelPort (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1004,6 +1035,7 @@ DevPathFromTextParallelPort (
   return ConvertFromTextAcpi (TextDeviceNode, 0x040141d0);\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextAcpiEx (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1025,10 +1057,10 @@ DevPathFromTextAcpiEx (
   HIDSTRStr = GetNextParamStr (&TextDeviceNode);\r
   CIDSTRStr = GetNextParamStr (&TextDeviceNode);\r
   UIDSTRStr = GetNextParamStr (&TextDeviceNode);\r
-  Length    = sizeof (ACPI_EXTENDED_HID_DEVICE_PATH) + \r
-                  (UINT16) StrLen (HIDSTRStr) + 1 +\r
-                  (UINT16) StrLen (UIDSTRStr) + 1 +\r
-                  (UINT16) StrLen (CIDSTRStr) + 1;\r
+\r
+  Length    = (UINT16) (sizeof (ACPI_EXTENDED_HID_DEVICE_PATH) + StrLen (HIDSTRStr) + 1);\r
+  Length    = (UINT16) (Length + StrLen (UIDSTRStr) + 1);\r
+  Length    = (UINT16) (Length + StrLen (CIDSTRStr) + 1);\r
   AcpiExt = (ACPI_EXTENDED_HID_DEVICE_PATH_WITH_STR *) CreateDeviceNode (\r
                                                          ACPI_DEVICE_PATH,\r
                                                          ACPI_EXTENDED_DP,\r
@@ -1053,6 +1085,7 @@ DevPathFromTextAcpiEx (
   return (EFI_DEVICE_PATH_PROTOCOL *) AcpiExt;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextAcpiExp (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1103,6 +1136,7 @@ DevPathFromTextAcpiExp (
   return (EFI_DEVICE_PATH_PROTOCOL *) AcpiExt;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextAta (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1123,13 +1157,14 @@ DevPathFromTextAta (
   SlaveMasterStr          = GetNextParamStr (&TextDeviceNode);\r
   LunStr                  = GetNextParamStr (&TextDeviceNode);\r
 \r
-  Atapi->PrimarySecondary = (StrCmp (PrimarySecondaryStr, L"Primary") == 0) ? (UINT8) 0 : (UINT8) 1;\r
-  Atapi->SlaveMaster      = (StrCmp (SlaveMasterStr, L"Master") == 0) ? (UINT8) 0 : (UINT8) 1;\r
+  Atapi->PrimarySecondary = (UINT8) ((StrCmp (PrimarySecondaryStr, L"Primary") == 0) ? 0 : 1);\r
+  Atapi->SlaveMaster      = (UINT8) ((StrCmp (SlaveMasterStr, L"Master") == 0) ? 0 : 1);\r
   Atapi->Lun              = (UINT16) Xtoi (LunStr);\r
 \r
   return (EFI_DEVICE_PATH_PROTOCOL *) Atapi;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextScsi (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1153,6 +1188,7 @@ DevPathFromTextScsi (
   return (EFI_DEVICE_PATH_PROTOCOL *) Scsi;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextFibre (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1177,6 +1213,7 @@ DevPathFromTextFibre (
   return (EFI_DEVICE_PATH_PROTOCOL *) Fibre;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromText1394 (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1198,6 +1235,7 @@ DevPathFromText1394 (
   return (EFI_DEVICE_PATH_PROTOCOL *) F1394;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextUsb (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1221,6 +1259,7 @@ DevPathFromTextUsb (
   return (EFI_DEVICE_PATH_PROTOCOL *) Usb;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextI2O (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1241,6 +1280,7 @@ DevPathFromTextI2O (
   return (EFI_DEVICE_PATH_PROTOCOL *) I2O;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextInfiniband (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1275,6 +1315,7 @@ DevPathFromTextInfiniband (
   return (EFI_DEVICE_PATH_PROTOCOL *) InfiniBand;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextVenMsg (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1287,6 +1328,7 @@ DevPathFromTextVenMsg (
             );\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextVenPcAnsi (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1303,6 +1345,7 @@ DevPathFromTextVenPcAnsi (
   return (EFI_DEVICE_PATH_PROTOCOL *) Vendor;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextVenVt100 (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1319,6 +1362,7 @@ DevPathFromTextVenVt100 (
   return (EFI_DEVICE_PATH_PROTOCOL *) Vendor;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextVenVt100Plus (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1335,6 +1379,7 @@ DevPathFromTextVenVt100Plus (
   return (EFI_DEVICE_PATH_PROTOCOL *) Vendor;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextVenUtf8 (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1351,6 +1396,7 @@ DevPathFromTextVenUtf8 (
   return (EFI_DEVICE_PATH_PROTOCOL *) Vendor;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextUartFlowCtrl (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1378,6 +1424,7 @@ DevPathFromTextUartFlowCtrl (
   return (EFI_DEVICE_PATH_PROTOCOL *) UartFlowControl;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextSAS (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1442,6 +1489,7 @@ DevPathFromTextSAS (
   return (EFI_DEVICE_PATH_PROTOCOL *) Sas;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextDebugPort (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1460,6 +1508,7 @@ DevPathFromTextDebugPort (
   return (EFI_DEVICE_PATH_PROTOCOL *) Vend;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextMAC (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1486,6 +1535,7 @@ DevPathFromTextMAC (
   return (EFI_DEVICE_PATH_PROTOCOL *) MAC;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextIPv4 (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1508,7 +1558,7 @@ DevPathFromTextIPv4 (
                                                  );\r
 \r
   StrToIPv4Addr (&RemoteIPStr, &IPv4->RemoteIpAddress);\r
-  IPv4->Protocol = (StrCmp (ProtocolStr, L"UDP") == 0) ? (UINT16) 0 : (UINT16) 1;\r
+  IPv4->Protocol = (UINT16) ((StrCmp (ProtocolStr, L"UDP") == 0) ? 0 : 1);\r
   if (StrCmp (TypeStr, L"Static") == 0) {\r
     IPv4->StaticIpAddress = TRUE;\r
   } else {\r
@@ -1523,6 +1573,7 @@ DevPathFromTextIPv4 (
   return (EFI_DEVICE_PATH_PROTOCOL *) IPv4;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextIPv6 (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1545,7 +1596,7 @@ DevPathFromTextIPv6 (
                                                  );\r
 \r
   StrToIPv6Addr (&RemoteIPStr, &IPv6->RemoteIpAddress);\r
-  IPv6->Protocol        = (StrCmp (ProtocolStr, L"UDP") == 0) ? (UINT16) 0 : (UINT16) 1;\r
+  IPv6->Protocol        = (UINT16) ((StrCmp (ProtocolStr, L"UDP") == 0) ? 0 : 1);\r
   if (StrCmp (TypeStr, L"Static") == 0) {\r
     IPv6->StaticIpAddress = TRUE;\r
   } else {\r
@@ -1560,6 +1611,7 @@ DevPathFromTextIPv6 (
   return (EFI_DEVICE_PATH_PROTOCOL *) IPv6;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextUart (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1582,7 +1634,7 @@ DevPathFromTextUart (
                                            );\r
 \r
   Uart->BaudRate  = (StrCmp (BaudStr, L"DEFAULT") == 0) ? 115200 : Atoi (BaudStr);\r
-  Uart->DataBits  = (StrCmp (DataBitsStr, L"DEFAULT") == 0) ? (UINT8) 8 : (UINT8) Atoi (DataBitsStr);\r
+  Uart->DataBits  = (UINT8) ((StrCmp (DataBitsStr, L"DEFAULT") == 0) ? 8 : Atoi (DataBitsStr));\r
   switch (*ParityStr) {\r
   case L'D':\r
     Uart->Parity = 0;\r
@@ -1626,6 +1678,7 @@ DevPathFromTextUart (
   return (EFI_DEVICE_PATH_PROTOCOL *) Uart;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 ConvertFromTextUsbClass (\r
   IN CHAR16         *TextDeviceNode,\r
@@ -1670,6 +1723,7 @@ ConvertFromTextUsbClass (
 }\r
 \r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextUsbClass (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1683,6 +1737,7 @@ DevPathFromTextUsbClass (
   return ConvertFromTextUsbClass (TextDeviceNode, &UsbClassText);\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextUsbAudio (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1697,6 +1752,7 @@ DevPathFromTextUsbAudio (
   return ConvertFromTextUsbClass (TextDeviceNode, &UsbClassText);\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextUsbCDCControl (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1711,6 +1767,7 @@ DevPathFromTextUsbCDCControl (
   return ConvertFromTextUsbClass (TextDeviceNode, &UsbClassText);\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextUsbHID (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1725,6 +1782,7 @@ DevPathFromTextUsbHID (
   return ConvertFromTextUsbClass (TextDeviceNode, &UsbClassText);\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextUsbImage (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1739,6 +1797,7 @@ DevPathFromTextUsbImage (
   return ConvertFromTextUsbClass (TextDeviceNode, &UsbClassText);\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextUsbPrinter (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1753,6 +1812,7 @@ DevPathFromTextUsbPrinter (
   return ConvertFromTextUsbClass (TextDeviceNode, &UsbClassText);\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextUsbMassStorage (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1767,6 +1827,7 @@ DevPathFromTextUsbMassStorage (
   return ConvertFromTextUsbClass (TextDeviceNode, &UsbClassText);\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextUsbHub (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1781,6 +1842,7 @@ DevPathFromTextUsbHub (
   return ConvertFromTextUsbClass (TextDeviceNode, &UsbClassText);\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextUsbCDCData (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1795,6 +1857,7 @@ DevPathFromTextUsbCDCData (
   return ConvertFromTextUsbClass (TextDeviceNode, &UsbClassText);\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextUsbSmartCard (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1809,6 +1872,7 @@ DevPathFromTextUsbSmartCard (
   return ConvertFromTextUsbClass (TextDeviceNode, &UsbClassText);\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextUsbVideo (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1823,6 +1887,7 @@ DevPathFromTextUsbVideo (
   return ConvertFromTextUsbClass (TextDeviceNode, &UsbClassText);\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextUsbDiagnostic (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1837,6 +1902,7 @@ DevPathFromTextUsbDiagnostic (
   return ConvertFromTextUsbClass (TextDeviceNode, &UsbClassText);\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextUsbWireless (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1851,6 +1917,7 @@ DevPathFromTextUsbWireless (
   return ConvertFromTextUsbClass (TextDeviceNode, &UsbClassText);\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextUsbDeviceFirmwareUpdate (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1866,6 +1933,7 @@ DevPathFromTextUsbDeviceFirmwareUpdate (
   return ConvertFromTextUsbClass (TextDeviceNode, &UsbClassText);\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextUsbIrdaBridge (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1881,6 +1949,7 @@ DevPathFromTextUsbIrdaBridge (
   return ConvertFromTextUsbClass (TextDeviceNode, &UsbClassText);\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextUsbTestAndMeasurement (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1896,6 +1965,7 @@ DevPathFromTextUsbTestAndMeasurement (
   return ConvertFromTextUsbClass (TextDeviceNode, &UsbClassText);\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextUsbWwid (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1922,6 +1992,7 @@ DevPathFromTextUsbWwid (
   return (EFI_DEVICE_PATH_PROTOCOL *) UsbWwid;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextUnit (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1942,6 +2013,7 @@ DevPathFromTextUnit (
   return (EFI_DEVICE_PATH_PROTOCOL *) LogicalUnit;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextiSCSI (\r
   IN CHAR16 *TextDeviceNode\r
@@ -1999,6 +2071,7 @@ DevPathFromTextiSCSI (
   return (EFI_DEVICE_PATH_PROTOCOL *) iSCSI;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextHD (\r
   IN CHAR16 *TextDeviceNode\r
@@ -2054,6 +2127,7 @@ DevPathFromTextHD (
   return (EFI_DEVICE_PATH_PROTOCOL *) Hd;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextCDROM (\r
   IN CHAR16 *TextDeviceNode\r
@@ -2080,6 +2154,7 @@ DevPathFromTextCDROM (
   return (EFI_DEVICE_PATH_PROTOCOL *) CDROM;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextVenMEDIA (\r
   IN CHAR16 *TextDeviceNode\r
@@ -2092,6 +2167,7 @@ DevPathFromTextVenMEDIA (
            );\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextFilePath (\r
   IN CHAR16 *TextDeviceNode\r
@@ -2110,6 +2186,7 @@ DevPathFromTextFilePath (
   return (EFI_DEVICE_PATH_PROTOCOL *) File;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextMedia (\r
   IN CHAR16 *TextDeviceNode\r
@@ -2130,6 +2207,7 @@ DevPathFromTextMedia (
   return (EFI_DEVICE_PATH_PROTOCOL *) Media;\r
 }\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 DevPathFromTextBBS (\r
   IN CHAR16 *TextDeviceNode\r
index 25c88e0..0315070 100644 (file)
@@ -21,6 +21,7 @@ Abstract:
 \r
 #include "DevicePath.h"\r
 \r
+STATIC\r
 EFI_DEVICE_PATH_PROTOCOL *\r
 UnpackDevicePath (\r
   IN EFI_DEVICE_PATH_PROTOCOL  *DevPath\r
@@ -95,6 +96,7 @@ UnpackDevicePath (
   return NewPath;\r
 }\r
 \r
+STATIC\r
 VOID *\r
 ReallocatePool (\r
   IN VOID                 *OldPool,\r
@@ -136,6 +138,7 @@ ReallocatePool (
   return NewPool;\r
 }\r
 \r
+STATIC\r
 CHAR16 *\r
 CatPrint (\r
   IN OUT POOL_PRINT   *Str,\r
@@ -177,7 +180,8 @@ CatPrint (
     Str->Str  = AllocateZeroPool (Size);\r
     ASSERT (Str->Str != NULL);\r
   } else {\r
-    Size = StrSize (AppendStr) + StrSize (Str->Str) - sizeof (UINT16);\r
+    Size = StrSize (AppendStr)  - sizeof (UINT16);\r
+    Size = Size + StrSize (Str->Str);\r
     Str->Str = ReallocatePool (\r
                 Str->Str,\r
                 StrSize (Str->Str),\r
@@ -196,6 +200,7 @@ CatPrint (
   return Str->Str;\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextPci (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -210,6 +215,7 @@ DevPathToTextPci (
   CatPrint (Str, L"Pci(%x,%x)", Pci->Function, Pci->Device);\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextPccard (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -224,6 +230,7 @@ DevPathToTextPccard (
   CatPrint (Str, L"PcCard(%x)", Pccard->FunctionNumber);\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextMemMap (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -243,6 +250,7 @@ DevPathToTextMemMap (
     );\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextVendor (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -357,6 +365,7 @@ DevPathToTextVendor (
   CatPrint (Str, L")");\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextController (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -375,6 +384,7 @@ DevPathToTextController (
     );\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextAcpi (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -424,6 +434,7 @@ DevPathToTextAcpi (
 \r
 #define NextStrA(a) ((UINT8 *) (((UINT8 *) (a)) + AsciiStrLen ((CHAR8 *) (a)) + 1))\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextExtAcpi (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -433,12 +444,14 @@ DevPathToTextExtAcpi (
   )\r
 {\r
   ACPI_EXTENDED_HID_DEVICE_PATH_WITH_STR  *AcpiExt;\r
+  UINT8                                   *NextString;\r
 \r
   AcpiExt = DevPath;\r
 \r
   if (AllowShortcuts) {\r
+    NextString = NextStrA (AcpiExt->HidUidCidStr);\r
     if ((*(AcpiExt->HidUidCidStr) == '\0') &&\r
-        (*(NextStrA (NextStrA (AcpiExt->HidUidCidStr))) == '\0') &&\r
+        (*(NextStrA (NextString)) == '\0') &&\r
         (AcpiExt->UID == 0)\r
         ) {\r
       if ((AcpiExt->HID & PNP_EISA_ID_MASK) == PNP_EISA_ID_CONST) {\r
@@ -462,6 +475,8 @@ DevPathToTextExtAcpi (
     return ;\r
   }\r
 \r
+  NextString = NextStrA (AcpiExt->HidUidCidStr);\r
+  NextString = NextStrA (NextString);\r
   if ((AcpiExt->HID & PNP_EISA_ID_MASK) == PNP_EISA_ID_CONST) {\r
     CatPrint (\r
       Str,\r
@@ -470,7 +485,7 @@ DevPathToTextExtAcpi (
       AcpiExt->CID,\r
       AcpiExt->UID,\r
       AcpiExt->HidUidCidStr,\r
-      NextStrA (NextStrA (AcpiExt->HidUidCidStr)),\r
+      NextString,\r
       NextStrA (AcpiExt->HidUidCidStr)\r
       );\r
   } else {\r
@@ -481,12 +496,13 @@ DevPathToTextExtAcpi (
       AcpiExt->CID,\r
       AcpiExt->UID,\r
       AcpiExt->HidUidCidStr,\r
-      NextStrA (NextStrA (AcpiExt->HidUidCidStr)),\r
+      NextString,\r
       NextStrA (AcpiExt->HidUidCidStr)\r
       );\r
   }\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextAtapi (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -512,6 +528,7 @@ DevPathToTextAtapi (
   }\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextScsi (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -526,6 +543,7 @@ DevPathToTextScsi (
   CatPrint (Str, L"Scsi(%x,%x)", Scsi->Pun, Scsi->Lun);\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextFibre (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -540,6 +558,7 @@ DevPathToTextFibre (
   CatPrint (Str, L"Fibre(%lx,%lx)", Fibre->WWN, Fibre->Lun);\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToText1394 (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -554,6 +573,7 @@ DevPathToText1394 (
   CatPrint (Str, L"I1394(%lx)", F1394->Guid);\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextUsb (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -568,6 +588,7 @@ DevPathToTextUsb (
   CatPrint (Str, L"USB(%x,%x)", Usb->ParentPortNumber, Usb->InterfaceNumber);\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextUsbWWID (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -588,6 +609,7 @@ DevPathToTextUsbWWID (
     );\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextLogicalUnit (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -602,6 +624,7 @@ DevPathToTextLogicalUnit (
   CatPrint (Str, L"Unit(%x)", LogicalUnit->Lun);\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextUsbClass (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -794,6 +817,7 @@ DevPathToTextUsbClass (
     );\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextI2O (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -808,6 +832,7 @@ DevPathToTextI2O (
   CatPrint (Str, L"I2O(%x)", I2O->Tid);\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextMacAddr (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -836,6 +861,7 @@ DevPathToTextMacAddr (
   CatPrint (Str, L",%x)", MAC->IfType);\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextIPv4 (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -875,6 +901,7 @@ DevPathToTextIPv4 (
     );\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextIPv6 (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -950,6 +977,7 @@ DevPathToTextIPv6 (
     );\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextInfiniBand (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -972,6 +1000,7 @@ DevPathToTextInfiniBand (
     );\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextUart (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -1051,6 +1080,7 @@ DevPathToTextUart (
   }\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextiSCSI (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -1086,6 +1116,7 @@ DevPathToTextiSCSI (
   CatPrint (Str, L"%s)", (iSCSI->NetworkProtocol == 0) ? L"TCP" : L"reserved");\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextHardDrive (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -1134,6 +1165,7 @@ DevPathToTextHardDrive (
   CatPrint (Str, L"%lx,%lx)", Hd->PartitionStart, Hd->PartitionSize);\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextCDROM (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -1153,6 +1185,7 @@ DevPathToTextCDROM (
   CatPrint (Str, L"CDROM(%x,%lx,%lx)", Cd->BootEntry, Cd->PartitionStart, Cd->PartitionSize);\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextFilePath (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -1167,6 +1200,7 @@ DevPathToTextFilePath (
   CatPrint (Str, L"%s", Fp->PathName);\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextMediaProtocol (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -1181,6 +1215,7 @@ DevPathToTextMediaProtocol (
   CatPrint (Str, L"Media(%g)", &MediaProt->Protocol);\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextBBS (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -1233,6 +1268,7 @@ DevPathToTextBBS (
   CatPrint (Str, L",%x)", Bbs->StatusFlag);\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextEndInstance (\r
   IN OUT POOL_PRINT  *Str,\r
@@ -1244,6 +1280,7 @@ DevPathToTextEndInstance (
   CatPrint (Str, L",");\r
 }\r
 \r
+STATIC\r
 VOID\r
 DevPathToTextNodeUnknown (\r
   IN OUT POOL_PRINT  *Str,\r
index ce6b0d5..8dd0b4b 100644 (file)
@@ -31,6 +31,7 @@ Abstract:
 \r
 #include "Partition.h"\r
 \r
+STATIC\r
 BOOLEAN\r
 PartitionValidMbr (\r
   IN  MASTER_BOOT_RECORD      *Mbr,\r
index 77cee24..6a896e4 100644 (file)
@@ -437,6 +437,7 @@ PartitionDriverBindingStop (
   return EFI_SUCCESS;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 PartitionReset (\r
@@ -469,6 +470,7 @@ PartitionReset (
                                   );\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 PartitionReadBlocks (\r
@@ -524,6 +526,7 @@ PartitionReadBlocks (
   return Private->DiskIo->ReadDisk (Private->DiskIo, MediaId, Offset, BufferSize, Buffer);\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 PartitionWriteBlocks (\r
@@ -580,6 +583,7 @@ PartitionWriteBlocks (
   return Private->DiskIo->WriteDisk (Private->DiskIo, MediaId, Offset, BufferSize, Buffer);\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 PartitionFlushBlocks (\r
index 70db157..ce9b3aa 100644 (file)
@@ -1098,7 +1098,6 @@ Returns:
   UINT8       Operands;\r
   VOID        *EbcEntryPoint;\r
   VOID        *Thunk;\r
-  EFI_STATUS  Status;\r
   UINT64      U64EbcEntryPoint;\r
   INT32       Offset;\r
 \r
@@ -1162,7 +1161,7 @@ Returns:
     //\r
     // Now create a new thunk\r
     //\r
-    Status = EbcCreateThunks (VmPtr->ImageHandle, EbcEntryPoint, &Thunk, 0);\r
+    EbcCreateThunks (VmPtr->ImageHandle, EbcEntryPoint, &Thunk, 0);\r
 \r
     //\r
     // Finally replace the EBC entry point memory with the thunk address\r
index eceb591..283cdaa 100644 (file)
@@ -76,6 +76,7 @@ PushU64 (
   *(UINT64 *) VmPtr->R[0] = Arg;\r
 }\r
 \r
+STATIC\r
 UINT64\r
 EbcInterpret (\r
   UINT64      Arg1,\r
@@ -468,14 +469,14 @@ Returns:
   //\r
   // Next is jumbled data, including opcode and rest of address\r
   //\r
-  Code[2] = LShiftU64 (Imm7b, 13)\r
-          | LShiftU64 (0x00, 20)   // vc\r
-          | LShiftU64 (Ic, 21)\r
-          | LShiftU64 (Imm5c, 22)\r
-          | LShiftU64 (Imm9d, 27)\r
-          | LShiftU64 (I, 36)\r
-          | LShiftU64 ((UINT64)MOVL_OPCODE, 37)\r
-          | LShiftU64 ((RegNum & 0x7F), 6);\r
+  Code[2] = LShiftU64 (Imm7b, 13);\r
+  Code[2] = Code[2] | LShiftU64 (0x00, 20);   // vc\r
+  Code[2] = Code[2] | LShiftU64 (Ic, 21);\r
+  Code[2] = Code[2] | LShiftU64 (Imm5c, 22);\r
+  Code[2] = Code[2] | LShiftU64 (Imm9d, 27);\r
+  Code[2] = Code[2] | LShiftU64 (I, 36);\r
+  Code[2] = Code[2] | LShiftU64 ((UINT64)MOVL_OPCODE, 37);\r
+  Code[2] = Code[2] | LShiftU64 ((RegNum & 0x7F), 6);\r
 \r
   WriteBundle ((VOID *) Ptr, 0x05, Code[0], Code[1], Code[2]);\r
 \r
@@ -533,14 +534,14 @@ Returns:
   //\r
   // Next is jumbled data, including opcode and rest of address\r
   //\r
-  Code[2] = LShiftU64 (Imm7b, 13)\r
-          | LShiftU64 (0x00, 20)   // vc\r
-          | LShiftU64 (Ic, 21)\r
-          | LShiftU64 (Imm5c, 22)\r
-          | LShiftU64 (Imm9d, 27)\r
-          | LShiftU64 (I, 36)\r
-          | LShiftU64 ((UINT64)MOVL_OPCODE, 37)\r
-          | LShiftU64 ((RegNum & 0x7F), 6);\r
+  Code[2] = LShiftU64 (Imm7b, 13);\r
+  Code[2] = Code[2] | LShiftU64 (0x00, 20);   // vc\r
+  Code[2] = Code[2] | LShiftU64 (Ic, 21);\r
+  Code[2] = Code[2] | LShiftU64 (Imm5c, 22);\r
+  Code[2] = Code[2] | LShiftU64 (Imm9d, 27);\r
+  Code[2] = Code[2] | LShiftU64 (I, 36);\r
+  Code[2] = Code[2] | LShiftU64 ((UINT64)MOVL_OPCODE, 37);\r
+  Code[2] = Code[2] | LShiftU64 ((RegNum & 0x7F), 6);\r
 \r
   WriteBundle ((VOID *) Ptr, 0x05, Code[0], Code[1], Code[2]);\r
 \r
@@ -604,14 +605,14 @@ Returns:
   //\r
   // Next is jumbled data, including opcode and rest of address\r
   //\r
-  Code[2] =   LShiftU64(Imm7b, 13)\r
-          | LShiftU64 (0x00, 20)   // vc\r
-          | LShiftU64 (Ic, 21)\r
-          | LShiftU64 (Imm5c, 22)\r
-          | LShiftU64 (Imm9d, 27)\r
-          | LShiftU64 (I, 36)\r
-          | LShiftU64 ((UINT64)MOVL_OPCODE, 37)\r
-          | LShiftU64 ((RegNum & 0x7F), 6);\r
+  Code[2] =   LShiftU64(Imm7b, 13);\r
+  Code[2] = Code[2] | LShiftU64 (0x00, 20);   // vc\r
+  Code[2] = Code[2] | LShiftU64 (Ic, 21);\r
+  Code[2] = Code[2] | LShiftU64 (Imm5c, 22);\r
+  Code[2] = Code[2] | LShiftU64 (Imm9d, 27);\r
+  Code[2] = Code[2] | LShiftU64 (I, 36);\r
+  Code[2] = Code[2] | LShiftU64 ((UINT64)MOVL_OPCODE, 37);\r
+  Code[2] = Code[2] | LShiftU64 ((RegNum & 0x7F), 6);\r
 \r
   WriteBundle ((VOID *) Ptr, 0x05, Code[0], Code[1], Code[2]);\r
 \r
@@ -722,8 +723,11 @@ Returns:
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
-  Low64   = LShiftU64 (Slot1, 46) | LShiftU64 (Slot0, 5) | Template;\r
-  High64  = RShiftU64 (Slot1, 18) | LShiftU64 (Slot2, 23);\r
+  Low64   = LShiftU64 (Slot1, 46);\r
+  Low64   = Low64 | LShiftU64 (Slot0, 5) | Template;\r
+\r
+  High64  = RShiftU64 (Slot1, 18);\r
+  High64  = High64 | LShiftU64 (Slot2, 23);\r
 \r
   //\r
   // Now write it all out\r
@@ -816,14 +820,13 @@ Returns:
   CodeTwo5c = RShiftU64 (*((UINT64 *)CalleeAddr + 3), 45) & 0x1F;\r
   CodeTwo9d = RShiftU64 (*((UINT64 *)CalleeAddr + 3), 50) & 0x1FF;\r
 \r
-  TargetEbcAddr = CodeTwo7b\r
-                  | LShiftU64 (CodeTwo9d, 7)\r
-                  | LShiftU64 (CodeTwo5c, 16)\r
-                  | LShiftU64 (CodeTwoIc, 21)\r
-                  | LShiftU64 (CodeOne18, 22)\r
-                  | LShiftU64 (CodeOne23, 40)\r
-                  | LShiftU64 (CodeTwoI, 63)\r
-                  ;\r
+  TargetEbcAddr = CodeTwo7b;\r
+  TargetEbcAddr = TargetEbcAddr | LShiftU64 (CodeTwo9d, 7);\r
+  TargetEbcAddr = TargetEbcAddr | LShiftU64 (CodeTwo5c, 16);\r
+  TargetEbcAddr = TargetEbcAddr | LShiftU64 (CodeTwoIc, 21);\r
+  TargetEbcAddr = TargetEbcAddr | LShiftU64 (CodeOne18, 22);\r
+  TargetEbcAddr = TargetEbcAddr | LShiftU64 (CodeOne23, 40);\r
+  TargetEbcAddr = TargetEbcAddr | LShiftU64 (CodeTwoI, 63);\r
 \r
 Action:\r
   if (IsThunk == 1){\r
index 0eef1ac..b9563f1 100644 (file)
@@ -26,6 +26,7 @@ Revision History
 //\r
 ESAL_VARIABLE_GLOBAL  *mVariableModuleGlobal;\r
 \r
+STATIC\r
 UINT32\r
 EFIAPI\r
 ArrayLength (\r
@@ -62,6 +63,7 @@ Returns:
   return (Count * 2) + 2;\r
 }\r
 \r
+STATIC\r
 VARIABLE_STORE_STATUS\r
 EFIAPI\r
 GetVariableStoreStatus (\r
@@ -103,6 +105,7 @@ Returns:
   }\r
 }\r
 \r
+STATIC\r
 UINT8 *\r
 EFIAPI\r
 GetVariableDataPtr (\r
@@ -133,6 +136,7 @@ Returns:
   return (UINT8 *) ((UINTN) GET_VARIABLE_NAME_PTR (Variable) + Variable->NameSize + GET_PAD_SIZE (Variable->NameSize));\r
 }\r
 \r
+STATIC\r
 VARIABLE_HEADER *\r
 EFIAPI\r
 GetNextVariablePtr (\r
@@ -173,6 +177,7 @@ Returns:
   return VarHeader;\r
 }\r
 \r
+STATIC\r
 VARIABLE_HEADER *\r
 EFIAPI\r
 GetEndPointer (\r
@@ -200,6 +205,7 @@ Returns:
   return (VARIABLE_HEADER *) ((UINTN) VolHeader + VolHeader->Size);\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 FindVariable (\r
@@ -413,7 +419,7 @@ Returns:
         Variable.StartPtr = (VARIABLE_HEADER *) ((UINTN) (Global->VolatileVariableBase + sizeof (VARIABLE_STORE_HEADER)));\r
         Variable.EndPtr = (VARIABLE_HEADER *) GetEndPointer ((VARIABLE_STORE_HEADER *) ((UINTN) Global->VolatileVariableBase));\r
       } else {\r
-        return EFI_NOT_FOUND;\r
+        goto Error;\r
       }\r
 \r
       Variable.CurrPtr = Variable.StartPtr;\r
@@ -449,6 +455,7 @@ Returns:
     }\r
   }\r
 \r
+Error:\r
   return EFI_NOT_FOUND;\r
 }\r
 \r
@@ -755,6 +762,7 @@ Returns:
 }\r
 #endif\r
 \r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 InitializeVariableStore (\r
index ee2a485..da71705 100644 (file)
@@ -26,6 +26,7 @@ Revision History
 //\r
 extern ESAL_VARIABLE_GLOBAL *mVariableModuleGlobal;\r
 \r
+STATIC\r
 SAL_RETURN_REGS\r
 EsalVariableCommonEntry (\r
   IN  UINT64                                      FunctionId,\r
index 820655c..40e56ad 100644 (file)
@@ -193,7 +193,7 @@ Notes:
   }\r
 \r
   State ^= FTW_POLARITY_REVERT;\r
-  State |= NewBit;\r
+  State = (UINT8) (State | NewBit);\r
   State ^= FTW_POLARITY_REVERT;\r
 \r
   //\r
index e091ae1..48ed607 100644 (file)
@@ -91,7 +91,6 @@ Returns:
 --*/    \r
 {\r
   UINT32                TestPattern;\r
-  UINT32                TestMask;\r
   EFI_PHYSICAL_ADDRESS  TempAddress;\r
   UINT32                SpanSize;\r
 \r
@@ -101,7 +100,6 @@ Returns:
     );\r
 \r
   TestPattern = TEST_PATTERN;\r
-  TestMask    = 0;\r
   SpanSize    = 0;\r
 \r
   //\r
index f790386..478729e 100644 (file)
@@ -51,6 +51,7 @@ EFI_GUID    mEfiMtcGuid = { 0xeb704011, 0x1402, 0x11d3, { 0x8e, 0x77, 0x0, 0xa0,
 //\r
 // Worker functions\r
 //\r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 MonotonicCounterDriverGetNextMonotonicCount (\r
@@ -132,6 +133,7 @@ Returns:
   @retval EFI_DEVICE_ERROR      The variable could not be saved due to a hardware failure.\r
 \r
 **/\r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 MonotonicCounterDriverGetNextHighMonotonicCount (\r
@@ -181,6 +183,7 @@ Returns:
 \r
 }\r
 \r
+STATIC\r
 VOID\r
 EFIAPI\r
 EfiMtcEventHandler (\r
index a391709..6d5a616 100644 (file)
@@ -494,6 +494,23 @@ typedef union {
 #include "dhcp.h"\r
 #include "tftp.h"\r
 \r
+VOID\r
+InitArpHeader (\r
+  VOID\r
+  )\r
+/*++\r
+Routine description:\r
+  Initialize ARP packet header.\r
+\r
+Parameters:\r
+  none\r
+\r
+Returns:\r
+  none\r
+\r
+--*/\r
+;\r
+\r
 #endif /* _BC_H */\r
 \r
 /* EOF - bc.h */\r
index 303f354..f3969b9 100644 (file)
@@ -45,17 +45,6 @@ PxeBcDriverStop (
   IN  EFI_HANDLE                     *ChildHandleBuffer\r
   );\r
 \r
-extern\r
-VOID\r
-InitArpHeader (\r
-  VOID\r
-  );\r
-extern\r
-VOID\r
-OptionsStrucInit (\r
-  VOID\r
-  );\r
-\r
 //\r
 // helper routines\r
 //\r
@@ -310,8 +299,8 @@ IpChecksum2 (
 {\r
   UINT32  Sum;\r
 \r
-  Sum = (UINT16)~IpChecksum (Header, HeaderLen) + (UINT16)~IpChecksum (Message, MessageLen);\r
-\r
+  Sum = (UINT16)~IpChecksum (Header, HeaderLen);\r
+  Sum = Sum + (UINT16)~IpChecksum (Message, MessageLen);\r
   //\r
   // in case above carried\r
   //\r
@@ -1597,7 +1586,6 @@ BcStop (
   //\r
   // Lock the instance data\r
   //\r
-  EFI_PXE_BASE_CODE_MODE      *PxebcMode;\r
   EFI_SIMPLE_NETWORK_PROTOCOL *SnpPtr;\r
   EFI_SIMPLE_NETWORK_MODE     *SnpModePtr;\r
   EFI_STATUS                  StatCode;\r
@@ -1619,7 +1607,6 @@ BcStop (
 \r
   EfiAcquireLock (&Private->Lock);\r
 \r
-  PxebcMode   = Private->EfiBc.Mode;\r
   SnpPtr      = Private->SimpleNetwork;\r
   SnpModePtr  = SnpPtr->Mode;\r
 \r
@@ -2069,13 +2056,11 @@ BcSetStationIP (
 --*/\r
 {\r
   EFI_PXE_BASE_CODE_MODE  *PxebcMode;\r
-  EFI_STATUS              StatCode;\r
   PXE_BASECODE_DEVICE     *Private;\r
 \r
   //\r
   // Lock the instance data and make sure started\r
   //\r
-  StatCode = EFI_SUCCESS;\r
 \r
   if (This == NULL) {\r
     DEBUG ((EFI_D_ERROR, "BC *This pointer == NULL"));\r
index cd448b3..102d30b 100644 (file)
@@ -622,6 +622,11 @@ typedef struct DhcpReceiveBufferStruct {
 #define CREDENTIALS_TYPE  (1 << 3)\r
 #define USE_THREE_BYTE    (1 << 4)\r
 \r
+VOID\r
+OptionsStrucInit (\r
+  VOID\r
+  );\r
+\r
 #endif // _DHCP_H\r
 \r
 /* EOF - dhcp.h */\r
index 57664d8..bb7022b 100644 (file)
@@ -211,7 +211,7 @@ OptionsStrucInit (
   CopyMem (DHCPOpStart.DhcpClassIdentifier.Data.UndiMajor, "yyy", sizeof ("yyy"));\r
   CopyMem (DHCPOpStart.DhcpClassIdentifier.Data.UndiMinor, "xxx", sizeof ("xxx"));\r
   DHCPOpStart.End[0] = OP_END;\r
-};\r
+}\r
 \r
 /* - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - */\r
 \r
@@ -399,6 +399,7 @@ STATIC UINT8  OurDhcpOptions[MAX_OUR_OPT] = {
 #pragma pack()\r
 \r
 /* - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - */\r
+STATIC\r
 CHAR8 *\r
 PxeBcLibGetSmbiosString (\r
   IN  SMBIOS_STRUCTURE_POINTER  *Smbios,\r
@@ -855,6 +856,7 @@ DecodeOptions (
 }\r
 \r
 /* - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - */\r
+STATIC\r
 VOID\r
 Parse (\r
   DHCP_RECEIVE_BUFFER *RxBufPtr,\r
@@ -958,6 +960,7 @@ CopyParse (
 }\r
 \r
 /* - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - */\r
+STATIC\r
 BOOLEAN\r
 AckEdit (\r
   DHCP_RECEIVE_BUFFER *DhcpRxBuf\r
@@ -984,6 +987,7 @@ AckEdit (
 //\r
 // if a discover type packet, make sure all required fields are present\r
 //\r
+STATIC\r
 BOOLEAN\r
 DHCPOfferAckEdit (\r
   DHCP_RECEIVE_BUFFER *DhcpRxBuf\r
@@ -1126,6 +1130,7 @@ DHCPOfferAckEdit (
 }\r
 \r
 /* - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - */\r
+STATIC\r
 BOOLEAN\r
 DHCPAckEdit (\r
   DHCP_RECEIVE_BUFFER *RxBufPtr\r
@@ -1139,6 +1144,7 @@ DHCPAckEdit (
 //\r
 // get an offer/ack\r
 //\r
+STATIC\r
 EFI_STATUS\r
 GetOfferAck (\r
   PXE_BASECODE_DEVICE          *Private,\r
@@ -1249,6 +1255,7 @@ Returns:
 //\r
 // get DHCPOFFER's\r
 //\r
+STATIC\r
 EFI_STATUS\r
 GetOffers (\r
   PXE_BASECODE_DEVICE *Private\r
@@ -2146,6 +2153,7 @@ DoDhcpDora (
 //\r
 // determine if the server ip is in the ip list\r
 //\r
+STATIC\r
 BOOLEAN\r
 InServerList (\r
   EFI_IP_ADDRESS    *ServerIpPtr,\r
@@ -2172,6 +2180,7 @@ InServerList (
 }\r
 \r
 /* - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - */\r
+STATIC\r
 BOOLEAN\r
 ExtractBootServerList (\r
   UINT16            Type,\r
@@ -2209,6 +2218,7 @@ ExtractBootServerList (
 }\r
 \r
 /* - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - */\r
+STATIC\r
 VOID\r
 FreeMem (\r
   PXE_BASECODE_DEVICE *Private\r
@@ -2226,6 +2236,7 @@ FreeMem (
 }\r
 \r
 /* - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - */\r
+STATIC\r
 BOOLEAN\r
 GetMem (\r
   PXE_BASECODE_DEVICE *Private\r
@@ -2443,6 +2454,7 @@ VerifyCredentialOption (
 }\r
 \r
 /* - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - */\r
+STATIC\r
 EFI_STATUS\r
 DoDiscover (\r
   PXE_BASECODE_DEVICE *Private,\r
@@ -3210,13 +3222,11 @@ Returns:
 {\r
   EFI_PXE_BASE_CODE_MODE  *PxebcMode;\r
   EFI_STATUS              Status;\r
-  EFI_STATUS              StatCode;\r
   PXE_BASECODE_DEVICE     *Private;\r
 \r
   //\r
   // Lock the instance data and make sure started\r
   //\r
-  StatCode = EFI_SUCCESS;\r
 \r
   if (This == NULL) {\r
     DEBUG ((EFI_D_ERROR, "BC *This pointer == NULL"));\r
index e6e91c3..b99f2d3 100644 (file)
@@ -36,6 +36,7 @@ Revision History
 #include "Bc.h"\r
 \r
 /* - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - */\r
+STATIC\r
 UINT64\r
 Swap64 (\r
   UINT64 n\r
@@ -1182,6 +1183,7 @@ Returns:
 }\r
 \r
 /* - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - */\r
+STATIC\r
 EFI_STATUS\r
 MtftpDownload (\r
   PXE_BASECODE_DEVICE           *Private,\r
@@ -1930,6 +1932,7 @@ Returns:
 }\r
 \r
 /* - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - */\r
+STATIC\r
 EFI_STATUS\r
 PxeBcMtftp (\r
   PXE_BASECODE_DEVICE               *Private,\r
index 182462c..6659bc4 100644 (file)
@@ -393,24 +393,23 @@ Returns:
     if (gST->ConIn->ReadKeyStroke (gST->ConIn, &Key) == EFI_NOT_READY) {\r
       UINT8       Buffer[512];\r
       UINTN       BufferSize;\r
-      EFI_STATUS  Status;\r
 \r
       BufferSize = sizeof Buffer;\r
 \r
-      Status = Private->EfiBc.UdpRead (\r
-                                &Private->EfiBc,\r
-                                EFI_PXE_BASE_CODE_UDP_OPFLAGS_ANY_SRC_IP |\r
-                                EFI_PXE_BASE_CODE_UDP_OPFLAGS_ANY_SRC_PORT |\r
-                                EFI_PXE_BASE_CODE_UDP_OPFLAGS_ANY_DEST_PORT,\r
-                                NULL, /* dest ip */\r
-                                NULL, /* dest port */\r
-                                NULL, /* src ip */\r
-                                NULL, /* src port */\r
-                                NULL, /* hdr size */\r
-                                NULL, /* hdr ptr */\r
-                                &BufferSize,\r
-                                Buffer\r
-                                );\r
+      Private->EfiBc.UdpRead (\r
+                       &Private->EfiBc,\r
+                       EFI_PXE_BASE_CODE_UDP_OPFLAGS_ANY_SRC_IP |\r
+                       EFI_PXE_BASE_CODE_UDP_OPFLAGS_ANY_SRC_PORT |\r
+                       EFI_PXE_BASE_CODE_UDP_OPFLAGS_ANY_DEST_PORT,\r
+                       NULL, /* dest ip */\r
+                       NULL, /* dest port */\r
+                       NULL, /* src ip */\r
+                       NULL, /* src port */\r
+                       NULL, /* hdr size */\r
+                       NULL, /* hdr ptr */\r
+                       &BufferSize,\r
+                       Buffer\r
+                       );\r
 \r
       continue;\r
     }\r
@@ -870,6 +869,7 @@ Returns:
   return NULL;\r
 }\r
 \r
+STATIC\r
 UINT8 *\r
 PxeBcFindDhcpOpt (\r
   EFI_PXE_BASE_CODE_PACKET  *PacketPtr,\r
@@ -1088,7 +1088,6 @@ Returns:
 \r
   if (Private->EfiBc.Mode->BisSupported && Private->EfiBc.Mode->BisDetected && Private->EfiBc.Mode->PxeBisReplyReceived) {\r
     UINT64  CredentialLen;\r
-    UINTN   BlockSize;\r
     UINT8   CredentialFilename[256];\r
     UINT8   *op;\r
     VOID    *CredentialBuffer;\r
@@ -1398,11 +1397,6 @@ Returns:
 \r
   AsciiPrint ("Running LoadFile()\n");\r
 \r
-  //\r
-  // Resolve Warning 4 unreferenced parameter problem\r
-  //\r
-  FilePath = NULL;\r
-\r
   //\r
   // If either if these parameters are NULL, we cannot continue.\r
   //\r
index 05c5a8b..3055b86 100644 (file)
@@ -385,11 +385,45 @@ snp_undi32_receive (
   )\r
 ;\r
 \r
+VOID\r
+EFIAPI\r
+SnpWaitForPacketNotify (\r
+  IN EFI_EVENT  Event,\r
+  IN VOID       *SnpPtr\r
+  );\r
+\r
+EFI_STATUS\r
+pxe_start (\r
+  SNP_DRIVER *snp\r
+  );\r
+\r
+EFI_STATUS\r
+pxe_stop (\r
+  SNP_DRIVER *snp\r
+  );\r
+\r
+EFI_STATUS\r
+pxe_init (\r
+  SNP_DRIVER *snp,\r
+  UINT16     OpFlags\r
+  );\r
+\r
+EFI_STATUS\r
+pxe_shutdown (\r
+  SNP_DRIVER *snp\r
+  );\r
+\r
+EFI_STATUS\r
+pxe_get_stn_addr (\r
+  SNP_DRIVER *snp\r
+  );\r
+\r
 typedef\r
 EFI_STATUS\r
 (*issue_undi32_command) (\r
   UINT64 cdb\r
   );\r
+\r
 typedef\r
 VOID\r
 (*ptr) (\r
index 65a4d58..54574cb 100644 (file)
@@ -182,19 +182,19 @@ Returns:
 \r
   switch (NumBytes) {\r
   case 2:\r
-    Width = 1;\r
+    Width = (EFI_PCI_IO_PROTOCOL_WIDTH) 1;\r
     break;\r
 \r
   case 4:\r
-    Width = 2;\r
+    Width = (EFI_PCI_IO_PROTOCOL_WIDTH) 2;\r
     break;\r
 \r
   case 8:\r
-    Width = 3;\r
+    Width = (EFI_PCI_IO_PROTOCOL_WIDTH) 3;\r
     break;\r
 \r
   default:\r
-    Width = 0;\r
+    Width = (EFI_PCI_IO_PROTOCOL_WIDTH) 0;\r
   }\r
 \r
   switch (ReadOrWrite) {\r
@@ -344,18 +344,18 @@ Returns:
 \r
   snp   = (SNP_DRIVER *) (UINTN) UniqueId;\r
 \r
-  Width = 0;\r
+  Width = (EFI_PCI_IO_PROTOCOL_WIDTH) 0;\r
   switch (NumBytes) {\r
   case 2:\r
-    Width = 1;\r
+    Width = (EFI_PCI_IO_PROTOCOL_WIDTH) 1;\r
     break;\r
 \r
   case 4:\r
-    Width = 2;\r
+    Width = (EFI_PCI_IO_PROTOCOL_WIDTH) 2;\r
     break;\r
 \r
   case 8:\r
-    Width = 3;\r
+    Width = (EFI_PCI_IO_PROTOCOL_WIDTH) 3;\r
     break;\r
   }\r
 \r
index 6d89d4c..c301b92 100644 (file)
@@ -20,6 +20,7 @@ Revision history:
 \r
 #include "Snp.h"\r
 \r
+STATIC\r
 EFI_STATUS\r
 pxe_getstatus (\r
   SNP_DRIVER *snp,\r
index cb80171..80e6033 100644 (file)
@@ -20,13 +20,6 @@ Revision history:
 \r
 #include "Snp.h"\r
 \r
-VOID\r
-EFIAPI\r
-SnpWaitForPacketNotify (\r
-  IN EFI_EVENT  Event,\r
-  IN VOID       *SnpPtr\r
-  );\r
-\r
 EFI_STATUS\r
 pxe_init (\r
   SNP_DRIVER *snp,\r
index 11b29f4..e768b85 100644 (file)
@@ -20,6 +20,7 @@ Revision history:
 \r
 #include "Snp.h"\r
 \r
+STATIC\r
 EFI_STATUS\r
 pxe_ip2mac (\r
   IN SNP_DRIVER          *snp,\r
index e9302da..ada3069 100644 (file)
@@ -20,6 +20,7 @@ Revision history:
 \r
 #include "Snp.h"\r
 \r
+STATIC\r
 EFI_STATUS\r
 pxe_nvdata_read (\r
   IN SNP_DRIVER *snp,\r
index 2d92dde..7c9cf5e 100644 (file)
@@ -20,6 +20,7 @@ Revision history:
 \r
 #include "Snp.h"\r
 \r
+STATIC\r
 EFI_STATUS\r
 pxe_receive (\r
   SNP_DRIVER      *snp,\r
index 6e234de..258c4a9 100644 (file)
@@ -20,6 +20,7 @@ Revision history:
 \r
 #include "Snp.h"\r
 \r
+STATIC\r
 EFI_STATUS\r
 pxe_rcvfilter_enable (\r
   SNP_DRIVER      *snp,\r
@@ -112,6 +113,7 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 pxe_rcvfilter_disable (\r
   SNP_DRIVER *snp,\r
@@ -191,6 +193,7 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 pxe_rcvfilter_read (\r
   SNP_DRIVER *snp\r
index a8dace6..757e8e0 100644 (file)
@@ -20,6 +20,7 @@ Revision history:
 \r
 #include "Snp.h"\r
 \r
+STATIC\r
 EFI_STATUS\r
 pxe_reset (\r
   SNP_DRIVER *snp\r
@@ -96,11 +97,6 @@ Returns:
 {\r
   SNP_DRIVER  *snp;\r
 \r
-  //\r
-  // Resolve Warning 4 unreferenced parameter problem\r
-  //\r
-  ExtendedVerification = 0;\r
-\r
   if (this == NULL) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
index c83ca37..7146e46 100644 (file)
@@ -18,72 +18,6 @@ Abstract:
 \r
 #include "Snp.h"\r
 \r
-EFI_STATUS\r
-pxe_start (\r
-  SNP_DRIVER *snp\r
-  );\r
-EFI_STATUS\r
-pxe_stop (\r
-  SNP_DRIVER *snp\r
-  );\r
-EFI_STATUS\r
-pxe_init (\r
-  SNP_DRIVER *snp,\r
-  UINT16     OpFlags\r
-  );\r
-EFI_STATUS\r
-pxe_shutdown (\r
-  SNP_DRIVER *snp\r
-  );\r
-EFI_STATUS\r
-pxe_get_stn_addr (\r
-  SNP_DRIVER *snp\r
-  );\r
-\r
-EFI_STATUS\r
-EFIAPI\r
-InitializeSnpNiiDriver (\r
-  IN EFI_HANDLE       image_handle,\r
-  IN EFI_SYSTEM_TABLE *system_table\r
-  );\r
-\r
-EFI_STATUS\r
-EFIAPI\r
-SimpleNetworkDriverSupported (\r
-  IN EFI_DRIVER_BINDING_PROTOCOL    *This,\r
-  IN EFI_HANDLE                     Controller,\r
-  IN EFI_DEVICE_PATH_PROTOCOL       *RemainingDevicePath\r
-  );\r
-\r
-EFI_STATUS\r
-EFIAPI\r
-SimpleNetworkDriverStart (\r
-  IN EFI_DRIVER_BINDING_PROTOCOL    *This,\r
-  IN EFI_HANDLE                     Controller,\r
-  IN EFI_DEVICE_PATH_PROTOCOL       *RemainingDevicePath\r
-  );\r
-\r
-EFI_STATUS\r
-EFIAPI\r
-SimpleNetworkDriverStop (\r
-  IN  EFI_DRIVER_BINDING_PROTOCOL    *This,\r
-  IN  EFI_HANDLE                     Controller,\r
-  IN  UINTN                          NumberOfChildren,\r
-  IN  EFI_HANDLE                     *ChildHandleBuffer\r
-  );\r
-\r
-//\r
-// Simple Network Protocol Driver Global Variables\r
-//\r
-EFI_DRIVER_BINDING_PROTOCOL mSimpleNetworkDriverBinding = {\r
-  SimpleNetworkDriverSupported,\r
-  SimpleNetworkDriverStart,\r
-  SimpleNetworkDriverStop,\r
-  0xa,\r
-  NULL,\r
-  NULL\r
-};\r
-\r
 //\r
 //  Module global variables needed to support undi 3.0 interface\r
 //\r
@@ -91,202 +25,6 @@ EFI_PCI_IO_PROTOCOL         *mPciIoFncs;
 struct s_v2p                *_v2p = NULL; // undi3.0 map_list head\r
 // End Global variables\r
 //\r
-EFI_STATUS\r
-add_v2p (\r
-  IN OUT struct s_v2p           **v2p,\r
-  EFI_PCI_IO_PROTOCOL_OPERATION type,\r
-  VOID                          *vaddr,\r
-  UINTN                         bsize\r
-  )\r
-/*++\r
-\r
-Routine Description:\r
- This routine maps the given CPU address to a Device address. It creates a\r
- an entry in the map list with the virtual and physical addresses and the \r
- un map cookie.\r
-\r
-Arguments:\r
- v2p - pointer to return a map list node pointer.\r
- type - the direction in which the data flows from the given virtual address\r
-        device->cpu or cpu->device or both ways.\r
- vaddr - virtual address (or CPU address) to be mapped\r
- bsize - size of the buffer to be mapped.\r
-\r
-Returns:\r
-\r
-  EFI_SUCEESS - routine has completed the mapping\r
-  other - error as indicated.\r
-\r
---*/\r
-{\r
-  EFI_STATUS  Status;\r
-\r
-  if ((v2p == NULL) || (vaddr == NULL) || (bsize == 0)) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  Status = gBS->AllocatePool (\r
-                  EfiBootServicesData,\r
-                  sizeof (struct s_v2p),\r
-                  (VOID **) v2p\r
-                  );\r
-\r
-  if (Status != EFI_SUCCESS) {\r
-    return Status;\r
-  }\r
-\r
-  Status = mPciIoFncs->Map (\r
-                        mPciIoFncs,\r
-                        type,\r
-                        vaddr,\r
-                        &bsize,\r
-                        &(*v2p)->paddr,\r
-                        &(*v2p)->unmap\r
-                        );\r
-  if (Status != EFI_SUCCESS) {\r
-    gBS->FreePool (*v2p);\r
-    return Status;\r
-  }\r
-  (*v2p)->vaddr = vaddr;\r
-  (*v2p)->bsize = bsize;\r
-  (*v2p)->next  = _v2p;\r
-  _v2p          = *v2p;\r
-\r
-  return EFI_SUCCESS;\r
-}\r
-\r
-EFI_STATUS\r
-find_v2p (\r
-  struct s_v2p **v2p,\r
-  VOID         *vaddr\r
-  )\r
-/*++\r
-\r
-Routine Description:\r
- This routine searches the linked list of mapped address nodes (for undi3.0 \r
- interface) to find the node that corresponds to the given virtual address and\r
- returns a pointer to that node.\r
-\r
-Arguments:\r
- v2p - pointer to return a map list node pointer.\r
- vaddr - virtual address (or CPU address) to be searched in the map list\r
-\r
-Returns:\r
-\r
-  EFI_SUCEESS - if a match found!\r
-  Other       - match not found\r
-\r
---*/\r
-{\r
-  struct s_v2p  *v;\r
-\r
-  if (v2p == NULL || vaddr == NULL) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  for (v = _v2p; v != NULL; v = v->next) {\r
-    if (v->vaddr == vaddr) {\r
-      *v2p = v;\r
-      return EFI_SUCCESS;\r
-    }\r
-  }\r
-\r
-  return EFI_NOT_FOUND;\r
-}\r
-\r
-EFI_STATUS\r
-del_v2p (\r
-  VOID *vaddr\r
-  )\r
-/*++\r
-\r
-Routine Description:\r
- This routine unmaps the given virtual address and frees the memory allocated \r
- for the map list node corresponding to that address.\r
\r
-Arguments:\r
- vaddr - virtual address (or CPU address) to be unmapped\r
-\r
-Returns:\r
- EFI_SUCEESS -  if successfully unmapped\r
- Other - as indicated by the error\r
-\r
-\r
---*/\r
-{\r
-  struct s_v2p  *v;\r
-  struct s_v2p  *t;\r
-  EFI_STATUS    Status;\r
-\r
-  if (vaddr == NULL) {\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
-  if (_v2p == NULL) {\r
-    return EFI_NOT_FOUND;\r
-  }\r
-  //\r
-  // Is our node at the head of the list??\r
-  //\r
-  if ((v = _v2p)->vaddr == vaddr) {\r
-    _v2p    = _v2p->next;\r
-\r
-    Status  = mPciIoFncs->Unmap (mPciIoFncs, v->unmap);\r
-\r
-    gBS->FreePool (v);\r
-\r
-#if SNP_DEBUG\r
-    if (Status) {\r
-      Print (L"Unmap failed with status = %x\n", Status);\r
-    }\r
-#endif\r
-    return Status;\r
-  }\r
-\r
-  for (; v->next != NULL; v = t) {\r
-    if ((t = v->next)->vaddr == vaddr) {\r
-      v->next = t->next;\r
-      Status  = mPciIoFncs->Unmap (mPciIoFncs, t->unmap);\r
-      gBS->FreePool (t);\r
-#if SNP_DEBUG\r
-      if (Status) {\r
-        Print (L"Unmap failed with status = %x\n", Status);\r
-      }\r
-#endif\r
-      return Status;\r
-    }\r
-  }\r
-\r
-  return EFI_NOT_FOUND;\r
-}\r
-\r
-#if SNP_DEBUG\r
-VOID\r
-snp_wait_for_key (\r
-  VOID\r
-  )\r
-/*++\r
-\r
-Routine Description:\r
- Wait for a key stroke, used for debugging purposes\r
-\r
-Arguments:\r
- none\r
-\r
-Returns:\r
- none\r
-\r
---*/\r
-{\r
-  EFI_INPUT_KEY key;\r
-\r
-  Aprint ("\nPress any key to continue\n");\r
-\r
-  while (gST->ConIn->ReadKeyStroke (gST->ConIn, &key) == EFI_NOT_READY) {\r
-    ;\r
-  }\r
-}\r
-#endif\r
 \r
 STATIC\r
 EFI_STATUS\r
@@ -355,6 +93,7 @@ Returns:
   return cksum;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 SimpleNetworkDriverSupported (\r
@@ -531,6 +270,7 @@ Done:
   return Status;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 SimpleNetworkDriverStart (\r
@@ -1190,6 +930,7 @@ NiiError:
   return Status;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 SimpleNetworkDriverStop (\r
@@ -1313,3 +1054,211 @@ Returns:
   return Status;\r
 }\r
 \r
+//\r
+// Simple Network Protocol Driver Global Variables\r
+//\r
+EFI_DRIVER_BINDING_PROTOCOL mSimpleNetworkDriverBinding = {\r
+  SimpleNetworkDriverSupported,\r
+  SimpleNetworkDriverStart,\r
+  SimpleNetworkDriverStop,\r
+  0xa,\r
+  NULL,\r
+  NULL\r
+};\r
+\r
+EFI_STATUS\r
+add_v2p (\r
+  IN OUT struct s_v2p           **v2p,\r
+  EFI_PCI_IO_PROTOCOL_OPERATION type,\r
+  VOID                          *vaddr,\r
+  UINTN                         bsize\r
+  )\r
+/*++\r
+\r
+Routine Description:\r
+ This routine maps the given CPU address to a Device address. It creates a\r
+ an entry in the map list with the virtual and physical addresses and the \r
+ un map cookie.\r
+\r
+Arguments:\r
+ v2p - pointer to return a map list node pointer.\r
+ type - the direction in which the data flows from the given virtual address\r
+        device->cpu or cpu->device or both ways.\r
+ vaddr - virtual address (or CPU address) to be mapped\r
+ bsize - size of the buffer to be mapped.\r
+\r
+Returns:\r
+\r
+  EFI_SUCEESS - routine has completed the mapping\r
+  other - error as indicated.\r
+\r
+--*/\r
+{\r
+  EFI_STATUS  Status;\r
+\r
+  if ((v2p == NULL) || (vaddr == NULL) || (bsize == 0)) {\r
+    return EFI_INVALID_PARAMETER;\r
+  }\r
+\r
+  Status = gBS->AllocatePool (\r
+                  EfiBootServicesData,\r
+                  sizeof (struct s_v2p),\r
+                  (VOID **) v2p\r
+                  );\r
+\r
+  if (Status != EFI_SUCCESS) {\r
+    return Status;\r
+  }\r
+\r
+  Status = mPciIoFncs->Map (\r
+                        mPciIoFncs,\r
+                        type,\r
+                        vaddr,\r
+                        &bsize,\r
+                        &(*v2p)->paddr,\r
+                        &(*v2p)->unmap\r
+                        );\r
+  if (Status != EFI_SUCCESS) {\r
+    gBS->FreePool (*v2p);\r
+    return Status;\r
+  }\r
+  (*v2p)->vaddr = vaddr;\r
+  (*v2p)->bsize = bsize;\r
+  (*v2p)->next  = _v2p;\r
+  _v2p          = *v2p;\r
+\r
+  return EFI_SUCCESS;\r
+}\r
+\r
+EFI_STATUS\r
+find_v2p (\r
+  struct s_v2p **v2p,\r
+  VOID         *vaddr\r
+  )\r
+/*++\r
+\r
+Routine Description:\r
+ This routine searches the linked list of mapped address nodes (for undi3.0 \r
+ interface) to find the node that corresponds to the given virtual address and\r
+ returns a pointer to that node.\r
+\r
+Arguments:\r
+ v2p - pointer to return a map list node pointer.\r
+ vaddr - virtual address (or CPU address) to be searched in the map list\r
+\r
+Returns:\r
+\r
+  EFI_SUCEESS - if a match found!\r
+  Other       - match not found\r
+\r
+--*/\r
+{\r
+  struct s_v2p  *v;\r
+\r
+  if (v2p == NULL || vaddr == NULL) {\r
+    return EFI_INVALID_PARAMETER;\r
+  }\r
+\r
+  for (v = _v2p; v != NULL; v = v->next) {\r
+    if (v->vaddr == vaddr) {\r
+      *v2p = v;\r
+      return EFI_SUCCESS;\r
+    }\r
+  }\r
+\r
+  return EFI_NOT_FOUND;\r
+}\r
+\r
+EFI_STATUS\r
+del_v2p (\r
+  VOID *vaddr\r
+  )\r
+/*++\r
+\r
+Routine Description:\r
+ This routine unmaps the given virtual address and frees the memory allocated \r
+ for the map list node corresponding to that address.\r
\r
+Arguments:\r
+ vaddr - virtual address (or CPU address) to be unmapped\r
+\r
+Returns:\r
+ EFI_SUCEESS -  if successfully unmapped\r
+ Other - as indicated by the error\r
+\r
+\r
+--*/\r
+{\r
+  struct s_v2p  *v;\r
+  struct s_v2p  *t;\r
+  EFI_STATUS    Status;\r
+\r
+  if (vaddr == NULL) {\r
+    return EFI_INVALID_PARAMETER;\r
+  }\r
+\r
+  if (_v2p == NULL) {\r
+    return EFI_NOT_FOUND;\r
+  }\r
+  //\r
+  // Is our node at the head of the list??\r
+  //\r
+  if ((v = _v2p)->vaddr == vaddr) {\r
+    _v2p    = _v2p->next;\r
+\r
+    Status  = mPciIoFncs->Unmap (mPciIoFncs, v->unmap);\r
+\r
+    gBS->FreePool (v);\r
+\r
+#if SNP_DEBUG\r
+    if (Status) {\r
+      Print (L"Unmap failed with status = %x\n", Status);\r
+    }\r
+#endif\r
+    return Status;\r
+  }\r
+\r
+  for (; v->next != NULL; v = t) {\r
+    if ((t = v->next)->vaddr == vaddr) {\r
+      v->next = t->next;\r
+      Status  = mPciIoFncs->Unmap (mPciIoFncs, t->unmap);\r
+      gBS->FreePool (t);\r
+#if SNP_DEBUG\r
+      if (Status) {\r
+        Print (L"Unmap failed with status = %x\n", Status);\r
+      }\r
+#endif\r
+      return Status;\r
+    }\r
+  }\r
+\r
+  return EFI_NOT_FOUND;\r
+}\r
+\r
+#if SNP_DEBUG\r
+VOID\r
+snp_wait_for_key (\r
+  VOID\r
+  )\r
+/*++\r
+\r
+Routine Description:\r
+ Wait for a key stroke, used for debugging purposes\r
+\r
+Arguments:\r
+ none\r
+\r
+Returns:\r
+ none\r
+\r
+--*/\r
+{\r
+  EFI_INPUT_KEY key;\r
+\r
+  Aprint ("\nPress any key to continue\n");\r
+\r
+  while (gST->ConIn->ReadKeyStroke (gST->ConIn, &key) == EFI_NOT_READY) {\r
+    ;\r
+  }\r
+}\r
+#endif\r
index 404c75d..3c722c9 100644 (file)
@@ -21,6 +21,7 @@ Revision history:
 \r
 #include "Snp.h"\r
 \r
+STATIC\r
 EFI_STATUS\r
 pxe_fillheader (\r
   SNP_DRIVER      *snp,\r
@@ -191,6 +192,7 @@ Returns:
   }\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 pxe_transmit (\r
   SNP_DRIVER *snp,\r
index 641cb70..2fd9cab 100644 (file)
@@ -525,12 +525,17 @@ DxePcdGetNextToken (
   )\r
 {\r
   EFI_STATUS          Status;\r
+  BOOLEAN             PeiExMapTableEmpty;\r
+  BOOLEAN             DxeExMapTableEmpty;\r
 \r
   if (!FeaturePcdGet (PcdDxePcdDatabaseTraverseEnabled)) {\r
     return EFI_UNSUPPORTED;\r
   }\r
 \r
   Status = EFI_NOT_FOUND;\r
+  PeiExMapTableEmpty = PEI_EXMAP_TABLE_EMPTY;\r
+  DxeExMapTableEmpty = DXE_EXMAP_TABLE_EMPTY;\r
+\r
   //\r
   // Scan the local token space\r
   //\r
@@ -557,12 +562,12 @@ DxePcdGetNextToken (
     return EFI_SUCCESS;\r
   }\r
 \r
-  if (PEI_EXMAP_TABLE_EMPTY && DXE_EXMAP_TABLE_EMPTY) {\r
+  if (PeiExMapTableEmpty && DxeExMapTableEmpty) {\r
     *TokenNumber = PCD_INVALID_TOKEN_NUMBER;\r
     return EFI_NOT_FOUND;\r
   }\r
 \r
-  if (!PEI_EXMAP_TABLE_EMPTY) {\r
+  if (!PeiExMapTableEmpty) {\r
     Status = ExGetNextTokeNumber (\r
                         Guid,\r
                         TokenNumber,\r
@@ -577,7 +582,7 @@ DxePcdGetNextToken (
     return Status;\r
   }\r
 \r
-  if (!DXE_EXMAP_TABLE_EMPTY) {\r
+  if (!DxeExMapTableEmpty) {\r
     Status = ExGetNextTokeNumber (\r
                         Guid,\r
                         TokenNumber,\r
@@ -591,7 +596,7 @@ DxePcdGetNextToken (
   return Status;\r
 }\r
 \r
-\r
+STATIC\r
 EFI_GUID **\r
 GetDistinctTokenSpace (\r
   IN OUT    UINTN             *ExMapTableSize,\r
@@ -649,6 +654,8 @@ DxePcdGetNextTokenSpace (
   EFI_GUID            **PeiTokenSpaceTable;\r
   EFI_GUID            **DxeTokenSpaceTable;\r
   BOOLEAN             Match;\r
+  BOOLEAN             PeiExMapTableEmpty;\r
+  BOOLEAN             DxeExMapTableEmpty;\r
 \r
   if (!FeaturePcdGet (PcdDxePcdDatabaseTraverseEnabled)) {\r
     return EFI_UNSUPPORTED;\r
@@ -656,7 +663,10 @@ DxePcdGetNextTokenSpace (
 \r
   ASSERT (Guid != NULL);\r
   \r
-  if (PEI_EXMAP_TABLE_EMPTY && DXE_EXMAP_TABLE_EMPTY) {\r
+  PeiExMapTableEmpty = PEI_EXMAP_TABLE_EMPTY;\r
+  DxeExMapTableEmpty = DXE_EXMAP_TABLE_EMPTY;\r
+\r
+  if (PeiExMapTableEmpty && DxeExMapTableEmpty) {\r
     if (*Guid != NULL) {\r
       return EFI_NOT_FOUND;\r
     } else {\r
@@ -668,7 +678,7 @@ DxePcdGetNextTokenSpace (
   if (TmpTokenSpaceBuffer[0] == NULL) {\r
     PeiTokenSpaceTableSize = 0;\r
 \r
-    if (!PEI_EXMAP_TABLE_EMPTY) {\r
+    if (!PeiExMapTableEmpty) {\r
       PeiTokenSpaceTableSize = PEI_EXMAPPING_TABLE_SIZE;\r
       PeiTokenSpaceTable = GetDistinctTokenSpace (&PeiTokenSpaceTableSize,\r
                             mPcdDatabase->PeiDb.Init.ExMapTable,\r
@@ -677,7 +687,7 @@ DxePcdGetNextTokenSpace (
       CopyMem (TmpTokenSpaceBuffer, PeiTokenSpaceTable, sizeof (EFI_GUID*) * PeiTokenSpaceTableSize);\r
     }\r
 \r
-    if (!DXE_EXMAP_TABLE_EMPTY) {\r
+    if (!DxeExMapTableEmpty) {\r
       DxeTokenSpaceTableSize = DXE_EXMAPPING_TABLE_SIZE;\r
       DxeTokenSpaceTable = GetDistinctTokenSpace (&DxeTokenSpaceTableSize,\r
                             mPcdDatabase->DxeDb.Init.ExMapTable,\r
index 51e1662..8ff45d3 100644 (file)
@@ -28,7 +28,6 @@ GetWorker (
   )\r
 {\r
   UINT32              *LocalTokenNumberTable;\r
-  UINT16              *SizeTable;\r
   EFI_GUID            *GuidTable;\r
   UINT16              *StringTable;\r
   EFI_GUID            *Guid;\r
@@ -76,14 +75,11 @@ GetWorker (
   // EBC compiler is very choosy. It may report warning about comparison\r
   // between UINTN and 0 . So we add 1 in each size of the \r
   // comparison.\r
-  IsPeiDb = (TokenNumber + 1 < PEI_LOCAL_TOKEN_NUMBER + 1) ? TRUE : FALSE;\r
+  IsPeiDb = (BOOLEAN) ((TokenNumber + 1 < PEI_LOCAL_TOKEN_NUMBER + 1) ? TRUE : FALSE);\r
 \r
   LocalTokenNumberTable  = IsPeiDb ? mPcdDatabase->PeiDb.Init.LocalTokenNumberTable : \r
                                      mPcdDatabase->DxeDb.Init.LocalTokenNumberTable;\r
 \r
-  SizeTable              = IsPeiDb ? mPcdDatabase->PeiDb.Init.SizeTable: \r
-                                     mPcdDatabase->DxeDb.Init.SizeTable;\r
-\r
   TokenNumber            = IsPeiDb ? TokenNumber :\r
                                      TokenNumber - PEI_LOCAL_TOKEN_NUMBER;\r
 \r
@@ -496,7 +492,7 @@ GetSkuEnabledTokenNumber (
 \r
 \r
 \r
-\r
+STATIC\r
 VOID\r
 InvokeCallbackOnSet (\r
   UINT32            ExTokenNumber,\r
@@ -597,7 +593,7 @@ SetWorker (
   // EBC compiler is very choosy. It may report warning about comparison\r
   // between UINTN and 0 . So we add 1 in each size of the \r
   // comparison.\r
-  IsPeiDb = (TokenNumber + 1 < PEI_LOCAL_TOKEN_NUMBER + 1) ? TRUE : FALSE;\r
+  IsPeiDb = (BOOLEAN) ((TokenNumber + 1 < PEI_LOCAL_TOKEN_NUMBER + 1) ? TRUE : FALSE);\r
 \r
   LocalTokenNumberTable  = IsPeiDb ? mPcdDatabase->PeiDb.Init.LocalTokenNumberTable : \r
                                      mPcdDatabase->DxeDb.Init.LocalTokenNumberTable;\r
@@ -895,7 +891,7 @@ GetExPcdTokenNumber (
 }\r
 \r
 \r
-\r
+STATIC\r
 SKU_ID *\r
 GetSkuIdArray (\r
   IN    UINTN             LocalTokenNumberTableIdx,\r
@@ -923,7 +919,7 @@ GetSkuIdArray (
 }\r
 \r
 \r
-\r
+STATIC\r
 UINTN\r
 GetSizeTableIndexA (\r
   IN UINTN        LocalTokenNumberTableIdx,\r
@@ -980,7 +976,7 @@ GetSizeTableIndexA (
 \r
 \r
 \r
-\r
+STATIC\r
 UINTN\r
 GetSizeTableIndex (\r
   IN    UINTN             LocalTokenNumberTableIdx,\r
index 33bc4e1..b8ed5b2 100644 (file)
@@ -502,11 +502,14 @@ PeiPcdGetNextToken (
   DYNAMICEX_MAPPING   *ExMapTable;\r
   UINTN               i;\r
   BOOLEAN             Found;\r
-    \r
+  BOOLEAN             PeiExMapTableEmpty;\r
+\r
   if (!FeaturePcdGet (PcdPeiPcdDatabaseTraverseEnabled)) {\r
     return EFI_UNSUPPORTED;\r
   }\r
-    \r
+\r
+  PeiExMapTableEmpty = PEI_EXMAP_TABLE_EMPTY;\r
+\r
   if (Guid == NULL) {\r
     if (*TokenNumber > PEI_NEX_TOKEN_NUMBER) {\r
       return EFI_NOT_FOUND;\r
@@ -517,7 +520,7 @@ PeiPcdGetNextToken (
     }\r
     return EFI_SUCCESS;\r
   } else {\r
-    if (PEI_EXMAP_TABLE_EMPTY) {\r
+    if (PeiExMapTableEmpty) {\r
       *TokenNumber = PCD_INVALID_TOKEN_NUMBER;\r
       return EFI_SUCCESS;\r
     }\r
@@ -597,6 +600,7 @@ PeiPcdGetNextTokenSpace (
   DYNAMICEX_MAPPING   *ExMapTable;\r
   UINTN               i;\r
   BOOLEAN             Found;\r
+  BOOLEAN             PeiExMapTableEmpty;\r
 \r
   if (!FeaturePcdGet (PcdPeiPcdDatabaseTraverseEnabled)) {\r
     return EFI_UNSUPPORTED;\r
@@ -604,7 +608,9 @@ PeiPcdGetNextTokenSpace (
 \r
   ASSERT (Guid != NULL);\r
 \r
-  if (PEI_EXMAP_TABLE_EMPTY) {\r
+  PeiExMapTableEmpty = PEI_EXMAP_TABLE_EMPTY;\r
+\r
+  if (PeiExMapTableEmpty) {\r
     if (*Guid != NULL) {\r
       return EFI_NOT_FOUND;\r
     } else {\r
index 521c88b..c228d3b 100644 (file)
@@ -154,6 +154,7 @@ BuildPcdDatabase (
   @retval EFI_SUCCESS         Operation successful.\r
   @retval EFI_NOT_FOUND         Variablel not found.\r
 --*/\r
+STATIC\r
 EFI_STATUS\r
 GetHiiVariable (\r
   IN  CONST EFI_GUID      *VariableGuid,\r
@@ -205,7 +206,7 @@ GetHiiVariable (
 \r
 }\r
 \r
-\r
+STATIC\r
 UINT32\r
 GetSkuEnabledTokenNumber (\r
   UINT32 LocalTokenNumber,\r
@@ -261,7 +262,7 @@ GetSkuEnabledTokenNumber (
 \r
 \r
 \r
-\r
+STATIC\r
 VOID\r
 InvokeCallbackOnSet (\r
   UINTN             ExTokenNumber,\r
@@ -651,7 +652,7 @@ GetPcdDatabase (
 }\r
 \r
 \r
-\r
+STATIC\r
 SKU_ID *\r
 GetSkuIdArray (\r
   IN    UINTN             LocalTokenNumberTableIdx,\r
@@ -672,7 +673,7 @@ GetSkuIdArray (
 }\r
 \r
 \r
-\r
+STATIC\r
 UINTN\r
 GetSizeTableIndex (\r
   IN    UINTN             LocalTokenNumberTableIdx,\r
index 3692d97..dc22fa0 100644 (file)
@@ -74,6 +74,7 @@ Returns:
   return EFI_SUCCESS;\r
 }\r
 \r
+STATIC\r
 UINT32\r
 ReverseBits (\r
   UINT32  Value\r
index 0b6965c..f24d2a5 100644 (file)
@@ -91,6 +91,7 @@ EFI_RUNTIME_ARCH_PROTOCOL     mRuntime = {
 //\r
 // Worker Functions\r
 //\r
+STATIC\r
 VOID\r
 RuntimeDriverCalculateEfiHdrCrc (\r
   IN OUT EFI_TABLE_HEADER  *Hdr\r
@@ -224,6 +225,7 @@ Returns:
   return EFI_NOT_FOUND;\r
 }\r
 \r
+STATIC\r
 EFI_STATUS\r
 RuntimeDriverConvertInternalPointer (\r
   IN OUT VOID   **ConvertAddress\r
index efd4f52..85e552b 100644 (file)
@@ -40,6 +40,7 @@ EFI_DATA_HUB_PROTOCOL     *mDataHubProtocol;
   @retval  !NULL  Point to buffer of record.\r
 \r
 **/\r
+STATIC\r
 DATAHUB_STATUSCODE_RECORD *\r
 AcquireRecordBuffer (\r
   VOID\r
@@ -92,6 +93,7 @@ AcquireRecordBuffer (
   @return   NULL    the FIFO of record is empty.\r
  \r
 **/\r
+STATIC\r
 DATAHUB_STATUSCODE_RECORD *\r
 RetrieveRecord (\r
   VOID\r
@@ -229,6 +231,7 @@ DataHubStatusCodeReportWorker (
   @param  Context     Context of the event.\r
 \r
 **/\r
+STATIC\r
 VOID\r
 EFIAPI\r
 LogDataHubEventCallBack (\r
index 506a2a1..eee0c94 100644 (file)
@@ -182,6 +182,7 @@ ReportDispatcher (
   @return Value returned in SAL_RETURN_REGS\r
 \r
 --*/\r
+STATIC\r
 SAL_RETURN_REGS\r
 EFIAPI\r
 ReportEsalServiceEntry (\r
index 9b70387..c49750b 100644 (file)
@@ -14,6 +14,8 @@
 \r
 **/\r
 \r
+#include "PeiStatusCode.h"\r
+\r
 /**\r
   Create one memory status code GUID'ed HOB, use PacketIndex \r
   to identify the packet.\r
@@ -23,6 +25,7 @@
   @return                 Always return pointer of memory status code packet.\r
 \r
 **/\r
+STATIC\r
 MEMORY_STATUSCODE_PACKET_HEADER *\r
 CreateMemoryStatusCodePacket (\r
   UINT16 PacketIndex\r
index 3cd072c..0e3f47e 100644 (file)
@@ -15,6 +15,8 @@
 \r
 **/\r
 \r
+#include "PeiStatusCode.h"\r
+\r
 /**\r
   Convert status code value and extended data to readable ASCII string, send string to serial I/O device.\r
  \r
index dd31611..f78d422 100644 (file)
@@ -30,6 +30,7 @@ Abstract:
 EFI_GUID  mFormSetGuid    = FORMSET_GUID;\r
 EFI_GUID  mStringPackGuid = STRING_PACK_GUID; \r
 \r
+STATIC\r
 EFI_STATUS\r
 EFIAPI\r
 DriverCallback (\r
index d7daa64..ebab3bc 100644 (file)
@@ -210,14 +210,17 @@ Returns:
 \r
     if (Narrow) {\r
       for (Value = 0; Value != Count; Value++) {\r
-        *GlyphBuffer[Location + Value] |= GlobalData->NarrowGlyphs[Character].GlyphCol1[Value];\r
+        *GlyphBuffer[Location + Value] = (UINT8) (*GlyphBuffer[Location + Value] |\r
+                                                  GlobalData->NarrowGlyphs[Character].GlyphCol1[Value]);\r
       }\r
 \r
       Attributes = GlobalData->NarrowGlyphs[Character].Attributes & EFI_GLYPH_NON_SPACING;\r
     } else {\r
       for (Value = 0; Value != Count; Value++) {\r
-        *GlyphBuffer[Location + Value] |= GlobalData->WideGlyphs[Character].GlyphCol1[Value];\r
-        *GlyphBuffer[Location + Value + Count] |= GlobalData->WideGlyphs[Character].GlyphCol2[Value];\r
+        *GlyphBuffer[Location + Value] = (UINT8) (*GlyphBuffer[Location + Value] | \r
+                                                  GlobalData->WideGlyphs[Character].GlyphCol1[Value]);\r
+        *GlyphBuffer[Location + Value + Count] = (UINT8) (*GlyphBuffer[Location + Value + Count] |\r
+                                                          GlobalData->WideGlyphs[Character].GlyphCol2[Value]);\r
       }\r
 \r
       Attributes = GlobalData->WideGlyphs[Character].Attributes & EFI_GLYPH_NON_SPACING;\r
index 4178284..0c8629c 100644 (file)
@@ -22,6 +22,7 @@ Abstract:
 \r
 #include "HiiDatabase.h"\r
 \r
+STATIC\r
 CHAR16*\r
 Ascii2Unicode (\r
   OUT CHAR16         *UnicodeStr,\r
@@ -54,6 +55,7 @@ Ascii2Unicode (
   }\r
 }\r
 \r
+STATIC\r
 CHAR8*\r
 Unicode2Ascii (\r
   OUT CHAR8          *AsciiStr,\r
@@ -86,6 +88,7 @@ Unicode2Ascii (
   }\r
 }\r
 \r
+STATIC\r
 VOID\r
 ExtractDevicePathData (\r
   IN     EFI_HII_DATA_TABLE   *DataTable,\r
@@ -129,6 +132,7 @@ Returns:
   *ExportBufferPtr  = ExportBuffer;\r
 }\r
 \r
+STATIC\r
 VOID\r
 ExtractVariableData (\r
   IN OUT EFI_HII_DATA_TABLE   *DataTable,\r
@@ -205,6 +209,7 @@ Returns:
                         &gEfiFormCallbackProtocolGuid,\r
                         (VOID *) &FormCallback\r
                         );\r
+        ASSERT_EFI_ERROR (Status);\r
       }\r
       //\r
       // Since we have a "Setup" variable that wasn't specified by a variable op-code\r
@@ -240,6 +245,7 @@ Returns:
                                  &TempValue,\r
                                  ExportBuffer\r
                                  );\r
+        ASSERT_EFI_ERROR (Status);\r
       } else {\r
         Status = gRT->GetVariable (\r
                         (CHAR16 *) SETUP_MAP_NAME,\r
@@ -248,6 +254,7 @@ Returns:
                         &TempValue,\r
                         ExportBuffer\r
                         );\r
+        ASSERT_EFI_ERROR (Status);\r
       }\r
 \r
       ExportBuffer = (UINT8 *) (UINTN) (((UINTN) ExportBuffer) + TempValue);\r
@@ -306,6 +313,7 @@ Returns:
                                  &TempValue,\r
                                  ExportBuffer\r
                                  );\r
+        ASSERT_EFI_ERROR (Status);\r
       } else {\r
         Status = gRT->GetVariable (\r
                         String,\r
@@ -314,6 +322,7 @@ Returns:
                         &TempValue,\r
                         ExportBuffer\r
                         );\r
+        ASSERT_EFI_ERROR (Status);\r
       }\r
 \r
       ExportBuffer = (UINT8 *) (UINTN) (((UINTN) ExportBuffer) + TempValue);\r
@@ -367,7 +376,6 @@ Returns:
   UINT8                     *ExportBuffer;\r
   EFI_HII_EXPORT_TABLE      *ExportTable;\r
   EFI_HII_DATA_TABLE        *DataTable;\r
-  BOOLEAN                   InsufficientSize;\r
   BOOLEAN                   VariableExist;\r
   UINT16                    NumberOfHiiDataTables;\r
   UINTN                     SizeNeeded;\r
@@ -386,7 +394,6 @@ Returns:
   FormPack              = NULL;\r
   RawData               = NULL;\r
   PackageInstance       = NULL;\r
-  InsufficientSize      = FALSE;\r
   NumberOfHiiDataTables = 0;\r
   VariableSize          = 0;\r
   TempValue             = 0;\r
index ce34eff..bcec42b 100644 (file)
@@ -154,7 +154,6 @@ Returns:
 \r
 --*/\r
 {\r
-  EFI_HII_GLOBAL_DATA     *GlobalData;\r
   EFI_HII_HANDLE_DATABASE *Database;\r
   EFI_HII_DATA            *HiiData;\r
   UINTN                   HandleCount;\r
@@ -165,8 +164,6 @@ Returns:
 \r
   HiiData     = EFI_HII_DATA_FROM_THIS (This);\r
 \r
-  GlobalData  = HiiData->GlobalData;\r
-\r
   Database    = HiiData->DatabaseHead;\r
 \r
   if (Database == NULL) {\r
@@ -333,7 +330,6 @@ Returns:
   EFI_HII_DATA              *HiiData;\r
   EFI_HII_HANDLE_DATABASE   *HandleDatabase;\r
   EFI_HII_STRING_PACK       *