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
> >
>