Browse Source

WhitleyOpenBoardPkg/Build: Reduce unnecessary code

Use MinPlatformFeaturesPcd.dsc.inc to initialize feature PCD
Move individual feature enabling examples to BoardPortTemplate

Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
Cc: Chasel Chiu <chasel.chiu@intel.com>
Signed-off-by: Isaac Oram <isaac.w.oram@intel.com>
Reviewed-by: Nate DeSimone <nathaniel.l.desimone@intel.com>
Isaac Oram 1 year ago
parent
commit
1f0f52230b

+ 0 - 3
Platform/Intel/WhitleyOpenBoardPkg/Aowanda/PlatformPkg.dsc

@@ -26,9 +26,6 @@
 !if $(gMinPlatformPkgTokenSpaceGuid.PcdBootStage) >= 5
   gIpmiFeaturePkgTokenSpaceGuid.PcdIpmiFeatureEnable        |TRUE
   gNetworkFeaturePkgTokenSpaceGuid.PcdNetworkFeatureEnable  |TRUE
-!else
-  gIpmiFeaturePkgTokenSpaceGuid.PcdIpmiFeatureEnable        |FALSE
-  gNetworkFeaturePkgTokenSpaceGuid.PcdNetworkFeatureEnable  |FALSE
 !endif
 
   !include AdvancedFeaturePkg/Include/AdvancedFeatures.dsc

+ 50 - 0
Platform/Intel/WhitleyOpenBoardPkg/BoardPortTemplate/PlatformPkg.dsc

@@ -15,6 +15,56 @@
 
 !include $(RP_PKG)/PlatformPkg.dsc
 
+#
+# Feature enable/disable flags
+#
+[PcdsFeatureFlag]
+  #
+  # MinPlatform control flags
+  #
+  gMinPlatformPkgTokenSpaceGuid.PcdStopAfterDebugInit                       |FALSE
+  gMinPlatformPkgTokenSpaceGuid.PcdStopAfterMemInit                         |FALSE
+  gMinPlatformPkgTokenSpaceGuid.PcdBootToShellOnly                          |FALSE
+  gMinPlatformPkgTokenSpaceGuid.PcdPerformanceEnable                        |TRUE
+
+  #
+  # Debugging features
+  #
+  gAcpiDebugFeaturePkgTokenSpaceGuid.PcdAcpiDebugFeatureEnable              |FALSE
+  gBeepDebugFeaturePkgTokenSpaceGuid.PcdBeepDebugFeatureEnable              |FALSE
+  gPostCodeDebugFeaturePkgTokenSpaceGuid.PcdPostCodeDebugFeatureEnable      |FALSE
+  gUsb3DebugFeaturePkgTokenSpaceGuid.PcdUsb3DebugFeatureEnable              |FALSE
+
+  #
+  # Network features
+  #
+  gNetworkFeaturePkgTokenSpaceGuid.PcdNetworkFeatureEnable                  |FALSE
+
+  #
+  # OutOfBandManagement features
+  #
+  gIpmiFeaturePkgTokenSpaceGuid.PcdIpmiFeatureEnable                        |FALSE
+  gSpcrFeaturePkgTokenSpaceGuid.PcdSpcrFeatureEnable                        |FALSE
+
+  #
+  # PowerManagement features
+  #
+  gS3FeaturePkgTokenSpaceGuid.PcdS3FeatureEnable                            |FALSE
+
+  #
+  # SystemInformation features
+  #
+  gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosFeatureEnable                    |FALSE
+
+  #
+  # UserInterface features
+  #
+  gLogoFeaturePkgTokenSpaceGuid.PcdLogoFeatureEnable                        |FALSE
+  gUserAuthFeaturePkgTokenSpaceGuid.PcdUserAuthenticationFeatureEnable      |FALSE
+  gVirtualKeyboardFeaturePkgTokenSpaceGuid.PcdVirtualKeyboardFeatureEnable  |FALSE
+
+!include AdvancedFeaturePkg/Include/AdvancedFeatures.dsc
+
 [Defines]
   DEFINE BOARD_NAME                   = BoardPortTemplate
   PLATFORM_GUID                       = f3518dd4-5dec-4d2c-9ac9-561121e2628b

+ 0 - 3
Platform/Intel/WhitleyOpenBoardPkg/JunctionCity/PlatformPkg.dsc

@@ -26,9 +26,6 @@
 !if $(gMinPlatformPkgTokenSpaceGuid.PcdBootStage) >= 5
   gIpmiFeaturePkgTokenSpaceGuid.PcdIpmiFeatureEnable        |TRUE
   gNetworkFeaturePkgTokenSpaceGuid.PcdNetworkFeatureEnable  |TRUE
-!else
-  gIpmiFeaturePkgTokenSpaceGuid.PcdIpmiFeatureEnable        |FALSE
-  gNetworkFeaturePkgTokenSpaceGuid.PcdNetworkFeatureEnable  |FALSE
 !endif
 
   !include AdvancedFeaturePkg/Include/AdvancedFeatures.dsc

+ 0 - 51
Platform/Intel/WhitleyOpenBoardPkg/PlatformPkg.dsc

@@ -59,57 +59,6 @@
 [PcdsFixedAtBuild]
   gUsb3DebugFeaturePkgTokenSpaceGuid.PcdUsb3DebugPortLibInstance|1
 
-#
-# Feature enable/disable flags
-#
-[PcdsFeatureFlag]
-  #
-  # MinPlatform control flags
-  #
-  gMinPlatformPkgTokenSpaceGuid.PcdStopAfterDebugInit                       |FALSE
-  gMinPlatformPkgTokenSpaceGuid.PcdStopAfterMemInit                         |FALSE
-  gMinPlatformPkgTokenSpaceGuid.PcdBootToShellOnly                          |FALSE
-  gMinPlatformPkgTokenSpaceGuid.PcdSmiHandlerProfileEnable                  |TRUE
-  gMinPlatformPkgTokenSpaceGuid.PcdPerformanceEnable                        |TRUE
-
-  #
-  # Debugging features
-  #
-  gAcpiDebugFeaturePkgTokenSpaceGuid.PcdAcpiDebugFeatureEnable              |FALSE
-  gBeepDebugFeaturePkgTokenSpaceGuid.PcdBeepDebugFeatureEnable              |FALSE
-  gPostCodeDebugFeaturePkgTokenSpaceGuid.PcdPostCodeDebugFeatureEnable      |FALSE
-  gUsb3DebugFeaturePkgTokenSpaceGuid.PcdUsb3DebugFeatureEnable              |FALSE
-
-  #
-  # Network features
-  #
-  gNetworkFeaturePkgTokenSpaceGuid.PcdNetworkFeatureEnable                  |FALSE
-
-  #
-  # OutOfBandManagement features
-  #
-  gIpmiFeaturePkgTokenSpaceGuid.PcdIpmiFeatureEnable                        |FALSE
-  gSpcrFeaturePkgTokenSpaceGuid.PcdSpcrFeatureEnable                        |FALSE
-
-  #
-  # PowerManagement features
-  #
-  gS3FeaturePkgTokenSpaceGuid.PcdS3FeatureEnable                            |FALSE
-
-  #
-  # SystemInformation features
-  #
-  gSmbiosFeaturePkgTokenSpaceGuid.PcdSmbiosFeatureEnable                    |FALSE
-
-  #
-  # UserInterface features
-  #
-  gLogoFeaturePkgTokenSpaceGuid.PcdLogoFeatureEnable                        |FALSE
-  gUserAuthFeaturePkgTokenSpaceGuid.PcdUserAuthenticationFeatureEnable      |FALSE
-  gVirtualKeyboardFeaturePkgTokenSpaceGuid.PcdVirtualKeyboardFeatureEnable  |FALSE
-
-!include AdvancedFeaturePkg/Include/AdvancedFeatures.dsc
-
   #
   # Platform On/Off features are defined here
   #