Great commit message, thanks Sunil! Maintainers, please take a look and let us know if there's any other concern. This patch lets us build the RISC-V platforms using modern toolchains that are

Reminder: TianoCore Bug Triage - APAC / NAMO When: 06/15/2021 6:30pm to 7:30pm (UTC-07:00) America/Los

Hi I am not sure why we hardcode 3 items for each. Can we move this fdf to platform pkg, instead of security pkg ? Thank you Yao Jiewen

Add Mike Turner for review.

Reviewed-by: Jiewen Yao <Jiewen.yao@...>

Hi Marcin, Congratulations! :) Is FreeBSD expected to follow suit? The patches themselves look straightforward enough. I *would* prefer some tested-by, for these sources rather than the binary,

Drop the comment. With that: Reviewed-by: Leif Lindholm <leif@...>

This patch does not touch ConfigurationManager. Please describe what this patch does. My guess is it's along the lines of: This set enables use of the ConfigurationManager framework for

Does it intend to add, or does it add? / Leif

This tells me nothing about what this patch does. Neither does this. And this sound like it should be a separate patch. *However* when I look at the code, this does look like a single change. And

Many of the cache definitions in ArmLibPrivate.h are being used outside of ArmLib, in Universal/Smbios. Move them into ArmCache.h to make them public, and remove the include of ArmLibPrivate.h from

Hi Ard, Sorry for the confusion and thanks for providing the r-b tag. This should be fixed in v2 patches: https://edk2.groups.io/g/devel/message/76479 Regards, Kun

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3445 Per update from Cspell tool, the minimal requirement of Cspell 5.x regarding Node is 12 and above. This has caused multple Cspell

From: Sean Brogan <sean.brogan@...> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3445 Spellcheck was not covering all specified files due to CSpell v5 and Node v10 incompatibility

From: Sean Brogan <sean.brogan@...> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3445 Spellcheck was not covering all specified files due to CSpell v5 and Node v10 incompatibility

From: Sean Brogan <sean.brogan@...> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3445 This change fixed a misspelling that was not caught by spell check. Cc: Sami Mujawar

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3445 This patch series is a follow up of previous submission: https://edk2.groups.io/g/devel/message/76419 v2 patches mainly focus on feedback for

It's neccessary to allocate a Graphics Stolen Memory area to enable GPU-Passthrough for integrated Intel GPUs. Therefore, use a new memory layout with a static Pci32Baseaddress. Old layout: [...