Fix RVCT compiler warnings, and a bug in the GdbStub
authorAJFISH <AJFISH@6f19259b-4bc3-4df7-8a09-765794883524>
Sun, 17 Jan 2010 04:44:33 +0000 (04:44 +0000)
committerAJFISH <AJFISH@6f19259b-4bc3-4df7-8a09-765794883524>
Sun, 17 Jan 2010 04:44:33 +0000 (04:44 +0000)
git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@9790 6f19259b-4bc3-4df7-8a09-765794883524

ArmPkg/Drivers/CpuDxe/Mmu.c
BeagleBoardPkg/PciEmulation/PciEmulation.c
BeagleBoardPkg/Sec/Cache.c
BeagleBoardPkg/Sec/Sec.c
BeagleBoardPkg/Sec/Sec.inf
EmbeddedPkg/GdbStub/GdbStub.c

index 6d51259..f2669fc 100644 (file)
@@ -438,6 +438,8 @@ UpdatePageEntries (
   volatile ARM_FIRST_LEVEL_DESCRIPTOR   *FirstLevelTable;\r
   volatile ARM_PAGE_TABLE_ENTRY         *PageTable;\r
 \r
+  Status = EFI_SUCCESS;\r
+\r
   // EntryMask: bitmask of values to change (1 = change this value, 0 = leave alone)\r
   // EntryValue: values at bit positions specified by EntryMask\r
   EntryMask = ARM_PAGE_DESC_TYPE_MASK;\r
index f558597..fa44570 100644 (file)
@@ -436,7 +436,6 @@ PciIoAttributes (
   case EfiPciIoAttributeOperationDisable:
     // Since we are not a real PCI device no enable/set or disable operations exist.
     return EFI_SUCCESS;
-    break;
     
   default:
   ASSERT (FALSE);
index d7c9062..12b585e 100755 (executable)
@@ -38,14 +38,11 @@ InitCache (
   IN  UINT32  MemoryLength\r
   )\r
 {\r
-  UINTN                         UncachedMemoryMask;\r
   UINT32                        CacheAttributes;\r
   ARM_MEMORY_REGION_DESCRIPTOR  MemoryTable[5];\r
   VOID                          *TranslationTableBase;\r
   UINTN                         TranslationTableSize;\r
 \r
-  UncachedMemoryMask = PcdGet64(PcdArmUncachedMemoryMask);\r
-\r
   if (FeaturePcdGet(PcdCacheEnable) == TRUE) {\r
     CacheAttributes = DDR_ATTRIBUTES_CACHED;\r
   } else {\r
index 5b4d294..5374752 100755 (executable)
@@ -239,7 +239,7 @@ CEntryPoint (
         //
 #ifdef __CC_ARM
         // Print out the command for the RVD debugger to load symbols for this image
-        DEBUG ((EFI_D_ERROR, "load /a /ni /np %a &0x%08x\n", SecDeCygwinPathIfNeeded (FilePath), PeCoffImage + Offset));
+        DEBUG ((EFI_D_ERROR, "load /a /ni /np %a &0x%08x\n", SecDeCygwinPathIfNeeded (FilePath), (CHAR8 *)PeCoffImage + Offset));
 #elif __GNUC__\r
         // This may not work correctly if you generate PE/COFF directlyas then the Offset would not be required
         DEBUG ((EFI_D_ERROR, "add-symbol-file %a 0x%08x\n", FilePath, PeCoffImage + Offset));
index 79a5eb4..0f611e3 100755 (executable)
@@ -52,8 +52,6 @@
   gEmbeddedTokenSpaceGuid.PcdCacheEnable
 
 [FixedPcd]
-  gArmTokenSpaceGuid.PcdArmUncachedMemoryMask
-
   gEmbeddedTokenSpaceGuid.PcdEmbeddedFdBaseAddress
   gEmbeddedTokenSpaceGuid.PcdEmbeddedFdSize
   gEmbeddedTokenSpaceGuid.PcdFlashFvMainBase
index b121e41..cc0e118 100644 (file)
@@ -976,7 +976,7 @@ PeCoffLoaderGetDebuggerInfo (
         case CODEVIEW_SIGNATURE_RSDS:
           return (VOID *) ((CHAR8 *)CodeViewEntryPointer + sizeof (EFI_IMAGE_DEBUG_CODEVIEW_RSDS_ENTRY));
         case CODEVIEW_SIGNATURE_MTOC:
-          *DebugBase = (VOID *)(UINTN)((UINTN)DebugBase + SizeOfHeaders);
+          *DebugBase = (VOID *)(UINTN)((UINTN)DebugBase - SizeOfHeaders);
           return (VOID *) ((CHAR8 *)CodeViewEntryPointer + sizeof (EFI_IMAGE_DEBUG_CODEVIEW_MTOC_ENTRY));
         default:
           break;