]> git.proxmox.com Git - mirror_edk2.git/commitdiff
StandaloneMmPkg/StandaloneMmCore: Fix compiler warning
authorMichael Kubacki <michael.kubacki@microsoft.com>
Thu, 11 Feb 2021 01:04:20 +0000 (17:04 -0800)
committermergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Thu, 11 Feb 2021 08:52:04 +0000 (08:52 +0000)
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3204

Fixes the following compiler warning in VS2019 by changing defining
the MmramRangeCount variable to be UINTN and type casting prior
to value assignment.

\edk2\StandaloneMmPkg\Core\StandaloneMmCore.c(570): error C2220:
  the following warning is treated as an error
\edk2\StandaloneMmPkg\Core\StandaloneMmCore.c(570): warning C4244:
  '=': conversion from 'UINT64' to 'UINT32', possible loss of data

Signed-off-by: Michael Kubacki <michael.kubacki@microsoft.com>
Reviewed-by: Sami Mujawar <sami.mujawar@arm.com>
StandaloneMmPkg/Core/StandaloneMmCore.c

index 8388ec289ca8d26cff02960aeeb579f418fba4fe..fbb0ec75e557622db8abe6493242b71b31adbdb5 100644 (file)
@@ -511,7 +511,7 @@ StandaloneMmMain (
   EFI_HOB_GUID_TYPE               *MmramRangesHob;\r
   EFI_MMRAM_HOB_DESCRIPTOR_BLOCK  *MmramRangesHobData;\r
   EFI_MMRAM_DESCRIPTOR            *MmramRanges;\r
-  UINT32                          MmramRangeCount;\r
+  UINT                          MmramRangeCount;\r
   EFI_HOB_FIRMWARE_VOLUME         *BfvHob;\r
 \r
   ProcessLibraryConstructorList (HobStart, &gMmCoreMmst);\r
@@ -546,7 +546,7 @@ StandaloneMmMain (
     MmramRangesHobData = GET_GUID_HOB_DATA (MmramRangesHob);\r
     ASSERT (MmramRangesHobData != NULL);\r
     MmramRanges = MmramRangesHobData->Descriptor;\r
-    MmramRangeCount = MmramRangesHobData->NumberOfMmReservedRegions;\r
+    MmramRangeCount = (UINTN)MmramRangesHobData->NumberOfMmReservedRegions;\r
     ASSERT (MmramRanges);\r
     ASSERT (MmramRangeCount);\r
 \r
@@ -554,7 +554,7 @@ StandaloneMmMain (
     // Copy the MMRAM ranges into MM_CORE_PRIVATE_DATA table just in case any\r
     // code relies on them being present there\r
     //\r
-    gMmCorePrivate->MmramRangeCount = MmramRangeCount;\r
+    gMmCorePrivate->MmramRangeCount = (UINT64)MmramRangeCount;\r
     gMmCorePrivate->MmramRanges =\r
       (EFI_PHYSICAL_ADDRESS)(UINTN)AllocatePool (MmramRangeCount * sizeof (EFI_MMRAM_DESCRIPTOR));\r
     ASSERT (gMmCorePrivate->MmramRanges != 0);\r
@@ -567,7 +567,7 @@ StandaloneMmMain (
     DataInHob       = GET_GUID_HOB_DATA (GuidHob);\r
     gMmCorePrivate = (MM_CORE_PRIVATE_DATA *)(UINTN)DataInHob->Address;\r
     MmramRanges     = (EFI_MMRAM_DESCRIPTOR *)(UINTN)gMmCorePrivate->MmramRanges;\r
-    MmramRangeCount = gMmCorePrivate->MmramRangeCount;\r
+    MmramRangeCount = (UINTN)gMmCorePrivate->MmramRangeCount;\r
   }\r
 \r
   //\r