Reviewed-by: Ray Ni
> -Original Message-
> From: Lou, Yun
> Sent: Monday, March 15, 2021 9:49 PM
> To: devel@edk2.groups.io
> Cc: Lou, Yun ; Ni, Ray ; Dong, Eric
> ; Laszlo Ersek ; Kumar, Rahul1
>
> Subject: [PATCH v2 1/1] UefiCpuPkg/CpuCacheInfoLib: Collect cache associative
> type
>
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3233
GDT needs to be allocated below 4GB in 64bit environment
because AP needs it for entering to protected mode.
CPU running in big real mode cannot access above 4GB GDT.
But CpuDxe driver contains below code:
gdt = AllocateRuntimePool
The change doesn't impact any functionality.
Signed-off-by: Ray Ni
Cc: Eric Dong
Cc: Laszlo Ersek
Cc: Rahul Kumar
---
UefiCpuPkg/CpuDxe/CpuGdt.c | 27 +--
1 file changed, 13 insertions(+), 14 deletions(-)
diff --git a/UefiCpuPkg/CpuDxe/CpuGdt.c
Ray Ni (2):
UefiCpuPkg/CpuDxe: Rename variables to follow EDKII coding standard
UefiCpuPkg/CpuDxe: Guarantee GDT is below 4GB
UefiCpuPkg/CpuDxe/CpuGdt.c | 42 +++---
1 file changed, 26 insertions(+), 16 deletions(-)
--
2.27.0.windows.1
Hi Liming,
Thanks for the comments. This patch is merged before this comment, but I can
still send another patch to modify if needed.
However, I think the implement in this patch is more simple.
The implement in QuarkPlatformPkg need changes in inf, dec and dsc files, and
is not as intuitive
Hi Bob,
I have created new patch file. PatchCheck is pass.
Using Git send-email send to edk2.group.io
Thanks,
Greg
-Original Message-
From: Feng, Bob C
Sent: Monday, March 15, 2021 12:35 PM
To: Yeh, GregX ; devel@edk2.groups.io; Yunhua Feng
Cc: Liming Gao
Subject: RE: [PATCH]
MpInitLib contains a function MicrocodeDetect() which is called by
all threads as an AP procedure.
Today this function contains below code:
if (CurrentRevision != LatestRevision) {
AcquireSpinLock(>MpLock);
DEBUG ((
EFI_D_ERROR,
"Updated microcode signature
Fix replace file failure when FFS in multiple level FV and FV level over 2
Cc: Bob Feng
Cc: Liming Gao
Signed-off-by: GregX Yeh
---
BaseTools/Source/C/FMMT/FmmtLib.c | 56 ++-
1 file changed, 37 insertions(+), 19 deletions(-)
diff --git
*Reminder:* TianoCore Bug Triage - APAC / NAMO
*When:* Tuesday, 16 March 2021, 6:30pm to 7:30pm, (GMT-07:00) America/Los
Angeles
*Where:*
https://meetingsamer34.webex.com/meetingsamer34/j.php?MTID=mb96c5bd411bd010e1e6d43a6f6c65f45
View Event (
Hi Bob,
I have no concern about this patch.
Thanks,
Yunhua
-邮件原件-
发件人: devel@edk2.groups.io 代表 Bob Feng
发送时间: 2021年3月15日 12:35
收件人: Yeh, GregX ; devel@edk2.groups.io; Yunhua Feng
抄送: Liming Gao
主题: Re: [edk2-devel] [PATCH] [edk2-staging] BaseTools/FMMT: Replace file
failure when FV
Abner has reviewed and approved everything now.
Other people only reviewed v2 5/5.
Does anybody else have any comments?
From: devel@edk2.groups.io on behalf of Daniel Schaefer
Sent: Thursday, March 4, 2021 11:21
To: devel@edk2.groups.io
Cc: G Edhaya Chandran ;
Hi Pedro,
Great to meet you and welcome to the TianoCore project! Glad you hear you are
interested! A MinPlatform board port can go in two wildly different directions,
depending on the answer to one question: Does MinPlatform currently have
support for the Processor/Chipset/SoC used by the
From: Matthew Carlson
Signed-off-by: Matthew Carlson
This added the compiler flags that were already defined for X64 and IA32.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2821
Ref: https://github.com/tianocore/edk2/pull/1493
Cc: Jiewen Yao
Cc: Jian J Wang
Cc: Xiaoyu Lu
Cc:
Small patch series that fixes a bugzilla.
V2: Actually added the patch information.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2821
Cc: Jiewen Yao
Cc: Jian J Wang
Cc: Xiaoyu Lu
Cc: Guomin Jiang
Cc: devel@edk2.groups.io
Matthew Carlson (1):
CryptoPkg: Added CC flags for ARM
May I know where the patch is?
> -Original Message-
> From: devel@edk2.groups.io On Behalf Of Matthew
> Carlson
> Sent: Tuesday, March 16, 2021 5:09 AM
> To: devel@edk2.groups.io
> Cc: Matthew Carlson ; Matthew Carlson
> ; Yao, Jiewen ; Wang,
> Jian J ; Lu, XiaoyuX ; Jiang,
> Guomin
>
From: Matthew Carlson
Signed-off-by: Matthew Carlson
This added the compiler flags that were already defined for X64 and IA32.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2821
Ref: https://github.com/tianocore/edk2/pull/1493
Cc: Jiewen Yao
Cc: Jian J Wang
Cc: Xiaoyu Lu
Cc:
Small patch series that fixes a bugzilla.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2821
Cc: Jiewen Yao
Cc: Jian J Wang
Cc: Xiaoyu Lu
Cc: Guomin Jiang
Cc: devel@edk2.groups.io
Matthew Carlson (1):
CryptoPkg: Added CC flags for ARM on IntrinsicLib
From: Matthew Carlson
Signed-off-by: Matthew Carlson
This added the compiler flags that were already defined for X64 and IA32.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2821
Ref: https://github.com/tianocore/edk2/pull/1493
Cc: Jiewen Yao
Cc: Jian J Wang
Cc: Xiaoyu Lu
Cc:
Small patch series that fixes a bugzilla.
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=2821
Cc: Jiewen Yao
Cc: Jian J Wang
Cc: Xiaoyu Lu
Cc: Guomin Jiang
Cc: devel@edk2.groups.io
Matthew Carlson (1):
CryptoPkg: Added CC flags for ARM on IntrinsicLib
Signed-off-by: Ling Jia
Reviewed-by: Leif Lindholm
---
Maintainers.txt | 8
1 file changed, 8 insertions(+)
diff --git a/Maintainers.txt b/Maintainers.txt
index 2e6e87bb6dab..e81e55c456dd 100644
--- a/Maintainers.txt
+++ b/Maintainers.txt
@@ -322,3 +322,11 @@ F: Silicon/SiFive/
M:
This changes modify ArmPkg to prepare building StandaloneMm
firmware for 32bit Arm architectures, factorizing the AArch64
implementation.
This change adds MmCommunicationDxe driver and ArmMmuPeiLib
and ArmmmuStandaloneMmLib libraries to the list of the
standard components build for ArmPkg on when
The modules could be runed at the silicon of FT2000/4.
They supported Acpi parameter configuration, Pci bus scaning,
flash read-write and erase abd operating system boot function.
Maintainers.txt: Added maintainers and reviewers for the DurianPkg.
The public git repository is :
Use intermediate (UINTN) cast when casting int from/to pointer. This
is needed as UINT64 values cast from/to 32bit pointer for 32bit
architectures.
Signed-off-by: Etienne Carriere
---
.../StandaloneMmCpu/AArch64/StandaloneMmCpu.c | 8
.../AArch64/CreateHobList.c
This driver installs SMBIOS information for FT2000/4.
Signed-off-by: Ling Jia
Reviewed-by: Leif Lindholm
---
Platform/Phytium/DurianPkg/DurianPkg.dsc|
6 +
Platform/Phytium/DurianPkg/DurianPkg.fdf|
6 +
Signed-off-by: Ling Jia
Reviewed-by: Leif Lindholm
---
Maintainers.txt | 8
1 file changed, 8 insertions(+)
diff --git a/Maintainers.txt b/Maintainers.txt
index afbd2cff0e9c..b6cfe74e09da 100644
--- a/Maintainers.txt
+++ b/Maintainers.txt
@@ -321,3 +321,11 @@ F: Silicon/SiFive/
M:
The RealTimeClockLib implemented EFI RealTimeClock
runtime services via RTC Lib.
v3:
Optimized the codes to conform to specifications.
Signed-off-by: Ling Jia
Reviewed-by: Leif Lindholm
---
Silicon/Phytium/PhytiumCommonPkg/PhytiumCommonPkg.dec |
1 +
The modules could be runed at the silicon of FT2000/4.
They supported Acpi parameter configuration, Pci bus scaning,
flash read-write and erase abd operating system boot function.
Maintainers.txt: Added maintainers and reviewers for the DurianPkg.
The public git repository is :
This change allows to build StandaloneMmPkg components for 32bit Arm
StandaloneMm firmware.
This change mainly moves AArch64/ source files to Arm/ side directory
for several components: StandaloneMmCpu, StandaloneMmCoreEntryPoint
and StandaloneMmMemLib. The source file is built for both 32b and
Change GenFv for Arm architecture to generate a specific jump
instruction as image entry instruction when the target entry label
is assembled with Thumb instruction set. This is possible since
SecCoreEntryAddress value fetched from the PE32 as its LSBit set when
the entry instruction executes in
The FlashFvbDxe provided the fvb protocol,
which requested by the flash operators.
v3:
Optimized the codes to conform to specifications.
Signed-off-by: Ling Jia
Reviewed-by: Leif Lindholm
---
Platform/Phytium/DurianPkg/DurianPkg.dsc |1 +
Defines ARM_SVC_ID_FFA_* and ARM_SVC_ID_SP_* identifiers for 32bit
function IDs as per FF-A specification [1]. Defines also generic ARM
SVC identifier macros to wrap 32bit or 64bit identifiers upon target
built architecture. This will allow to factorize 64bit and 32bit in
packages implementation.
The Pci host bridge library is mainly
to get Pci bridge information.
v3:
Optimize the codes of PciHostBridgeLib.c to conform to specifications.
Signed-off-by: Ling Jia
Reviewed-by: Leif Lindholm
---
Platform/Phytium/DurianPkg/DurianPkg.dsc |
9 +
The FlashFvbDxe provided the fvb protocol,
which requested by the flash operators.
v3:
Optimized the codes to conform to specifications.
Signed-off-by: Ling Jia
Reviewed-by: Leif Lindholm
---
Platform/Phytium/DurianPkg/DurianPkg.dsc |1 +
This driver installs SMBIOS information for FT2000/4.
Signed-off-by: Ling Jia
Reviewed-by: Leif Lindholm
---
Platform/Phytium/DurianPkg/DurianPkg.dsc|
6 +
Platform/Phytium/DurianPkg/DurianPkg.fdf|
6 +
Added Acpi driver and table to FT2000/4,
the ACPI Tables providing library AcpiTables.inf uses
a lot of information that is available in the form of PCDs
for differnt platforms.
v3:
Optimize code to conform to specifications.
Signed-off-by: Ling Jia
Reviewed-by: Leif Lindholm
---
Added Acpi driver and table to FT2000/4,
the ACPI Tables providing library AcpiTables.inf uses
a lot of information that is available in the form of PCDs
for differnt platforms.
v3:
Optimize code to conform to specifications.
Signed-off-by: Ling Jia
Reviewed-by: Leif Lindholm
---
The PCI Segment Library for Phytium platform.
with multiple RCs.
Signed-off-by: Ling Jia
Reviewed-by: Leif Lindholm
---
Platform/Phytium/DurianPkg/DurianPkg.dsc|9 +-
Silicon/Phytium/FT2000-4Pkg/Library/PciSegmentLib/PciSegmentLib.inf | 28 +
The RealTimeClockLib implemented EFI RealTimeClock
runtime services via RTC Lib.
v3:
Optimized the codes to conform to specifications.
Signed-off-by: Ling Jia
Reviewed-by: Leif Lindholm
---
Silicon/Phytium/PhytiumCommonPkg/PhytiumCommonPkg.dec |
1 +
The SpiNorFlashDxe provided norflash initialization,
read-write, erase and other interfaces.
v3:
Optimized the codes to conform to specifications.
Signed-off-by: Ling Jia
Reviewed-by: Leif Lindholm
---
Silicon/Phytium/PhytiumCommonPkg/PhytiumCommonPkg.dec | 1 +
From: Takuto Naito
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3224
- Fix the path of TigerLakeFspBinPkg
- Fix misuse of RETURN_ERROR
- Remove unused function CheckNationalSio.
Cc: Sai Chaganty
Cc: Nate DeSimone
Cc: Heng Luo
Signed-off-by: Takuto Naito
Reviewed-by: Heng Luo
The SpiDxe is to provide Spi bus read-write interfaces.
v3:
Optimized the codes to conform to specifications.
Signed-off-by: Ling Jia
Reviewed-by: Leif Lindholm
---
Silicon/Phytium/PhytiumCommonPkg/PhytiumCommonPkg.dec | 9 +
Platform/Phytium/DurianPkg/DurianPkg.dsc
The SpiNorFlashDxe provided norflash initialization,
read-write, erase and other interfaces.
v3:
Optimized the codes to conform to specifications.
Signed-off-by: Ling Jia
Reviewed-by: Leif Lindholm
---
Silicon/Phytium/PhytiumCommonPkg/PhytiumCommonPkg.dec | 1 +
The SpiDxe is to provide Spi bus read-write interfaces.
v3:
Optimized the codes to conform to specifications.
Signed-off-by: Ling Jia
Reviewed-by: Leif Lindholm
---
Silicon/Phytium/PhytiumCommonPkg/PhytiumCommonPkg.dec | 9 +
Platform/Phytium/DurianPkg/DurianPkg.dsc
The PCI Segment Library for Phytium platform.
with multiple RCs.
Signed-off-by: Ling Jia
Reviewed-by: Leif Lindholm
---
Platform/Phytium/DurianPkg/DurianPkg.dsc|9 +-
Silicon/Phytium/FT2000-4Pkg/Library/PciSegmentLib/PciSegmentLib.inf | 28 +
The PlatformLib supported the system library for FT2000/4 chip.
Platform/Phytium: Added the dsc and fdf files of DurianPkg.
v3:
DurianPkg.dsc:Added OrderedCollectionLib to upstream changes in
edk2, and some parameters omitted in V2 version.
Signed-off-by: Ling Jia
Reviewed-by: Leif Lindholm
From: "Liu, Zhiguang"
Currently, load option is only sorted when setup is the first priority in boot
option. However, Below change in UefiBootManagerLib puts setup in the end, which
causes the sort function won't be called.
MdeModulePkg/UefiBootManagerLib: Put BootMenu at the end of BootOrder
The PlatformLib supported the system library for FT2000/4 chip.
Platform/Phytium: Added the dsc and fdf files of DurianPkg.
v3:
DurianPkg.dsc:Added OrderedCollectionLib to upstream changes in
edk2, and some parameters omitted in V2 version.
Signed-off-by: Ling Jia
Reviewed-by: Leif Lindholm
Added Acpi driver and table to FT2000/4,
the ACPI Tables providing library AcpiTables.inf uses
a lot of information that is available in the form of PCDs
for differnt platforms.
v3:
Optimize code to conform to specifications.
Signed-off-by: Ling Jia
Reviewed-by: Leif Lindholm
---
The Pci host bridge library is mainly
to get Pci bridge information.
v3:
Optimize the codes of PciHostBridgeLib.c to conform to specifications.
Signed-off-by: Ling Jia
Reviewed-by: Leif Lindholm
---
Platform/Phytium/DurianPkg/DurianPkg.dsc |
9 +
Re-sent because I forgot to add Liming's 'reviewd-by'.
Sorry for the spam.
Regards,
Pierre
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#72808): https://edk2.groups.io/g/devel/message/72808
Mute This Topic:
From: Pierre Gondois
The Ecc tool checks the format of the include guard. This check is
currently done on all the names following the '#ifndef' statement.
It should only be done on the first include guard.
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3252
Signed-off-by: Pierre Gondois
Hi Ard/Leif,
Could you please help review the patch when you have a chance? Thanks in
advance.
Regards,
Kun
On 03/10/2021 12:07, Kun Qin wrote:
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3250
This change added NULL MmUnblockMemoryLib instance in DeveloperBox.dsc to
resolve new
Hi Zailiang/Yi,
Could you please help review the patch below? Any input is appreciated.
Thanks,
Kun
On 03/10/2021 12:07, Kun Qin wrote:
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3250
This change added NULL MmUnblockMemoryLib instance in PlatformPkg dsc
file to resolve new
Reviewed-by: Maurice Ma
Regards
Maurice
> -Original Message-
> From: Bi, Dandan
> Sent: Sunday, March 14, 2021 20:59
> To: devel@edk2.groups.io
> Cc: Ma, Maurice ; Dong, Guo
> ; You, Benjamin
> Subject: [patch 25/28] UefiPayloadPkg: Consume RegisterFilterLibNull
> instance
>
> REF:
Hi Ling,
It would appear that you have merged master into your branch instead
of rebasing your branch on top of current master.
The most practical solution I can think of for getting out of this
bind is to:
- Create a new branch, based on current master.
- Use git cherry-pick to copy your
Hello Ard, Leif,
Does the patchset look ok for you ?
Just in case: there is a dependency of this patchset over a patch for
edk2-platforms with: https://edk2.groups.io/g/devel/message/71618
Regards,
Pierre
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3265
Support collecting cache associative type in CpuCacheInfoLib.
This prevents the user from using additional code to obtain the
same information.
Signed-off-by: Jason Lou
Cc: Ray Ni
Cc: Eric Dong
Cc: Laszlo Ersek
Cc: Rahul Kumar
---
Hello Ard, Leif,
Does the patchset look ok for you ?
Regards,
Pierre
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#72800): https://edk2.groups.io/g/devel/message/72800
Mute This Topic: https://groups.io/mt/80905918/21656
Group
From: Pierre Gondois
Correct some minor ordering/comments issues in ArmNameSpaceObjects.h.
The changes can be seen at:
https://github.com/PierreARM/edk2/tree/1685_CheckUpArmNameSpaceObjects_v1
Pierre Gondois (2):
DynamicTablesPkg: Re-order GicItsIdentifierArray struct
DynamicTablesPkg:
From: Pierre Gondois
The EArmObjExtendedInterruptInfo doesn't exist. Remove any reference
to this enum.
Signed-off-by: Pierre Gondois
---
The changes can be seen at:
https://github.com/PierreARM/edk2/tree/1685_CheckUpArmNameSpaceObjects_v1
DynamicTablesPkg/Include/ArmNameSpaceObjects.h | 2
From: Pierre Gondois
The structure is not correctly placed in the file. Move it
so it follows the EARM_OBJECT_ID enum order.
Signed-off-by: Pierre Gondois
---
The changes can be seen at:
https://github.com/PierreARM/edk2/tree/1685_CheckUpArmNameSpaceObjects_v1
Hi,
On Thu, Feb 25, 2021 at 4:38 PM PierreGondois wrote:
>
> From: Pierre Gondois
>
> The DependencyCheck available in .pytool detects an unnecessary
> dependency of the NorFlashStandaloneMm.inf module over the
> EmbeddedPkg package.
> This patch removes this dependency.
>
> Signed-off-by:
> -Original Message-
> From: Chang, Abner (HPS SW/FW Technologist)
> Sent: Monday, March 15, 2021 1:21 PM
> To: devel@edk2.groups.io; Bi, Dandan
> Cc: Kinney, Michael D ; Liming Gao
> ; Liu, Zhiguang
> Subject: RE: [edk2-devel] [patch 01/28] MdePkg: Add RegisterFilterLib class
> and
From: Pierre Gondois
The Ecc tool checks the format of the include guard. This check is
currently done on all the names following the '#ifndef' statement.
It should only be done on the first include guard.
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3252
Signed-off-by: Pierre Gondois
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3265
Support collecting cache associative type in CpuCacheInfoLib.
This prevents the user from using additional code to obtain the
same information.
Signed-off-by: Jason Lou
Cc: Ray Ni
Cc: Eric Dong
Cc: Laszlo Ersek
Cc: Rahul Kumar
---
> -Original Message-
> From: Bi, Dandan
> Sent: Monday, March 15, 2021 11:58 AM
> To: devel@edk2.groups.io
> Cc: Wang, Jian J ; Wu, Hao A
> Subject: [patch 13/28] MdeModulePkg: Consume RegisterFilterLibNull
> instance
>
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
>
> Add
> -Original Message-
> From: Bi, Dandan
> Sent: Monday, March 15, 2021 11:59 AM
> To: devel@edk2.groups.io
> Cc: Wu, Hao A
> Subject: [patch 22/28] SourceLevelDebugPkg: Consume RegisterFilterLibNull
> instance
>
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
>
> Add
67 matches
Mail list logo
