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>,
Jeremy Linton <jeremy.linton@arm.com>
Subject: [edk2-devel] [PATCH edk2-platforms v2 0/8] RPi: Drop EmbeddedPkg reset runtime
Date: Sun, 28 Jul 2024 22:44:27 +0200 [thread overview]
Message-ID: <20240728204437.4064847-1-ardb+git@google.com> (raw)
From: Ard Biesheuvel <ardb@kernel.org>
The EmbeddedPkg runtime DXE is being retired in favour of the generic
one in MdeModulePkg which is actually being maintained.
RPi uses this driver and the associated EfiResetSystemLib, of which it
has an implementation with value-add for reset notification. So this
logic needs to be moved elsewhere and hooked up to the generic protocols
that implement the same.
Changes since v1:
- boot tested
- add patch to fix pre-existing issue that causes a crash when DmaLib
attempts to set EFI_MEMORY_XP on allocated buffers
- add a patch to force the correct dispatch order for the varstore
related drivers
- fix line endings
Cc: Leif Lindholm <quic_llindhol@quicinc.com>
Cc: Jeremy Linton <jeremy.linton@arm.com>
Ard Biesheuvel (8):
Platform/RaspberryPi: Mark RAM regions as write/execute protectable
Platform/RaspberryPi: Fix line endings in DSCs
Platform/RaspberryPi: Use depex based dispatch order for varstore
Platform/RaspberryPi/VarBlockServiceDxe: Refactor DumpVars event
handler
Platform/RaspberryPi/VarBlockServiceDxe: Register for reset
notification
Platform/RaspberryPi/PlatformBootManagerLib: Reimplement reset hook
Platform/RaspberryPi: Switch to generic reset runtime
Platform/RaspberryPi: Drop platform specific EfiResetSystemLib
Platform/RaspberryPi/RaspberryPi.dec | 1 -
Platform/RaspberryPi/RPi3/RPi3.dsc | 15 +-
Platform/RaspberryPi/RPi4/RPi4.dsc | 15 +-
Platform/RaspberryPi/RPi3/RPi3.fdf | 2 +-
Platform/RaspberryPi/RPi4/RPi4.fdf | 2 +-
Platform/RaspberryPi/Drivers/VarBlockServiceDxe/VarBlockServiceDxe.inf | 6 +-
Platform/RaspberryPi/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf | 6 +
Platform/RaspberryPi/Library/ResetLib/ResetLib.inf | 45 ------
Platform/RaspberryPi/Drivers/VarBlockServiceDxe/VarBlockServiceDxe.c | 65 ++++++---
Platform/RaspberryPi/Library/MemoryInitPeiLib/MemoryInitPeiLib.c | 2 +
Platform/RaspberryPi/Library/PlatformBootManagerLib/PlatformBm.c | 77 ++++++++++
Platform/RaspberryPi/Library/ResetLib/ResetLib.c | 151 --------------------
12 files changed, 156 insertions(+), 231 deletions(-)
delete mode 100644 Platform/RaspberryPi/Library/ResetLib/ResetLib.inf
delete mode 100644 Platform/RaspberryPi/Library/ResetLib/ResetLib.c
--
2.46.0.rc1.232.g9752f9e123-goog
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#120100): https://edk2.groups.io/g/devel/message/120100
Mute This Topic: https://groups.io/mt/107626473/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-
next reply other threads:[~2024-07-30 15:08 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-07-28 20:44 Ard Biesheuvel via groups.io [this message]
2024-07-28 20:44 ` [edk2-devel] [PATCH edk2-platforms v2 1/8] Platform/RaspberryPi: Mark RAM regions as write/execute protectable Ard Biesheuvel via groups.io
2024-07-28 20:44 ` [edk2-devel] [PATCH edk2-platforms v2 2/8] Platform/RaspberryPi: Fix line endings in DSCs Ard Biesheuvel via groups.io
2024-07-28 20:44 ` [edk2-devel] [PATCH edk2-platforms v2 3/8] Platform/RaspberryPi: Use depex based dispatch order for varstore Ard Biesheuvel via groups.io
2024-07-28 20:44 ` [edk2-devel] [PATCH edk2-platforms v2 4/8] Platform/RaspberryPi/VarBlockServiceDxe: Refactor DumpVars event handler Ard Biesheuvel via groups.io
2024-07-28 20:44 ` [edk2-devel] [PATCH edk2-platforms v2 5/8] Platform/RaspberryPi/VarBlockServiceDxe: Register for reset notification Ard Biesheuvel via groups.io
2024-07-28 20:44 ` [edk2-devel] [PATCH edk2-platforms v2 6/8] Platform/RaspberryPi/PlatformBootManagerLib: Reimplement reset hook Ard Biesheuvel via groups.io
2024-07-28 20:44 ` [edk2-devel] [PATCH edk2-platforms v2 7/8] Platform/RaspberryPi: Switch to generic reset runtime Ard Biesheuvel via groups.io
2024-07-28 20:44 ` [edk2-devel] [PATCH edk2-platforms v2 8/8] Platform/RaspberryPi: Drop platform specific EfiResetSystemLib Ard Biesheuvel via groups.io
2024-07-30 12:40 ` [edk2-devel] [PATCH edk2-platforms v2 0/8] RPi: Drop EmbeddedPkg reset runtime Leif Lindholm
2024-07-30 15:43 ` 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=20240728204437.4064847-1-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