Re: [edk2] [Patch v3 2/5] UefiCpuPkg/AcpiCpuData.h: Remove AcpiNVS and Below 4G limitation.

2018-08-10 Thread Laszlo Ersek
On 08/10/18 06:19, Eric Dong wrote: > ACPI_CPU_DATA structure first introduced to save data in > normal boot phase. Also this data will be used in S3 phase > by one PEI driver. So in first phase, this data is been > defined to use ACPI NVS memory type and must below 4G. > > Later in order to fix

Re: [edk2] [Patch v3 5/5] UefiCpuPkg/RegisterCpuFeaturesLib: Combine implementation.

2018-08-10 Thread Laszlo Ersek
On 08/10/18 06:19, Eric Dong wrote: > V1 changes: >> Current code logic can't confirm CpuS3DataDxe driver start before >> CpuFeaturesDxe driver. So the assumption in CpuFeaturesDxe not valid. >> Add implementation for AllocateAcpiCpuData function to remove this >> assumption. > > V2 changes: >>

Re: [edk2] [Patch v3 1/5] UefiCpuPkg/PiSmmCpuDxeSmm: Use GDT/IDT saved in Smram.

2018-08-10 Thread Laszlo Ersek
On 08/10/18 06:19, Eric Dong wrote: > Current implementation copies GDT/IDT at SmmReadyToLock point > from ACPI NVS memory to Smram. Later at S3 resume phase, it restores > memory saved in Smram to ACPI NVS. Driver can directly use GDT/IDT > saved in Smram instead of restore the original ACPI NVS

Re: [edk2] [Patch v3 3/5] UefiCpuPkg/CpuS3DataDxe: Change Memory Type and address limitation.

2018-08-10 Thread Laszlo Ersek
On 08/10/18 06:19, Eric Dong wrote: > Because CpuS3Data memory will be copy to smram at SmmReadyToLock point, > the memory type no need to be ACPI NVS type, also the address not > limit to below 4G. > > This change remove the limit of ACPI NVS memory type and below 4G. > > Pass OS boot and

Re: [edk2] [Patch v3 0/5] Change CpuS3Data memory type and address limitation

2018-08-10 Thread Laszlo Ersek
Eric, On 08/10/18 06:19, Eric Dong wrote: > Because CpuS3Data memory will be copy to smram at SmmReadToLock point by > PiSmmCpuDxeSmm driver, > the memory type no need to be ACPI NVS type, also the address not limit to > below 4G. > > This change remove the limit of ACPI NVS memory type and

Re: [edk2] [PATCH 0/2] UefiCpuPkg: Add RSB stuffing before rsm instruction

2018-08-10 Thread Laszlo Ersek
On 08/10/18 03:43, Hao Wu wrote: > The series will add RSB stuffing logics to avoid RSB underflow on return > from SMM (rsm instruction). > > Cc: Jiewen Yao > Cc: Eric Dong > Cc: Laszlo Ersek > > Hao Wu (2): > UefiCpuPkg/SmmCpuFeaturesLib: Add RSB stuffing before rsm instruction >

Re: [edk2] [Patch v3 3/5] UefiCpuPkg/CpuS3DataDxe: Change Memory Type and address limitation.

2018-08-10 Thread Laszlo Ersek
On 08/10/18 18:12, Laszlo Ersek wrote: > On 08/10/18 06:19, Eric Dong wrote: >> Because CpuS3Data memory will be copy to smram at SmmReadyToLock point, >> the memory type no need to be ACPI NVS type, also the address not >> limit to below 4G. >> >> This change remove the limit of ACPI NVS memory

Re: [edk2] [PATCH] MdeModulePkg: Remove redundant things in inf and .h files

2018-08-10 Thread Laszlo Ersek
On 08/10/18 10:47, Zeng, Star wrote: > I suggest merging this patch with > https://lists.01.org/pipermail/edk2-devel/2018-August/027953.html to one > patch. > And remember to update the new patch's version. There's another thing to remember please, when posting the new version of the series:

Re: [edk2] [Patch v3 4/5] UefiCpuPkg/CpuS3DataDxe: Remove below 4G limitation.

2018-08-10 Thread Laszlo Ersek
On 08/10/18 06:19, Eric Dong wrote: > Because PrepareApStartupVector() stores StackAddress to > "mExchangeInfo->StackStart" (which has type (VOID*)), and because > "UefiCpuPkg/PiSmmCpuDxeSmm/X64/MpFuncs.nasm" reads the latter with: > > add edi, StackStartAddressLocation > add

Re: [edk2] [PATCH] SignedCapsulePkg: Remove redundant things in inf, .h and .c files

2018-08-10 Thread Yao, Jiewen
Reviewed-by: jiewen@intel.com > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > shenglei > Sent: Friday, August 10, 2018 2:11 PM > To: edk2-devel@lists.01.org > Cc: Yao, Jiewen > Subject: [edk2] [PATCH] SignedCapsulePkg: Remove redundant

Re: [edk2] [PATCH] SignedCapsulePkg: Remove redundant things in inf, .h and .c files

2018-08-10 Thread Zeng, Star
Hi Shenglei, I know PrintLib is used by ParseConfigProfile.c that is included by SystemFirmwareUpdateDxe.inf. But could you evaluate whether DxeServicesLib could be removed from SystemFirmwareUpdateDxe.inf or not? Thanks, Star -Original Message- From: edk2-devel

Re: [edk2] [PATCH 06/26] MdeModulePkg PciBusDxe: Remove redundant functions

2018-08-10 Thread Zeng, Star
Reviewed-by: Star Zeng -Original Message- From: Zhang, Shenglei Sent: Wednesday, August 8, 2018 4:47 PM To: edk2-devel@lists.01.org Cc: Zeng, Star ; Dong, Eric Subject: [PATCH 06/26] MdeModulePkg PciBusDxe: Remove redundant functions The functions that are never called have been

Re: [edk2] [PATCH 10/26] MdeModulePkg XhciPei: Remove redundant functions

2018-08-10 Thread Zeng, Star
Reviewed-by: Star Zeng -Original Message- From: Zhang, Shenglei Sent: Wednesday, August 8, 2018 4:47 PM To: edk2-devel@lists.01.org Cc: Zeng, Star ; Dong, Eric Subject: [PATCH 10/26] MdeModulePkg XhciPei: Remove redundant functions The functions that are never called have been

Re: [edk2] [PATCH 09/26] MdeModulePkg XhciDxe: Remove redundant functions

2018-08-10 Thread Zeng, Star
Reviewed-by: Star Zeng -Original Message- From: Zhang, Shenglei Sent: Wednesday, August 8, 2018 4:47 PM To: edk2-devel@lists.01.org Cc: Zeng, Star ; Dong, Eric Subject: [PATCH 09/26] MdeModulePkg XhciDxe: Remove redundant functions The functions that are never called have been

Re: [edk2] [PATCH 12/26] MdeModulePkg UfsPassThruDxe: Remove redundant functions

2018-08-10 Thread Zeng, Star
Reviewed-by: Star Zeng -Original Message- From: Zhang, Shenglei Sent: Wednesday, August 8, 2018 4:47 PM To: edk2-devel@lists.01.org Cc: Zeng, Star ; Dong, Eric Subject: [PATCH 12/26] MdeModulePkg UfsPassThruDxe: Remove redundant functions The functions that are never called have been

Re: [edk2] [PATCH 13/26] MdeModulePkg UsbBotPei: Remove redundant functions

2018-08-10 Thread Zeng, Star
Reviewed-by: Star Zeng -Original Message- From: Zhang, Shenglei Sent: Wednesday, August 8, 2018 4:47 PM To: edk2-devel@lists.01.org Cc: Zeng, Star ; Dong, Eric Subject: [PATCH 13/26] MdeModulePkg UsbBotPei: Remove redundant functions The functions that are never called have been

Re: [edk2] [PATCH 15/26] MdeModulePkg UsbBusPei: Remove redundant functions

2018-08-10 Thread Zeng, Star
Reviewed-by: Star Zeng -Original Message- From: Zhang, Shenglei Sent: Wednesday, August 8, 2018 4:47 PM To: edk2-devel@lists.01.org Cc: Zeng, Star ; Dong, Eric Subject: [PATCH 15/26] MdeModulePkg UsbBusPei: Remove redundant functions The functions that are never called have been

Re: [edk2] Question about SecurityPkg/DxeTcg2PhysicalPresenceLib

2018-08-10 Thread Yao, Jiewen
by design a platform need define a trusted console and only connect this trusted console before endofdxe thank you! Yao, Jiewen > 在 2018年8月10日,下午4:50,"heyi@linaro.org" 写道: > > Hi folks, > > The function Tcg2PhysicalPresenceLibProcessRequest in > DxeTcg2PhysicalPresenceLib > requires

Re: [edk2] [PATCH 16/26] MdeModulePkg Core/Dxe: Remove redundant functions

2018-08-10 Thread Zeng, Star
Reviewed-by: Star Zeng -Original Message- From: Zhang, Shenglei Sent: Wednesday, August 8, 2018 4:47 PM To: edk2-devel@lists.01.org Cc: Zeng, Star ; Dong, Eric Subject: [PATCH 16/26] MdeModulePkg Core/Dxe: Remove redundant functions The functions that are never called have been

Re: [edk2] [PATCH 17/26] MdeModulePkg PiSmmCore: Remove redundant functions

2018-08-10 Thread Zeng, Star
Reviewed-by: Star Zeng -Original Message- From: Zhang, Shenglei Sent: Wednesday, August 8, 2018 4:47 PM To: edk2-devel@lists.01.org Cc: Zeng, Star ; Dong, Eric Subject: [PATCH 17/26] MdeModulePkg PiSmmCore: Remove redundant functions The functions that are never called have been

[edk2] [PATCH v1 1/2] EmbeddedPkg: add NonDiscoverableDeviceDxe driver

2018-08-10 Thread Haojian Zhuang
It's used to create NonDiscoverableDevice in embedded platform. Since there's no PCI bus. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Haojian Zhuang --- EmbeddedPkg/EmbeddedPkg.dec | 1 +

[edk2] [PATCH v1 0/2] add DwMmcHcDxe driver

2018-08-10 Thread Haojian Zhuang
Changelog: v1: *Add NonDiscoverableDeviceDxe for embedded platform. Make DwMmcHcDxe driver to support both eMMC and SD controller. Haojian Zhuang (2): EmbeddedPkg: add NonDiscoverableDeviceDxe driver EmbeddedPkg: add DwMmcHcDxe driver EmbeddedPkg/Drivers/DwMmcHcDxe/DwMmcHcDxe.dec

Re: [edk2] [PATCH] MdeModulePkg: Remove redundant things in inf and .h files

2018-08-10 Thread Zeng, Star
I suggest merging this patch with https://lists.01.org/pipermail/edk2-devel/2018-August/027953.html to one patch. And remember to update the new patch's version. Thanks, Star -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of shenglei Sent:

Re: [edk2] [PATCH 08/26] MdeModulePkg UhciPei: Remove redundant functions

2018-08-10 Thread Zeng, Star
Reviewed-by: Star Zeng -Original Message- From: Zhang, Shenglei Sent: Wednesday, August 8, 2018 4:47 PM To: edk2-devel@lists.01.org Cc: Zeng, Star ; Dong, Eric Subject: [PATCH 08/26] MdeModulePkg UhciPei: Remove redundant functions The functions that are never called have been

Re: [edk2] [PATCH 11/26] MdeModulePkg UfsBlockIoPei: Remove redundant functions

2018-08-10 Thread Zeng, Star
I agree with Hao. Reviewed-by: Star Zeng . Thanks, Star -Original Message- From: Wu, Hao A Sent: Friday, August 10, 2018 10:58 AM To: Zhang, Shenglei ; edk2-devel@lists.01.org Cc: Dong, Eric ; Zeng, Star Subject: RE: [edk2] [PATCH 11/26] MdeModulePkg UfsBlockIoPei: Remove redundant

Re: [edk2] [PATCH 22/26] MdeModulePkg HiiDatabaseDxe: Remove a redundant function

2018-08-10 Thread Zeng, Star
Sorry, I meant 'called'. -Original Message- From: Zeng, Star Sent: Thursday, August 9, 2018 6:24 PM To: Zhang, Shenglei ; edk2-devel@lists.01.org Cc: Dong, Eric ; Zeng, Star Subject: RE: [PATCH 22/26] MdeModulePkg HiiDatabaseDxe: Remove a redundant function There is a typo in commit

Re: [edk2] [PATCH 01/26] MdeModulePkg CapsuleApp: Remove a redundant function

2018-08-10 Thread Zeng, Star
Sorry, I meant 'called'. -Original Message- From: Zeng, Star Sent: Thursday, August 9, 2018 6:10 PM To: Zhang, Shenglei ; edk2-devel@lists.01.org Cc: Dong, Eric ; Zeng, Star Subject: RE: [PATCH 01/26] MdeModulePkg CapsuleApp: Remove a redundant function There is a typo in commit log.

[edk2] [Patch][edk2-platforms/devel-IntelAtomProcessorE3900] UP2 Board Enabling.

2018-08-10 Thread zwei4
Add board specific code for AAEON UP2 board. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: David Wei CC: Mike Wu CC: Mang Guo --- .../Board/UP2/BoardInitDxe/BoardInitDxe.c | 68 +++ .../Board/UP2/BoardInitDxe/BoardInitDxe.h | 37 ++

[edk2] [PATCH v1 edk-platforms 2/3] Platform/HiKey960: enable SD controller

2018-08-10 Thread Haojian Zhuang
Enable DwMmcHcDxe driver on HiKey960 platform. Cc: Leif Lindholm Cc: Ard Biesheuvel Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Haojian Zhuang --- Platform/Hisilicon/HiKey960/HiKey960.dsc | 8 ++ Platform/Hisilicon/HiKey960/HiKey960.fdf

[edk2] [PATCH v1 edk-platforms 0/3] enable eMMC/SD driver on HiKey

2018-08-10 Thread Haojian Zhuang
Changelog: v1: * Enable eMMC/SD driver on HiKey. * Enable SD driver on HiKey960. Haojian Zhuang (3): Silicon/Hi3660: fix LDO9_VSET register definition Platform/HiKey960: enable SD controller Platform/HiKey: enable SD/MMC controller on HiKey Platform/Hisilicon/HiKey/HiKey.dsc

[edk2] [PATCH v1 edk-platforms 3/3] Platform/HiKey: enable SD/MMC controller on HiKey

2018-08-10 Thread Haojian Zhuang
Replace DwEmmcDxe driver by DwMmcHcDxe driver on HiKey platform. Since the new driver could work on both eMMC and SD controller. Cc: Leif Lindholm Cc: Ard Biesheuvel Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Haojian Zhuang --- Platform/Hisilicon/HiKey/HiKey.dsc

[edk2] Question about SecurityPkg/DxeTcg2PhysicalPresenceLib

2018-08-10 Thread heyi . guo
Hi folks, The function Tcg2PhysicalPresenceLibProcessRequest in DxeTcg2PhysicalPresenceLib requires to be invoked after console is ready, and in the function it will call VariableLockProtocol->RequestToLock(), while variable RequestToLock() requires to be called before "End Of Dxe" event, or else

Re: [edk2] [PATCH] MdeModulePkg: Remove some comments

2018-08-10 Thread Zeng, Star
I suggest merging this patch to thread https://lists.01.org/pipermail/edk2-devel/2018-August/028167.html. Thanks, Star -Original Message- From: Zhang, Shenglei Sent: Friday, August 10, 2018 2:30 PM To: edk2-devel@lists.01.org Cc: Zeng, Star ; Dong, Eric Subject: [PATCH] MdeModulePkg:

Re: [edk2] [PATCH] MdeModulePkg UiApp: Remove something redundant

2018-08-10 Thread Zeng, Star
I suggest merging this patch to thread https://lists.01.org/pipermail/edk2-devel/2018-August/028167.html. Thanks, Star -Original Message- From: Zhang, Shenglei Sent: Thursday, August 9, 2018 2:54 PM To: edk2-devel@lists.01.org Cc: Zeng, Star ; Dong, Eric Subject: [PATCH] MdeModulePkg

[edk2] [PATCH v1 edk-platforms 1/3] Silicon/Hi3660: fix LDO9_VSET register definition

2018-08-10 Thread Haojian Zhuang
Fix the LDO9_VSET register definition in PMIC. Cc: Leif Lindholm Cc: Ard Biesheuvel Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Haojian Zhuang --- Silicon/Hisilicon/Hi3660/Include/Hi3660.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [edk2] [PATCH 14/26] MdeModulePkg UsbBusDxe: Remove redundant functions

2018-08-10 Thread Zeng, Star
Reviewed-by: Star Zeng -Original Message- From: Zhang, Shenglei Sent: Wednesday, August 8, 2018 4:47 PM To: edk2-devel@lists.01.org Cc: Zeng, Star ; Dong, Eric Subject: [PATCH 14/26] MdeModulePkg UsbBusDxe: Remove redundant functions The functions that are never called have been

Re: [edk2] [PATCH 21/26] MdeModulePkg TerminalDxe: Remove a redundant function

2018-08-10 Thread Zeng, Star
Sorry, I meant 'called'. -Original Message- From: Zeng, Star Sent: Thursday, August 9, 2018 6:25 PM To: Zhang, Shenglei ; edk2-devel@lists.01.org Cc: Dong, Eric ; Zeng, Star Subject: RE: [PATCH 21/26] MdeModulePkg TerminalDxe: Remove a redundant function There is a typo in commit log.

Re: [edk2] [PATCH 19/26] MdeModulePkg EmmcBlockIoPei: Remove a redundant function

2018-08-10 Thread Zeng, Star
I agree with Hao. And there is a typo in commit log. 'calld' should be 'called'. With the typo fixed, Reviewed-by: Star Zeng Thanks, Star -Original Message- From: Wu, Hao A Sent: Friday, August 10, 2018 10:58 AM To: Zhang, Shenglei ; edk2-devel@lists.01.org Cc: Dong, Eric ; Zeng, Star

Re: [edk2] [PATCH 18/26] MdeModulePkg Ps2KeyboardDxe: Remove a redundant function

2018-08-10 Thread Zeng, Star
There is a typo in commit log. 'calld' should be 'called'. With the typo fixed, Reviewed-by: Star Zeng Thanks, Star -Original Message- From: Zhang, Shenglei Sent: Wednesday, August 8, 2018 4:47 PM To: edk2-devel@lists.01.org Cc: Zeng, Star ; Dong, Eric Subject: [PATCH 18/26]

Re: [edk2] [PATCH 20/26] MdeModulePkg SdBlockIoPei: Remove a redundant function

2018-08-10 Thread Zeng, Star
I agree with Hao. And there is a typo in commit log. 'calld' should be 'called'. With the typo fixed, Reviewed-by: Star Zeng Thanks, Star -Original Message- From: Wu, Hao A Sent: Friday, August 10, 2018 10:58 AM To: Zhang, Shenglei ; edk2-devel@lists.01.org Cc: Dong, Eric ; Zeng, Star

Re: [edk2] [Patch 0/4] Vlv2TbltDevicePkg: Add FmpDevicePkg support

2018-08-10 Thread Guo, Mang
Reviewed-by: Guo Mang -Original Message- From: Kinney, Michael D Sent: Friday, August 10, 2018 9:28 AM To: edk2-devel@lists.01.org Cc: Wei, David; Guo, Mang Subject: [Patch 0/4] Vlv2TbltDevicePkg: Add FmpDevicePkg support Update Vlv2TbltDevicePkg to build UEFI capsules for system

[edk2] [PATCH] MdeModulePkg: Remove some comments

2018-08-10 Thread shenglei
Some comments including the redundant functions removed before have been removed. https://bugzilla.tianocore.org/show_bug.cgi?id=1062 Cc: Star Zeng Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: shenglei --- MdeModulePkg/Bus/Sd/EmmcBlockIoPei/EmmcHcMem.c

[edk2] [Patch] BaseTools: Remove the redundant if statement

2018-08-10 Thread Yonghong Zhu
after analysis the BuildOptionValue function, we found the if statement IsFieldValueAnArray is redundant because ValueExpressionEx will handle it. Cc: Liming Gao Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Yonghong Zhu --- BaseTools/Source/Python/Common/Expression.py

[edk2] [PATCH] SignedCapsulePkg: Remove redundant things in inf, .h and .c files

2018-08-10 Thread shenglei
Two redundant library classes have been removed. The redundant function "IsSystemFmp" has been removed. https://bugzilla.tianocore.org/show_bug.cgi?id=1067 Cc: Jiewen Yao Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: shenglei --- .../SystemFirmwareCommonDxe.c