public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [Patch][edk2-platforms/devel-MinnowBoard3-UDK2017] Fix some GCC build error.
@ 2017-05-12  5:47 zwei4
  2017-05-12  8:34 ` Lu, ShifeiX A
  0 siblings, 1 reply; 2+ messages in thread
From: zwei4 @ 2017-05-12  5:47 UTC (permalink / raw)
  To: edk2-devel

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: zwei4 <david.wei@intel.com>
---
 Platform/BroxtonPlatformPkg/BuildBios.sh                              | 2 +-
 .../DxeFspWrapperPlatformResetLib/DxeFspWrapperPlatformResetLib.c     | 2 +-
 .../Common/PlatformSettings/PlatformPreMemPei/PlatformInitPreMem.c    | 4 ++--
 Platform/BroxtonPlatformPkg/PlatformPkgX64Gcc.dsc                     | 2 +-
 .../SampleCode/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c  | 2 +-
 5 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/Platform/BroxtonPlatformPkg/BuildBios.sh b/Platform/BroxtonPlatformPkg/BuildBios.sh
index 8702fbeaa..3963c887a 100644
--- a/Platform/BroxtonPlatformPkg/BuildBios.sh
+++ b/Platform/BroxtonPlatformPkg/BuildBios.sh
@@ -211,7 +211,7 @@ fi
 ## Additional EDK Build Setup/Configuration
 ##**********************************************************************
 echo "Ensuring correct build directory is present for GenBiosId..."
-ACTIVE_PLATFORM=$PLATFORM_PACKAGE/PlatformPkgX64.dsc
+ACTIVE_PLATFORM=$PLATFORM_PACKAGE/PlatformPkgX64Gcc.dsc
 #TOOL_CHAIN_TAG=GCC47
 MAX_CONCURRENT_THREAD_NUMBER=1
 sed -i '/^ACTIVE_PLATFORM/d' $WORKSPACE/Conf/target.txt
diff --git a/Platform/BroxtonPlatformPkg/Common/FspSupport/Library/DxeFspWrapperPlatformResetLib/DxeFspWrapperPlatformResetLib.c b/Platform/BroxtonPlatformPkg/Common/FspSupport/Library/DxeFspWrapperPlatformResetLib/DxeFspWrapperPlatformResetLib.c
index 4474eafaf..b7b57c517 100644
--- a/Platform/BroxtonPlatformPkg/Common/FspSupport/Library/DxeFspWrapperPlatformResetLib/DxeFspWrapperPlatformResetLib.c
+++ b/Platform/BroxtonPlatformPkg/Common/FspSupport/Library/DxeFspWrapperPlatformResetLib/DxeFspWrapperPlatformResetLib.c
@@ -33,7 +33,7 @@ CallFspWrapperResetSystem (
   IN UINT32    ResetType
   )
 {
-  EFI_RESET_TYPE             EfiResetType;
+  EFI_RESET_TYPE             EfiResetType = EfiResetCold;
 
   switch (ResetType) {
     case FSP_STATUS_RESET_REQUIRED_COLD:
diff --git a/Platform/BroxtonPlatformPkg/Common/PlatformSettings/PlatformPreMemPei/PlatformInitPreMem.c b/Platform/BroxtonPlatformPkg/Common/PlatformSettings/PlatformPreMemPei/PlatformInitPreMem.c
index 630c2c1d1..d7ef4970b 100644
--- a/Platform/BroxtonPlatformPkg/Common/PlatformSettings/PlatformPreMemPei/PlatformInitPreMem.c
+++ b/Platform/BroxtonPlatformPkg/Common/PlatformSettings/PlatformPreMemPei/PlatformInitPreMem.c
@@ -893,7 +893,7 @@ PlatformInitPreMemEntryPoint (
   DRAM_POLICY_PPI                  *DramPolicy;
   EFI_PEI_PPI_DESCRIPTOR           *NewPeiPpiDescriptor;
   EFI_BOOT_MODE                    BootMode;
-  CarMapStruc                      *CarMap;
+  CarMapStruc                      *CarMap = NULL;
   SYSTEM_CONFIGURATION             SystemConfiguration;
   UINTN                            VariableSize;
   EFI_PEI_HOB_POINTERS             Hob;
@@ -938,7 +938,7 @@ PlatformInitPreMemEntryPoint (
                  &gBoardPreMemInitPpiGuid,
                  Instance,
                  &PeiPpiDescriptor,
-                 &BoardPreMemInitPpi
+                 (VOID **)&BoardPreMemInitPpi
                  );
 
       if (Status == EFI_NOT_FOUND) {
diff --git a/Platform/BroxtonPlatformPkg/PlatformPkgX64Gcc.dsc b/Platform/BroxtonPlatformPkg/PlatformPkgX64Gcc.dsc
index 1339b44b2..42f068a38 100644
--- a/Platform/BroxtonPlatformPkg/PlatformPkgX64Gcc.dsc
+++ b/Platform/BroxtonPlatformPkg/PlatformPkgX64Gcc.dsc
@@ -177,5 +177,5 @@
 #[BuildOptions.Common]
 
 [BuildOptions.Common.EDKII]
-   !include PlatformDsc/BuildOptions.EDKII.dsc
+   !include PlatformDsc/BuildOptions.EDKII.Gcc.dsc
 
diff --git a/Silicon/BroxtonSoC/BroxtonSiPkg/SampleCode/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c b/Silicon/BroxtonSoC/BroxtonSiPkg/SampleCode/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c
index 9fcb55197..316e5c745 100644
--- a/Silicon/BroxtonSoC/BroxtonSiPkg/SampleCode/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c
+++ b/Silicon/BroxtonSoC/BroxtonSiPkg/SampleCode/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c
@@ -1122,7 +1122,7 @@ Reclaim (
   VARIABLE_POINTER_TRACK        VarErrFlagPtr;
 
   Status = EFI_SUCCESS;
-
+  VariableSize = 0;
   UpdatingVariable            = NULL;
   UpdatingInDeletedTransition = NULL;
 
-- 
2.11.0.windows.1



^ permalink raw reply related	[flat|nested] 2+ messages in thread
* Re: [Patch][edk2-platforms/devel-MinnowBoard3-UDK2017] Fix some GCC build error.
  2017-05-12  5:47 [Patch][edk2-platforms/devel-MinnowBoard3-UDK2017] Fix some GCC build error zwei4
@ 2017-05-12  8:34 ` Lu, ShifeiX A
  0 siblings, 0 replies; 2+ messages in thread
From: Lu, ShifeiX A @ 2017-05-12  8:34 UTC (permalink / raw)
  To: Wei, David; +Cc: edk2-devel@lists.01.org, Lu, ShifeiX A

Reviewed-by: lushifex <shifeix.a.lu@intel.com>


Shifei

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of zwei4
Sent: Friday, May 12, 2017 1:47 PM
To: edk2-devel@lists.01.org
Subject: [edk2] [Patch][edk2-platforms/devel-MinnowBoard3-UDK2017] Fix some GCC build error.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: zwei4 <david.wei@intel.com>
---
 Platform/BroxtonPlatformPkg/BuildBios.sh                              | 2 +-
 .../DxeFspWrapperPlatformResetLib/DxeFspWrapperPlatformResetLib.c     | 2 +-
 .../Common/PlatformSettings/PlatformPreMemPei/PlatformInitPreMem.c    | 4 ++--
 Platform/BroxtonPlatformPkg/PlatformPkgX64Gcc.dsc                     | 2 +-
 .../SampleCode/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c  | 2 +-
 5 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/Platform/BroxtonPlatformPkg/BuildBios.sh b/Platform/BroxtonPlatformPkg/BuildBios.sh
index 8702fbeaa..3963c887a 100644
--- a/Platform/BroxtonPlatformPkg/BuildBios.sh
+++ b/Platform/BroxtonPlatformPkg/BuildBios.sh
@@ -211,7 +211,7 @@ fi
 ## Additional EDK Build Setup/Configuration
 ##**********************************************************************
 echo "Ensuring correct build directory is present for GenBiosId..."
-ACTIVE_PLATFORM=$PLATFORM_PACKAGE/PlatformPkgX64.dsc
+ACTIVE_PLATFORM=$PLATFORM_PACKAGE/PlatformPkgX64Gcc.dsc
 #TOOL_CHAIN_TAG=GCC47
 MAX_CONCURRENT_THREAD_NUMBER=1
 sed -i '/^ACTIVE_PLATFORM/d' $WORKSPACE/Conf/target.txt diff --git a/Platform/BroxtonPlatformPkg/Common/FspSupport/Library/DxeFspWrapperPlatformResetLib/DxeFspWrapperPlatformResetLib.c b/Platform/BroxtonPlatformPkg/Common/FspSupport/Library/DxeFspWrapperPlatformResetLib/DxeFspWrapperPlatformResetLib.c
index 4474eafaf..b7b57c517 100644
--- a/Platform/BroxtonPlatformPkg/Common/FspSupport/Library/DxeFspWrapperPlatformResetLib/DxeFspWrapperPlatformResetLib.c
+++ b/Platform/BroxtonPlatformPkg/Common/FspSupport/Library/DxeFspWrappe
+++ rPlatformResetLib/DxeFspWrapperPlatformResetLib.c
@@ -33,7 +33,7 @@ CallFspWrapperResetSystem (
   IN UINT32    ResetType
   )
 {
-  EFI_RESET_TYPE             EfiResetType;
+  EFI_RESET_TYPE             EfiResetType = EfiResetCold;
 
   switch (ResetType) {
     case FSP_STATUS_RESET_REQUIRED_COLD:
diff --git a/Platform/BroxtonPlatformPkg/Common/PlatformSettings/PlatformPreMemPei/PlatformInitPreMem.c b/Platform/BroxtonPlatformPkg/Common/PlatformSettings/PlatformPreMemPei/PlatformInitPreMem.c
index 630c2c1d1..d7ef4970b 100644
--- a/Platform/BroxtonPlatformPkg/Common/PlatformSettings/PlatformPreMemPei/PlatformInitPreMem.c
+++ b/Platform/BroxtonPlatformPkg/Common/PlatformSettings/PlatformPreMem
+++ Pei/PlatformInitPreMem.c
@@ -893,7 +893,7 @@ PlatformInitPreMemEntryPoint (
   DRAM_POLICY_PPI                  *DramPolicy;
   EFI_PEI_PPI_DESCRIPTOR           *NewPeiPpiDescriptor;
   EFI_BOOT_MODE                    BootMode;
-  CarMapStruc                      *CarMap;
+  CarMapStruc                      *CarMap = NULL;
   SYSTEM_CONFIGURATION             SystemConfiguration;
   UINTN                            VariableSize;
   EFI_PEI_HOB_POINTERS             Hob;
@@ -938,7 +938,7 @@ PlatformInitPreMemEntryPoint (
                  &gBoardPreMemInitPpiGuid,
                  Instance,
                  &PeiPpiDescriptor,
-                 &BoardPreMemInitPpi
+                 (VOID **)&BoardPreMemInitPpi
                  );
 
       if (Status == EFI_NOT_FOUND) {
diff --git a/Platform/BroxtonPlatformPkg/PlatformPkgX64Gcc.dsc b/Platform/BroxtonPlatformPkg/PlatformPkgX64Gcc.dsc
index 1339b44b2..42f068a38 100644
--- a/Platform/BroxtonPlatformPkg/PlatformPkgX64Gcc.dsc
+++ b/Platform/BroxtonPlatformPkg/PlatformPkgX64Gcc.dsc
@@ -177,5 +177,5 @@
 #[BuildOptions.Common]
 
 [BuildOptions.Common.EDKII]
-   !include PlatformDsc/BuildOptions.EDKII.dsc
+   !include PlatformDsc/BuildOptions.EDKII.Gcc.dsc
 
diff --git a/Silicon/BroxtonSoC/BroxtonSiPkg/SampleCode/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c b/Silicon/BroxtonSoC/BroxtonSiPkg/SampleCode/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c
index 9fcb55197..316e5c745 100644
--- a/Silicon/BroxtonSoC/BroxtonSiPkg/SampleCode/MdeModulePkg/Universal/Variable/RuntimeDxe/Variable.c
+++ b/Silicon/BroxtonSoC/BroxtonSiPkg/SampleCode/MdeModulePkg/Universal/
+++ Variable/RuntimeDxe/Variable.c
@@ -1122,7 +1122,7 @@ Reclaim (
   VARIABLE_POINTER_TRACK        VarErrFlagPtr;
 
   Status = EFI_SUCCESS;
-
+  VariableSize = 0;
   UpdatingVariable            = NULL;
   UpdatingInDeletedTransition = NULL;
 
--
2.11.0.windows.1

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply related	[flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-05-12  8:35 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-05-12  5:47 [Patch][edk2-platforms/devel-MinnowBoard3-UDK2017] Fix some GCC build error zwei4
2017-05-12  8:34 ` Lu, ShifeiX A

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox