From: "Ard Biesheuvel via groups.io" <ardb+git=google.com@groups.io>
To: devel@edk2.groups.io
Cc: Ard Biesheuvel <ardb@kernel.org>,
Leif Lindholm <quic_llindhol@quicinc.com>,
Rebecca Cran <rebecca@os.amperecomputing.com>,
Nhi Pham <nhi@os.amperecomputing.com>,
Chuong Tran <chuong@os.amperecomputing.com>,
Wenyi Xie <xiewenyi2@huawei.com>,
Peng Xie <xiepeng@phytium.com.cn>,
Ling Jia <jialing@phytium.com.cn>,
Yiqi Shu <shuyiqi@phytium.com.cn>
Subject: [edk2-devel] [PATCH RFC edk2-platforms 2/5] Platform AARCH64: Remove bogus references to MPCore stack
Date: Mon, 29 Jul 2024 14:22:12 +0200 [thread overview]
Message-ID: <20240729122215.400402-3-ardb+git@google.com> (raw)
In-Reply-To: <20240729122215.400402-1-ardb+git@google.com>
From: Ard Biesheuvel <ardb@kernel.org>
Remove references to the MPCore stack from platforms that do not use the
MPCore SEC implementations to begin with.
Signed-off-by: Ard Biesheuvel <ardb@kernel.org>
---
Platform/ARM/Morello/MorelloPlatform.dsc.inc | 2 --
Platform/ARM/SgiPkg/SgiPlatform.dsc.inc | 2 --
Platform/AMD/OverdriveBoard/OverdriveBoard.dsc | 2 --
Platform/ARM/N1Sdp/N1SdpPlatform.dsc | 2 --
Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc | 2 --
Platform/LeMaker/CelloBoard/CelloBoard.dsc | 2 --
Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc | 2 --
7 files changed, 14 deletions(-)
diff --git a/Platform/ARM/Morello/MorelloPlatform.dsc.inc b/Platform/ARM/Morello/MorelloPlatform.dsc.inc
index d2f885d1c10f..5407185e853c 100644
--- a/Platform/ARM/Morello/MorelloPlatform.dsc.inc
+++ b/Platform/ARM/Morello/MorelloPlatform.dsc.inc
@@ -75,10 +75,8 @@ [PcdsFeatureFlag.common]
[PcdsFixedAtBuild.common]
gArmTokenSpaceGuid.PcdVFPEnabled|1
- # Stacks for MPCores in Normal World
gArmPlatformTokenSpaceGuid.PcdCPUCoresStackBase|0x80000000
gArmPlatformTokenSpaceGuid.PcdCPUCorePrimaryStackSize|0x40000
- gArmPlatformTokenSpaceGuid.PcdCPUCoreSecondaryStackSize|0x0
# System Memory (2GB - 16MB)
gArmTokenSpaceGuid.PcdSystemMemoryBase|0x80000000
diff --git a/Platform/ARM/SgiPkg/SgiPlatform.dsc.inc b/Platform/ARM/SgiPkg/SgiPlatform.dsc.inc
index a793e96f8b0f..87dcfbc4ce32 100644
--- a/Platform/ARM/SgiPkg/SgiPlatform.dsc.inc
+++ b/Platform/ARM/SgiPkg/SgiPlatform.dsc.inc
@@ -134,10 +134,8 @@ [PcdsFixedAtBuild.common]
gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwWorkingSize|0x01400000
gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwSpareSize|0x01400000
- # Stacks for MPCores in Normal World
gArmPlatformTokenSpaceGuid.PcdCPUCoresStackBase|0x80000000
gArmPlatformTokenSpaceGuid.PcdCPUCorePrimaryStackSize|0x40000
- gArmPlatformTokenSpaceGuid.PcdCPUCoreSecondaryStackSize|0x0
# ACPI Table Version
gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiExposedTableVersions|0x20
diff --git a/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc b/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
index 5a1fdb8bfa1f..1778f69ca6a3 100644
--- a/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
+++ b/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc
@@ -354,10 +354,8 @@ [PcdsFixedAtBuild.common]
# Number of configured cores
gArmPlatformTokenSpaceGuid.PcdCoreCount|$(NUM_CORES)
- # Stacks for MPCores in Normal World
gArmPlatformTokenSpaceGuid.PcdCPUCoresStackBase|0x8001680000
gArmPlatformTokenSpaceGuid.PcdCPUCorePrimaryStackSize|0x10000
- gArmPlatformTokenSpaceGuid.PcdCPUCoreSecondaryStackSize|0x800
# Declare system memory base
gArmTokenSpaceGuid.PcdSystemMemoryBase|0x8000000000
diff --git a/Platform/ARM/N1Sdp/N1SdpPlatform.dsc b/Platform/ARM/N1Sdp/N1SdpPlatform.dsc
index b14ece1b8f61..63e885320651 100644
--- a/Platform/ARM/N1Sdp/N1SdpPlatform.dsc
+++ b/Platform/ARM/N1Sdp/N1SdpPlatform.dsc
@@ -104,10 +104,8 @@ [PcdsFixedAtBuild.common]
gArmN1SdpTokenSpaceGuid.PcdRamDiskBase|0x88000000
gArmN1SdpTokenSpaceGuid.PcdRamDiskSize|0x18000000
- # Stacks for MPCores in Normal World
gArmPlatformTokenSpaceGuid.PcdCPUCoresStackBase|0x80000000
gArmPlatformTokenSpaceGuid.PcdCPUCorePrimaryStackSize|0x40000
- gArmPlatformTokenSpaceGuid.PcdCPUCoreSecondaryStackSize|0x0
# System Memory (2GB) - Reserved Secure Memory (16MB)
gArmTokenSpaceGuid.PcdSystemMemoryBase|0x80000000
diff --git a/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc b/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc
index 60556f6661c4..e1e85b4f2ec2 100644
--- a/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc
+++ b/Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc
@@ -132,11 +132,9 @@ [PcdsFixedAtBuild.common]
gArmTokenSpaceGuid.PcdVFPEnabled|1
- # Stacks for MPCores in Normal World
# Non-Trusted SRAM
gArmPlatformTokenSpaceGuid.PcdCPUCoresStackBase|0x2E000000
gArmPlatformTokenSpaceGuid.PcdCPUCorePrimaryStackSize|0x4000
- gArmPlatformTokenSpaceGuid.PcdCPUCoreSecondaryStackSize|0x0
# System Memory
# When RME is supported by the FVP the top 64MB of DRAM1 (i.e. at the top
diff --git a/Platform/LeMaker/CelloBoard/CelloBoard.dsc b/Platform/LeMaker/CelloBoard/CelloBoard.dsc
index 7b791d1635e9..5f6f42a3bf53 100644
--- a/Platform/LeMaker/CelloBoard/CelloBoard.dsc
+++ b/Platform/LeMaker/CelloBoard/CelloBoard.dsc
@@ -344,10 +344,8 @@ [PcdsFixedAtBuild.common]
# Number of configured cores
gArmPlatformTokenSpaceGuid.PcdCoreCount|$(NUM_CORES)
- # Stacks for MPCores in Normal World
gArmPlatformTokenSpaceGuid.PcdCPUCoresStackBase|0x8001680000
gArmPlatformTokenSpaceGuid.PcdCPUCorePrimaryStackSize|0x10000
- gArmPlatformTokenSpaceGuid.PcdCPUCoreSecondaryStackSize|0x800
# Declare system memory base
gArmTokenSpaceGuid.PcdSystemMemoryBase|0x8000000000
diff --git a/Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc b/Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc
index 843d18bc746b..adae7e71d3bf 100644
--- a/Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc
+++ b/Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc
@@ -342,10 +342,8 @@ [PcdsFixedAtBuild.common]
# Number of configured cores
gArmPlatformTokenSpaceGuid.PcdCoreCount|$(NUM_CORES)
- # Stacks for MPCores in Normal World
gArmPlatformTokenSpaceGuid.PcdCPUCoresStackBase|0x8001680000
gArmPlatformTokenSpaceGuid.PcdCPUCorePrimaryStackSize|0x10000
- gArmPlatformTokenSpaceGuid.PcdCPUCoreSecondaryStackSize|0x800
# Declare system memory base
gArmTokenSpaceGuid.PcdSystemMemoryBase|0x8000000000
--
2.46.0.rc1.232.g9752f9e123-goog
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#120111): https://edk2.groups.io/g/devel/message/120111
Mute This Topic: https://groups.io/mt/107628985/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
next prev parent reply other threads:[~2024-07-30 15:08 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-07-29 12:22 [edk2-devel] [PATCH RFC edk2-platforms 0/5] Phase out MPCore SEC drivers Ard Biesheuvel via groups.io
2024-07-29 12:22 ` [edk2-devel] [PATCH RFC edk2-platforms 1/5] Platform AARCH64: Drop leftover references to deleted timer PCD Ard Biesheuvel via groups.io
2024-07-29 12:22 ` Ard Biesheuvel via groups.io [this message]
2024-07-30 19:16 ` [edk2-devel] [PATCH RFC edk2-platforms 2/5] Platform AARCH64: Remove bogus references to MPCore stack Sami Mujawar
2024-07-29 12:22 ` [edk2-devel] [PATCH RFC edk2-platforms 3/5] Platform/Ampere: Switch to unicore SEC implementation Ard Biesheuvel via groups.io
2024-07-31 8:21 ` Nhi Pham via groups.io
2024-07-29 12:22 ` [edk2-devel] [PATCH RFC edk2-platforms 4/5] Platform/Durian: " Ard Biesheuvel via groups.io
2024-07-29 12:22 ` [edk2-devel] [PATCH RFC edk2-platforms 5/5] Platform/HiSilicon/D0x: " Ard Biesheuvel via groups.io
2024-07-30 12:42 ` [edk2-devel] [PATCH RFC edk2-platforms 0/5] Phase out MPCore SEC drivers Leif Lindholm
2024-07-30 14:30 ` Rebecca Cran
2024-07-30 14:48 ` Leif Lindholm
2024-07-30 15:00 ` Ard Biesheuvel
2024-07-30 15:09 ` Leif Lindholm
2024-07-30 15:18 ` Ard Biesheuvel
2024-07-30 15:34 ` Leif Lindholm
2024-07-30 14:18 ` Rebecca Cran via groups.io
2024-07-30 15:48 ` Ard Biesheuvel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240729122215.400402-3-ardb+git@google.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox