From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 134.134.136.126, mailfrom: liming.gao@intel.com) Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by groups.io with SMTP; Tue, 07 May 2019 07:27:28 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga004.jf.intel.com ([10.7.209.38]) by orsmga106.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 07 May 2019 07:27:27 -0700 X-ExtLoop1: 1 Received: from lgao4-mobl1.ccr.corp.intel.com ([10.255.29.55]) by orsmga004.jf.intel.com with ESMTP; 07 May 2019 07:27:26 -0700 From: "Liming Gao" To: devel@edk2.groups.io Subject: [Patch v5 0/3] Add package level include DSC/FDF in NetworkPkg Date: Tue, 7 May 2019 22:27:02 +0800 Message-Id: <20190507142705.20092-1-liming.gao@intel.com> X-Mailer: git-send-email 2.16.2.windows.1 v5: Update Network.fdf.inc to include the missing IScsiDxe Update NetworkLibs.dsc.inc to add the comments for HttpLib v4: Move library instance BaseStackCheckLib to ARM, AARCH64 arch in Patch 1/3 Move PLATFORMX64_ENABLE build flag to package level include DSC file Correct typo in Patch 3/3 BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1293 BZ 1293 requests to move Network modules from MdeModulePkg to NetworkPkg. To keep the backword compatiblity, Network package level include DSC/FDF are introduced to be used in the platform DSC/FDF files. When Network modules are moved from MdeModulePkg to NetworkPkg, Network package level include DSC/FDF will be updated together. There is no impact on the platform DSC/FDF file. This patch set is to introduce network package level include DSC/FDF files. Bases on previous discussion and the existing usage case, build flag will be used to enable/disable the network features. PCD control feature way can be discussed later. And, to meet with the different usages, this patch set introduces the separate DSC for Defines/Pcds/Libraries/Components (Patch 2) , and also adds the package level combined DSC to include them all (Patch 3). If platform wants to use the flexible way to enable Network feature, it can use the separate DSCs. If the platform wants to directly enable Network feature, it can use the combined package DSC file. This patch set is to update NetworkPkg only. If there is no objection on this proposal, the following changes to platform DSC/FDF will be made and sent for review. By default, the platform DSC/FDF will use the combined DSC/FDF file. If the platform owner wants to use the flexible way to enable Network feature, please reply this mail. Feedback from Laszlo, OvmfPkg and ArmVirtPkg platforms to use the standalone include files from patch #2. Liming Gao (3): NetworkPkg DSC: Add the required ARM library to pass ARM build NetworkPkg: Add DSC/FDF include segment files to NetworkPkg. NetworkPkg: Add package level include DSC file NetworkPkg/Network.dsc.inc | 40 ++++++++++++ NetworkPkg/Network.fdf.inc | 60 ++++++++++++++++++ NetworkPkg/NetworkComponents.dsc.inc | 61 ++++++++++++++++++ NetworkPkg/NetworkDefines.dsc.inc | 118 +++++++++++++++++++++++++++++++++++ NetworkPkg/NetworkLibs.dsc.inc | 20 ++++++ NetworkPkg/NetworkPcds.dsc.inc | 16 +++++ NetworkPkg/NetworkPkg.dsc | 24 +------ 7 files changed, 318 insertions(+), 21 deletions(-) create mode 100644 NetworkPkg/Network.dsc.inc create mode 100644 NetworkPkg/Network.fdf.inc create mode 100644 NetworkPkg/NetworkComponents.dsc.inc create mode 100644 NetworkPkg/NetworkDefines.dsc.inc create mode 100644 NetworkPkg/NetworkLibs.dsc.inc create mode 100644 NetworkPkg/NetworkPcds.dsc.inc -- 2.13.0.windows.1