If the implementation didn't make so many calls into Arm-specific libs for
SMC servicing, it could stay right where it is.
It's just that untangling that Gordian knot is outside my ken.
On Tue, Nov 2, 2021 at 3:00 AM Leif Lindholm <leif@nuviainc.com> wrote:
> On Mon, Nov 01, 2021 at 12:56:44 -0700, brbarkel@microsoft.com wrote:
> > From: Bret Barkelew <brbarkel@microsoft.com>
> >
> > This aligns better with Mu's philosophy around dependency structuring
> > and is one of the steps to enable Basecore to have zero CI dependencies
> > on other Mu repos.
> >
> > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3652
>
> I much prefer moving things out of ArmPkg (which shouldn't exist),
> than moving things into it. Where would be the logical place for this
> to go if ARM wasn't so special it needed its own top-level package?
>
> /
> Leif
>
> > Cc: Leif Lindholm <leif@nuviainc.com>
> > Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
> > Cc: Sami Mujawar <sami.mujawar@arm.com>
> > Cc: Jiewen Yao <jiewen.yao@intel.com>
> > Cc: Supreeth Venkatesh <supreeth.venkatesh@arm.com>
> > Cc: Sean Brogan <sean.brogan@microsoft.com>
> > Signed-off-by: Bret Barkelew <bret.barkelew@microsoft.com>
> > ---
> > {StandaloneMmPkg => ArmPkg}/Drivers/StandaloneMmCpu/EventHandle.c
> | 0
> > {StandaloneMmPkg => ArmPkg}/Drivers/StandaloneMmCpu/StandaloneMmCpu.c
> | 0
> > {StandaloneMmPkg => ArmPkg}/Drivers/StandaloneMmCpu/StandaloneMmCpu.h
> | 0
> > {StandaloneMmPkg => ArmPkg}/Drivers/StandaloneMmCpu/StandaloneMmCpu.inf
> | 0
> > 4 files changed, 0 insertions(+), 0 deletions(-)
> >
> > diff --git a/StandaloneMmPkg/Drivers/StandaloneMmCpu/EventHandle.c
> b/ArmPkg/Drivers/StandaloneMmCpu/EventHandle.c
> > similarity index 100%
> > rename from StandaloneMmPkg/Drivers/StandaloneMmCpu/EventHandle.c
> > rename to ArmPkg/Drivers/StandaloneMmCpu/EventHandle.c
> > diff --git a/StandaloneMmPkg/Drivers/StandaloneMmCpu/StandaloneMmCpu.c
> b/ArmPkg/Drivers/StandaloneMmCpu/StandaloneMmCpu.c
> > similarity index 100%
> > rename from StandaloneMmPkg/Drivers/StandaloneMmCpu/StandaloneMmCpu.c
> > rename to ArmPkg/Drivers/StandaloneMmCpu/StandaloneMmCpu.c
> > diff --git a/StandaloneMmPkg/Drivers/StandaloneMmCpu/StandaloneMmCpu.h
> b/ArmPkg/Drivers/StandaloneMmCpu/StandaloneMmCpu.h
> > similarity index 100%
> > rename from StandaloneMmPkg/Drivers/StandaloneMmCpu/StandaloneMmCpu.h
> > rename to ArmPkg/Drivers/StandaloneMmCpu/StandaloneMmCpu.h
> > diff --git a/StandaloneMmPkg/Drivers/StandaloneMmCpu/StandaloneMmCpu.inf
> b/ArmPkg/Drivers/StandaloneMmCpu/StandaloneMmCpu.inf
> > similarity index 100%
> > rename from StandaloneMmPkg/Drivers/StandaloneMmCpu/StandaloneMmCpu.inf
> > rename to ArmPkg/Drivers/StandaloneMmCpu/StandaloneMmCpu.inf
> > --
> > 2.31.1.windows.1
> >
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#83157): https://edk2.groups.io/g/devel/message/83157
Mute This Topic: https://groups.io/mt/86750659/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-