public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
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]
-=-=-=-=-=-=-=-=-=-=-=-



             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