Could someone review these, and if they look good push them please?
Thanks.
Rebecca Cran
On 3/19/22 14:13, Rebecca Cran wrote:
I noticed Linux reports a firmware bug with the current Juno ACPI
tables. These patches fix it by reserving the ECAM area with a RES0
device, while also converting
(defined(__MAC_OS_X_VERSION_MIN_REQUIRED) &&
__MAC_OS_X_VERSION_MIN_REQUIRED >= 101200) || \
(defined(__IPHONE_OS_VERSION_MIN_REQUIRED) &&
__IPHONE_OS_VERSION_MIN_REQUIRED >= 8)
# define OPENSSL_APPLE_CRYPTO_RANDOM 1
# include
# include
#
]
--
Rebecca Cran
On 3/24/22 6:04 AM, Gerd Hoffmann wrote:
In function ‘SetDevicePathEndNode’,
inlined from ‘FileDevicePath’ at DevicePathUtilities.c:857:5:
DevicePathUtilities.c:321:3: error: writing 4 bytes into a region of size 1
[-Werror=stringop-overflow=]
321 | memcpy (Node
emoved (so I can build NOOPT and debug without optimizations
getting in the way).
--
Rebecca Cran
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#88011): https://edk2.groups.io/g/devel/message/88011
Mute This Topic: https://groups.io/m
Lindholm
Cc: Michael D Kinney
Cc: Hao A Wu
Cc: Bob Feng
Cc: Liming Gao
Cc: Yuwei Chen
Signed-off-by: Rebecca Cran
---
BaseTools/Scripts/efi_lldb.py | 1044
1 file changed, 1044 insertions(+)
diff --git a/BaseTools/Scripts/efi_lldb.py b/BaseTools/Scripts/efi_lldb
of "RISCV" as "RISKV".
Testing:
- Tested gdb on Ubuntu and lldb on macOS for IA32 and X64.
- Tested gdb on openSUSE for AARCH64.
Rebecca Cran (3):
BaseTools: efi_debugging.py: Add debugger agnostic dbg Python Classes
BaseTools: Scripts/efi_gdb.py: Add gdb EFI commands and pret
Gao
Cc: Yuwei Chen
Signed-off-by: Rebecca Cran
---
BaseTools/Scripts/efi_gdb.py | 918
1 file changed, 918 insertions(+)
diff --git a/BaseTools/Scripts/efi_gdb.py b/BaseTools/Scripts/efi_gdb.py
new file mode 100755
index ..e9bae8e9b913
--- /dev/null
+++
Add Scripts/efi_debugging.py to provide debugger agnostic debugging
utility Python classes.
Cc: Leif Lindholm
Cc: Michael D Kinney
Cc: Hao A Wu
Cc: Bob Feng
Cc: Liming Gao
Cc: Yuwei Chen
Signed-off-by: Rebecca Cran
---
BaseTools/Scripts/efi_debugging.py | 2185
1 file
To build TF-A for QEMU Sbsa PLAT=qemu_sbsa should now be specified.
Signed-off-by: Rebecca Cran
---
Platform/Qemu/SbsaQemu/Readme.md | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/Platform/Qemu/SbsaQemu/Readme.md b/Platform/Qemu/SbsaQemu/Readme.md
index 692f9702de56
The canonical QEMU Git URL isn't GitHub, but GitLab.
Signed-off-by: Rebecca Cran
---
Platform/Qemu/SbsaQemu/Readme.md | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Platform/Qemu/SbsaQemu/Readme.md b/Platform/Qemu/SbsaQemu/Readme.md
index c514c33b7c2f..692f9702de56 100644
Fix some issues with the SbsaQemu Readme.md file.
Rebecca Cran (3):
Platform/Qemu: Update SbsaQemu Readme.md to change 'ATF' to 'TF-A'
Platform/Qemu: Update SbsaQemu Readme.md with current QEMU git URL
Platform/Qemu: Update SbsaQemu Readme.md with current TF-A build PLAT
Platform/Qemu
Trusted Firmware is no longer called ATF (ARM Trusted Firmware) but is
now TF-A (Trusted Firmware for A-profile architecture). Update
Readme.md to match.
Signed-off-by: Rebecca Cran
---
Platform/Qemu/SbsaQemu/Readme.md | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
nt warning is displayed:
"system 00:00: [mem 0x4000-0x4fff window] could not be reserved"
It appears this is a harmless message though: a discussion about it can
be seen at https://lore.kernel.org/all/20210603141641.GA17284@lpieralisi/#t
Signed-off-by: Rebecca Cran
Tested-by: Sa
Instead of using a custom Pcd for the ECAM address
(gArmJunoTokenSpaceGuid.PcdPciConfigurationSpaceBaseAddress),
use the Pcd from MdePkg.
Signed-off-by: Rebecca Cran
Reviewed-by: Sami Mujawar
---
Platform/ARM/JunoPkg/ArmJuno.dec
| 4
Other .asl files in Platform/ARM/JunoPkg/AcpiTables use spaces, while
AcpiSsdtRootPci.asl uses tabs. To be consistent, convert it to spaces.
Signed-off-by: Rebecca Cran
Reviewed-by: Sami Mujawar
---
Platform/ARM/JunoPkg/AcpiTables/AcpiSsdtRootPci.asl | 288 ++--
1 file changed
:
- Removed the namespace prefix from the Pcd call in 2/3.
- Updated the commit message in 3/3 to link to discussion about the new
warning message.
Rebecca Cran (3):
Platform/ARM/JunoPkg: Convert AcpiSsdtRootPci.asl from tabs to spaces
Platform/ARM/JunoPkg: Use MdePkg PcdPciExpressBaseAddress
On 3/17/22 03:55, Sami Mujawar wrote:
On 05/03/2022 04:19 AM, Rebecca Cran wrote:
Add a RES0 device to the SSDT to reserve the PCI ECAM area.
This fixes the warning that Linux prints:
acpi PNP0A08:00: [Firmware Bug]: ECAM area [mem 0x4000-0x4fff]
not reserved in ACPI namespace
[SAMI
(PcdPciExpressBaseAddress).
Oops, yes. I've fixed it in v2.
--
Rebecca Cran
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#87762): https://edk2.groups.io/g/devel/message/87762
Mute This Topic: https://groups.io/mt/89565201/21656
Group Owner: devel+ow
Given the lack of progress on this for many months, I was wondering if I
should perhaps take this over and work on the changes needed to get this
functionality committed?
--
Rebecca Cran
On 2/10/22 12:13, Rebecca Cran wrote:
Andrew: We discussed your patch from August last year in today's
Could someone review this please?
Thanks.
Rebecca Cran
On 3/4/22 21:19, Rebecca Cran wrote:
I noticed Linux reports a firmware bug with the current Juno ACPI
tables. These patches fix it by reserving the ECAM area with a RES0
device, while also converting AcpiSsdtRootPci.asl from tabs
Add a RES0 device to the SSDT to reserve the PCI ECAM area.
This fixes the warning that Linux prints:
acpi PNP0A08:00: [Firmware Bug]: ECAM area [mem 0x4000-0x4fff]
not reserved in ACPI namespace
Signed-off-by: Rebecca Cran
---
Platform/ARM/JunoPkg/AcpiTables/AcpiTables.inf | 4
Other .asl files in Platform/ARM/JunoPkg/AcpiTables use spaces, while
AcpiSsdtRootPci.asl uses tabs. To be consistent, convert it to spaces.
Signed-off-by: Rebecca Cran
---
Platform/ARM/JunoPkg/AcpiTables/AcpiSsdtRootPci.asl | 288 ++--
1 file changed, 144 insertions(+), 144
I noticed Linux reports a firmware bug with the current Juno ACPI
tables. These patches fix it by reserving the ECAM area with a RES0
device, while also converting AcpiSsdtRootPci.asl from tabs to spaces
and using the standard Pcd from MdePkg for the ECAM base address.
Rebecca Cran (3
Instead of using a custom Pcd for the ECAM address
(gArmJunoTokenSpaceGuid.PcdPciConfigurationSpaceBaseAddress),
use the Pcd from MdePkg.
Signed-off-by: Rebecca Cran
---
Platform/ARM/JunoPkg/ArmJuno.dec
| 4 ++--
Platform/ARM
The Readme file in the edk2 repo was changed from Readme.md to
Readme.rst.
Signed-off-by: Rebecca Cran
---
Readme.md | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Readme.md b/Readme.md
index 62876b4b7d75..991acd53ae03 100644
--- a/Readme.md
+++ b/Readme.md
@@ -9,7 +9,7
The Readme file in the edk2 repo was changed from Readme.md to
Readme.rst.
Signed-off-by: Rebecca Cran
---
Readme.md | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/Readme.md b/Readme.md
index 62876b4b7d75..58ab835b1174 100644
--- a/Readme.md
+++ b/Readme.md
@@ -4,12
onnect to azure.archive.ubuntu.com:80 (52.250.76.244),
connection timed out
--
Rebecca Cran
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#86994): https://edk2.groups.io/g/devel/message/86994
Mute This Topic: https://groups.io/mt/89377682/21656
Group Owner: de
ontrollerInit.
--
Rebecca Cran
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#86991): https://edk2.groups.io/g/devel/message/86991
Mute This Topic: https://groups.io/mt/89364574/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe:
Reviewed-by: Rebecca Cran
On 2/7/22 05:21, Leif Lindholm wrote:
NUVIA inc. was acquired by Qualcomm in March 2021, but we continued
contributions under the existing IDs until the start of this year.
We are now switching to use Qualcomm Innovation Center email, so
update Maintainers.txt
any formatted_files?
--
Rebecca Cran
On 2/10/22 09:19, Michael Kubacki wrote:
From: Michael Kubacki
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3808
Changes the default for the "OutputFileDiffs" configuration option
to "True" so the formatting suggestions detected by Un
there weren't actually any reviews on it, so it'll
need to wait until after the freeze.
--
Rebecca Cran
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#86572): https://edk2.groups.io/g/devel/message/86572
Mute This Topic: https://groups.io/mt
Thanks!
Reviewed-by: Rebecca Cran
--
Rebecca Cran
On 2/10/22 09:19, Michael Kubacki wrote:
From: Michael Kubacki
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3808
Changes the default for the "OutputFileDiffs" configuration option
to "True" so the formatting
What version of iasl are you using?
—
Rebecca Cran
On Mon, Feb 7, 2022, at 10:38 PM, Akber Basha J via groups.io wrote:
> Team,
>
> We are facing build issues while trying to build BIOS image on "EDK2
> for Intel TigerlakeURvp platform".
>
> Have attached "
did notice thatDmaLib is missing from ArmPkg/ArmPkg.dsc. -- Rebecca Cran
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#86398): https://edk2.groups.io/g/devel/message/86398
Mute This Topic: https://groups.io/mt/87760630/21656
Group Owner
XCODE5 reported the following warning:
OvmfPkg/Library/VmgExitLib/VmgExitVcHandler.c:1895:12: note:
uninitialized use occurs here
Compacted
^
Initialize the 'Compacted' variable to fix the warning.
Signed-off-by: Rebecca Cran
---
OvmfPkg/Library/VmgExitLib
.
Signed-off-by: Rebecca Cran
---
BaseTools/Conf/tools_def.template | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/BaseTools/Conf/tools_def.template
b/BaseTools/Conf/tools_def.template
index 2e6b382ab623..e839e8602a0b 100755
--- a/BaseTools/Conf/tools_def.template
+++ b
Reviewed-by: Rebecca Cran
On 11/10/21 03:55, Sami Mujawar wrote:
Bugzilla: 3458 - Add support IORT Rev E.b specification updates
(https://bugzilla.tianocore.org/show_bug.cgi?id=3458)
The IO Remapping Table, Platform Design Document, Revision E.b,
Feb 2021 (https
with the file in general, excluding the patch?
No, the patch is wrong: Uncrustify wants the equals signs to line up.
XssMsr.Uint64 = 0;
+Compacted = FALSE;
Should be:
XssMsr.Uint64 = 0;
+Compacted = FALSE;
--
Rebecca Cran
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all
/Library/VmgExitLib/VmgExitVcHandler.c:1871:37: note:
initialize the variable 'Compacted' to silence this warning
BOOLEAN Compacted;
^
= '\0'
--
Rebecca Cran
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You
: variable 'Status' set but not used
[-Werror,-Wunused-but-set-variable]
EFI_STATUS Status;
--
Rebecca Cran
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#86344): https://edk2.groups.io/g/devel/message/86344
Mute This Topic: https
code, before merging.
[/SAMI]
Yes, please do.
Thanks.
Rebecca Cran
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#86340): https://edk2.groups.io/g/devel/message/86340
Mute This Topic: https://groups.io/mt/88400646/21656
Group Owner
change the above line to
RangeDesc.Header.Length = sizeof
(EFI_ACPI_32_BIT_FIXED_MEMORY_RANGE_DESCRIPTOR) -
sizeof (ACPI_LARGE_RESOURCE_HEADER);
before merging?
[/SAMI]
Yes, that would be good.
Thanks.
Rebecca Cran
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io
Sorry, I forgot to add a cover letter.
I noticed some breakages in Arm platforms caused by additions of
VariablePolicyHelperLib: the 3 patches fix them by adding it to
LibraryClasses.common.
--
Rebecca Cran
On 2/1/22 09:56, Rebecca Cran wrote:
The VariablePolicyHelperLib is now used
The VariablePolicyHelperLib is now used by a number of driver types, so
instead of duplicating it, move it into the LibraryClasses.common section.
Signed-off-by: Rebecca Cran
---
Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc | 5 ++---
1 file changed, 2 insertions(+), 3 deletions
The VariablePolicyHelperLib is now used by a number of driver types, so
instead of duplicating it, move it into the LibraryClasses.common section.
Signed-off-by: Rebecca Cran
---
Platform/AMD/OverdriveBoard/OverdriveBoard.dsc | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git
The VariablePolicyHelperLib is now used by a number of driver types, so
instead of duplicating it, move it into the LibraryClasses.common section.
Signed-off-by: Rebecca Cran
---
Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git
It looks like this hasn't been committed yet - I still get a crash when
running acpiview on my Overdrive 1000 system, and see the original code
is still on the latest master.
Could someone commit it please?
--
Rebecca Cran
On 12/14/21 09:08, Sami Mujawar wrote:
Hi Chris,
Thanks
The VariablePolicyHelperLib is now used by a number of driver types, so
instead of duplicating it, move it into the LibraryClasses.common section.
Signed-off-by: Rebecca Cran
---
Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc | 2 +-
1 file changed, 1 insertion(+), 1 deletion
interpreter to use full 64-bit
integers and math.
--
Rebecca Cran
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#86255): https://edk2.groups.io/g/devel/message/86255
Mute This Topic: https://groups.io/mt/88746974/21656
Group Owner: devel
I'm still seeing this error when building using XCode 13.2.1, so I guess
this patch hasn't been committed yet.
--
Rebecca Cran
On 12/13/21 12:38 PM, Brijesh Singh via groups.io wrote:
The XCODE5 reported the below warning
OvmfPkg/Library/VmgExitLib/VmgExitVcHandler.c:1895:12: note
It's been a week since I sent this out: could someone review it please?
Thanks.
Rebecca Cran
On 1/19/22 14:01, Rebecca Cran wrote:
Since the Variable Lock protocol is deprecated, convert locking of
PlatformRecovery in EfiBootManagerLoadOptionToVariable to use the
Variable Policy protocol
Oh, thanks! It turns out there was one remaining use in MdeModulePkg, by
UefiBootManagerLib.
I've sent the following patch to fix it: [PATCH 1/1]
MdeModulePkg/UefiBootManagerLib: Convert BmLoadOption to Variable Policy .
--
Rebecca Cran
On 1/18/22 23:30, gaoliming wrote:
Rebecca
Since the Variable Lock protocol is deprecated, convert locking of
PlatformRecovery in EfiBootManagerLoadOptionToVariable to use the
Variable Policy protocol.
Cc: Jian J Wang
Cc: Liming Gao
Cc: Zhichao Gao
Cc: Ray Ni
Signed-off-by: Rebecca Cran
---
MdeModulePkg/Library
It looks like the BZ associated with this change is
https://bugzilla.tianocore.org/show_bug.cgi?id=3421 .
It would be nice to get this fix committed, to stop the loud deprecated
interface messages.
--
Rebecca Cran
On 6/16/21 18:58, gaoliming wrote:
Ken:
Thanks for your update. Please
[+Leif, Ard]
Kun,
There's often so much mailing list traffic that you need to CC the
maintainers for them to see your emails.
--
Rebecca Cran
On 1/18/22 11:40, Kun Qin wrote:
Hi ArmPkg maintainers,
It has been almost a month since this v2 patch series has been sent
out for review
Add AmlCodeGenMethodRetInteger function to generate AML code for
a Method returning an Integer.
Signed-off-by: Rebecca Cran
Reviewed-by: Pierre Gondois
---
DynamicTablesPkg/Include/Library/AmlLib/AmlLib.h| 47 ++
DynamicTablesPkg/Library/Common/AmlLib/CodeGen/AmlCodeGen.c
In AmlCodeGenReturn, the cast to AML_NODE_HEADER* in the call to
AmlSetFixedArgument is redundant because ReturnNode is already a
AML_NODE_HEADER* .
Signed-off-by: Rebecca Cran
Reviewed-by: Pierre Gondois
---
DynamicTablesPkg/Library/Common/AmlLib/CodeGen/AmlCodeGen.c | 2 +-
1 file changed, 1
Add functions to generate code for the Memory32Fixed ASL macro and a
method returning an Integer.
Remove a redundant cast from AmlCodeGenReturn.
Changes from v2 to v3:
o Rename function to AmlCodeGenRdMemory32Fixed.
o Use define ACPI_32_BIT_FIXED_MEMORY_RANGE_DESCRIPTOR.
Rebecca Cran (3
Add a Memory32Fixed function to generate code for the corresponding
Memory32Fixed macro in AML.
Signed-off-by: Rebecca Cran
Reviewed-by: Pierre Gondois
---
DynamicTablesPkg/Include/Library/AmlLib/AmlLib.h| 33
DynamicTablesPkg/Library/Common/AmlLib/CodeGen
Add AmlCodeGenMethodRetInteger function to generate AML code for
a Method returning an Integer.
Signed-off-by: Rebecca Cran
Reviewed-by: Pierre Gondois
---
DynamicTablesPkg/Include/Library/AmlLib/AmlLib.h| 47 ++
DynamicTablesPkg/Library/Common/AmlLib/CodeGen/AmlCodeGen.c
In AmlCodeGenReturn, the cast to AML_NODE_HEADER* in the call to
AmlSetFixedArgument is redundant because ReturnNode is already a
AML_NODE_HEADER* .
Signed-off-by: Rebecca Cran
Reviewed-by: Pierre Gondois
---
DynamicTablesPkg/Library/Common/AmlLib/CodeGen/AmlCodeGen.c | 2 +-
1 file changed, 1
EFI_ACPI_32_BIT_MEMORY_RANGE_DESCRIPTOR instead of raw
buffer.
Rebecca Cran (3):
DynamicTablesPkg: Add Memory32Fixed function
DynamicTablesPkg: Remove redundant cast in AmlCodeGenReturn
DynamicTablesPkg: Add AmlCodeGenMethodRetInteger function
DynamicTablesPkg/Include/Library/AmlLib/AmlLib.h
Add a Memory32Fixed function to generate code for the corresponding
Memory32Fixed macro in AML.
Signed-off-by: Rebecca Cran
---
DynamicTablesPkg/Include/Library/AmlLib/AmlLib.h| 33
DynamicTablesPkg/Library/Common/AmlLib/CodeGen/AmlResourceDataCodeGen.c
Thanks. I've fixed these issues in the v2 patch which I'll send out soon.
--
Rebecca Cran
On 1/11/22 04:34, Pierre Gondois wrote:
Hello Rebecca,
On 1/8/22 10:57 PM, Rebecca Cran via groups.io wrote:
Add a Memory32Fixed function to generate code for the corresponding
Memory32Fixed macro
Thanks! It looks like the existing changes covered
Platform/Marvell/Cn913xDb/Cn913xDbA.dsc and Armada7k8kCapsule isn't
something that would be broken by my changes.
--
Rebecca Cran
On 12/19/21 03:32, Marcin Wojtas wrote:
Hi,
czw., 16 gru 2021 o 23:09 Rebecca Cran napisał(a):
On 12/16
Add a Memory32Fixed function to generate code for the corresponding
Memory32Fixed macro in AML.
Signed-off-by: Rebecca Cran
---
DynamicTablesPkg/Include/Library/AmlLib/AmlLib.h| 33
+++
DynamicTablesPkg/Library/Common/AmlLib/CodeGen/AmlResourceDataCodeGen.c | 59
In AmlCodeGenReturn, the cast to AML_NODE_HEADER* in the call to
AmlSetFixedArgument is redundant because ReturnNode is already a
AML_NODE_HEADER* .
Signed-off-by: Rebecca Cran
---
DynamicTablesPkg/Library/Common/AmlLib/CodeGen/AmlCodeGen.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
Add AmlCodeGenMethodRetInteger function to generate AML code for
a Method returning an Integer.
Signed-off-by: Rebecca Cran
---
DynamicTablesPkg/Include/Library/AmlLib/AmlLib.h| 47 ++
DynamicTablesPkg/Library/Common/AmlLib/CodeGen/AmlCodeGen.c | 156
2
Add functions to generate code for the Memory32Fixed ASL macro and a
method returning an Integer.
Remove a redundant cast from AmlCodeGenReturn.
Rebecca Cran (3):
DynamicTablesPkg: Add Memory32Fixed function
DynamicTablesPkg: Remove redundant cast in AmlCodeGenReturn
DynamicTablesPkg: Add
Since it's now the new year, I thought it might be worth pinging people
again to try and get these patches pushed.
--
Rebecca Cran
On 12/13/21 14:59, Rebecca Cran wrote:
(cc other TianoCore stewards)
With edk2-stable202111 just tagged, now would be a good time to get
the patches pushed
No, I expect there to only ever be a single MP services instance.
I agree that the AP function should do something, but I'm hoping to get
this initial implementation committed first.
--
Rebecca Cran
On 12/29/21 20:21, Jeff Fan wrote:
Hi,Rebecca
I'd like to give two comments on this patch
Jian, Liming:
Could you review this please?
--
Rebecca Cran
On 12/12/21 11:08, Rebecca Cran via groups.io wrote:
Add a new MpServicesTest application under MdeModulePkg/Application that
exercises the EFI_MP_SERVICES_PROTOCOL.
Signed-off-by: Rebecca Cran
Reviewed-by: Sami Mujawar
Reviewed-by: Rebecca Cran
On 12/17/21 2:34 PM, Ard Biesheuvel wrote:
Remove the obsolete ARM_PROCESSOR_TABLE definition, and all the pieces
that are related to it.
Cc: Rebecca Cran
Cc: Leif Lindholm
Cc: Sami Mujawar
Ard Biesheuvel (2):
ArmPkg/MpInitLib: avoid ARM_PROCESSOR_TABLE
The Ssdt.asl file isn't used, so delete it.
Signed-off-by: Rebecca Cran
---
OvmfPkg/Bhyve/AcpiTables/Ssdt.asl | 15 ---
1 file changed, 15 deletions(-)
diff --git a/OvmfPkg/Bhyve/AcpiTables/Ssdt.asl
b/OvmfPkg/Bhyve/AcpiTables/Ssdt.asl
deleted file mode 100644
index 175ab3b7e66d
On 12/21/21 09:31, Ard Biesheuvel wrote:
On Tue, 21 Dec 2021 at 16:06, Rebecca Cran wrote:
According to the ACPI Specification version 6.0 and newer, the SSDT
Revision field should be 2. Fix AcpiTables/Ssdt.aml.
Signed-off-by: Rebecca Cran
---
OvmfPkg/Bhyve/AcpiTables/Ssdt.asl | 2 +-
1
for EFI_ACPI_6_3_PROCESSOR_LOCAL_APIC.
--
Rebecca Cran
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#85136): https://edk2.groups.io/g/devel/message/85136
Mute This Topic: https://groups.io/mt/87879480/21656
Group Owner: devel+ow...@edk2
According to the ACPI Specification version 6.0 and newer, the SSDT
Revision field should be 2. Fix AcpiTables/Ssdt.aml.
Signed-off-by: Rebecca Cran
---
OvmfPkg/Bhyve/AcpiTables/Ssdt.asl | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/OvmfPkg/Bhyve/AcpiTables/Ssdt.asl
b
, PrePi etc. since they have obsolete code to handle
secondary cores from before TF-A.
--
Rebecca Cran
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#85085): https://edk2.groups.io/g/devel/message/85085
Mute This Topic: https://groups.io/mt
/Cn913xDbA.dsc for AARCH64 and ARM: tries to
use literal $(PLATFORM_NAME).
o Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc for AARCH64
and ARM: VariablePolicyHelperLib issue.
o Platform/SolidRun/Cn913xCEx7Eval/Cn913xCEx7Eval.dsc for AARCH64 and ARM:
T91 not found.
--
Rebecca Cran
MpInitLib is now required for all users of ArmPkg/Drivers/CpuDxe. Add
ArmPkg/Library/MpInitLib/DxeMpInitLib.inf .
Signed-off-by: Rebecca Cran
---
Silicon/Hisilicon/Hisilicon.dsc.inc | 1 +
1 file changed, 1 insertion(+)
diff --git a/Silicon/Hisilicon/Hisilicon.dsc.inc
b/Silicon/Hisilicon
The ARM_CORE_INFO struct has been updated so the MPIDR is now a single
field instead of separate cluster/core fields. Update FT2000-4Pkg
PlatformLib.
Signed-off-by: Rebecca Cran
---
Silicon/Phytium/FT2000-4Pkg/Library/PlatformLib/PlatformLib.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
MpInitLib is now required for all users of ArmPkg/Drivers/CpuDxe. Add
ArmPkg/Library/MpInitLib/DxeMpInitLib.inf .
Signed-off-by: Rebecca Cran
---
Silicon/Phytium/PhytiumCommonPkg/PhytiumCommonPkg.dsc.inc | 1 +
1 file changed, 1 insertion(+)
diff --git a/Silicon/Phytium/PhytiumCommonPkg
The ARM_CORE_INFO struct has been updated so the MPIDR is now a single
field instead of separate cluster/core fields. Update ArmPlatformLib.
Signed-off-by: Rebecca Cran
---
Silicon/Ampere/AmpereAltraPkg/Library/ArmPlatformLib/ArmPlatformLib.c | 5 ++---
1 file changed, 2 insertions(+), 3
MpInitLib is now required for all users of ArmPkg/Drivers/CpuDxe. Add
ArmPkg/Library/MpInitLib/DxeMpInitLib.inf .
Signed-off-by: Rebecca Cran
---
Silicon/Ampere/AmpereAltraPkg/AmpereAltraPkg.dsc.inc | 1 +
1 file changed, 1 insertion(+)
diff --git a/Silicon/Ampere/AmpereAltraPkg
The ARM_CORE_INFO struct has been updated so the MPIDR is now a single
field instead of separate cluster/core fields. Update the Styx code in
AcpiPlatformDxe, PlatInitPei and StyxDtbLoaderLib.
Signed-off-by: Rebecca Cran
---
Silicon/AMD/Styx/Drivers/AcpiPlatformDxe/AcpiPlatform.c | 3
MpInitLib is now required for all users of ArmPkg/Drivers/CpuDxe. Add
ArmPkg/Library/MpInitLib/DxeMpInitLib.inf .
Signed-off-by: Rebecca Cran
---
Platform/RaspberryPi/RPi3/RPi3.dsc | 1 +
Platform/RaspberryPi/RPi4/RPi4.dsc | 1 +
2 files changed, 2 insertions(+)
diff --git a/Platform
MpInitLib is now required for all users of ArmPkg/Drivers/CpuDxe. Add
ArmPkg/Library/MpInitLib/DxeMpInitLib.inf .
Signed-off-by: Rebecca Cran
---
Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc | 1 +
1 file changed, 1 insertion(+)
diff --git a/Platform/SoftIron/Overdrive1000Board
MpInitLib is now required for all users of ArmPkg/Drivers/CpuDxe. Add
ArmPkg/Library/MpInitLib/DxeMpInitLib.inf .
Signed-off-by: Rebecca Cran
---
Platform/AMD/OverdriveBoard/OverdriveBoard.dsc | 1 +
1 file changed, 1 insertion(+)
diff --git a/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
b
The ARM_CORE_INFO struct has been updated so the MPIDR is now a single
field instead of separate cluster/core fields. Update the initializer.
Signed-off-by: Rebecca Cran
---
Silicon/Qemu/SbsaQemu/Library/SbsaQemuLib/SbsaQemuLib.c | 8
1 file changed, 4 insertions(+), 4 deletions
The ARM_CORE_INFO struct has been updated so the MPIDR is now a single
field instead of separate cluster/core fields. Update the initializer.
Signed-off-by: Rebecca Cran
---
Silicon/Socionext/SynQuacer/Library/SynQuacerLib/SynQuacer.c | 48
++--
1 file changed, 24 insertions
MpInitLib is now required for all users of ArmPkg/Drivers/CpuDxe. Add
ArmPkg/Library/MpInitLib/DxeMpInitLib.inf .
Signed-off-by: Rebecca Cran
---
Platform/Socionext/DeveloperBox/DeveloperBox.dsc | 1 +
Platform/Socionext/SynQuacerEvalBoard/SynQuacerEvalBoard.dsc | 1 +
2 files
The ARM_CORE_INFO struct has been updated so the MPIDR is now a single
field instead of separate cluster/core fields. Update the initializer.
Signed-off-by: Rebecca Cran
---
Silicon/Marvell/Armada7k8k/Library/Armada7k8kLib/Armada7k8kLib.c | 8
1 file changed, 4 insertions(+), 4
The ARM_CORE_INFO struct has been updated so the MPIDR is now a single
field instead of separate cluster/core fields. Update the initializer.
Signed-off-by: Rebecca Cran
---
Platform/ARM/JunoPkg/Library/ArmJunoLib/ArmJuno.c | 12
++--
Platform/ARM/VExpressPkg/Library
MpInitLib is now required for all users of ArmPkg/Drivers/CpuDxe. Add
ArmPkg/Library/MpInitLib/DxeMpInitLib.inf .
Signed-off-by: Rebecca Cran
---
Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc | 1 +
1 file changed, 1 insertion(+)
diff --git a/Silicon/Marvell/Armada7k8k/Armada7k8k.dsc.inc
b
MpInitLib is now required for all users of ArmPkg/Drivers/CpuDxe. Add
ArmPkg/Library/MpInitLib/DxeMpInitLib.inf .
Signed-off-by: Rebecca Cran
---
Platform/Qemu/SbsaQemu/SbsaQemu.dsc | 1 +
1 file changed, 1 insertion(+)
diff --git a/Platform/Qemu/SbsaQemu/SbsaQemu.dsc
b/Platform/Qemu/SbsaQemu
MpInitLib is now required for all users of ArmPkg/Drivers/CpuDxe. Add
ArmPkg/Library/MpInitLib/DxeMpInitLib.inf .
Signed-off-by: Rebecca Cran
---
Platform/ARM/JunoPkg/ArmJuno.dsc | 1 +
Platform/ARM/Morello/MorelloPlatform.dsc.inc | 1 +
Platform/ARM/N1Sdp/N1SdpPlatform.dsc
ore fields. This
series updates the initializers and other code for the various instances.
Rebecca Cran (17):
Platform/ARM: Add MpInitLib instance
Platform/Socionext: Add instance of MpInitLib
Silicon/Marvell: Add instance of MpInitLib
Platform/Qemu: Add instance of MpInitLib
Platform/A
,
for now I'll plan on doing my own thing with shell scripts etc.
--
Rebecca Cran
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#85007): https://edk2.groups.io/g/devel/message/85007
Mute This Topic: https://groups.io/mt/87734393/21656
these patches in a separate series and post them
to the list.
Actually could you re-send them please, cc'ing the platform maintainers
(Ard, Leif and Masami)? I suspect that's why they were missed.
Thanks,
Rebecca Cran
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent
hoping to find some
time next week to work on it. A useful step, even if it's just the first
of many, would be to simply run lots of 'build' commands for each platform.
--
Rebecca Cran
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online
801 - 900 of 1401 matches
Mail list logo