* [edk2-devel][edk2-platforms][PATCH V1 1/1] Features/Intel: Prepare for edk2-platforms CI
[not found] <cover.1673309073.git.isaac.w.oram@intel.com>
@ 2023-01-10 0:05 ` Isaac Oram
2023-01-11 22:10 ` Chaganty, Rangasai V
2023-01-13 14:46 ` Rebecca Cran
0 siblings, 2 replies; 5+ messages in thread
From: Isaac Oram @ 2023-01-10 0:05 UTC (permalink / raw)
To: devel
Cc: Isaac Oram, Sai Chaganty, Nate DeSimone, Liming Gao, Eric Dong,
Dandan Bi
Fix some build issues with GCC5 targets
Add a Readme.md for AdvancedFeaturePkg
Add VS2019, CLANGPDB, and GCC5 build targets to Readme.md
for each feature
Cc: Sai Chaganty <rangasai.v.chaganty@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Eric Dong <eric.dong@intel.com>
Cc: Dandan Bi <dandan.bi@intel.com>
Signed-off-by: Isaac Oram <isaac.w.oram@intel.com>
---
.../AdvancedFeaturePkg/AdvancedFeaturePkg.dsc | 6 ++-
Features/Intel/AdvancedFeaturePkg/Readme.md | 38 +++++++++++++++++++
.../AcpiDebugFeaturePkg.dsc | 3 ++
.../Debugging/AcpiDebugFeaturePkg/Readme.md | 9 ++---
.../Debugging/BeepDebugFeaturePkg/Readme.md | 6 +++
.../PostCodeDebugFeaturePkg/Readme.md | 5 ++-
.../Debugging/Usb3DebugFeaturePkg/Readme.md | 9 ++---
.../Intel/Network/NetworkFeaturePkg/Readme.md | 9 ++---
.../AsfFeaturePkg/Readme.md | 5 ++-
.../GenericIpmi/Common/IpmiHooks.c | 2 +
.../GenericIpmi/Common/IpmiHooks.h | 2 +
.../GenericIpmi/Dxe/GenericIpmi.c | 2 +-
.../Library/IpmiBaseLib/IpmiBaseLib.c | 2 +-
.../IpmiFeaturePkg/Readme.md | 9 ++---
.../SpcrFeaturePkg/Readme.md | 5 ++-
.../Library/PcdInitLib/PcdInitLib.c | 2 +-
.../PowerManagement/S3FeaturePkg/Readme.md | 9 ++---
.../SmbiosFeaturePkg/Readme.md | 9 ++---
Features/Intel/TemplateFeaturePkg/Readme.md | 9 ++++-
.../UserInterface/LogoFeaturePkg/Readme.md | 6 ++-
.../UserAuthFeaturePkg/Readme.md | 9 ++---
.../VirtualKeyboardFeaturePkg/Readme.md | 5 ++-
.../VirtualKeyboardFeaturePkg.dsc | 4 ++
23 files changed, 119 insertions(+), 46 deletions(-)
create mode 100644 Features/Intel/AdvancedFeaturePkg/Readme.md
diff --git a/Features/Intel/AdvancedFeaturePkg/AdvancedFeaturePkg.dsc b/Features/Intel/AdvancedFeaturePkg/AdvancedFeaturePkg.dsc
index 18929589c5..226d41522d 100644
--- a/Features/Intel/AdvancedFeaturePkg/AdvancedFeaturePkg.dsc
+++ b/Features/Intel/AdvancedFeaturePkg/AdvancedFeaturePkg.dsc
@@ -25,7 +25,7 @@
DSC_SPECIFICATION = 0x00010005
OUTPUT_DIRECTORY = Build/AdvancedFeaturePkg
SUPPORTED_ARCHITECTURES = IA32|X64
- BUILD_TARGETS = DEBUG|RELEASE
+ BUILD_TARGETS = DEBUG|RELEASE|NOOPT
SKUID_IDENTIFIER = DEFAULT
PEI_ARCH = IA32
DXE_ARCH = X64
@@ -95,6 +95,10 @@
[PcdsFixedAtBuild]
gUsb3DebugFeaturePkgTokenSpaceGuid.PcdUsb3DebugPortLibInstance|1
+[PcdsDynamicExDefault.X64]
+ gEfiMdeModulePkgTokenSpaceGuid.PcdConOutRow
+ gEfiMdeModulePkgTokenSpaceGuid.PcdSetupConOutRow
+
#
# MinPlatform common include for required feature PCD
# These PCD must be set before the core include files, CoreCommonLib,
diff --git a/Features/Intel/AdvancedFeaturePkg/Readme.md b/Features/Intel/AdvancedFeaturePkg/Readme.md
new file mode 100644
index 0000000000..ba50560328
--- /dev/null
+++ b/Features/Intel/AdvancedFeaturePkg/Readme.md
@@ -0,0 +1,38 @@
+# Overview
+Build all advanced features for testing build and integration.
+
+Please refer to individual feature packages for details on intended uses.
+
+# High-Level Theory of Operation
+Enable all features and build a reasonable default configuration. This is not expected to produce binaries that are directly usable in a product as features may require board or silicon specific configuration and libraries.
+
+## Firmware Volumes
+Produces:
+* FvAdvancedPreMemory
+* FvAdvancedUncompressed
+* FvAdvanced
+
+## Build
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
+
+## Windows Example:
+With current working directory at O:\
+
+git clone https://github.com/tianocore/edk2.git
+
+git clone https://github.com/tianocore/edk2-platforms.git
+
+set workspace=O:\
+
+set EDK_TOOLS_PATH=O:\Edk2\BaseTools
+
+set packages_path=O:\edk2;O:\edk2-non-osi;O:\edk2-platforms\Platform\Intel;O:\edk2-platforms\Silicon\Intel;O:\edk2-platforms\Platform\Qemu;O:\edk2-platforms\Silicon\Qemu;O:\edk2-platforms\Features\Intel;O:\edk2-platforms\Features\Intel\Debugging;O:\edk2-platforms\Features\Intel\Network;O:\edk2-platforms\Features\Intel\OutOfBandManagement;O:\edk2-platforms\Features\Intel\PowerManagement;O:\edk2-platforms\Features\Intel\SystemInformation;O:\edk2-platforms\Features\Intel\UserInterface
+
+cd \edk2
+
+edksetup.bat Rebuild
+
+build -a IA32 -a X64 -t CLANGPDB -b NOOPT -p AdvancedFeaturePkg\AdvancedFeaturePkg.dsc
diff --git a/Features/Intel/Debugging/AcpiDebugFeaturePkg/AcpiDebugFeaturePkg.dsc b/Features/Intel/Debugging/AcpiDebugFeaturePkg/AcpiDebugFeaturePkg.dsc
index 306d8bce16..ffe8c427ff 100644
--- a/Features/Intel/Debugging/AcpiDebugFeaturePkg/AcpiDebugFeaturePkg.dsc
+++ b/Features/Intel/Debugging/AcpiDebugFeaturePkg/AcpiDebugFeaturePkg.dsc
@@ -34,6 +34,9 @@
#
!include MinPlatformPkg/Include/Dsc/MinPlatformFeaturesPcd.dsc.inc
+[PcdsFeatureFlag.X64]
+ gAcpiDebugFeaturePkgTokenSpaceGuid.PcdUseSmmVersion |FALSE
+
#
# Include common libraries
#
diff --git a/Features/Intel/Debugging/AcpiDebugFeaturePkg/Readme.md b/Features/Intel/Debugging/AcpiDebugFeaturePkg/Readme.md
index 9868d6d816..b6cb054efa 100644
--- a/Features/Intel/Debugging/AcpiDebugFeaturePkg/Readme.md
+++ b/Features/Intel/Debugging/AcpiDebugFeaturePkg/Readme.md
@@ -91,11 +91,10 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-*_TODO_*
-Any special build flows should be described in this section.
-
-This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the
-standard flow in the feature package template is used, this section may be empty.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
*_TODO_*
diff --git a/Features/Intel/Debugging/BeepDebugFeaturePkg/Readme.md b/Features/Intel/Debugging/BeepDebugFeaturePkg/Readme.md
index 6626c9b764..143699dbb3 100644
--- a/Features/Intel/Debugging/BeepDebugFeaturePkg/Readme.md
+++ b/Features/Intel/Debugging/BeepDebugFeaturePkg/Readme.md
@@ -120,11 +120,17 @@ Status Code (ReportStatusCode) -> Beep Value (GetBeepValueFromStatusCode).
ReportStatusCode () -> BeepStatusCodeReportWorker () -> GetBeepValueFromStatusCode () -> Beep ()
## Build Flows
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
+
Standalone build
* build -a IA32 -a X64 -p Debugging\BeepDebugFeaturePkg\BeepDebugFeaturePkg.dsc
AdvanceFeaturePkg build
* build -a IA32 -a X64 -p AdvancedFeaturePkg/AdvancedFeaturePkg.dsc
+
## Test Point Results
None
diff --git a/Features/Intel/Debugging/PostCodeDebugFeaturePkg/Readme.md b/Features/Intel/Debugging/PostCodeDebugFeaturePkg/Readme.md
index 3fc52f1c00..d0d8d36fd7 100644
--- a/Features/Intel/Debugging/PostCodeDebugFeaturePkg/Readme.md
+++ b/Features/Intel/Debugging/PostCodeDebugFeaturePkg/Readme.md
@@ -111,7 +111,10 @@ Status Code (ReportStatusCode) -> Post Code (GetPostCodeFromStatusCode).
ReportStatusCode () -> PostCodeStatusCodeReportWorker () -> GetPostCodeFromStatusCode () -> PostCode ()
## Build Flows
-There are not special build flows.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
There are no test points defined.
diff --git a/Features/Intel/Debugging/Usb3DebugFeaturePkg/Readme.md b/Features/Intel/Debugging/Usb3DebugFeaturePkg/Readme.md
index c8afcf9fd2..740bfb6db0 100644
--- a/Features/Intel/Debugging/Usb3DebugFeaturePkg/Readme.md
+++ b/Features/Intel/Debugging/Usb3DebugFeaturePkg/Readme.md
@@ -72,11 +72,10 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-*_TODO_*
-Any special build flows should be described in this section.
-
-This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the
-standard flow in the feature package template is used, this section may be empty.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
No test points implemented
diff --git a/Features/Intel/Network/NetworkFeaturePkg/Readme.md b/Features/Intel/Network/NetworkFeaturePkg/Readme.md
index af59fc427a..e953bbcbda 100644
--- a/Features/Intel/Network/NetworkFeaturePkg/Readme.md
+++ b/Features/Intel/Network/NetworkFeaturePkg/Readme.md
@@ -69,11 +69,10 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-*_TODO_*
-Any special build flows should be described in this section.
-
-This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the
-standard flow in the feature package template is used, this section may be empty.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
*_TODO_*
diff --git a/Features/Intel/OutOfBandManagement/AsfFeaturePkg/Readme.md b/Features/Intel/OutOfBandManagement/AsfFeaturePkg/Readme.md
index f18b1bfb07..35f5fc202d 100644
--- a/Features/Intel/OutOfBandManagement/AsfFeaturePkg/Readme.md
+++ b/Features/Intel/OutOfBandManagement/AsfFeaturePkg/Readme.md
@@ -44,7 +44,10 @@ AsfPushEvent() -> through SmBus -> NIC -> Remote management consoles
N/A now.
## Build Flows
-There is not special build flow.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
There are not test points implemented.
diff --git a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Common/IpmiHooks.c b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Common/IpmiHooks.c
index 86ff7c0fdf..b2788e5a4c 100644
--- a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Common/IpmiHooks.c
+++ b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Common/IpmiHooks.c
@@ -9,6 +9,7 @@
#include "IpmiHooks.h"
EFI_STATUS
+EFIAPI
IpmiSendCommand (
IN IPMI_TRANSPORT *This,
IN UINT8 NetFunction,
@@ -64,6 +65,7 @@ Returns:
} // IpmiSendCommand()
EFI_STATUS
+EFIAPI
IpmiGetBmcStatus (
IN IPMI_TRANSPORT *This,
OUT BMC_STATUS *BmcStatus,
diff --git a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Common/IpmiHooks.h b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Common/IpmiHooks.h
index 083c9e70b0..823cc08c61 100644
--- a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Common/IpmiHooks.h
+++ b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Common/IpmiHooks.h
@@ -15,6 +15,7 @@
// Internal(hook) function list
//
EFI_STATUS
+EFIAPI
IpmiSendCommand (
IN IPMI_TRANSPORT *This,
IN UINT8 NetFunction,
@@ -54,6 +55,7 @@ Returns:
;
EFI_STATUS
+EFIAPI
IpmiGetBmcStatus (
IN IPMI_TRANSPORT *This,
OUT BMC_STATUS *BmcStatus,
diff --git a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Dxe/GenericIpmi.c b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Dxe/GenericIpmi.c
index 957a9c8e6e..d973db3680 100644
--- a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Dxe/GenericIpmi.c
+++ b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Dxe/GenericIpmi.c
@@ -28,6 +28,7 @@
@retval EFI_UNSUPPORTED IPMI is not available.
**/
EFI_STATUS
+EFIAPI
LocateIpmiInterface (
IN EFI_HANDLE ImageHandle,
IN EFI_SYSTEM_TABLE *SystemTable
@@ -43,4 +44,3 @@ LocateIpmiInterface (
return Status;
} // LocateIpmiInterface()
-
diff --git a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiBaseLib/IpmiBaseLib.c b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiBaseLib/IpmiBaseLib.c
index 6b2e0a9e1e..5df9d861c6 100644
--- a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiBaseLib/IpmiBaseLib.c
+++ b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiBaseLib/IpmiBaseLib.c
@@ -132,6 +132,7 @@ IpmiSubmitCommand (
**/
EFI_STATUS
+EFIAPI
IpmiGetBmcStatus (
OUT BMC_STATUS *BmcStatus,
OUT SM_COM_ADDRESS *ComAddress
@@ -152,4 +153,3 @@ IpmiGetBmcStatus (
);
return Status;
}
-
diff --git a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Readme.md b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Readme.md
index d383150e3f..7551fcc64f 100644
--- a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Readme.md
+++ b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Readme.md
@@ -58,11 +58,10 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-*_TODO_*
-Any special build flows should be described in this section.
-
-This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the
-standard flow in the feature package template is used, this section may be empty.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
*_TODO_*
diff --git a/Features/Intel/OutOfBandManagement/SpcrFeaturePkg/Readme.md b/Features/Intel/OutOfBandManagement/SpcrFeaturePkg/Readme.md
index 66138ee0f6..173fb3e491 100644
--- a/Features/Intel/OutOfBandManagement/SpcrFeaturePkg/Readme.md
+++ b/Features/Intel/OutOfBandManagement/SpcrFeaturePkg/Readme.md
@@ -46,7 +46,10 @@ GetSpcrDevice() -> serial port device path -> get PCI info and serial port info
GetSpcrDevice () in SpcrDeviceLib -> SpcrAcpiDxe.
## Build Flows
-There is not special build flow.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
There are not test points implemented.
diff --git a/Features/Intel/PlatformPayloadFeaturePkg/Library/PcdInitLib/PcdInitLib.c b/Features/Intel/PlatformPayloadFeaturePkg/Library/PcdInitLib/PcdInitLib.c
index 4ea89582f3..e3d305d01f 100644
--- a/Features/Intel/PlatformPayloadFeaturePkg/Library/PcdInitLib/PcdInitLib.c
+++ b/Features/Intel/PlatformPayloadFeaturePkg/Library/PcdInitLib/PcdInitLib.c
@@ -21,6 +21,7 @@
**/
EFI_STATUS
+EFIAPI
PcdInitConstructor (
VOID
)
@@ -86,4 +87,3 @@ PcdInitConstructor (
return EFI_SUCCESS;
}
-
diff --git a/Features/Intel/PowerManagement/S3FeaturePkg/Readme.md b/Features/Intel/PowerManagement/S3FeaturePkg/Readme.md
index 95ce2ddc15..8f96d29ce1 100644
--- a/Features/Intel/PowerManagement/S3FeaturePkg/Readme.md
+++ b/Features/Intel/PowerManagement/S3FeaturePkg/Readme.md
@@ -63,11 +63,10 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-*_TODO_*
-Any special build flows should be described in this section.
-
-This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the
-standard flow in the feature package template is used, this section may be empty.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
*_TODO_*
diff --git a/Features/Intel/SystemInformation/SmbiosFeaturePkg/Readme.md b/Features/Intel/SystemInformation/SmbiosFeaturePkg/Readme.md
index 6ecca8af4b..46d1744414 100644
--- a/Features/Intel/SystemInformation/SmbiosFeaturePkg/Readme.md
+++ b/Features/Intel/SystemInformation/SmbiosFeaturePkg/Readme.md
@@ -65,11 +65,10 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-*_TODO_*
-Any special build flows should be described in this section.
-
-This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the
-standard flow in the feature package template is used, this section may be empty.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
*_TODO_*
diff --git a/Features/Intel/TemplateFeaturePkg/Readme.md b/Features/Intel/TemplateFeaturePkg/Readme.md
index fc77340684..a777eccc29 100644
--- a/Features/Intel/TemplateFeaturePkg/Readme.md
+++ b/Features/Intel/TemplateFeaturePkg/Readme.md
@@ -45,11 +45,18 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-Any special build flows should be described in this section.
+Standard and special build flows should be described in this section.
This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the
standard flow in the feature package template is used, this section may be empty.
+Example:
+
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
+
## Test Point Results
The test(s) that can verify porting is complete for the feature.
diff --git a/Features/Intel/UserInterface/LogoFeaturePkg/Readme.md b/Features/Intel/UserInterface/LogoFeaturePkg/Readme.md
index bf1a61ba95..33f2ffe410 100644
--- a/Features/Intel/UserInterface/LogoFeaturePkg/Readme.md
+++ b/Features/Intel/UserInterface/LogoFeaturePkg/Readme.md
@@ -78,8 +78,10 @@ Architecturally defined data structures and flows for the feature.
Consumer gets image from the logo driver via EDKII_PLATFORM_LOGO_PROTOCOL.
## Build Flows
-
-No any special build flows is needed.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
diff --git a/Features/Intel/UserInterface/UserAuthFeaturePkg/Readme.md b/Features/Intel/UserInterface/UserAuthFeaturePkg/Readme.md
index b236d2ad30..a5c1d3cb96 100644
--- a/Features/Intel/UserInterface/UserAuthFeaturePkg/Readme.md
+++ b/Features/Intel/UserInterface/UserAuthFeaturePkg/Readme.md
@@ -59,11 +59,10 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-*_TODO_*
-Any special build flows should be described in this section.
-
-This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the
-standard flow in the feature package template is used, this section may be empty.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
*_TODO_*
diff --git a/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/Readme.md b/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/Readme.md
index ae7f778748..5e750cd923 100644
--- a/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/Readme.md
+++ b/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/Readme.md
@@ -43,7 +43,10 @@ User touch the touch panel -> absolute pointer and status -> virtual keyboard ev
This driver use AbsolutePointer protocol to get the info of user input, and use GraphicsOutput protocol to show the picture.
## Build Flows
-There is not special build flow.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
Test this driver in a device with touch panel and the BIOS support touch panel driver.
diff --git a/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/VirtualKeyboardFeaturePkg.dsc b/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/VirtualKeyboardFeaturePkg.dsc
index d13bd673d3..a025cc755d 100644
--- a/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/VirtualKeyboardFeaturePkg.dsc
+++ b/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/VirtualKeyboardFeaturePkg.dsc
@@ -34,6 +34,10 @@
#
!include MinPlatformPkg/Include/Dsc/MinPlatformFeaturesPcd.dsc.inc
+[PcdsDynamicExDefault.X64]
+ gEfiMdeModulePkgTokenSpaceGuid.PcdConOutRow
+ gEfiMdeModulePkgTokenSpaceGuid.PcdSetupConOutRow
+
#
# Include common libraries
#
--
2.39.0.windows.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [edk2-devel][edk2-platforms][PATCH V1 1/1] Features/Intel: Prepare for edk2-platforms CI
2023-01-10 0:05 ` [edk2-devel][edk2-platforms][PATCH V1 1/1] Features/Intel: Prepare for edk2-platforms CI Isaac Oram
@ 2023-01-11 22:10 ` Chaganty, Rangasai V
2023-01-13 0:45 ` Isaac Oram
2023-01-13 14:46 ` Rebecca Cran
1 sibling, 1 reply; 5+ messages in thread
From: Chaganty, Rangasai V @ 2023-01-11 22:10 UTC (permalink / raw)
To: Oram, Isaac W, devel@edk2.groups.io
Cc: Desimone, Nathaniel L, Gao, Liming, Dong, Eric, Bi, Dandan
Reviewed-by: Sai Chaganty <rangasai.v.chaganty@intel.com>
-----Original Message-----
From: Oram, Isaac W <isaac.w.oram@intel.com>
Sent: Monday, January 09, 2023 4:05 PM
To: devel@edk2.groups.io
Cc: Oram, Isaac W <isaac.w.oram@intel.com>; Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>; Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Gao, Liming <gaoliming@byosoft.com.cn>; Dong, Eric <eric.dong@intel.com>; Bi, Dandan <dandan.bi@intel.com>
Subject: [edk2-devel][edk2-platforms][PATCH V1 1/1] Features/Intel: Prepare for edk2-platforms CI
Fix some build issues with GCC5 targets
Add a Readme.md for AdvancedFeaturePkg
Add VS2019, CLANGPDB, and GCC5 build targets to Readme.md for each feature
Cc: Sai Chaganty <rangasai.v.chaganty@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Eric Dong <eric.dong@intel.com>
Cc: Dandan Bi <dandan.bi@intel.com>
Signed-off-by: Isaac Oram <isaac.w.oram@intel.com>
---
.../AdvancedFeaturePkg/AdvancedFeaturePkg.dsc | 6 ++-
Features/Intel/AdvancedFeaturePkg/Readme.md | 38 +++++++++++++++++++
.../AcpiDebugFeaturePkg.dsc | 3 ++
.../Debugging/AcpiDebugFeaturePkg/Readme.md | 9 ++---
.../Debugging/BeepDebugFeaturePkg/Readme.md | 6 +++
.../PostCodeDebugFeaturePkg/Readme.md | 5 ++-
.../Debugging/Usb3DebugFeaturePkg/Readme.md | 9 ++---
.../Intel/Network/NetworkFeaturePkg/Readme.md | 9 ++---
.../AsfFeaturePkg/Readme.md | 5 ++-
.../GenericIpmi/Common/IpmiHooks.c | 2 +
.../GenericIpmi/Common/IpmiHooks.h | 2 +
.../GenericIpmi/Dxe/GenericIpmi.c | 2 +-
.../Library/IpmiBaseLib/IpmiBaseLib.c | 2 +-
.../IpmiFeaturePkg/Readme.md | 9 ++---
.../SpcrFeaturePkg/Readme.md | 5 ++-
.../Library/PcdInitLib/PcdInitLib.c | 2 +-
.../PowerManagement/S3FeaturePkg/Readme.md | 9 ++---
.../SmbiosFeaturePkg/Readme.md | 9 ++---
Features/Intel/TemplateFeaturePkg/Readme.md | 9 ++++-
.../UserInterface/LogoFeaturePkg/Readme.md | 6 ++-
.../UserAuthFeaturePkg/Readme.md | 9 ++---
.../VirtualKeyboardFeaturePkg/Readme.md | 5 ++-
.../VirtualKeyboardFeaturePkg.dsc | 4 ++
23 files changed, 119 insertions(+), 46 deletions(-) create mode 100644 Features/Intel/AdvancedFeaturePkg/Readme.md
diff --git a/Features/Intel/AdvancedFeaturePkg/AdvancedFeaturePkg.dsc b/Features/Intel/AdvancedFeaturePkg/AdvancedFeaturePkg.dsc
index 18929589c5..226d41522d 100644
--- a/Features/Intel/AdvancedFeaturePkg/AdvancedFeaturePkg.dsc
+++ b/Features/Intel/AdvancedFeaturePkg/AdvancedFeaturePkg.dsc
@@ -25,7 +25,7 @@
DSC_SPECIFICATION = 0x00010005
OUTPUT_DIRECTORY = Build/AdvancedFeaturePkg
SUPPORTED_ARCHITECTURES = IA32|X64
- BUILD_TARGETS = DEBUG|RELEASE
+ BUILD_TARGETS = DEBUG|RELEASE|NOOPT
SKUID_IDENTIFIER = DEFAULT
PEI_ARCH = IA32
DXE_ARCH = X64
@@ -95,6 +95,10 @@
[PcdsFixedAtBuild]
gUsb3DebugFeaturePkgTokenSpaceGuid.PcdUsb3DebugPortLibInstance|1
+[PcdsDynamicExDefault.X64]
+ gEfiMdeModulePkgTokenSpaceGuid.PcdConOutRow
+ gEfiMdeModulePkgTokenSpaceGuid.PcdSetupConOutRow
+
#
# MinPlatform common include for required feature PCD # These PCD must be set before the core include files, CoreCommonLib, diff --git a/Features/Intel/AdvancedFeaturePkg/Readme.md b/Features/Intel/AdvancedFeaturePkg/Readme.md
new file mode 100644
index 0000000000..ba50560328
--- /dev/null
+++ b/Features/Intel/AdvancedFeaturePkg/Readme.md
@@ -0,0 +1,38 @@
+# Overview
+Build all advanced features for testing build and integration.
+
+Please refer to individual feature packages for details on intended uses.
+
+# High-Level Theory of Operation
+Enable all features and build a reasonable default configuration. This is not expected to produce binaries that are directly usable in a product as features may require board or silicon specific configuration and libraries.
+
+## Firmware Volumes
+Produces:
+* FvAdvancedPreMemory
+* FvAdvancedUncompressed
+* FvAdvanced
+
+## Build
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
+
+## Windows Example:
+With current working directory at O:\
+
+git clone https://github.com/tianocore/edk2.git
+
+git clone https://github.com/tianocore/edk2-platforms.git
+
+set workspace=O:\
+
+set EDK_TOOLS_PATH=O:\Edk2\BaseTools
+
+set
+packages_path=O:\edk2;O:\edk2-non-osi;O:\edk2-platforms\Platform\Intel;
+O:\edk2-platforms\Silicon\Intel;O:\edk2-platforms\Platform\Qemu;O:\edk2
+-platforms\Silicon\Qemu;O:\edk2-platforms\Features\Intel;O:\edk2-platfo
+rms\Features\Intel\Debugging;O:\edk2-platforms\Features\Intel\Network;O
+:\edk2-platforms\Features\Intel\OutOfBandManagement;O:\edk2-platforms\F
+eatures\Intel\PowerManagement;O:\edk2-platforms\Features\Intel\SystemIn
+formation;O:\edk2-platforms\Features\Intel\UserInterface
+
+cd \edk2
+
+edksetup.bat Rebuild
+
+build -a IA32 -a X64 -t CLANGPDB -b NOOPT -p
+AdvancedFeaturePkg\AdvancedFeaturePkg.dsc
diff --git a/Features/Intel/Debugging/AcpiDebugFeaturePkg/AcpiDebugFeaturePkg.dsc b/Features/Intel/Debugging/AcpiDebugFeaturePkg/AcpiDebugFeaturePkg.dsc
index 306d8bce16..ffe8c427ff 100644
--- a/Features/Intel/Debugging/AcpiDebugFeaturePkg/AcpiDebugFeaturePkg.dsc
+++ b/Features/Intel/Debugging/AcpiDebugFeaturePkg/AcpiDebugFeaturePkg.d
+++ sc
@@ -34,6 +34,9 @@
#
!include MinPlatformPkg/Include/Dsc/MinPlatformFeaturesPcd.dsc.inc
+[PcdsFeatureFlag.X64]
+ gAcpiDebugFeaturePkgTokenSpaceGuid.PcdUseSmmVersion |FALSE
+
#
# Include common libraries
#
diff --git a/Features/Intel/Debugging/AcpiDebugFeaturePkg/Readme.md b/Features/Intel/Debugging/AcpiDebugFeaturePkg/Readme.md
index 9868d6d816..b6cb054efa 100644
--- a/Features/Intel/Debugging/AcpiDebugFeaturePkg/Readme.md
+++ b/Features/Intel/Debugging/AcpiDebugFeaturePkg/Readme.md
@@ -91,11 +91,10 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-*_TODO_*
-Any special build flows should be described in this section.
-
-This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the -standard flow in the feature package template is used, this section may be empty.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
*_TODO_*
diff --git a/Features/Intel/Debugging/BeepDebugFeaturePkg/Readme.md b/Features/Intel/Debugging/BeepDebugFeaturePkg/Readme.md
index 6626c9b764..143699dbb3 100644
--- a/Features/Intel/Debugging/BeepDebugFeaturePkg/Readme.md
+++ b/Features/Intel/Debugging/BeepDebugFeaturePkg/Readme.md
@@ -120,11 +120,17 @@ Status Code (ReportStatusCode) -> Beep Value (GetBeepValueFromStatusCode).
ReportStatusCode () -> BeepStatusCodeReportWorker () -> GetBeepValueFromStatusCode () -> Beep ()
## Build Flows
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
+
Standalone build
* build -a IA32 -a X64 -p Debugging\BeepDebugFeaturePkg\BeepDebugFeaturePkg.dsc
AdvanceFeaturePkg build
* build -a IA32 -a X64 -p AdvancedFeaturePkg/AdvancedFeaturePkg.dsc
+
## Test Point Results
None
diff --git a/Features/Intel/Debugging/PostCodeDebugFeaturePkg/Readme.md b/Features/Intel/Debugging/PostCodeDebugFeaturePkg/Readme.md
index 3fc52f1c00..d0d8d36fd7 100644
--- a/Features/Intel/Debugging/PostCodeDebugFeaturePkg/Readme.md
+++ b/Features/Intel/Debugging/PostCodeDebugFeaturePkg/Readme.md
@@ -111,7 +111,10 @@ Status Code (ReportStatusCode) -> Post Code (GetPostCodeFromStatusCode).
ReportStatusCode () -> PostCodeStatusCodeReportWorker () -> GetPostCodeFromStatusCode () -> PostCode ()
## Build Flows
-There are not special build flows.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
There are no test points defined.
diff --git a/Features/Intel/Debugging/Usb3DebugFeaturePkg/Readme.md b/Features/Intel/Debugging/Usb3DebugFeaturePkg/Readme.md
index c8afcf9fd2..740bfb6db0 100644
--- a/Features/Intel/Debugging/Usb3DebugFeaturePkg/Readme.md
+++ b/Features/Intel/Debugging/Usb3DebugFeaturePkg/Readme.md
@@ -72,11 +72,10 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-*_TODO_*
-Any special build flows should be described in this section.
-
-This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the -standard flow in the feature package template is used, this section may be empty.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
No test points implemented
diff --git a/Features/Intel/Network/NetworkFeaturePkg/Readme.md b/Features/Intel/Network/NetworkFeaturePkg/Readme.md
index af59fc427a..e953bbcbda 100644
--- a/Features/Intel/Network/NetworkFeaturePkg/Readme.md
+++ b/Features/Intel/Network/NetworkFeaturePkg/Readme.md
@@ -69,11 +69,10 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-*_TODO_*
-Any special build flows should be described in this section.
-
-This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the -standard flow in the feature package template is used, this section may be empty.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
*_TODO_*
diff --git a/Features/Intel/OutOfBandManagement/AsfFeaturePkg/Readme.md b/Features/Intel/OutOfBandManagement/AsfFeaturePkg/Readme.md
index f18b1bfb07..35f5fc202d 100644
--- a/Features/Intel/OutOfBandManagement/AsfFeaturePkg/Readme.md
+++ b/Features/Intel/OutOfBandManagement/AsfFeaturePkg/Readme.md
@@ -44,7 +44,10 @@ AsfPushEvent() -> through SmBus -> NIC -> Remote management consoles N/A now.
## Build Flows
-There is not special build flow.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
There are not test points implemented.
diff --git a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Common/IpmiHooks.c b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Common/IpmiHooks.c
index 86ff7c0fdf..b2788e5a4c 100644
--- a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Common/IpmiHooks.c
+++ b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Comm
+++ on/IpmiHooks.c
@@ -9,6 +9,7 @@
#include "IpmiHooks.h"
EFI_STATUS
+EFIAPI
IpmiSendCommand (
IN IPMI_TRANSPORT *This,
IN UINT8 NetFunction,
@@ -64,6 +65,7 @@ Returns:
} // IpmiSendCommand()
EFI_STATUS
+EFIAPI
IpmiGetBmcStatus (
IN IPMI_TRANSPORT *This,
OUT BMC_STATUS *BmcStatus,
diff --git a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Common/IpmiHooks.h b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Common/IpmiHooks.h
index 083c9e70b0..823cc08c61 100644
--- a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Common/IpmiHooks.h
+++ b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Comm
+++ on/IpmiHooks.h
@@ -15,6 +15,7 @@
// Internal(hook) function list
//
EFI_STATUS
+EFIAPI
IpmiSendCommand (
IN IPMI_TRANSPORT *This,
IN UINT8 NetFunction,
@@ -54,6 +55,7 @@ Returns:
;
EFI_STATUS
+EFIAPI
IpmiGetBmcStatus (
IN IPMI_TRANSPORT *This,
OUT BMC_STATUS *BmcStatus,
diff --git a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Dxe/GenericIpmi.c b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Dxe/GenericIpmi.c
index 957a9c8e6e..d973db3680 100644
--- a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Dxe/GenericIpmi.c
+++ b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Dxe/
+++ GenericIpmi.c
@@ -28,6 +28,7 @@
@retval EFI_UNSUPPORTED IPMI is not available.
**/
EFI_STATUS
+EFIAPI
LocateIpmiInterface (
IN EFI_HANDLE ImageHandle,
IN EFI_SYSTEM_TABLE *SystemTable
@@ -43,4 +44,3 @@ LocateIpmiInterface (
return Status;
} // LocateIpmiInterface()
-
diff --git a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiBaseLib/IpmiBaseLib.c b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiBaseLib/IpmiBaseLib.c
index 6b2e0a9e1e..5df9d861c6 100644
--- a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiBaseLib/IpmiBaseLib.c
+++ b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiBase
+++ Lib/IpmiBaseLib.c
@@ -132,6 +132,7 @@ IpmiSubmitCommand (
**/
EFI_STATUS
+EFIAPI
IpmiGetBmcStatus (
OUT BMC_STATUS *BmcStatus,
OUT SM_COM_ADDRESS *ComAddress
@@ -152,4 +153,3 @@ IpmiGetBmcStatus (
);
return Status;
}
-
diff --git a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Readme.md b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Readme.md
index d383150e3f..7551fcc64f 100644
--- a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Readme.md
+++ b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Readme.md
@@ -58,11 +58,10 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-*_TODO_*
-Any special build flows should be described in this section.
-
-This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the -standard flow in the feature package template is used, this section may be empty.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
*_TODO_*
diff --git a/Features/Intel/OutOfBandManagement/SpcrFeaturePkg/Readme.md b/Features/Intel/OutOfBandManagement/SpcrFeaturePkg/Readme.md
index 66138ee0f6..173fb3e491 100644
--- a/Features/Intel/OutOfBandManagement/SpcrFeaturePkg/Readme.md
+++ b/Features/Intel/OutOfBandManagement/SpcrFeaturePkg/Readme.md
@@ -46,7 +46,10 @@ GetSpcrDevice() -> serial port device path -> get PCI info and serial port info GetSpcrDevice () in SpcrDeviceLib -> SpcrAcpiDxe.
## Build Flows
-There is not special build flow.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
There are not test points implemented.
diff --git a/Features/Intel/PlatformPayloadFeaturePkg/Library/PcdInitLib/PcdInitLib.c b/Features/Intel/PlatformPayloadFeaturePkg/Library/PcdInitLib/PcdInitLib.c
index 4ea89582f3..e3d305d01f 100644
--- a/Features/Intel/PlatformPayloadFeaturePkg/Library/PcdInitLib/PcdInitLib.c
+++ b/Features/Intel/PlatformPayloadFeaturePkg/Library/PcdInitLib/PcdIni
+++ tLib.c
@@ -21,6 +21,7 @@
**/
EFI_STATUS
+EFIAPI
PcdInitConstructor (
VOID
)
@@ -86,4 +87,3 @@ PcdInitConstructor (
return EFI_SUCCESS;
}
-
diff --git a/Features/Intel/PowerManagement/S3FeaturePkg/Readme.md b/Features/Intel/PowerManagement/S3FeaturePkg/Readme.md
index 95ce2ddc15..8f96d29ce1 100644
--- a/Features/Intel/PowerManagement/S3FeaturePkg/Readme.md
+++ b/Features/Intel/PowerManagement/S3FeaturePkg/Readme.md
@@ -63,11 +63,10 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-*_TODO_*
-Any special build flows should be described in this section.
-
-This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the -standard flow in the feature package template is used, this section may be empty.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
*_TODO_*
diff --git a/Features/Intel/SystemInformation/SmbiosFeaturePkg/Readme.md b/Features/Intel/SystemInformation/SmbiosFeaturePkg/Readme.md
index 6ecca8af4b..46d1744414 100644
--- a/Features/Intel/SystemInformation/SmbiosFeaturePkg/Readme.md
+++ b/Features/Intel/SystemInformation/SmbiosFeaturePkg/Readme.md
@@ -65,11 +65,10 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-*_TODO_*
-Any special build flows should be described in this section.
-
-This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the -standard flow in the feature package template is used, this section may be empty.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
*_TODO_*
diff --git a/Features/Intel/TemplateFeaturePkg/Readme.md b/Features/Intel/TemplateFeaturePkg/Readme.md
index fc77340684..a777eccc29 100644
--- a/Features/Intel/TemplateFeaturePkg/Readme.md
+++ b/Features/Intel/TemplateFeaturePkg/Readme.md
@@ -45,11 +45,18 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-Any special build flows should be described in this section.
+Standard and special build flows should be described in this section.
This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the standard flow in the feature package template is used, this section may be empty.
+Example:
+
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
+
## Test Point Results
The test(s) that can verify porting is complete for the feature.
diff --git a/Features/Intel/UserInterface/LogoFeaturePkg/Readme.md b/Features/Intel/UserInterface/LogoFeaturePkg/Readme.md
index bf1a61ba95..33f2ffe410 100644
--- a/Features/Intel/UserInterface/LogoFeaturePkg/Readme.md
+++ b/Features/Intel/UserInterface/LogoFeaturePkg/Readme.md
@@ -78,8 +78,10 @@ Architecturally defined data structures and flows for the feature.
Consumer gets image from the logo driver via EDKII_PLATFORM_LOGO_PROTOCOL.
## Build Flows
-
-No any special build flows is needed.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
diff --git a/Features/Intel/UserInterface/UserAuthFeaturePkg/Readme.md b/Features/Intel/UserInterface/UserAuthFeaturePkg/Readme.md
index b236d2ad30..a5c1d3cb96 100644
--- a/Features/Intel/UserInterface/UserAuthFeaturePkg/Readme.md
+++ b/Features/Intel/UserInterface/UserAuthFeaturePkg/Readme.md
@@ -59,11 +59,10 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-*_TODO_*
-Any special build flows should be described in this section.
-
-This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the -standard flow in the feature package template is used, this section may be empty.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
*_TODO_*
diff --git a/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/Readme.md b/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/Readme.md
index ae7f778748..5e750cd923 100644
--- a/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/Readme.md
+++ b/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/Readme.md
@@ -43,7 +43,10 @@ User touch the touch panel -> absolute pointer and status -> virtual keyboard ev This driver use AbsolutePointer protocol to get the info of user input, and use GraphicsOutput protocol to show the picture.
## Build Flows
-There is not special build flow.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
Test this driver in a device with touch panel and the BIOS support touch panel driver.
diff --git a/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/VirtualKeyboardFeaturePkg.dsc b/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/VirtualKeyboardFeaturePkg.dsc
index d13bd673d3..a025cc755d 100644
--- a/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/VirtualKeyboardFeaturePkg.dsc
+++ b/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/VirtualKeyb
+++ oardFeaturePkg.dsc
@@ -34,6 +34,10 @@
#
!include MinPlatformPkg/Include/Dsc/MinPlatformFeaturesPcd.dsc.inc
+[PcdsDynamicExDefault.X64]
+ gEfiMdeModulePkgTokenSpaceGuid.PcdConOutRow
+ gEfiMdeModulePkgTokenSpaceGuid.PcdSetupConOutRow
+
#
# Include common libraries
#
--
2.39.0.windows.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [edk2-devel][edk2-platforms][PATCH V1 1/1] Features/Intel: Prepare for edk2-platforms CI
2023-01-11 22:10 ` Chaganty, Rangasai V
@ 2023-01-13 0:45 ` Isaac Oram
0 siblings, 0 replies; 5+ messages in thread
From: Isaac Oram @ 2023-01-13 0:45 UTC (permalink / raw)
To: Chaganty, Rangasai V, devel@edk2.groups.io
Cc: Desimone, Nathaniel L, Gao, Liming, Dong, Eric, Bi, Dandan
Pushed as 3e0f5b21c9..c227f900f5
-----Original Message-----
From: Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>
Sent: Wednesday, January 11, 2023 2:10 PM
To: Oram, Isaac W <isaac.w.oram@intel.com>; devel@edk2.groups.io
Cc: Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Gao, Liming <gaoliming@byosoft.com.cn>; Dong, Eric <eric.dong@intel.com>; Bi, Dandan <dandan.bi@intel.com>
Subject: RE: [edk2-devel][edk2-platforms][PATCH V1 1/1] Features/Intel: Prepare for edk2-platforms CI
Reviewed-by: Sai Chaganty <rangasai.v.chaganty@intel.com>
-----Original Message-----
From: Oram, Isaac W <isaac.w.oram@intel.com>
Sent: Monday, January 09, 2023 4:05 PM
To: devel@edk2.groups.io
Cc: Oram, Isaac W <isaac.w.oram@intel.com>; Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>; Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Gao, Liming <gaoliming@byosoft.com.cn>; Dong, Eric <eric.dong@intel.com>; Bi, Dandan <dandan.bi@intel.com>
Subject: [edk2-devel][edk2-platforms][PATCH V1 1/1] Features/Intel: Prepare for edk2-platforms CI
Fix some build issues with GCC5 targets
Add a Readme.md for AdvancedFeaturePkg
Add VS2019, CLANGPDB, and GCC5 build targets to Readme.md for each feature
Cc: Sai Chaganty <rangasai.v.chaganty@intel.com>
Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Eric Dong <eric.dong@intel.com>
Cc: Dandan Bi <dandan.bi@intel.com>
Signed-off-by: Isaac Oram <isaac.w.oram@intel.com>
---
.../AdvancedFeaturePkg/AdvancedFeaturePkg.dsc | 6 ++-
Features/Intel/AdvancedFeaturePkg/Readme.md | 38 +++++++++++++++++++
.../AcpiDebugFeaturePkg.dsc | 3 ++
.../Debugging/AcpiDebugFeaturePkg/Readme.md | 9 ++---
.../Debugging/BeepDebugFeaturePkg/Readme.md | 6 +++
.../PostCodeDebugFeaturePkg/Readme.md | 5 ++-
.../Debugging/Usb3DebugFeaturePkg/Readme.md | 9 ++---
.../Intel/Network/NetworkFeaturePkg/Readme.md | 9 ++---
.../AsfFeaturePkg/Readme.md | 5 ++-
.../GenericIpmi/Common/IpmiHooks.c | 2 +
.../GenericIpmi/Common/IpmiHooks.h | 2 +
.../GenericIpmi/Dxe/GenericIpmi.c | 2 +-
.../Library/IpmiBaseLib/IpmiBaseLib.c | 2 +-
.../IpmiFeaturePkg/Readme.md | 9 ++---
.../SpcrFeaturePkg/Readme.md | 5 ++-
.../Library/PcdInitLib/PcdInitLib.c | 2 +-
.../PowerManagement/S3FeaturePkg/Readme.md | 9 ++---
.../SmbiosFeaturePkg/Readme.md | 9 ++---
Features/Intel/TemplateFeaturePkg/Readme.md | 9 ++++-
.../UserInterface/LogoFeaturePkg/Readme.md | 6 ++-
.../UserAuthFeaturePkg/Readme.md | 9 ++---
.../VirtualKeyboardFeaturePkg/Readme.md | 5 ++-
.../VirtualKeyboardFeaturePkg.dsc | 4 ++
23 files changed, 119 insertions(+), 46 deletions(-) create mode 100644 Features/Intel/AdvancedFeaturePkg/Readme.md
diff --git a/Features/Intel/AdvancedFeaturePkg/AdvancedFeaturePkg.dsc b/Features/Intel/AdvancedFeaturePkg/AdvancedFeaturePkg.dsc
index 18929589c5..226d41522d 100644
--- a/Features/Intel/AdvancedFeaturePkg/AdvancedFeaturePkg.dsc
+++ b/Features/Intel/AdvancedFeaturePkg/AdvancedFeaturePkg.dsc
@@ -25,7 +25,7 @@
DSC_SPECIFICATION = 0x00010005
OUTPUT_DIRECTORY = Build/AdvancedFeaturePkg
SUPPORTED_ARCHITECTURES = IA32|X64
- BUILD_TARGETS = DEBUG|RELEASE
+ BUILD_TARGETS = DEBUG|RELEASE|NOOPT
SKUID_IDENTIFIER = DEFAULT
PEI_ARCH = IA32
DXE_ARCH = X64
@@ -95,6 +95,10 @@
[PcdsFixedAtBuild]
gUsb3DebugFeaturePkgTokenSpaceGuid.PcdUsb3DebugPortLibInstance|1
+[PcdsDynamicExDefault.X64]
+ gEfiMdeModulePkgTokenSpaceGuid.PcdConOutRow
+ gEfiMdeModulePkgTokenSpaceGuid.PcdSetupConOutRow
+
#
# MinPlatform common include for required feature PCD # These PCD must be set before the core include files, CoreCommonLib, diff --git a/Features/Intel/AdvancedFeaturePkg/Readme.md b/Features/Intel/AdvancedFeaturePkg/Readme.md
new file mode 100644
index 0000000000..ba50560328
--- /dev/null
+++ b/Features/Intel/AdvancedFeaturePkg/Readme.md
@@ -0,0 +1,38 @@
+# Overview
+Build all advanced features for testing build and integration.
+
+Please refer to individual feature packages for details on intended uses.
+
+# High-Level Theory of Operation
+Enable all features and build a reasonable default configuration. This is not expected to produce binaries that are directly usable in a product as features may require board or silicon specific configuration and libraries.
+
+## Firmware Volumes
+Produces:
+* FvAdvancedPreMemory
+* FvAdvancedUncompressed
+* FvAdvanced
+
+## Build
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
+
+## Windows Example:
+With current working directory at O:\
+
+git clone https://github.com/tianocore/edk2.git
+
+git clone https://github.com/tianocore/edk2-platforms.git
+
+set workspace=O:\
+
+set EDK_TOOLS_PATH=O:\Edk2\BaseTools
+
+set
+packages_path=O:\edk2;O:\edk2-non-osi;O:\edk2-platforms\Platform\Intel;
+O:\edk2-platforms\Silicon\Intel;O:\edk2-platforms\Platform\Qemu;O:\edk2
+-platforms\Silicon\Qemu;O:\edk2-platforms\Features\Intel;O:\edk2-platfo
+rms\Features\Intel\Debugging;O:\edk2-platforms\Features\Intel\Network;O
+:\edk2-platforms\Features\Intel\OutOfBandManagement;O:\edk2-platforms\F
+eatures\Intel\PowerManagement;O:\edk2-platforms\Features\Intel\SystemIn
+formation;O:\edk2-platforms\Features\Intel\UserInterface
+
+cd \edk2
+
+edksetup.bat Rebuild
+
+build -a IA32 -a X64 -t CLANGPDB -b NOOPT -p
+AdvancedFeaturePkg\AdvancedFeaturePkg.dsc
diff --git a/Features/Intel/Debugging/AcpiDebugFeaturePkg/AcpiDebugFeaturePkg.dsc b/Features/Intel/Debugging/AcpiDebugFeaturePkg/AcpiDebugFeaturePkg.dsc
index 306d8bce16..ffe8c427ff 100644
--- a/Features/Intel/Debugging/AcpiDebugFeaturePkg/AcpiDebugFeaturePkg.dsc
+++ b/Features/Intel/Debugging/AcpiDebugFeaturePkg/AcpiDebugFeaturePkg.d
+++ sc
@@ -34,6 +34,9 @@
#
!include MinPlatformPkg/Include/Dsc/MinPlatformFeaturesPcd.dsc.inc
+[PcdsFeatureFlag.X64]
+ gAcpiDebugFeaturePkgTokenSpaceGuid.PcdUseSmmVersion |FALSE
+
#
# Include common libraries
#
diff --git a/Features/Intel/Debugging/AcpiDebugFeaturePkg/Readme.md b/Features/Intel/Debugging/AcpiDebugFeaturePkg/Readme.md
index 9868d6d816..b6cb054efa 100644
--- a/Features/Intel/Debugging/AcpiDebugFeaturePkg/Readme.md
+++ b/Features/Intel/Debugging/AcpiDebugFeaturePkg/Readme.md
@@ -91,11 +91,10 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-*_TODO_*
-Any special build flows should be described in this section.
-
-This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the -standard flow in the feature package template is used, this section may be empty.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
*_TODO_*
diff --git a/Features/Intel/Debugging/BeepDebugFeaturePkg/Readme.md b/Features/Intel/Debugging/BeepDebugFeaturePkg/Readme.md
index 6626c9b764..143699dbb3 100644
--- a/Features/Intel/Debugging/BeepDebugFeaturePkg/Readme.md
+++ b/Features/Intel/Debugging/BeepDebugFeaturePkg/Readme.md
@@ -120,11 +120,17 @@ Status Code (ReportStatusCode) -> Beep Value (GetBeepValueFromStatusCode).
ReportStatusCode () -> BeepStatusCodeReportWorker () -> GetBeepValueFromStatusCode () -> Beep ()
## Build Flows
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
+
Standalone build
* build -a IA32 -a X64 -p Debugging\BeepDebugFeaturePkg\BeepDebugFeaturePkg.dsc
AdvanceFeaturePkg build
* build -a IA32 -a X64 -p AdvancedFeaturePkg/AdvancedFeaturePkg.dsc
+
## Test Point Results
None
diff --git a/Features/Intel/Debugging/PostCodeDebugFeaturePkg/Readme.md b/Features/Intel/Debugging/PostCodeDebugFeaturePkg/Readme.md
index 3fc52f1c00..d0d8d36fd7 100644
--- a/Features/Intel/Debugging/PostCodeDebugFeaturePkg/Readme.md
+++ b/Features/Intel/Debugging/PostCodeDebugFeaturePkg/Readme.md
@@ -111,7 +111,10 @@ Status Code (ReportStatusCode) -> Post Code (GetPostCodeFromStatusCode).
ReportStatusCode () -> PostCodeStatusCodeReportWorker () -> GetPostCodeFromStatusCode () -> PostCode ()
## Build Flows
-There are not special build flows.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
There are no test points defined.
diff --git a/Features/Intel/Debugging/Usb3DebugFeaturePkg/Readme.md b/Features/Intel/Debugging/Usb3DebugFeaturePkg/Readme.md
index c8afcf9fd2..740bfb6db0 100644
--- a/Features/Intel/Debugging/Usb3DebugFeaturePkg/Readme.md
+++ b/Features/Intel/Debugging/Usb3DebugFeaturePkg/Readme.md
@@ -72,11 +72,10 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-*_TODO_*
-Any special build flows should be described in this section.
-
-This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the -standard flow in the feature package template is used, this section may be empty.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
No test points implemented
diff --git a/Features/Intel/Network/NetworkFeaturePkg/Readme.md b/Features/Intel/Network/NetworkFeaturePkg/Readme.md
index af59fc427a..e953bbcbda 100644
--- a/Features/Intel/Network/NetworkFeaturePkg/Readme.md
+++ b/Features/Intel/Network/NetworkFeaturePkg/Readme.md
@@ -69,11 +69,10 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-*_TODO_*
-Any special build flows should be described in this section.
-
-This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the -standard flow in the feature package template is used, this section may be empty.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
*_TODO_*
diff --git a/Features/Intel/OutOfBandManagement/AsfFeaturePkg/Readme.md b/Features/Intel/OutOfBandManagement/AsfFeaturePkg/Readme.md
index f18b1bfb07..35f5fc202d 100644
--- a/Features/Intel/OutOfBandManagement/AsfFeaturePkg/Readme.md
+++ b/Features/Intel/OutOfBandManagement/AsfFeaturePkg/Readme.md
@@ -44,7 +44,10 @@ AsfPushEvent() -> through SmBus -> NIC -> Remote management consoles N/A now.
## Build Flows
-There is not special build flow.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
There are not test points implemented.
diff --git a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Common/IpmiHooks.c b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Common/IpmiHooks.c
index 86ff7c0fdf..b2788e5a4c 100644
--- a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Common/IpmiHooks.c
+++ b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Comm
+++ on/IpmiHooks.c
@@ -9,6 +9,7 @@
#include "IpmiHooks.h"
EFI_STATUS
+EFIAPI
IpmiSendCommand (
IN IPMI_TRANSPORT *This,
IN UINT8 NetFunction,
@@ -64,6 +65,7 @@ Returns:
} // IpmiSendCommand()
EFI_STATUS
+EFIAPI
IpmiGetBmcStatus (
IN IPMI_TRANSPORT *This,
OUT BMC_STATUS *BmcStatus,
diff --git a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Common/IpmiHooks.h b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Common/IpmiHooks.h
index 083c9e70b0..823cc08c61 100644
--- a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Common/IpmiHooks.h
+++ b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Comm
+++ on/IpmiHooks.h
@@ -15,6 +15,7 @@
// Internal(hook) function list
//
EFI_STATUS
+EFIAPI
IpmiSendCommand (
IN IPMI_TRANSPORT *This,
IN UINT8 NetFunction,
@@ -54,6 +55,7 @@ Returns:
;
EFI_STATUS
+EFIAPI
IpmiGetBmcStatus (
IN IPMI_TRANSPORT *This,
OUT BMC_STATUS *BmcStatus,
diff --git a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Dxe/GenericIpmi.c b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Dxe/GenericIpmi.c
index 957a9c8e6e..d973db3680 100644
--- a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Dxe/GenericIpmi.c
+++ b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/GenericIpmi/Dxe/
+++ GenericIpmi.c
@@ -28,6 +28,7 @@
@retval EFI_UNSUPPORTED IPMI is not available.
**/
EFI_STATUS
+EFIAPI
LocateIpmiInterface (
IN EFI_HANDLE ImageHandle,
IN EFI_SYSTEM_TABLE *SystemTable
@@ -43,4 +44,3 @@ LocateIpmiInterface (
return Status;
} // LocateIpmiInterface()
-
diff --git a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiBaseLib/IpmiBaseLib.c b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiBaseLib/IpmiBaseLib.c
index 6b2e0a9e1e..5df9d861c6 100644
--- a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiBaseLib/IpmiBaseLib.c
+++ b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Library/IpmiBase
+++ Lib/IpmiBaseLib.c
@@ -132,6 +132,7 @@ IpmiSubmitCommand (
**/
EFI_STATUS
+EFIAPI
IpmiGetBmcStatus (
OUT BMC_STATUS *BmcStatus,
OUT SM_COM_ADDRESS *ComAddress
@@ -152,4 +153,3 @@ IpmiGetBmcStatus (
);
return Status;
}
-
diff --git a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Readme.md b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Readme.md
index d383150e3f..7551fcc64f 100644
--- a/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Readme.md
+++ b/Features/Intel/OutOfBandManagement/IpmiFeaturePkg/Readme.md
@@ -58,11 +58,10 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-*_TODO_*
-Any special build flows should be described in this section.
-
-This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the -standard flow in the feature package template is used, this section may be empty.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
*_TODO_*
diff --git a/Features/Intel/OutOfBandManagement/SpcrFeaturePkg/Readme.md b/Features/Intel/OutOfBandManagement/SpcrFeaturePkg/Readme.md
index 66138ee0f6..173fb3e491 100644
--- a/Features/Intel/OutOfBandManagement/SpcrFeaturePkg/Readme.md
+++ b/Features/Intel/OutOfBandManagement/SpcrFeaturePkg/Readme.md
@@ -46,7 +46,10 @@ GetSpcrDevice() -> serial port device path -> get PCI info and serial port info GetSpcrDevice () in SpcrDeviceLib -> SpcrAcpiDxe.
## Build Flows
-There is not special build flow.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
There are not test points implemented.
diff --git a/Features/Intel/PlatformPayloadFeaturePkg/Library/PcdInitLib/PcdInitLib.c b/Features/Intel/PlatformPayloadFeaturePkg/Library/PcdInitLib/PcdInitLib.c
index 4ea89582f3..e3d305d01f 100644
--- a/Features/Intel/PlatformPayloadFeaturePkg/Library/PcdInitLib/PcdInitLib.c
+++ b/Features/Intel/PlatformPayloadFeaturePkg/Library/PcdInitLib/PcdIni
+++ tLib.c
@@ -21,6 +21,7 @@
**/
EFI_STATUS
+EFIAPI
PcdInitConstructor (
VOID
)
@@ -86,4 +87,3 @@ PcdInitConstructor (
return EFI_SUCCESS;
}
-
diff --git a/Features/Intel/PowerManagement/S3FeaturePkg/Readme.md b/Features/Intel/PowerManagement/S3FeaturePkg/Readme.md
index 95ce2ddc15..8f96d29ce1 100644
--- a/Features/Intel/PowerManagement/S3FeaturePkg/Readme.md
+++ b/Features/Intel/PowerManagement/S3FeaturePkg/Readme.md
@@ -63,11 +63,10 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-*_TODO_*
-Any special build flows should be described in this section.
-
-This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the -standard flow in the feature package template is used, this section may be empty.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
*_TODO_*
diff --git a/Features/Intel/SystemInformation/SmbiosFeaturePkg/Readme.md b/Features/Intel/SystemInformation/SmbiosFeaturePkg/Readme.md
index 6ecca8af4b..46d1744414 100644
--- a/Features/Intel/SystemInformation/SmbiosFeaturePkg/Readme.md
+++ b/Features/Intel/SystemInformation/SmbiosFeaturePkg/Readme.md
@@ -65,11 +65,10 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-*_TODO_*
-Any special build flows should be described in this section.
-
-This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the -standard flow in the feature package template is used, this section may be empty.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
*_TODO_*
diff --git a/Features/Intel/TemplateFeaturePkg/Readme.md b/Features/Intel/TemplateFeaturePkg/Readme.md
index fc77340684..a777eccc29 100644
--- a/Features/Intel/TemplateFeaturePkg/Readme.md
+++ b/Features/Intel/TemplateFeaturePkg/Readme.md
@@ -45,11 +45,18 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-Any special build flows should be described in this section.
+Standard and special build flows should be described in this section.
This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the standard flow in the feature package template is used, this section may be empty.
+Example:
+
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
+
## Test Point Results
The test(s) that can verify porting is complete for the feature.
diff --git a/Features/Intel/UserInterface/LogoFeaturePkg/Readme.md b/Features/Intel/UserInterface/LogoFeaturePkg/Readme.md
index bf1a61ba95..33f2ffe410 100644
--- a/Features/Intel/UserInterface/LogoFeaturePkg/Readme.md
+++ b/Features/Intel/UserInterface/LogoFeaturePkg/Readme.md
@@ -78,8 +78,10 @@ Architecturally defined data structures and flows for the feature.
Consumer gets image from the logo driver via EDKII_PLATFORM_LOGO_PROTOCOL.
## Build Flows
-
-No any special build flows is needed.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
diff --git a/Features/Intel/UserInterface/UserAuthFeaturePkg/Readme.md b/Features/Intel/UserInterface/UserAuthFeaturePkg/Readme.md
index b236d2ad30..a5c1d3cb96 100644
--- a/Features/Intel/UserInterface/UserAuthFeaturePkg/Readme.md
+++ b/Features/Intel/UserInterface/UserAuthFeaturePkg/Readme.md
@@ -59,11 +59,10 @@ Architecturally defined data structures and flows for the feature.
Key control flows for the feature.
## Build Flows
-*_TODO_*
-Any special build flows should be described in this section.
-
-This is particularly useful for features that use custom build tools or require non-standard tool configuration. If the -standard flow in the feature package template is used, this section may be empty.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
*_TODO_*
diff --git a/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/Readme.md b/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/Readme.md
index ae7f778748..5e750cd923 100644
--- a/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/Readme.md
+++ b/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/Readme.md
@@ -43,7 +43,10 @@ User touch the touch panel -> absolute pointer and status -> virtual keyboard ev This driver use AbsolutePointer protocol to get the info of user input, and use GraphicsOutput protocol to show the picture.
## Build Flows
-There is not special build flow.
+Supported build targets
+* VS2019
+* CLANGPDB
+* GCC5
## Test Point Results
Test this driver in a device with touch panel and the BIOS support touch panel driver.
diff --git a/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/VirtualKeyboardFeaturePkg.dsc b/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/VirtualKeyboardFeaturePkg.dsc
index d13bd673d3..a025cc755d 100644
--- a/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/VirtualKeyboardFeaturePkg.dsc
+++ b/Features/Intel/UserInterface/VirtualKeyboardFeaturePkg/VirtualKeyb
+++ oardFeaturePkg.dsc
@@ -34,6 +34,10 @@
#
!include MinPlatformPkg/Include/Dsc/MinPlatformFeaturesPcd.dsc.inc
+[PcdsDynamicExDefault.X64]
+ gEfiMdeModulePkgTokenSpaceGuid.PcdConOutRow
+ gEfiMdeModulePkgTokenSpaceGuid.PcdSetupConOutRow
+
#
# Include common libraries
#
--
2.39.0.windows.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [edk2-devel][edk2-platforms][PATCH V1 1/1] Features/Intel: Prepare for edk2-platforms CI
2023-01-10 0:05 ` [edk2-devel][edk2-platforms][PATCH V1 1/1] Features/Intel: Prepare for edk2-platforms CI Isaac Oram
2023-01-11 22:10 ` Chaganty, Rangasai V
@ 2023-01-13 14:46 ` Rebecca Cran
2023-01-18 23:20 ` Isaac Oram
1 sibling, 1 reply; 5+ messages in thread
From: Rebecca Cran @ 2023-01-13 14:46 UTC (permalink / raw)
To: devel, isaac.w.oram
Cc: Sai Chaganty, Nate DeSimone, Liming Gao, Eric Dong, Dandan Bi
On 1/9/23 17:05, Isaac Oram wrote:
> --- /dev/null
> +++ b/Features/Intel/AdvancedFeaturePkg/Readme.md
...
> +## Windows Example:
> +With current working directory at O:\
> +
> +git clone https://github.com/tianocore/edk2.git
> +
> +git clone https://github.com/tianocore/edk2-platforms.git
> +
> +set workspace=O:\
> +
> +set EDK_TOOLS_PATH=O:\Edk2\BaseTools
Would "set workspace=%cd%" work better? That would avoid hard-coding the
assumption that the workspace is on the O: drive.
Then, subsequent paths should be
%workspace%\edk2;%workspace%\edk2-non-osi etc.
> +
> +set packages_path=O:\edk2;O:\edk2-non-osi;O:\edk2-platforms\Platform\Intel;O:\edk2-platforms\Silicon\Intel;O:\edk2-platforms\Platform\Qemu;O:\edk2-platforms\Silicon\Qemu;O:\edk2-platforms\Features\Intel;O:\edk2-platforms\Features\Intel\Debugging;O:\edk2-platforms\Features\Intel\Network;O:\edk2-platforms\Features\Intel\OutOfBandManagement;O:\edk2-platforms\Features\Intel\PowerManagement;O:\edk2-platforms\Features\Intel\SystemInformation;O:\edk2-platforms\Features\Intel\UserInterface
> +
> +cd \edk2
Either here or earlier you probably want to add a step of running "git
submodule update --init" otherwise the Rebuild step below will fail.
--
Rebecca Cran
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [edk2-devel][edk2-platforms][PATCH V1 1/1] Features/Intel: Prepare for edk2-platforms CI
2023-01-13 14:46 ` Rebecca Cran
@ 2023-01-18 23:20 ` Isaac Oram
0 siblings, 0 replies; 5+ messages in thread
From: Isaac Oram @ 2023-01-18 23:20 UTC (permalink / raw)
To: devel@edk2.groups.io, quic_rcran@quicinc.com
New patch here since I had already pushed. https://edk2.groups.io/g/devel/message/98829
Thanks,
Isaac
-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Rebecca Cran
Sent: Friday, January 13, 2023 6:46 AM
To: devel@edk2.groups.io; Oram, Isaac W <isaac.w.oram@intel.com>
Cc: Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>; Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Gao, Liming <gaoliming@byosoft.com.cn>; Dong, Eric <eric.dong@intel.com>; Bi, Dandan <dandan.bi@intel.com>
Subject: Re: [edk2-devel][edk2-platforms][PATCH V1 1/1] Features/Intel: Prepare for edk2-platforms CI
On 1/9/23 17:05, Isaac Oram wrote:
> --- /dev/null
> +++ b/Features/Intel/AdvancedFeaturePkg/Readme.md
...
> +## Windows Example:
> +With current working directory at O:\
> +
> +git clone https://github.com/tianocore/edk2.git
> +
> +git clone https://github.com/tianocore/edk2-platforms.git
> +
> +set workspace=O:\
> +
> +set EDK_TOOLS_PATH=O:\Edk2\BaseTools
Would "set workspace=%cd%" work better? That would avoid hard-coding the assumption that the workspace is on the O: drive.
Then, subsequent paths should be
%workspace%\edk2;%workspace%\edk2-non-osi etc.
> +
> +set
> +packages_path=O:\edk2;O:\edk2-non-osi;O:\edk2-platforms\Platform\Inte
> +l;O:\edk2-platforms\Silicon\Intel;O:\edk2-platforms\Platform\Qemu;O:\
> +edk2-platforms\Silicon\Qemu;O:\edk2-platforms\Features\Intel;O:\edk2-
> +platforms\Features\Intel\Debugging;O:\edk2-platforms\Features\Intel\N
> +etwork;O:\edk2-platforms\Features\Intel\OutOfBandManagement;O:\edk2-p
> +latforms\Features\Intel\PowerManagement;O:\edk2-platforms\Features\In
> +tel\SystemInformation;O:\edk2-platforms\Features\Intel\UserInterface
> +
> +cd \edk2
Either here or earlier you probably want to add a step of running "git submodule update --init" otherwise the Rebuild step below will fail.
--
Rebecca Cran
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-01-18 23:20 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <cover.1673309073.git.isaac.w.oram@intel.com>
2023-01-10 0:05 ` [edk2-devel][edk2-platforms][PATCH V1 1/1] Features/Intel: Prepare for edk2-platforms CI Isaac Oram
2023-01-11 22:10 ` Chaganty, Rangasai V
2023-01-13 0:45 ` Isaac Oram
2023-01-13 14:46 ` Rebecca Cran
2023-01-18 23:20 ` Isaac Oram
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox