From 019feb42a1dd1136014b19df3fcb618861b621e3 Mon Sep 17 00:00:00 2001 From: Oliver Smith-Denny Date: Wed, 14 Feb 2024 15:20:09 -0800 Subject: [PATCH] MdeModulePkg: Remove ArmPkg Dependency With commita21a994f55e53325d3e060c435ca3a87fd7c2c79 MdeModulePkg no longer has a hard dependency on ArmMmuLib and therefore ArmLib. This is the final dependency on ArmPkg, so remove the unused libs and drop the allowed dependency on ArmPkg as MdeModulePkg should not depend on it as this is a circular dependency. Github PR: https://github.com/tianocore/edk2/pull/5361 BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3651 Cc: Leif Lindholm Cc: Ard Biesheuvel Cc: Sami Mujawar Cc: Liming Gao Signed-off-by: Oliver Smith-Denny Reviewed-by: Sean Brogan Acked-by: Ard Biesheuvel Reviewed-by: Liming Gao --- MdeModulePkg/MdeModulePkg.ci.yaml | 3 +-- MdeModulePkg/MdeModulePkg.dsc | 2 -- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/MdeModulePkg/MdeModulePkg.ci.yaml b/MdeModulePkg/MdeModulePkg.ci.yaml index 669f54e33c..a3de60a12c 100644 --- a/MdeModulePkg/MdeModulePkg.ci.yaml +++ b/MdeModulePkg/MdeModulePkg.ci.yaml @@ -53,8 +53,7 @@ "AcceptableDependencies": [ "MdePkg/MdePkg.dec", "MdeModulePkg/MdeModulePkg.dec", - "StandaloneMmPkg/StandaloneMmPkg.dec", - "ArmPkg/ArmPkg.dec" # this should be fixed by promoting an abstraction + "StandaloneMmPkg/StandaloneMmPkg.dec" ], # For host based unit tests "AcceptableDependencies-HOST_APPLICATION":[ diff --git a/MdeModulePkg/MdeModulePkg.dsc b/MdeModulePkg/MdeModulePkg.dsc index 6b3052ff46..c0f1df3bfb 100644 --- a/MdeModulePkg/MdeModulePkg.dsc +++ b/MdeModulePkg/MdeModulePkg.dsc @@ -179,8 +179,6 @@ MemLib|StandaloneMmPkg/Library/StandaloneMmMemLib/StandaloneMmMemLib.inf [LibraryClasses.ARM, LibraryClasses.AARCH64] - ArmLib|ArmPkg/Library/ArmLib/ArmBaseLib.inf - ArmMmuLib|ArmPkg/Library/ArmMmuLib/ArmMmuBaseLib.inf LockBoxLib|MdeModulePkg/Library/LockBoxNullLib/LockBoxNullLib.inf #