On Tue, 23 Jul 2024 at 17:32, Leif Lindholm <quic_llind...@quicinc.com> wrote: > > On Sun, Jul 21, 2024 at 16:51:54 +0200, Ard Biesheuvel wrote: > > From: Ard Biesheuvel <a...@kernel.org> > > > > The reset runtime DXE driver is deprecated and will be removed soon. It > > is superseded by a generic implementation in MdeModulePkg, which is > > shared between all architectures, and implements the notification > > protocols that the EFI spec describes. So move the Ampere Jade platform > > to this implementation the of ResetSystem EFI runtime service. > > > > Signed-off-by: Ard Biesheuvel <a...@kernel.org> > > --- > > Silicon/Ampere/AmpereAltraPkg/AmpereAltraPkg.dsc.inc | 3 +-- > > Platform/Ampere/JadePkg/Jade.fdf | 2 +- > > 2 files changed, 2 insertions(+), 3 deletions(-) > > > > diff --git a/Silicon/Ampere/AmpereAltraPkg/AmpereAltraPkg.dsc.inc > > b/Silicon/Ampere/AmpereAltraPkg/AmpereAltraPkg.dsc.inc > > index 1f705c68579a..fb170d436d00 100644 > > --- a/Silicon/Ampere/AmpereAltraPkg/AmpereAltraPkg.dsc.inc > > +++ b/Silicon/Ampere/AmpereAltraPkg/AmpereAltraPkg.dsc.inc > > @@ -259,7 +259,6 @@ [LibraryClasses.common.DXE_RUNTIME_DRIVER] > > !endif > > > > VariablePolicyLib|MdeModulePkg/Library/VariablePolicyLib/VariablePolicyLibRuntimeDxe.inf > > > > - > > EfiResetSystemLib|ArmPkg/Library/ArmPsciResetSystemLib/ArmPsciResetSystemLib.inf > > Would it make sense to move the existing ResetSystemLib mapping here? > Not for any particular reason other than it's not actually used > anywhere else.
There isn't an obvious reason imo why we would ever use another ResetSystemLib instance for other phases - the existing one is universal, i.e., it doesn't produce HOBs or consume PPIs or protocols and so it is usable from any context. So moving it here is possible but unnecessary. > Regardless, for the series: > Reviewed-by: Leif Lindholm <quic_llind...@quicinc.com> > Cheers. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#120015): https://edk2.groups.io/g/devel/message/120015 Mute This Topic: https://groups.io/mt/107476835/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
