[edk2] [Patch 3/3] MdeModulePkg: Update Guid/Protocol usages in INF files.

Cc: Star Zeng Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao --- .../Library/VarCheckUefiLib/VarCheckUefiLib.inf| 78 +++--- .../Variable/RuntimeDxe/VariableRuntimeDxe.inf | 10 +-- 2 files changed, 44 insertions(+), 44 deletions(-) dif

[edk2] [Patch 1/3] SecurityPkg: Update protocol usage in module INF files.

Update TCG and Library module uses gEdkiiVariableLockProtocolGuid as SOMETIMES_CONSUMES instead of CONSUMES to follow the code logic. Cc: Chao Zhang Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao --- .../DxeTcg2PhysicalPresenceLib/DxeTcg2PhysicalPresenceLib.in

[edk2] [Patch 2/3] ShellPkg: Update Guid/Protocol usages in INF files.

Cc: Jaben Carsey Cc: Shumin Qiu Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Liming Gao --- ShellPkg/Application/Shell/Shell.inf | 10 - .../UefiHandleParsingLib/UefiHandleParsingLib.inf | 24 +++--- .../UefiShellCommandLib/UefiSh

[edk2] [Patch 0/3] Update Guid/Protocol usages to match code logic.

Liming Gao (3): SecurityPkg: Update protocol usage in module INF files. ShellPkg: Update Guid/Protocol usages in INF files. MdeModulePkg: Update Guid/Protocol usages in INF files. .../Library/VarCheckUefiLib/VarCheckUefiLib.inf| 78 +++--- .../Variable/RuntimeDxe/Variabl

Re: [edk2] [PATCH v2] Merge 2-clause BSD licensed FatPkg

Pushed as 2777ed7068f0ce186761f5cfef8a0c2252e3e2a5. Cheers! -Jordan On 2016-04-04 21:43:01, Jordan Justen wrote: > === > > web: https://github.com/jljusten/edk2/tree/fatpkg-open-source-v2 > > git: https://github.com/jljusten/edk2.git fatpkg-open-source-v2 > > In this commit 4cefb4cd738fc2bba9

Re: [edk2] [PATCH 03/12] FatBinPkg: Change to 2-clause BSD license

(How about an r-b for some of the patches I Cc'd you on? :) On 2016-04-06 19:11:05, Ni, Ruiyu wrote: >My understanding is: > >Shell is a very standalone project so the binary package was >created for a certain level of isolation. And it does save build >time. It also assumes that

[edk2] [Patch 1/2] BaseTools/VolInfo: generate HASH value for each PE image

VolInfo Tool add new option --hash to use openssl to generate hash value for each PE image. If the image base address is not zero, we will rebase its base address to zero before generate hash value. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools

[edk2] [Patch 0/2] Tool generate hash value for each output EFI image

Build Tool add new report type 'HASH' to generate hash value in the build report. VolInfo Tool add new option --hash to use openssl to generate hash value for each PE image. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu Yonghong Zhu (2): BaseTools/VolInfo:

[edk2] [Patch 2/2] BaseTools: generate hash value in build report for each output EFI image

Build report add new report type 'HASH' to include the hash value for each output EFI image. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/build/BuildReport.py | 53 +++- BaseTools/Source/Python/build/build

[edk2] [Patch] SecurityPkg OpalPasswordDxe: Suppress option for special device.

According to current Pyrite SSC Spec 1.00, there is no parameter for RevertSP method. So suppress KeepUserData option for the Pyrite Ssc type device. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Eric Dong Cc: Feng Tian --- SecurityPkg/Tcg/Opal/OpalPasswordDxe/OpalHii.c

Re: [edk2] [Patch] BaseTools: Add support to merge Prebuild and Postbuild into build Process

Reviewed-by: Liming Gao > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Yonghong Zhu > Sent: Tuesday, April 05, 2016 3:04 PM > To: edk2-devel@lists.01.org > Subject: [edk2] [Patch] BaseTools: Add support to merge Prebuild and > Postbuild int

Re: [edk2] [Patch] BaseTools: Enhance --Pcd which override by build option

Reviewed-by: Liming Gao > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Yonghong Zhu > Sent: Tuesday, April 05, 2016 12:32 PM > To: edk2-devel@lists.01.org > Subject: [edk2] [Patch] BaseTools: Enhance --Pcd which override by build > option >

Re: [edk2] [Patch] MdeModulePkg/Bds: Fix build failures of VS tool chain

Reviewed-by: Qiu Shumin -Original Message- From: Ni, Ruiyu Sent: Thursday, April 07, 2016 10:14 AM To: edk2-devel@lists.01.org Cc: Ni, Ruiyu; Qiu, Shumin Subject: [Patch] MdeModulePkg/Bds: Fix build failures of VS tool chain Contributed-under: TianoCore Contribution Agreement 1.0 Signed

Re: [edk2] [Patch] MdeModulePkg/Bds: Fix build failures of VS tool chain

Thanks for fixing this. Reviewed-by: Sunny Wang -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ruiyu Ni Sent: Thursday, April 07, 2016 10:14 AM To: edk2-devel@lists.01.org Cc: Ruiyu Ni ; Shumin Qiu Subject: [edk2] [Patch] MdeModulePkg/Bds: Fix b

[edk2] [Patch] MdeModulePkg/Bds: Fix build failures of VS tool chain

Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Shumin Qiu --- MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c | 1 + MdeModulePkg/Library/UefiBootManagerLib/BmMisc.c | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/MdeModulePkg/Library/U

Re: [edk2] [PATCH 03/12] FatBinPkg: Change to 2-clause BSD license

My understanding is: Shell is a very standalone project so the binary package was created for a certain level of isolation. And it does save build time. It also assumes that most of the users don't need to source debug the Shell. Fat is similar to Shell. But I agree the build time saving is not no

Re: [edk2] [PATCH V2 0/8] Move S3Ready() functional code from AcpiS3SaveDxe to S3SaveStateDxe

MdeModelPkg/IntelFrameworkModulePkg reviewed by: jiewen@intel.com > -Original Message- > From: Zeng, Star > Sent: Wednesday, April 6, 2016 8:40 PM > To: edk2-devel@lists.01.org > Cc: Yao, Jiewen ; Tian, Feng ; > Fan, Jeff ; Laszlo Ersek ; Justen, > Jordan L > Subject: [PATCH V2 0/8] M

Re: [edk2] [PATCH] SecurityPkg/Tpm12CommandLib: Fix Tpm12NvWriteValue cmd.

Reviewed by: jiewen@intel.com > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Derek Lin > Sent: Thursday, April 7, 2016 9:55 AM > To: edk2-devel@lists.01.org > Cc: Derek Lin > Subject: [edk2] [PATCH] SecurityPkg/Tpm12CommandLib: Fix > Tp

[edk2] [PATCH] SecurityPkg/Tpm12CommandLib: Fix Tpm12NvWriteValue cmd.

Fix command length endian. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Derek Lin --- SecurityPkg/Library/Tpm12CommandLib/Tpm12NvStorage.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/SecurityPkg/Library/Tpm12CommandLib/Tpm12NvStorage.

[edk2] [PATCH] EmbeddedPkg: Add GICD table init macro for ACPI 6.0

Add macro to help initialize GICD structure in MADT table according to ACPI 6.0. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Heyi Guo Cc: Leif Lindholm Cc: Ard Biesheuvel --- EmbeddedPkg/Include/Library/AcpiLib.h | 7 +++ 1 file changed, 7 insertions(+) diff --g

Re: [edk2] [PATCH V2 0/8] Move S3Ready() functional code from AcpiS3SaveDxe to S3SaveStateDxe

On 2016/4/7 0:04, Laszlo Ersek wrote: Hi Star, On 04/06/16 14:40, Star Zeng wrote: The S3Ready() functional code in AcpiS3SaveDxe of IntelFrameworkModulePkg is to do ACPI S3 Context save. In fact, that is not really related to Intel framework ACPI S3 protocol. IntelFrameworkModulePkg will be d

Re: [edk2] [PATCH V2 3/8] OvmfPkg: Install LockBox protocol in constructor of LockBoxDxeLib

The comments below are fair enough, accepted. Thanks, Star On 2016/4/7 0:24, Laszlo Ersek wrote: On 04/06/16 14:40, Star Zeng wrote: Currently, the LockBox protocol is installed in entrypoint of OVMF AcpiS3SaveDxe, that is strange. (6) please remove the phrase that is strange It is not

Re: [edk2] [PATCH V2 2/8] OvmfPkg: Set PcdAcpiS3Enable according to QemuFwCfgS3Enabled()

On 2016/4/7 0:16, Laszlo Ersek wrote: On 04/06/16 14:40, Star Zeng wrote: Also need to declare PcdAcpiS3Enable as DynamicDefault in *.dsc. Cc: Laszlo Ersek Cc: Jordan Justen Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng --- OvmfPkg/OvmfPkg

[edk2] Building a SMM_CORE module as an XIP image

Hi All, I am prototyping the proposal made in ECR #1390 (MM in Standalone mode etc) on the Juno and FVP ARM development platforms. The prototype mainly has a module of type SMM_CORE that contains: a. The MM Foundation code in Standalone mode. Lets call the entry point of this module SmmMain().

Re: [edk2] [PATCH 04/21] ArmVirtPkg/FdtClientDxe: implement new driver

On 04/06/16 18:15, Ard Biesheuvel wrote: > This implements a new DXE driver FdtClientDxe to produce the FDT client > protocol based on a device tree image supplied by the virt host. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Ard Biesheuvel > --- > ArmVirtPkg/FdtC

Re: [edk2] [PATCH 03/21] ArmVirtPkg: drop dependency on PeiPcdLib for PEI Pcd.inf

On 04/06/16 18:15, Ard Biesheuvel wrote: > The PcdPeim dynamic PCD driver is dispatched explicitly via an 'A PRIORI' > declaration in the platform DSC. Without that declaration, the PEI module > can never be dispatched since it transitively depends on protocols it (4) s/protocols/the PPI/ with th

Re: [edk2] [PATCH 02/21] ArmVirtPkg: drop dependency on DxePcdLib for DXE Pcd.inf

On 04/06/16 18:15, Ard Biesheuvel wrote: > The PcdDxe dynamic PCD driver is dispatched explicitly via an 'A PRIORI' > declaration in the platform DSC. Without that declaration, the DXE driver > can never be dispatched since it transitively depends on protocols it > produces itself via DxePcdLib. So

Re: [edk2] [PATCH 01/21] ArmVirtPkg: introduce FdtClientProtocol

On 04/06/16 18:14, Ard Biesheuvel wrote: > This introduces the FdtClientProtocol, which will be used to expose the > device tree provided by the host to other DXE drivers. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Ard Biesheuvel > --- > ArmVirtPkg/ArmVirtPkg.de

Re: [edk2] [PATCH 11/12] OvmfPkg: Convert to using FatPkg in the EDK II tree

On 04/06/16 17:15, Jordan Justen wrote: > Now that FatPkg is open source (and therefore can be included in the > EDK II tree) we build and use it directly. > > Build tested with GCC 5.3 on IA32 and X64. Boot tested to UEFI Shell > on IA32 and UEFI Linux on X64. > > Contributed-under: TianoCore Co

Re: [edk2] [PATCH 04/12] ArmVirtPkg: Convert to build FatPkg from source

On 04/06/16 17:14, Jordan Justen wrote: > Now that FatPkg is open source (and therefore can be included in the > EDK II tree) we build and use it directly. > > Note: Not build tested > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Jordan Justen > Cc: Laszlo Ersek >

Re: [edk2] [PATCH 03/12] FatBinPkg: Change to 2-clause BSD license

On 04/06/16 17:14, Jordan Justen wrote: > The FatPkg is now open source. Update FatBinPkg license to match > FatPkg. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Jordan Justen > Cc: Ruiyu Ni > --- > FatBinPkg/FatBinPkg.dec | 2 +- > FatBinPkg/License.txt | 53

Re: [edk2] [PATCH 02/12] FatPkg: Add Contributions.txt

On 04/06/16 17:14, Jordan Justen wrote: > This is copied from MdePkg/Contributions.txt > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Jordan Justen > Cc: Ruiyu Ni > --- > FatPkg/Contributions.txt | 218 > +++ > 1 file ch

Re: [edk2] [PATCH 01/12] FatPkg: Update License.txt to have the full license text

On 04/06/16 17:14, Jordan Justen wrote: > The script replaced the contents of License.txt will the words the s/will the words/with the words of/ > shortened text that we include in the headers to files. Instead, use s/headers to/source/ > the full text. This is copied from MdePkg/License.txt.

Re: [edk2] [PATCH 1/2] OvmfPkg: disable PcdHiiOsRuntimeSupport

On 04/06/16 19:32, Jordan Justen wrote: > Reviewed-by: Jordan Justen Commit eccc28bfcb91b5d72adf173d7404652c7aa63c26. Thank you! Laszlo > > On 2016-03-29 10:51:29, Laszlo Ersek wrote: >> Edk2 commit 8a45f80edad4 ("MdeModulePkg: Make HII configuration settings >> available to OS runtime") imple

Re: [edk2] [PATCH] OvmfPkg: remove PcdMaxHardwareErrorVariableSize from the DSC files

On 04/06/16 19:34, Jordan Justen wrote: > Reviewed-by: Jordan Justen Commit 29d0259e06ee0d0772d1068e02fb2242542fad2a. Thanks! Laszlo > On 2016-03-25 04:02:25, Laszlo Ersek wrote: >> PcdMaxHardwareErrorVariableSize sets the size limit for individual >> Hardware Error Record Variables (see "7.2.3

Re: [edk2] [PATCH] OvmfPkg: remove PcdMaxHardwareErrorVariableSize from the DSC files

Reviewed-by: Jordan Justen On 2016-03-25 04:02:25, Laszlo Ersek wrote: > PcdMaxHardwareErrorVariableSize sets the size limit for individual > Hardware Error Record Variables (see "7.2.3 Hardware Error Record > Persistence" and "Appendix P, Hardware Error Record Persistence Usage" in > the UEFI-2.

Re: [edk2] [PATCH 1/2] OvmfPkg: disable PcdHiiOsRuntimeSupport

Reviewed-by: Jordan Justen On 2016-03-29 10:51:29, Laszlo Ersek wrote: > Edk2 commit 8a45f80edad4 ("MdeModulePkg: Make HII configuration settings > available to OS runtime") implements the optional UEFI feature described > in "31.2.11.1 OS Runtime Utilization" in UEFI v2.6. > > While this featur

Re: [edk2] [wave 3 PATCH v2 00/17] OvmfPkg, ArmVirtPkg: VirtIo 1.0

On 04/06/16 16:57, Jordan Justen wrote: > On 2016-04-06 05:40:10, Laszlo Ersek wrote: >> Version 2 of >> . >> >> Public branch: . >> >> * The following patches are new: >> >> v2

Re: [edk2] [PATCH V2 0/8] Move S3Ready() functional code from AcpiS3SaveDxe to S3SaveStateDxe

On 04/06/16 18:04, Laszlo Ersek wrote: > Hi Star, > > On 04/06/16 14:40, Star Zeng wrote: >> The S3Ready() functional code in AcpiS3SaveDxe of IntelFrameworkModulePkg >> is to do ACPI S3 Context save. In fact, that is not really related to >> Intel framework ACPI S3 protocol. >> >> IntelFrameworkM

Re: [edk2] [PATCH V2 6/8] OvmfPkg: Retire AcpiS3SaveDxe

On 04/06/16 14:40, Star Zeng wrote: > The same functional code has been in S3SaveStateDxe, > OVMF AcpiS3SaveDxe can be retired now. > > Cc: Laszlo Ersek > Cc: Jordan Justen > Cc: Jiewen Yao > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Star Zeng > --- > OvmfPkg/Ac

Re: [edk2] [PATCH 00/21] VirtFdtDxe refactoring

On 6 April 2016 at 18:14, Ard Biesheuvel wrote: > This is the first step towards getting rid of the kludge which is VirtFdtDxe. > > This series addresses the cases where other DXE drivers rely on dynamic PCDs > populated by VirtFdtDxe, which requires it to execute first using an A PRIORI > declara

Re: [edk2] [PATCH V2 4/8] MdeModulePkg S3SaveStateDxe: Consume PcdAcpiS3Enable to control the code

On 04/06/16 14:40, Star Zeng wrote: > Cc: Feng Tian > Cc: Jiewen Yao > Cc: Laszlo Ersek > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Star Zeng > --- > MdeModulePkg/Universal/Acpi/S3SaveStateDxe/S3SaveState.c | 5 - > MdeModulePkg/Universal/Acpi/S3SaveStat

Re: [edk2] [PATCH V2 3/8] OvmfPkg: Install LockBox protocol in constructor of LockBoxDxeLib

On 04/06/16 14:40, Star Zeng wrote: > Currently, the LockBox protocol is installed in entrypoint of > OVMF AcpiS3SaveDxe, that is strange. (6) please remove the phrase that is strange It is not strange; the commit message for the change justified it. (In other words, what the patch does is fin

[edk2] [PATCH 21/21] ArmVirtPkg: get rid of A PRIORI DXE declarations for VirtFdtDxe

We no longer care when VirtFdtDxe executes, since the dependencies are now all resolved via protocols. So drop the A PRIORI declaration. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel --- ArmVirtPkg/ArmVirtQemu.fdf | 4 ArmVirtPkg/ArmVirtQemuKern

Re: [edk2] [PATCH V2 2/8] OvmfPkg: Set PcdAcpiS3Enable according to QemuFwCfgS3Enabled()

On 04/06/16 14:40, Star Zeng wrote: > Also need to declare PcdAcpiS3Enable as DynamicDefault in *.dsc. > > Cc: Laszlo Ersek > Cc: Jordan Justen > Cc: Jiewen Yao > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Star Zeng > --- > OvmfPkg/OvmfPkgIa32.dsc | 1

[edk2] [PATCH 19/21] ArmVirtPkg: move QEMU based platforms to ArmVirtRtcFdtClientLib

This moves QEMU based platforms to ArmVirtRtcFdtClientLib, so that we no longer have to rely on VirtFdtDxe to execute first and set the RTC base address in a dynamic PCD. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel --- ArmVirtPkg/ArmVirtQemu.dsc |

[edk2] [PATCH 16/21] ArmVirtPkg/BaseCachingPciExpressLib: construct at first invocation

Instead of using a constructor, which may reference a dynamic PCD which is set by the DXE entry point of its user, defer the assignment of the global mPciExpressBaseAddress until the first the library is actually invoked. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard B

Re: [edk2] [PATCH 06/12] CorebootPayloadPkg: Convert to build FatPkg from source

On 2016-04-06 08:57:01, Agyeman, Prince wrote: > Jordan, >Will the FatPkg move into the EDKII repo then ? Yes. It should be added to EDK II later today. (Before this patch would be pushed.) -Jordan > > -Original Message- > From: Justen, Jordan L > Sent: Wednesday, April 6, 2016 8

[edk2] [PATCH 14/21] ArmVirtPkg/VirtFdtDxe: remove handling of fw_cfg DT node

Remove the handling of the fw_cfg DT node from VirtFdtDxe now that the fw_cfg client library has been moved to the FDT client protocol, and no longer relies on VirtFdtDxe to pass this information via dynamic PCDs. Since the PCDs in question are now no longer used, remove them from the various DEC a

[edk2] [PATCH 18/21] ArmVirtPkg: implement ArmVirtRtcFdtClientLib

This implements a library ArmVirtRtcFdtClientLib which is intended to be incorporated into RealTimeClockRuntimeDxe via NULL library class resolution. This allows us to make RealTimeClockRuntimeDxe depend on the FDT client protocol, and discover the RTC base address from the device tree directly rat

[edk2] [PATCH 15/21] ArmVirtPkg/PciHostBridgeDxe: move to FDT client protocol

Instead of relying on VirtFdtDxe to populate various dynamic PCDs with information retrieved from the host provided device tree, perform the PCI ECAM related DT node parsing directly in PciHostBridgeDxe. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel --- Ar

[edk2] [PATCH 20/21] ArmVirtPkg/VirtFdtDxe: drop RTC handling

The RTC driver no longer relies on VirtFdtDxe to set the pl031 RTC base address in a dynamic PCD, so drop the handling altogether. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel --- ArmVirtPkg/VirtFdtDxe/VirtFdtDxe.c | 36 ArmVirtPkg/

[edk2] [PATCH 13/21] ArmVirtPkg/QemuFwCfgLib: move to FDT client protocol

Make this library depend on the FDT client protocol to access the host supplied device tree directly rather than depending on VirtFdtDxe to set them using dynamic PCDs. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel --- ArmVirtPkg/Library/QemuFwCfgLib/QemuF

[edk2] [PATCH 17/21] ArmVirtPkg/VirtFdtDxe: drop PCI host bridge handling

Now that the PCI host bridge driver parses the DT node that describes the PCI host bridge directly via the FDT client protocol, we can drop the handling from VirtFdtDxe completely. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel --- ArmVirtPkg/ArmVirtQemu.ds

[edk2] [PATCH 08/21] ArmVirtPkg/ArmVirtPsciResetSystemLib: move to FDT client protocol

Instead of relying on VirtFdtDxe to detect the PSCI method, move our EfiResetSystemLib to the FDT client protocol to interrogate the device tree directly. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel --- ArmVirtPkg/Library/ArmVirtPsciResetSystemLib/ArmVir

[edk2] [PATCH 06/21] ArmVirtPkg/ArmGicArchLib: move to FdtClient protocol

Instead of relying on VirtFdtDxe to populate the GIC related PCDs, move this handling to our implementation of ArmGicArchLib, and retrieve the required DT info using the new FDT client protocol. This removes one of the reasons we need to load VirtFdtDxe first using an 'A PRIORI' declaration in the

[edk2] [PATCH 12/21] ArmVirtPkg/VirtFdtDxe: remove timer DT node handling

The timer code no longer relies on VirtFdtDxe to set the PCDs, so remove the handling of the timer node and the references to those PCDs. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel --- ArmVirtPkg/VirtFdtDxe/VirtFdtDxe.c | 39 +--- ArmV

[edk2] [PATCH 11/21] ArmVirtPkg: move TimerDxe to FDT client library

Move to the new dedicated to ArmVirtTimerFdtClientLib to populate the various timer related PCDs at driver load time rather than relying on VirtFdtDxe to do it. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel --- ArmVirtPkg/ArmVirtQemu.dsc

[edk2] [PATCH 05/21] ArmVirtPkg: add FdtClientDxe to the ArmVirtPkg platforms

Add FdtClientDxe to the various platforms under ArmVirtPkg, which will be required when we update the shared ArmGicArchLib to depend on it. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel --- ArmVirtPkg/ArmVirtQemu.dsc | 1 + ArmVirtPkg/ArmVirtQemu.fdf

[edk2] [PATCH 10/21] ArmVirtPkg: implement ArmVirtTimerFdtClientLib

This implements a library ArmVirtTimerFdtClientLib which is intended to be incorporated into TimerDxe via NULL library class resolution. This allows us to make TimerDxe depend on the FDT client protocol, and discover the timer interrupts from the device tree directly rather than relying on VirtFdtD

[edk2] [PATCH 03/21] ArmVirtPkg: drop dependency on PeiPcdLib for PEI Pcd.inf

The PcdPeim dynamic PCD driver is dispatched explicitly via an 'A PRIORI' declaration in the platform DSC. Without that declaration, the PEI module can never be dispatched since it transitively depends on protocols it produces itself via PeiPcdLib. So use the NULL PcdLib explicitly only for this dr

[edk2] [PATCH 07/21] ArmVirtPkg/VirtFdtDxe: remove GIC discovery

Now that we moved the GIC discovery to our ArmGicArchLib implementation, we can remove it from VirtFdtDxe, since it is no longer used. Remove the PcdArmGicRevision declaration and definitions as well: VirtFdtDxe no longer sets it, and no other drivers consume its value. Contributed-under: TianoCor

[edk2] [PATCH 09/21] ArmVirtPkg/VirtFdtDxe: drop detection of PSCI method

The detection of the PSCI method has been moved to the EfiResetSystemLib implementation, so drop the handling from VirtFdtDxe. Since no users remain of gArmVirtTokenSpaceGuid.PcdArmPsciMethod, remove that as well. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuve

[edk2] [PATCH 01/21] ArmVirtPkg: introduce FdtClientProtocol

This introduces the FdtClientProtocol, which will be used to expose the device tree provided by the host to other DXE drivers. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel --- ArmVirtPkg/ArmVirtPkg.dec | 3 + ArmVirtPkg/Include/Protocol/Fdt

[edk2] [PATCH 02/21] ArmVirtPkg: drop dependency on DxePcdLib for DXE Pcd.inf

The PcdDxe dynamic PCD driver is dispatched explicitly via an 'A PRIORI' declaration in the platform DSC. Without that declaration, the DXE driver can never be dispatched since it transitively depends on protocols it produces itself via DxePcdLib. So use the NULL PcdLib explicitly only for this dri

[edk2] [PATCH 00/21] VirtFdtDxe refactoring

This is the first step towards getting rid of the kludge which is VirtFdtDxe. This series addresses the cases where other DXE drivers rely on dynamic PCDs populated by VirtFdtDxe, which requires it to execute first using an A PRIORI declaration. Instead, this series moves the DT node parsing to th

[edk2] [PATCH 04/21] ArmVirtPkg/FdtClientDxe: implement new driver

This implements a new DXE driver FdtClientDxe to produce the FDT client protocol based on a device tree image supplied by the virt host. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ard Biesheuvel --- ArmVirtPkg/FdtClientDxe/FdtClientDxe.c | 236

Re: [edk2] [PATCH V2 0/8] Move S3Ready() functional code from AcpiS3SaveDxe to S3SaveStateDxe

Hi Star, On 04/06/16 14:40, Star Zeng wrote: > The S3Ready() functional code in AcpiS3SaveDxe of IntelFrameworkModulePkg > is to do ACPI S3 Context save. In fact, that is not really related to > Intel framework ACPI S3 protocol. > > IntelFrameworkModulePkg will be deprecated step by step, so move

[edk2] Performance identifier mismatch in TrEEDxe.c and TrEEPei.c

Hello all, I am studying about measuring the boot performance with PERF_START_EX and PERF_STOP_EX. I notice that in both files SecurityPkg/Tcg/TrEEDxe/TrEEDxe.c and SecurityPkg/Tcg/TrEEPei/TrEEPei.c the input identifier value of the PERF_STOP_EX is one bigger than the input identifier value of

Re: [edk2] [PATCH 04/12] ArmVirtPkg: Convert to build FatPkg from source

On 6 April 2016 at 17:14, Jordan Justen wrote: > Now that FatPkg is open source (and therefore can be included in the > EDK II tree) we build and use it directly. > > Note: Not build tested > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Jordan Justen > Cc: Laszlo Ers

Re: [edk2] [PATCH 05/12] BeagleBoardPkg: Convert to build FatPkg from source

On 6 April 2016 at 17:14, Jordan Justen wrote: > Now that FatPkg is open source (and therefore can be included in the > EDK II tree) we build and use it directly. > > Note: Not build tested > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Jordan Justen > Cc: Leif Lindh

Re: [edk2] [PATCH 09/12] EmulatorPkg: Convert to always build FatPkg

> On Apr 6, 2016, at 8:14 AM, Jordan Justen wrote: > > Now that FatPkg is open source (and therefore can be included in the > EDK II tree) we always build and use it directly. > > Build and boot tested with GCC 5.3 on X64, but I did not test > attaching a disk image with a FAT filesystem. > >

[edk2] [PATCH 05/12] BeagleBoardPkg: Convert to build FatPkg from source

Now that FatPkg is open source (and therefore can be included in the EDK II tree) we build and use it directly. Note: Not build tested Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jordan Justen Cc: Leif Lindholm Cc: Ard Biesheuvel --- BeagleBoardPkg/BeagleBoardPkg.ds

[edk2] [PATCH 09/12] EmulatorPkg: Convert to always build FatPkg

Now that FatPkg is open source (and therefore can be included in the EDK II tree) we always build and use it directly. Build and boot tested with GCC 5.3 on X64, but I did not test attaching a disk image with a FAT filesystem. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by:

[edk2] [PATCH 02/12] FatPkg: Add Contributions.txt

This is copied from MdePkg/Contributions.txt Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jordan Justen Cc: Ruiyu Ni --- FatPkg/Contributions.txt | 218 +++ 1 file changed, 218 insertions(+) create mode 100644 FatPkg/Contrib

[edk2] [PATCH 00/12] Cleanups to follow FatPkg license update

web: https://github.com/jljusten/edk2/tree/fatpkg-open-source-cleanups-v1 git: https://github.com/jljusten/edk2.git fatpkg-open-source-cleanups-v1 The license update to FatPkg is expected to be pushed later today. Following the license update, I recommend these changes. Jordan Justen (12): Fa

[edk2] [PATCH 11/12] OvmfPkg: Convert to using FatPkg in the EDK II tree

Now that FatPkg is open source (and therefore can be included in the EDK II tree) we build and use it directly. Build tested with GCC 5.3 on IA32 and X64. Boot tested to UEFI Shell on IA32 and UEFI Linux on X64. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jordan Justen

[edk2] [PATCH 10/12] Nt32Pkg: Convert to build FatPkg from source

Now that FatPkg is open source (and therefore can be included in the EDK II tree) we build and use it directly. Note: Not build tested Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jordan Justen Cc: Ruiyu Ni --- Nt32Pkg/Nt32Pkg.dsc | 3 ++- Nt32Pkg/Nt32Pkg.fdf | 4 ++--

[edk2] [PATCH 12/12] Vlv2TbltDevicePkg: Convert to build FatPkg from source

Now that FatPkg is open source (and therefore can be included in the EDK II tree) we build and use it directly. Note: Not build tested Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jordan Justen Cc: David Wei Cc: Tim He --- Vlv2TbltDevicePkg/PlatformPkg.fdf | 7

[edk2] [PATCH 06/12] CorebootPayloadPkg: Convert to build FatPkg from source

Now that FatPkg is open source (and therefore can be included in the EDK II tree) we build and use it directly. Note: Build tested with GCC 5.3 on IA32 and IA32+X64. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jordan Justen Cc: Maurice Ma Cc: Prince Agyeman --- Core

[edk2] [PATCH 03/12] FatBinPkg: Change to 2-clause BSD license

The FatPkg is now open source. Update FatBinPkg license to match FatPkg. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jordan Justen Cc: Ruiyu Ni --- FatBinPkg/FatBinPkg.dec | 2 +- FatBinPkg/License.txt | 53 ++--- 2 files

[edk2] [PATCH 04/12] ArmVirtPkg: Convert to build FatPkg from source

Now that FatPkg is open source (and therefore can be included in the EDK II tree) we build and use it directly. Note: Not build tested Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jordan Justen Cc: Laszlo Ersek Cc: Ard Biesheuvel --- ArmVirtPkg/ArmVirtQemu.dsc

[edk2] [PATCH 07/12] DuetPkg scripts: Support building with GCC45 - GCC49

Build tested with GCC 5.3 with both IA32 and X64. Unfortunately both seemed to hang very early when boot tested on QEMU. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jordan Justen Cc: Ruiyu Ni --- DuetPkg/PostBuild.sh | 8 DuetPkg/build32.sh | 24 ++

[edk2] [PATCH 08/12] DuetPkg: Convert to build FatPkg from source

Now that FatPkg is open source (and therefore can be included in the EDK II tree) we build and use it directly. Note: Build tested with GCC 5.3 on IA32 and X64. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jordan Justen Cc: Ruiyu Ni --- DuetPkg/DuetPkg.fdf | 5 ++-

[edk2] [PATCH 01/12] FatPkg: Update License.txt to have the full license text

The script replaced the contents of License.txt will the words the shortened text that we include in the headers to files. Instead, use the full text. This is copied from MdePkg/License.txt. Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jordan Justen Cc: Ruiyu Ni --- Fa

Re: [edk2] [wave 3 PATCH v2 00/17] OvmfPkg, ArmVirtPkg: VirtIo 1.0

On 2016-04-06 05:40:10, Laszlo Ersek wrote: > Version 2 of > . > > Public branch: . > > * The following patches are new: > > v2 06/17 OvmfPkg: IndustryStandard: factor out Vi

[edk2] 答复: 答复: [EDK2]an issue that PXE boot failed when received a NACK from the DHCP server

Hi Siyuan, Ard and Michael: I am very sorry to reply you so late because of busy work. The attachment is the DHCP log when the issue happens. First of all, Let me introduce the environment in detail: Multiple boards connect to the DHCP server through a switch . The MAC

[edk2] [PATCH V2 3/8] OvmfPkg: Install LockBox protocol in constructor of LockBoxDxeLib

Currently, the LockBox protocol is installed in entrypoint of OVMF AcpiS3SaveDxe, that is strange. We can let the first driver run with LockBoxDxeLib linked to have its library constructor to install LockBox protocol on the ImageHandle. As other drivers may have gEfiLockBoxProtocolGuid dependency,

[edk2] [PATCH V2 7/8] IntelFrameworkModulePkg AcpiS3SaveDxe: Consume PcdAcpiS3Enable to control the code

Cc: Jeff Fan Cc: Jiewen Yao Cc: Laszlo Ersek Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng --- IntelFrameworkModulePkg/Universal/Acpi/AcpiS3SaveDxe/AcpiS3Save.c | 4 .../Universal/Acpi/AcpiS3SaveDxe/AcpiS3SaveDxe.inf| 3 ++- 2 fil

[edk2] [PATCH V2 0/8] Move S3Ready() functional code from AcpiS3SaveDxe to S3SaveStateDxe

The S3Ready() functional code in AcpiS3SaveDxe of IntelFrameworkModulePkg is to do ACPI S3 Context save. In fact, that is not really related to Intel framework ACPI S3 protocol. IntelFrameworkModulePkg will be deprecated step by step, so move the functional code to MdeModulePkg and S3SaveStateDxe

[edk2] [wave 3 PATCH v2 12/17] OvmfPkg: VirtioRngDxe: adapt feature negotiation to virtio-1.0

Relative to virtio-0.9.5, virtio-1.0 reverses the order of queue discovery and feature negotiation. In virtio-1.0, feature negotiation has to complete first, and the device can also reject a self-inconsistent feature request through the new VSTAT_FEATURES_OK status bit. (For example if the driver r

[edk2] [wave 3 PATCH v2 09/17] OvmfPkg: VirtioLib: add Virtio10WriteFeatures() function

In VirtIo 1.0, a device can reject a self-inconsistent feature bitmap through the new VSTAT_FEATURES_OK status bit. (For example if the driver requests a higher level feature but clears a prerequisite feature.) This function is a small wrapper around VIRTIO_DEVICE_PROTOCOL.SetGuestFeatures() that a

[edk2] [PATCH V2 2/8] OvmfPkg: Set PcdAcpiS3Enable according to QemuFwCfgS3Enabled()

Also need to declare PcdAcpiS3Enable as DynamicDefault in *.dsc. Cc: Laszlo Ersek Cc: Jordan Justen Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng --- OvmfPkg/OvmfPkgIa32.dsc | 1 + OvmfPkg/OvmfPkgIa32X64.dsc | 1 + OvmfPkg

[edk2] [PATCH V2 6/8] OvmfPkg: Retire AcpiS3SaveDxe

The same functional code has been in S3SaveStateDxe, OVMF AcpiS3SaveDxe can be retired now. Cc: Laszlo Ersek Cc: Jordan Justen Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng --- OvmfPkg/AcpiS3SaveDxe/AcpiS3Save.c | 549

[edk2] [PATCH V2 5/8] MdeModulePkg S3SaveStateDxe: Move S3Ready() functional code from AcpiS3SaveDxe

The S3Ready() functional code in AcpiS3SaveDxe of IntelFrameworkModulePkg is to do ACPI S3 Context save. In fact, that is not really related to Intel framework ACPI S3 protocol. IntelFrameworkModulePkg will be deprecated step by step, so move the functional code to MdeModulePkg and S3SaveStateDxe

[edk2] [wave 3 PATCH v2 10/17] OvmfPkg: VirtioBlkDxe: adapt feature negotiation to virtio-1.0

Relative to virtio-0.9.5, virtio-1.0 reverses the order of queue discovery and feature negotiation. In virtio-1.0, feature negotiation has to complete first, and the device can also reject a self-inconsistent feature request through the new VSTAT_FEATURES_OK status bit. (For example if the driver r

[edk2] [PATCH V2 8/8] IntelFrameworkModulePkg AcpiS3SaveDxe: Remove S3Ready() functional code

The S3Ready() functional code has been moved to S3SaveStateDxe in MdeModulePkg, the ACPI global variable related code is leaved as is for compatibility. PcdS3BootScriptStackSize is also moved to MdeModulePkg. Cc: Jiewen Yao Cc: Jeff Fan Contributed-under: TianoCore Contribution Agreement 1.0 Sig

[edk2] [PATCH V2 1/8] MdeModulePkg: Introduce new PCD PcdAcpiS3Enable

Platform can configure the PCD statically or dynamically to control if ACPI S3 will be enabled. S3 related modules can consume the PCD to control the code. Cc: Feng Tian Cc: Jiewen Yao Cc: Laszlo Ersek Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng --- MdeMod

[edk2] [PATCH V2 4/8] MdeModulePkg S3SaveStateDxe: Consume PcdAcpiS3Enable to control the code

Cc: Feng Tian Cc: Jiewen Yao Cc: Laszlo Ersek Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Star Zeng --- MdeModulePkg/Universal/Acpi/S3SaveStateDxe/S3SaveState.c | 5 - MdeModulePkg/Universal/Acpi/S3SaveStateDxe/S3SaveStateDxe.inf | 5 - 2 files changed,

[edk2] [wave 3 PATCH v2 11/17] OvmfPkg: VirtioNetDxe: adapt feature negotiation to virtio-1.0

Relative to virtio-0.9.5, virtio-1.0 reverses the order of queue discovery and feature negotiation. In virtio-1.0, feature negotiation has to complete first, and the device can also reject a self-inconsistent feature request through the new VSTAT_FEATURES_OK status bit. (For example if the driver r

  1   2   >