[edk2-devel] [PATCH 1/1] WhitleyOpenBoardPkg/Build: Fix GCC build issues

[Oram, Isaac W]

Fix build warning in PlatformInfo.c.
Fix script invocation issues in build_board.py for AML
content build in WilsonCityRvp and JunctionCity.

Cc: Nate DeSimone <nathaniel.l.desim...@intel.com>
Cc: Chasel Chiu <chasel.c...@intel.com>
Signed-off-by: Isaac Oram <isaac.w.o...@intel.com>
---
 .../WhitleyOpenBoardPkg/JunctionCity/build_board.py   | 11 ++++++++---
 .../Platform/Pei/PlatformInfo/PlatformInfo.c          |  4 ++--
 .../WhitleyOpenBoardPkg/WilsonCityRvp/build_board.py  | 11 ++++++++---
 3 files changed, 18 insertions(+), 8 deletions(-)

diff --git a/Platform/Intel/WhitleyOpenBoardPkg/JunctionCity/build_board.py 
b/Platform/Intel/WhitleyOpenBoardPkg/JunctionCity/build_board.py
index 72d0c5089a..ccf7a8c55a 100644
--- a/Platform/Intel/WhitleyOpenBoardPkg/JunctionCity/build_board.py
+++ b/Platform/Intel/WhitleyOpenBoardPkg/JunctionCity/build_board.py
@@ -62,7 +62,8 @@ def pre_build_ex(config, functions):
 
     execute_script = functions.get("execute_script")
 
-    command = ["build", "-D", "MAX_SOCKET=" + config["MAX_SOCKET"]]
+    # AML offset arch is X64, not sure if it matters.
+    command = ["build", "-a", "X64", "-t", config["TOOL_CHAIN_TAG"], "-D", 
"MAX_SOCKET=" + config["MAX_SOCKET"]]
 
     if config["EXT_BUILD_FLAGS"] and config["EXT_BUILD_FLAGS"] != "":
         ext_build_flags = config["EXT_BUILD_FLAGS"].split(" ")
@@ -79,7 +80,11 @@ def pre_build_ex(config, functions):
     command.append(os.path.join(config["WORKSPACE"], "PreBuildReport.txt"))
     command.append("--log=" + os.path.join(config["WORKSPACE"], 
"PreBuild.log"))
 
-    _, _, _, code = execute_script(command, config)
+    shell = True
+    if os.name == "posix":  # linux
+        shell = False
+
+    _, _, _, code = execute_script(command, config, shell=shell)
     if code != 0:
         print(" ".join(command))
         print("Error re-generating PlatformOffset header files")
@@ -109,7 +114,7 @@ def pre_build_ex(config, functions):
                os.path.join(config["BUILD_X64"], aml_offsets_split[0], 
aml_offsets_split[1], aml_offsets_split[1], "OUTPUT", 
os.path.dirname(relative_dsdt_file_path), dsdt_file_root_ext[0] + ".offset.h")]
 
     # execute the command
-    _, _, _, code = execute_script(command, config)
+    _, _, _, code = execute_script(command, config, shell=shell)
     if code != 0:
         print(" ".join(command))
         print("Error re-generating PlatformOffset header files")
diff --git 
a/Platform/Intel/WhitleyOpenBoardPkg/Platform/Pei/PlatformInfo/PlatformInfo.c 
b/Platform/Intel/WhitleyOpenBoardPkg/Platform/Pei/PlatformInfo/PlatformInfo.c
index 0065819d83..439ce357f1 100644
--- 
a/Platform/Intel/WhitleyOpenBoardPkg/Platform/Pei/PlatformInfo/PlatformInfo.c
+++ 
b/Platform/Intel/WhitleyOpenBoardPkg/Platform/Pei/PlatformInfo/PlatformInfo.c
@@ -413,8 +413,8 @@ PdrGetPlatformInfo (
     return Status;
   }
 
-  if ((PlatformInfoHob->BoardId >= TypePlatformMin) && 
(PlatformInfoHob->BoardId <= TypePlatformMax) ||
-      (PlatformInfoHob->BoardId >= TypePlatformVendorMin) && 
(PlatformInfoHob->BoardId <= TypePlatformVendorMax)) {
+  if (((PlatformInfoHob->BoardId >= TypePlatformMin) && 
(PlatformInfoHob->BoardId <= TypePlatformMax)) ||
+      ((PlatformInfoHob->BoardId >= TypePlatformVendorMin) && 
(PlatformInfoHob->BoardId <= TypePlatformVendorMax))) {
     //
     // Valid Platform Identified
     //
diff --git a/Platform/Intel/WhitleyOpenBoardPkg/WilsonCityRvp/build_board.py 
b/Platform/Intel/WhitleyOpenBoardPkg/WilsonCityRvp/build_board.py
index 5f625f5f92..e2d32d3e13 100644
--- a/Platform/Intel/WhitleyOpenBoardPkg/WilsonCityRvp/build_board.py
+++ b/Platform/Intel/WhitleyOpenBoardPkg/WilsonCityRvp/build_board.py
@@ -60,7 +60,8 @@ def pre_build_ex(config, functions):
 
     execute_script = functions.get("execute_script")
 
-    command = ["build", "-D", "MAX_SOCKET=" + config["MAX_SOCKET"]]
+    # AML offset arch is X64, not sure if it matters.
+    command = ["build", "-a", "X64", "-t", config["TOOL_CHAIN_TAG"], "-D", 
"MAX_SOCKET=" + config["MAX_SOCKET"]]
 
     if config["EXT_BUILD_FLAGS"] and config["EXT_BUILD_FLAGS"] != "":
         ext_build_flags = config["EXT_BUILD_FLAGS"].split(" ")
@@ -77,7 +78,11 @@ def pre_build_ex(config, functions):
     command.append(os.path.join(config["WORKSPACE"], "PreBuildReport.txt"))
     command.append("--log=" + os.path.join(config["WORKSPACE"], 
"PreBuild.log"))
 
-    _, _, _, code = execute_script(command, config)
+    shell = True
+    if os.name == "posix":  # linux
+        shell = False
+
+    _, _, _, code = execute_script(command, config, shell=shell)
     if code != 0:
         print(" ".join(command))
         print("Error re-generating PlatformOffset header files")
@@ -107,7 +112,7 @@ def pre_build_ex(config, functions):
                os.path.join(config["BUILD_X64"], aml_offsets_split[0], 
aml_offsets_split[1], aml_offsets_split[1], "OUTPUT", 
os.path.dirname(relative_dsdt_file_path), dsdt_file_root_ext[0] + ".offset.h")]
 
     # execute the command
-    _, _, _, code = execute_script(command, config)
+    _, _, _, code = execute_script(command, config, shell=shell)
     if code != 0:
         print(" ".join(command))
         print("Error re-generating PlatformOffset header files")
-- 
2.27.0.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#89450): https://edk2.groups.io/g/devel/message/89450
Mute This Topic: https://groups.io/mt/90785171/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-