Re: [edk2-devel] [PATCH 2/5] Platform/RaspberryPi: Expand locking to cover return data

2021-10-01 Thread Andrei Warkentin
Yikes - that's embarrassing. Thanks for the fix. Reviewed-by: Andrei Warkentin From: Jeremy Linton Sent: Friday, October 1, 2021 7:52 PM To: devel@edk2.groups.io Cc: p...@akeo.ie ; ardb+tianoc...@kernel.org ; l...@nuviainc.com ; Andrei Warkentin ; sunny.w...@a

Re: [edk2-devel] [PATCH 5/5] Platform/RaspberryPi: Disconnect/shutdown all drivers before reboot

2021-10-01 Thread Andrei Warkentin
Seems smart to do. Reviewed-by: Andrei Warkentin From: Jeremy Linton Sent: Friday, October 1, 2021 7:52 PM To: devel@edk2.groups.io Cc: p...@akeo.ie ; ardb+tianoc...@kernel.org ; l...@nuviainc.com ; Andrei Warkentin ; sunny.w...@arm.com ; samer.el-haj-mahm...

Re: [edk2-devel] [PATCH 4/5] Platform/RaspberryPi: Normal memory should not be marked as uncached

2021-10-01 Thread Andrei Warkentin
I may have misunderstood the flags as being valid ways of mapping the added range. Should we also then take out WC and WT? From: Jeremy Linton Sent: Friday, October 1, 2021 7:52 PM To: devel@edk2.groups.io Cc: p...@akeo.ie ; ardb+tianoc...@kernel.org ; l...@nuvi

Re: [edk2-devel] [edk2-platforms][PATCH v2 1/1] MinPlatformPkg/AcpiTables: Update structures for ACPI 6.3

2021-10-01 Thread Nate DeSimone
Pushed: https://github.com/tianocore/edk2-platforms/commit/bbb431a > -Original Message- > From: mikub...@linux.microsoft.com > Sent: Friday, September 24, 2021 11:23 AM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Liming Gao > ; Dong, Eric ; Maddy, > Daniel ;

Re: [edk2-devel] [PATCH 3/5] Platform/RaspberryPi: Update Linux quirk name

2021-10-01 Thread Andrei Warkentin
Reviewed-by: Andrei Warkentin From: Jeremy Linton Sent: Friday, October 1, 2021 7:52 PM To: devel@edk2.groups.io Cc: p...@akeo.ie ; ardb+tianoc...@kernel.org ; l...@nuviainc.com ; Andrei Warkentin ; sunny.w...@arm.com ; samer.el-haj-mahm...@arm.com ; Jeremy Li

Re: [edk2-devel] [edk2-platforms][PATCH v2 1/1] MinPlatformPkg/AcpiTables: Update structures for ACPI 6.3

2021-10-01 Thread Nate DeSimone
Reviewed-by: Nate DeSimone > -Original Message- > From: mikub...@linux.microsoft.com > Sent: Friday, September 24, 2021 11:23 AM > To: devel@edk2.groups.io > Cc: Chiu, Chasel ; Desimone, Nathaniel L > ; Liming Gao > ; Dong, Eric ; Maddy, > Daniel ; Michael Kubacki > > Subject: [edk2-pla

Re: [edk2-devel] [PATCH 1/5] Platform/RaspberryPi: Fix vfr warning caused by two defaults

2021-10-01 Thread Andrei Warkentin
Reviewed-by: Andrei Warkentin From: Jeremy Linton Sent: Friday, October 1, 2021 7:52 PM To: devel@edk2.groups.io Cc: p...@akeo.ie ; ardb+tianoc...@kernel.org ; l...@nuviainc.com ; Andrei Warkentin ; sunny.w...@arm.com ; samer.el-haj-mahm...@arm.com ; Jeremy Li

[edk2-devel] [PATCH 5/5] Platform/RaspberryPi: Disconnect/shutdown all drivers before reboot

2021-10-01 Thread Jeremy Linton
In theory we should be properly cleaning up all the device drivers before pulling the big switch. Particularly the partition mgr will issue flush commands to attached disks as it goes down. This assures that devices running in WB mode (that correctly handle flush/sync/etc) commands are persisted to

[edk2-devel] [PATCH 4/5] Platform/RaspberryPi: Normal memory should not be marked as uncached

2021-10-01 Thread Jeremy Linton
The efi spec seems to indicate that the efi uncacheable attribute should be mapped to device memory rather than normal-nc. This means that the uefi mem attribute for the >3G ram doesn't match the remainder of the RAM in the machine. So, lets remove the uncacheable attribute to make it more consist

[edk2-devel] [PATCH 3/5] Platform/RaspberryPi: Update Linux quirk name

2021-10-01 Thread Jeremy Linton
During review/merge of the linux ecam quirk, some logic was added to require the quirk name to be exactly 6 characters, matching the MADT field its overriding. As such, the rpi quirk here needed to be shorted by a character to avoid confusion. Signed-off-by: Jeremy Linton --- Platform/Raspberry

[edk2-devel] [PATCH 1/5] Platform/RaspberryPi: Fix vfr warning caused by two defaults

2021-10-01 Thread Jeremy Linton
The build has been tossing a warning about having two defaults for a while now, lets fix it. Signed-off-by: Jeremy Linton --- Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxeHii.vfr | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Platform/RaspberryPi/Drivers/ConfigDxe/ConfigD

[edk2-devel] [PATCH 2/5] Platform/RaspberryPi: Expand locking to cover return data

2021-10-01 Thread Jeremy Linton
While debugging problems with the GET/SET_CLOCK mailbox calls it appeared that the locking in most of the mailbox commands isn't perfectly correct. All UEFI firmware calls to the RPi mailbox share a single mDmaBuffer which is used to fill out the command passed to the vc firmware, and record its re

[edk2-devel] [PATCH 0/5] Platform/Rpi: Various cleanups

2021-10-01 Thread Jeremy Linton
This set is a few patches I've been collecting to fix minor issues I've seen while debugging other problems, or just various things I think should probably be changed. Generally I don't think they fix anything that is currently user visible, although its quite possible some mysterious failures go a

[edk2-devel] [PATCH v2 1/1] Platform/RaspberryPi: Always use non translating DMA in DT mode

2021-10-01 Thread Jeremy Linton
One of the many issues with the PCIe on this platform is its inbound DMA is either constrained to the lower 3G, or on later SoC's a translation can be used. That translation was problematic with some of the OS's expected to boot on this platform. So, across the board a 3G DMA limit is enforced duri

[edk2-devel] [PATCH v2 0/1] Fix DT boot on rpi4

2021-10-01 Thread Jeremy Linton
Some rpi4's haven't booted cleanly using DT for a few months. The primarly problem is that the XHCI_NOTIFY_RESET mailbox call doesn't seem to be able to correctly read the pcie translation state after we have set it up. So, while linux appears to be able to adjust the PCIe subsystem to the demands

Re: [edk2-devel] [PATCH] IntelFsp2Pkg: Adopt FSP 2.3 specification.

2021-10-01 Thread Nate DeSimone
Hi Chasel, My only feedback is let's not add the EDK1 style GUID definition to FspNonVolatileStorageHob2.h. It is redundant with the EDK2 style definition. Please see inline. With that change... Reviewed-by: Nate DeSimone > -Original Message- > From: Chiu, Chasel > Sent: Friday, Oct

Re: [edk2-devel] [PATCH v1 9/9] ArmVirtPkg: Kvmtool: Add RNG support using FW-TRNG interface

2021-10-01 Thread Joey Gouly
Hi Sami, > > The EFI_RNG_PROTOCOL published by RngDxe has been updated to > implement the EFI_RNG_ALGORITHM_RAW using the Arm FW-TRNG > interface to provide access to entropy. > > Therefore, enable EFI_RNG_PROTOCOL for the Kvmtool guest/virtual > firmware. > > Signed-off-by: Sami Mujawar > --- >

Re: [edk2-devel] [PATCH edk2-platforms v1 2/2] Platform/ARM/Juno: Add RNG support using FW-TRNG interface

2021-10-01 Thread Joey Gouly
Hi Sami, > TF-A for Juno has been updated to implement the Arm FW-TRNG interface > that can be used to access entropy from the TRNG hardware on Juno. > > Similarly, the EFI_RNG_PROTOCOL in RngDxe has been updated to add > EFI_RNG_ALGORITHM_RAW support using the FW-TRNG interface. > > Therefore, en

Re: [edk2-devel] [PATCH v1 11/13] DynamicTablesPkg: AML code generation to add an _LPI state

2021-10-01 Thread Sami Mujawar
Hi Pierre, Please find my response inline marked [SAMI]. Regards, Sami Mujawar On 23/06/2021 12:40 PM, pierre.gond...@arm.com wrote: From: Pierre Gondois Add AmlAddLpiState() to generates AML code to add an _LPI state to an _LPI object created using AmlCreateLpiNode(). AmlAddLpiState incre

Re: [edk2-devel] [PATCH v1 10/13] DynamicTablesPkg: AML code generation for a _LPI object

2021-10-01 Thread Sami Mujawar
Hi Pierre, Please find my response inline marked [SAMI]. Regards, Sami Mujawar On 23/06/2021 12:40 PM, pierre.gond...@arm.com wrote: From: Pierre Gondois _LPI object provides a method to describe Low Power Idle states that define the local power states for each node in a hierarchical proce

Re: [edk2-devel] [PATCH v1 09/13] DynamicTablesPkg: AML code generation for a Method returning a NS

2021-10-01 Thread Sami Mujawar
Hi Pierre, Please find my response inline marked [SAMI]. Regards, Sami Mujawar On 23/06/2021 12:40 PM, pierre.gond...@arm.com wrote: From: Pierre Gondois Add AmlCodeGenMethodRetNameString() to generate AML code to create a Method returning a NameString (NS). AmlCodeGenMethodRetNameString

Re: [edk2-devel] [PATCH v1 08/13] DynamicTablesPkg: AML code generation to Return a NameString

2021-10-01 Thread Sami Mujawar
Hi Pierre, Please find my feedback inline marked [SAMI]. Regards, Sami Mujawar On 23/06/2021 12:40 PM, pierre.gond...@arm.com wrote: From: Pierre Gondois Add AmlCodeGenReturnNameString() to generate AML code for a Return object node, returning the object as a NameString. AmlCodeGenReturn

Re: [edk2-devel] [PATCH v1 07/13] DynamicTablesPkg: AML code generation for a Method

2021-10-01 Thread Sami Mujawar
Hi Pierre, I have a minor suggestion marked inline as [SAMI]. Otherwise this patch looks good to me. Reviewed-by: Sami Mujawar Regards, Sami Mujawar On 23/06/2021 12:40 PM, pierre.gond...@arm.com wrote: From: Pierre Gondois Add AmlCodeGenMethod() to generate code for a control method.

Re: [edk2-devel] [PATCH v1 01/13] DynamicTablesPkg: Make AmlNodeGetIntegerValue public

2021-10-01 Thread Sami Mujawar
Hi Pierre, Thank you for this patch. This patch looks good to me. Reviewed-by: Sami Mujawar Regards, Sami Mujawar On 23/06/2021 12:40 PM, pierre.gond...@arm.com wrote: From: Pierre Gondois Remove the STATIC qualifier for the AmlUtility function AmlNodeGetIntegerValue() and add the definit

Re: [edk2-devel] [PATCH v1 06/13] DynamicTablesPkg: AML code generation for a ResourceTemplate

2021-10-01 Thread Sami Mujawar
Hi Pierre, Thank you for this patch. Reviewed-by: Sami Mujawar Regards, Sami Mujawar On 23/06/2021 12:40 PM, pierre.gond...@arm.com wrote: From: Pierre Gondois ASL provides a ResourceTemplate macro that creates a Buffer in which resource descriptor macros can be listed. The ResourceTempl

Re: [edk2-devel] [PATCH v1 05/13] DynamicTablesPkg: Helper function to compute package length

2021-10-01 Thread Sami Mujawar
Hi Pierre, This patch looks good to me. Reviewed-by: Sami Mujawar Regards, Sami Mujawar On 23/06/2021 12:40 PM, pierre.gond...@arm.com wrote: From: Pierre Gondois Some AML object have a PkgLen which indicates the size of the AML object. The package length can be encoded in 1 to 4 bytes.

Re: [edk2-devel] [PATCH v1 04/13] DynamicTablesPkg: AML code generation for a Package

2021-10-01 Thread Sami Mujawar
Hi Pierre, Thank you for this patch. This patch looks good to me. Reviewed-by: Sami Mujawar Regards, Sami Mujawar On 23/06/2021 12:40 PM, pierre.gond...@arm.com wrote: From: Pierre Gondois Add AmlCodeGenPackage() to generate AML code for declaring a Package() object. This function genera

Re: [edk2-devel] [PATCH v1 03/13] DynamicTablesPkg: AML Code generation for Resource data EndTag

2021-10-01 Thread Sami Mujawar
Hi Pierre, Thank you for this patch. Please find my response inline marked [SAMI]. Regards, Sami Mujawar On 23/06/2021 12:40 PM, pierre.gond...@arm.com wrote: From: Pierre Gondois Add a helper function AmlCodeGenEndTag() to generate AML Resource Data EndTag. The EndTag resource data is au

Re: [edk2-devel] [PATCH v1 02/13] DynamicTablesPkg: AML Code generation for Register()

2021-10-01 Thread Sami Mujawar
Hi Pierre, Thank you for this patch. I have a minor suggestion marked inline as [SAMI]. With that changed, Reviewed-by: Sami Mujawar Regards, Sami Mujawar On 23/06/2021 12:40 PM, pierre.gond...@arm.com wrote: From: Pierre Gondois Add AmlCodeGenRegister() to generate AML code for the Ge

Re: [edk2-devel] [PATCH v2 00/10] Various DynamicTablesPkg modifications

2021-10-01 Thread Sami Mujawar
Merged as 422e5d2f7f1a..22873f58c40c Regards, Sami Mujawar On 30/09/2021 08:48 AM, pierre.gond...@arm.com wrote: From: Pierre Gondois This patch-set aggregates various modifications in the DynamicTablesPkg: - Extract an AcpiTableHelperLib from TableHelperLib to remove the dependency o

[edk2-devel] [PATCH] IntelFsp2Pkg: Adopt FSP 2.3 specification.

2021-10-01 Thread Chiu, Chasel
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3674 Add ExtendedImageRevision in FSP_INFO_HEADER structure, also add FSP_NON_VOLATILE_STORAGE_HOB2 header. Cc: Maurice Ma Cc: Nate DeSimone Cc: Star Zeng Signed-off-by: Chasel Chiu --- IntelFsp2Pkg/Include/Guid/FspHeaderFile.h |