[edk2] [Patch] Maintainers.txt: update Vlv2*Pkg maintainers.

2018-11-21 Thread Sun, Zailiang
Cc: David Wei Cc: Yi Qian Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Zailiang Sun --- Maintainers.txt | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Maintainers.txt b/Maintainers.txt index f3ead7c504..9a36f0232b 100644 --- a/Maintainers.t

Re: [edk2] [Patch] Maintainers.txt: update Vlv2*Pkg maintainers.

2018-11-21 Thread Wei, David
Reviewed-by: David Wei david@intel.com Thanks, David Wei Intel SSG Firmware Integration and Development -Original Message- From: Sun, Zailiang Sent: Wednesday, November 21, 2018 5:30 PM To: edk2-devel@lists.01.org Cc: Wei, David ; Qian, Yi ; Sun, Zailiang Subject: [Patch] Main

Re: [edk2] [PATCH 1/6] NetworkPkg: Add DSC/FDF include segment files to NetworkPkg.

2018-11-21 Thread Laszlo Ersek
On 11/21/18 06:28, Fu Siyuan wrote: > The "Network.dsc.inc" and "Network.fdf.inc" are added for platform owner to > easily enable/disable network stack support on their platform, by adding > !include NetworkPkg/Network.dsc.inc > and > !include NetworkPkg/Network.fdf.inc > to their platform DSC/

Re: [edk2] [PATCH 1/6] NetworkPkg: Add DSC/FDF include segment files to NetworkPkg.

2018-11-21 Thread Laszlo Ersek
On 11/21/18 11:46, Laszlo Ersek wrote: > On 11/21/18 06:28, Fu Siyuan wrote: >> The "Network.dsc.inc" and "Network.fdf.inc" are added for platform owner to >> easily enable/disable network stack support on their platform, by adding >> !include NetworkPkg/Network.dsc.inc >> and >> !include Netwo

Re: [edk2] [RFC] Proposal to remove DuetPkg

2018-11-21 Thread Ni, Ruiyu
On 11/21/2018 7:21 AM, Kinney, Michael D wrote: Ray, One additional note. There are some tools in BaseTools that are only used by Duet. We should remove those tools too. Good points. Thanks for the reminding. Mike -- Thanks, Ray ___ edk2-devel

Re: [edk2] [PATCH v2] ArmPkg/ArmSmcPsciResetSystemLib: add missing call to ExitBootServices()

2018-11-21 Thread Ard Biesheuvel
On Tue, 20 Nov 2018 at 17:17, Leif Lindholm wrote: > > On Tue, Nov 20, 2018 at 04:44:33PM +0100, Ard Biesheuvel wrote: > > Our poor man's implementation of EnterS3WithImmediateWake () currently > > sets a high TPL level to disable interrupts, and simply calls the > > PEI entrypoint again after dis

Re: [edk2] [PATCH 5/6] OvmfPkg: Update DSC/FDF to use NetworkPkg's include fragment file.

2018-11-21 Thread Laszlo Ersek
As I said, I wouldn't like to review this patch in detail right now. Just some light comments: On 11/21/18 06:28, Fu Siyuan wrote: > This patch updates the platform DSC/FDF files to use the include fragment > files provided by NetworkPkg. > The feature enabling flags in [Defines] section have been

Re: [edk2] [PATCH 3/6] ArmVirtPkg: Update DSC/FDF to use NetworkPkg's include fragment file.

2018-11-21 Thread Laszlo Ersek
some light comments for now: On 11/21/18 06:28, Fu Siyuan wrote: > This patch updates the platform DSC/FDF files to use the include fragment > files provided by NetworkPkg. > The feature enabling flags in [Defines] section have been updated to use > the NetworkPkg's terms, and the value has been o

Re: [edk2] [PATCH 1/6] NetworkPkg: Add DSC/FDF include segment files to NetworkPkg.

2018-11-21 Thread Fu, Siyuan
Hi, Laszlo > -Original Message- > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Wednesday, November 21, 2018 6:47 PM > To: Fu, Siyuan > Cc: edk2-devel@lists.01.org; Ye, Ting ; Wu, Jiaxin > > Subject: Re: [edk2] [PATCH 1/6] NetworkPkg: Add DSC/FDF include segment > files to Networ

[edk2] [PATCH v2 1/5] ArmPlatformPkg/NorFlashDxe: prepare for devicepath format change

2018-11-21 Thread Ard Biesheuvel
A subsequent patch will change the layout of devicepath nodes produced by this driver. In preparation, make some tweaks to the code to use a packed struct for the devicepath and to pass the device index to NorFlashCreateInstance(). These are cosmetic changes only, the resulting binaries should be i

[edk2] [PATCH v2 3/5] ArmVirtPkg/FdtClientDxe: take DT node 'status' properties into account

2018-11-21 Thread Ard Biesheuvel
DT has a [pseudo-]standardized 'status' property that can be set on any node, and which signifies that a node should be treated as absent unless it is set to 'ok' or 'okay'. So take this into account when iterating over nodes. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by:

[edk2] [PATCH v2 2/5] ArmPlatformPkg/NorFlashDxe: use one GUID plus index to identify flash banks

2018-11-21 Thread Ard Biesheuvel
Currently, each flash bank controlled by ArmPlatformPkg/NorFlashDxe has its own VendorHw GUID, and instances of NorFlashPlatformLib describe each bank to the driver, along with the GUID for each. This works ok for bare metal platforms, but it would be useful for virtual platforms if we could obtai

[edk2] [PATCH v2 0/5] ArmPlatformPkg, ArmVirtPkg: discover NOR flash banks from DTB

2018-11-21 Thread Ard Biesheuvel
This series fixes an issue reported by Hongbo and Philippe, where ArmVirtQemuKernel will crash on an attempt to access flash bank #0, which is secure-only when running QEMU with support for EL3. So let's switch to discovering the NOR flash banks from the device tree instead. This requires some pre

[edk2] [PATCH v2 5/5] ArmPlatformPkg/NorFlashPlatformLib: remove unused Guid member from struct

2018-11-21 Thread Ard Biesheuvel
We no longer use per-instance GUIDs to identify NOR flash banks so there is no longer a need to define them. Drop the Guid member from the NOR_FLASH_DESCRIPTION type. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ard Biesheuvel --- This patch should not be merged before u

[edk2] [PATCH v2 4/5] ArmVirtPkg/NorFlashQemuLib: discover NOR flash banks dynamically

2018-11-21 Thread Ard Biesheuvel
NorFlashQemuLib is one of the last remaining drivers in ArmVirtPkg that are not based on the device tree received from QEMU. For ArmVirtQemu, this does not really matter, given that the NOR flash banks are always the same: the PEI code is linked to execute in place from flash bank #0, and the fixe

[edk2] [PATCH edk2-platforms 3/3] Platform/ARM: drop per-bank NOR flash GUIDs

2018-11-21 Thread Ard Biesheuvel
Remove the GUID definitions from the NOR flash bank structure: the driver no longer uses them. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ard Biesheuvel --- Platform/ARM/JunoPkg/Library/NorFlashJunoLib/NorFlashJuno.c | 2 -- Platform/ARM/SgiPkg/Libr

[edk2] [PATCH edk2-platforms 1/3] Platform/ARM: replace hardcoded VenHW() device paths referring to NOR flash

2018-11-21 Thread Ard Biesheuvel
The ArmPlatformPkg NOR flash driver has been updated to use device paths consisting of a fixed GUID and a numeric index rather than a separate GUID for each flash bank on a given system. This means all explicit device path references to NOR flash banks have to be brought up to date as well. Contri

[edk2] [PATCH edk2-platforms 2/3] Silicon/SynQuacer: drop per-bank NOR flash GUIDs

2018-11-21 Thread Ard Biesheuvel
Give the SynQuacer NOR flash driver the same treatment as we gave the one in ArmPlatformPkg: identify NOR flash banks by a single GUID identifying the driver, and use indexes to identify each device instance owned by the driver. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by

[edk2] [PATCH edk2-platforms 0/3] drop GUIDs from NOR flash bank descriptors

2018-11-21 Thread Ard Biesheuvel
Align edk2-platform with upcoming changes to EDK2 to get rid of per-bank NOR flash GUIDs. Ard Biesheuvel (3): Platform/ARM: replace hardcoded VenHW() device paths referring to NOR flash Silicon/SynQuacer: drop per-bank NOR flash GUIDs Platform/ARM: drop per-bank NOR flash GUIDs Platfor

[edk2] [PATCH v3 edk2-platforms 0/6] Platform/ARM/Sgi: Add support for Clark.Ares and Clark.Helios platforms

2018-11-21 Thread Chandni Cherukuri
Changes since v1: - No code changes, posting this series again with correct patch subject This patch series adds support for two new Arm's SGI platforms - SGI-Clark.Ares and SGI-Clark.Helios. The first patch in this series adds support to use a new binding added to the system-id node for Platform

[edk2] [PATCH v3 edk2-platforms 1/6] Platform/ARM/Sgi: Adapt to changes in system-id DT node.

2018-11-21 Thread Chandni Cherukuri
The 'system-id' node of HW_CONFIG device tree has been updated to have a new property 'config-id' to hold the platform configuration value. To adapt to this change, SGI_PLATFORM_DESCRIPTOR gets a new member named 'ConfigId' and the function GetSgiPlatformId has been renamed to GetSgiSystemId which

[edk2] [PATCH v3 edk2-platforms 2/6] Platform/ARM/Sgi: Refactor ACPI tables for SGI platforms

2018-11-21 Thread Chandni Cherukuri
ACPI tables Dbg2.aslc, Fadt.aslc, Gtdt.aslc, Iort.aslc, Mcfg.aslc, Spcr.aslc and Ssdt.asl are common for SGI platforms. To reduce duplication of ACPI tables when new platforms are added these tables are moved to the AcpiTables folder. Only platform specific tables Dsdt.asl and Madt.aslc remain in t

[edk2] [PATCH v3 edk2-platforms 3/6] Platform/ARM/Sgi: Add ACPI tables for SGI-Clark.Ares platform

2018-11-21 Thread Chandni Cherukuri
Add ACPI tables specific for SGI-Clark.Ares platform and let this platform reuse the rest of the common SGI platform ACPI tables. Cc: Ard Biesheuvel Cc: Leif Lindholm Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Chandni Cherukuri --- Platform/ARM/SgiPkg/SgiPlatform.de

[edk2] [PATCH v3 edk2-platforms 4/6] Platform/ARM/Sgi: Add initial support for SGI-Clark.Ares platform

2018-11-21 Thread Chandni Cherukuri
Add the initial support for ARM's System Guidance for Infrastructure SGI-Clark.Ares platform. The ACPI tables for this platform are installed. Cc: Ard Biesheuvel Cc: Leif Lindholm Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Chandni Cherukuri --- Platform/ARM/SgiPkg/I

[edk2] [PATCH v3 edk2-platforms 5/6] Platform/ARM/Sgi: Add ACPI tables for SGI-Clark.Helios platform

2018-11-21 Thread Chandni Cherukuri
Add ACPI tables specific for SGI-Clark.Helios platform and let this platform reuse the rest of the common SGI platform ACPI tables. Cc: Ard Biesheuvel Cc: Leif Lindholm Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Chandni Cherukuri --- Platform/ARM/SgiPkg/SgiPlatform.

[edk2] [PATCH v3 edk2-platforms 6/6] Platform/ARM/Sgi: Add initial support for SGI-Clark.Helios platform

2018-11-21 Thread Chandni Cherukuri
Add the initial support for ARM's System Guidance for Infrastructure SGI-Clark.Helios platform. The ACPI tables for this platform are installed. Cc: Ard Biesheuvel Cc: Leif Lindholm Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Chandni Cherukuri --- Platform/ARM/SgiPkg

Re: [edk2] [PATCH edk2-platforms 0/3] drop GUIDs from NOR flash bank descriptors

2018-11-21 Thread Thomas Abraham
Hi Ard, On Wed, Nov 21, 2018 at 5:31 PM Ard Biesheuvel wrote: > > Align edk2-platform with upcoming changes to EDK2 to get rid of per-bank > NOR flash GUIDs. > > Ard Biesheuvel (3): > Platform/ARM: replace hardcoded VenHW() device paths referring to NOR > flash > Silicon/SynQuacer: drop p

Re: [edk2] [edk2-test][Patch] uefi-sct/SctPkg:Remove old unused files in EfiCompliant test

2018-11-21 Thread Supreeth Venkatesh
Reviewed-by: Supreeth Venkatesh -Original Message- From: Eric Jin Sent: Tuesday, November 20, 2018 8:13 PM To: edk2-devel@lists.01.org Cc: Supreeth Venkatesh Subject: [edk2-test][Patch] uefi-sct/SctPkg:Remove old unused files in EfiCompliant test Cc: Supreeth Venkatesh Contributed-un

[edk2] [PATCH v2] MdeModulePkg/Variable: add debug logs in VariableServiceSetVariable

2018-11-21 Thread Vijayenthiran Subramaniam
Print debug messages if size of the VariableName plus DataSize exceeds Max(Auth|Voltaile)VariableSize bytes. The messages will be useful if any platform specific value of Max(Auth|Voltaile)VariableSize PCDs have to be changed. Cc: Star Zeng Cc: Jian J Wang Cc: Ruiyu Ni Contributed-under: TianoC

Re: [edk2] [PATCH 1/6] NetworkPkg: Add DSC/FDF include segment files to NetworkPkg.

2018-11-21 Thread Laszlo Ersek
On 11/21/18 12:53, Fu, Siyuan wrote: >> -Original Message- >> From: Laszlo Ersek [mailto:ler...@redhat.com] >> Sent: Wednesday, November 21, 2018 6:47 PM >> To: Fu, Siyuan >> Cc: edk2-devel@lists.01.org; Ye, Ting ; Wu, Jiaxin >> >> Subject: Re: [edk2] [PATCH 1/6] NetworkPkg: Add DSC/FDF i

[edk2] [PATCH v2] NetworkPkg/IScsiDxe: add debug logs for failed SetVariable attempts

2018-11-21 Thread Vijayenthiran Subramaniam
Add debug messages for failed attempts to write to a variable. Cc: Siyuan Fu Cc: Jiaxin Wu Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Vijayenthiran Subramaniam --- Made the log messages bit more useful. Thanks, Vijayenthiran NetworkPkg/IScsiDxe/IScsiMisc.c | 8 ++

[edk2] [PATCH] Platform/ARM/Juno: increase max variable size to 8KB

2018-11-21 Thread Vijayenthiran Subramaniam
Commit dc37ca75 ("Edk2Platforms: Replace MdeModulePkg PXE/iSCSI/TCP with NetworkPkg drivers") switched to using iSCSI driver from the NetworkPkg package. This driver requires the platform to support a maximum variable size of atleast 4KB. So increase the maximum supported variable size to 8KB on t

Re: [edk2] [PATCH v3 0/6] ArmPkg related changes for StandaloneMM package

2018-11-21 Thread Thomas Abraham
Hi Ard, Leif, On Mon, Nov 19, 2018 at 10:39 AM Sughosh Ganu wrote: > > hi Ard, > > On Thu Oct 25, 2018 at 01:02:33PM +0530, Sughosh Ganu wrote: > > The following patch series adds support for Management Mode related > > changes for aarch64 based platforms. > > > > Changes since v2: > > Based on r

Re: [edk2] [PATCH] Platform/ARM/Juno: increase max variable size to 8KB

2018-11-21 Thread Thomas Abraham
On Wed, Nov 21, 2018 at 9:26 PM Vijayenthiran Subramaniam wrote: > > Commit dc37ca75 ("Edk2Platforms: Replace MdeModulePkg PXE/iSCSI/TCP with > NetworkPkg drivers") switched to using iSCSI driver from the NetworkPkg > package. This driver requires the platform to support a maximum variable > size

Re: [edk2] [PATCH v2 2/5] ArmPlatformPkg/NorFlashDxe: use one GUID plus index to identify flash banks

2018-11-21 Thread Laszlo Ersek
On 11/21/18 12:58, Ard Biesheuvel wrote: > Currently, each flash bank controlled by ArmPlatformPkg/NorFlashDxe > has its own VendorHw GUID, and instances of NorFlashPlatformLib > describe each bank to the driver, along with the GUID for each. > > This works ok for bare metal platforms, but it woul

Re: [edk2] [PATCH v2 3/5] ArmVirtPkg/FdtClientDxe: take DT node 'status' properties into account

2018-11-21 Thread Laszlo Ersek
On 11/21/18 12:58, Ard Biesheuvel wrote: > DT has a [pseudo-]standardized 'status' property that can be set on > any node, and which signifies that a node should be treated as > absent unless it is set to 'ok' or 'okay'. (I now really want "oktopus" to be [pseudo-]standardized too! ;) /jk) > So t

Re: [edk2] [PATCH v2 1/5] ArmPlatformPkg/NorFlashDxe: prepare for devicepath format change

2018-11-21 Thread Laszlo Ersek
On 11/21/18 12:58, Ard Biesheuvel wrote: > A subsequent patch will change the layout of devicepath nodes > produced by this driver. In preparation, make some tweaks to > the code to use a packed struct for the devicepath and to pass > the device index to NorFlashCreateInstance(). These are cosmetic

Re: [edk2] [PATCH v2 4/5] ArmVirtPkg/NorFlashQemuLib: discover NOR flash banks dynamically

2018-11-21 Thread Laszlo Ersek
On 11/21/18 12:58, Ard Biesheuvel wrote: > NorFlashQemuLib is one of the last remaining drivers in ArmVirtPkg > that are not based on the device tree received from QEMU. > > For ArmVirtQemu, this does not really matter, given that the NOR > flash banks are always the same: the PEI code is linked t

Re: [edk2] [PATCH v2 5/5] ArmPlatformPkg/NorFlashPlatformLib: remove unused Guid member from struct

2018-11-21 Thread Laszlo Ersek
On 11/21/18 12:58, Ard Biesheuvel wrote: > We no longer use per-instance GUIDs to identify NOR flash banks so > there is no longer a need to define them. Drop the Guid member from > the NOR_FLASH_DESCRIPTION type. > > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Ard Bie

Re: [edk2] [PATCH edk2-platforms 0/3] drop GUIDs from NOR flash bank descriptors

2018-11-21 Thread Ard Biesheuvel
On Wed, 21 Nov 2018 at 14:48, Thomas Abraham wrote: > > Hi Ard, > > On Wed, Nov 21, 2018 at 5:31 PM Ard Biesheuvel > wrote: > > > > Align edk2-platform with upcoming changes to EDK2 to get rid of per-bank > > NOR flash GUIDs. > > > > Ard Biesheuvel (3): > > Platform/ARM: replace hardcoded VenHW

Re: [edk2] [PATCH v3 0/6] ArmPkg related changes for StandaloneMM package

2018-11-21 Thread Ard Biesheuvel
On Wed, 21 Nov 2018 at 17:01, Thomas Abraham wrote: > > Hi Ard, Leif, > > On Mon, Nov 19, 2018 at 10:39 AM Sughosh Ganu wrote: > > > > hi Ard, > > > > On Thu Oct 25, 2018 at 01:02:33PM +0530, Sughosh Ganu wrote: > > > The following patch series adds support for Management Mode related > > > chang

[edk2] Hardcoded IDE Controller B/D/F (0/1/1) in BdsPlatform.c?

2018-11-21 Thread Bryan Rosario via edk2-devel
Hi all, I noticed that there is apparently a hardcoded B/D/F for an IDE Controller in BdsPlatform.c, when setting bit 15 of a particular register: https://github.com/tianocore/edk2/blob/14923c1a6bf9940b48feeaf47cb5d6c662b6528c/OvmfPkg/Library/PlatformBootManagerLib/BdsPlatform.c#L1612-L1617 . Why

Re: [edk2] [PATCH v2] MdeModulePkg/Variable: add debug logs in VariableServiceSetVariable

2018-11-21 Thread Zeng, Star
Reviewed-by: Star Zeng -Original Message- From: Vijayenthiran Subramaniam [mailto:vijayenthiran.subraman...@arm.com] Sent: Wednesday, November 21, 2018 11:29 PM To: edk2-devel@lists.01.org; Zeng, Star ; Wang, Jian J ; Ni, Ruiyu Cc: Vijayenthiran Subramaniam Subject: [PATCH v2] MdeModu

Re: [edk2] Hardcoded IDE Controller B/D/F (0/1/1) in BdsPlatform.c?

2018-11-21 Thread Bryan Rosario via edk2-devel
Hi Ken, That's what I had guessed -- thanks for confirming that it's platform specific. In my use case, I actually want the code to work on multiple hardware configurations, so I can't just replace the hardcoded B/D/F with the one used by a particular platform. Does the change to SataController.c

Re: [edk2] Hardcoded IDE Controller B/D/F (0/1/1) in BdsPlatform.c?

2018-11-21 Thread Ken Taylor
Hi Bryan, If you were talking about PC architecture, I would suggest that there should be a hierarchy of configuration that needs to occur. Specifically, there should be silicon / board / platform specific configuration that is architecturally independent of the SATA driver. Normally, there w

[edk2] [PATCH] IntelFsp2WrapperPkg: Fix line ending format issue

2018-11-21 Thread Chasel, Chiu
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1352 Fixed line ending format wrong issues on some files. Test: Verified building successfully. Cc: Jiewen Yao Cc: Desimone Nathaniel L Cc: Wu Hao A Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Chasel Chiu --- Int

Re: [edk2] Hardcoded IDE Controller B/D/F (0/1/1) in BdsPlatform.c?

2018-11-21 Thread Bryan Rosario via edk2-devel
This is for a virtual x86 platform using OVMF. I think I have enough info at this point to get things set up. Thanks for the help. Bryan On Wed, Nov 21, 2018 at 6:06 PM Ken Taylor wrote: > Hi Bryan, > > > > If you were talking about PC architecture, I would suggest that there > should be a hier

[edk2] [PATCH v2 2/2] IntelFsp2WrapperPkg: Fix constant if statements issue

2018-11-21 Thread Chasel, Chiu
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1351 Internal code quality scanning found 2 constant if statements related to FixedPcdGet8 () usage. Since the PCD can be PatchableInModule too, it should be changed to PcdGet8 () to fix this issue. Test: Verified on internal platform and booted

[edk2] [PATCH v2 0/2] Fix constant if statement issue

2018-11-21 Thread Chasel, Chiu
Internal code quality scanning found 2 constant if statements related to FixedPcdGet8 () usage. Since the PCD can be PatchableInModule too, it should be changed to PcdGet8 () to fix this issue. Also fixed the line ending format issue from previous patch. Patch1 fixed line ending format issue on so

[edk2] [PATCH v2 1/2] IntelFsp2WrapperPkg: Fix line ending format issue

2018-11-21 Thread Chasel, Chiu
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1352 Fixed line ending format wrong issues on some files. Test: Verified building successfully. Cc: Jiewen Yao Cc: Desimone Nathaniel L Cc: Wu Hao A Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Chasel Chiu --- Int

Re: [edk2] [PATCH edk2-platforms 0/3] drop GUIDs from NOR flash bank descriptors

2018-11-21 Thread Thomas Abraham
Hi Ard, On Thu, Nov 22, 2018 at 3:46 AM Ard Biesheuvel wrote: > > On Wed, 21 Nov 2018 at 14:48, Thomas Abraham wrote: > > > > Hi Ard, > > > > On Wed, Nov 21, 2018 at 5:31 PM Ard Biesheuvel > > wrote: > > > > > > Align edk2-platform with upcoming changes to EDK2 to get rid of per-bank > > > NOR

Re: [edk2] [PATCH v2 2/2] IntelFsp2WrapperPkg: Fix constant if statements issue

2018-11-21 Thread Wu, Hao A
> -Original Message- > From: Chiu, Chasel > Sent: Thursday, November 22, 2018 11:51 AM > To: edk2-devel@lists.01.org > Cc: Yao, Jiewen; Desimone, Nathaniel L; Wu, Hao A; Chiu, Chasel > Subject: [PATCH v2 2/2] IntelFsp2WrapperPkg: Fix constant if statements > issue > > REF: https://bugzilla

[edk2] [PATCH v2 2/6] Nt32Pkg: Update DSC/FDF to use NetworkPkg's include fragment file.

2018-11-21 Thread Fu Siyuan
This patch updates the platform DSC/FDF files to use the include fragment files provided by NetworkPkg. The feature enabling flags in [Defines] section have be updated to use the NetworkPkg's terms, and the value have been overridden with the original default value on this platform. Cc: Ruiyu Ni

[edk2] [PATCH v2 1/6] NetworkPkg: Add DSC/FDF include segment files to NetworkPkg.

2018-11-21 Thread Fu Siyuan
This patch provides a set of include segment files for platform owner to easily enable/disable network stack support on their platform. For DSC, there are: - a "NetworkDefines.dsc.inc" for the [Defines] section(s), - a "NetworkLibs.dsc.inc" for the [LibraryClasses*] section(s), - a "NetworkPcds.ds

[edk2] [PATCH v2 4/6] EmulatorPkg: Update DSC/FDF to use NetworkPkg's include fragment file.

2018-11-21 Thread Fu Siyuan
This patch updates the platform DSC/FDF files to use the include fragment files provided by NetworkPkg. The feature enabling flags in [Defines] section have been updated to use the NetworkPkg's terms, and the value has been overridden with the original default value on this platform. Cc: Jordan Ju

[edk2] [PATCH v2 6/6] Vlv2TbltDevicePkg: Update DSC/FDF to use NetworkPkg's include fragment file.

2018-11-21 Thread Fu Siyuan
This patch updates the platform DSC/FDF files to use the include fragment files provided by NetworkPkg. The feature enabling flags in [Defines] section have been updated to use the NetworkPkg's terms, and the value has been overridden with the original default value on this platform. Cc: David Wei

[edk2] [PATCH v2 0/6] Add DSC/FDF include segment files for network stack

2018-11-21 Thread Fu Siyuan
There is a patch to remove the redudant IP4 only iSCSI/PXE/TCP drivers from MdeModulePkg, which has been reviewed before edk2-stable201811 tag. And we also have plan to move all network related libraries/modules to NetworkPkg. In order to make these change more smoothly, a set of fragment files (2

[edk2] [PATCH v2 3/6] ArmVirtPkg: Update DSC/FDF to use NetworkPkg's include fragment file.

2018-11-21 Thread Fu Siyuan
This patch updates the platform DSC/FDF files to use the include fragment files provided by NetworkPkg. The feature enabling flags in [Defines] section have been updated to use the NetworkPkg's terms, and the value has been overridden with the original default value on this platform. Cc: Laszlo Er

[edk2] [PATCH v2 5/6] OvmfPkg: Update DSC/FDF to use NetworkPkg's include fragment file.

2018-11-21 Thread Fu Siyuan
This patch updates the platform DSC/FDF files to use the include fragment files provided by NetworkPkg. The feature enabling flags in [Defines] section have been updated to use the NetworkPkg's terms, and the value has been overridden with the original default value on this platform. This patch al

Re: [edk2] [PATCH v2 0/6] Add DSC/FDF include segment files for network stack

2018-11-21 Thread Gao, Liming
Siyuan: Thanks for your contribution. I really like this idea to share the common DSC/FDF between all platform DSC/FDFs. Now, FixedAtBuild PCD can also be used in the conditional statement. Its value can be specified by build command with --pcd option. So, I suggest to use FixedAtBuild PCD fo

[edk2] [PATCH 0/2] Remove DuetPkg and unused tools

2018-11-21 Thread Shenglei Zhang
DuetPkg depends on Legacy BIOS to provide a UEFI environment. It was invented in the era when UEFI environment is hard to find. Since now UEFI is very popular in PC area, we could stop the official support of this package and remove it from the master. And moreover, the tools only used by DuetPkg c

Re: [edk2] [PATCH edk2-platforms 0/3] drop GUIDs from NOR flash bank descriptors

2018-11-21 Thread Ard Biesheuvel
On Thu, 22 Nov 2018 at 05:01, Thomas Abraham wrote: > > Hi Ard, > > On Thu, Nov 22, 2018 at 3:46 AM Ard Biesheuvel > wrote: > > > > On Wed, 21 Nov 2018 at 14:48, Thomas Abraham wrote: > > > > > > Hi Ard, > > > > > > On Wed, Nov 21, 2018 at 5:31 PM Ard Biesheuvel > > > wrote: > > > > > > > > Ali