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 <l...@nuviainc.com> wrote: > On Mon, Nov 01, 2021 at 12:56:44 -0700, brbar...@microsoft.com wrote: > > From: Bret Barkelew <brbar...@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 <l...@nuviainc.com> > > Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org> > > Cc: Sami Mujawar <sami.muja...@arm.com> > > Cc: Jiewen Yao <jiewen....@intel.com> > > Cc: Supreeth Venkatesh <supreeth.venkat...@arm.com> > > Cc: Sean Brogan <sean.bro...@microsoft.com> > > Signed-off-by: Bret Barkelew <bret.barke...@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/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
