From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by spool.mail.gandi.net (Postfix) with ESMTPS id 3BD8E740041 for ; Wed, 13 Mar 2024 17:41:37 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=NEHHSGA3kVfP94YhLsVRprY50BFfNXx6GEvo23VtDPU=; c=relaxed/simple; d=groups.io; h=DKIM-Filter:Message-ID:Date:MIME-Version:User-Agent:Subject:From:To:Cc:Reply-To:References:In-Reply-To:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Resent-Date:List-Unsubscribe-Post:List-Unsubscribe:Content-Language:Content-Type:Content-Transfer-Encoding; s=20240206; t=1710351696; v=1; b=zlOsdhOSOh+AklhWOBke2q+SNf+kuUyBhG4qXw+eRS0+1XTS7AOnvWS7qTGA9fcDmLLAuZ+X sJEi+Kw6i5OQ6eIVFUjy0lP2Yntn7m9/3dunlJeD1iJRXu+MfUntIDHDFCru81J2ny8ykmZ/tcI s8bj0yBnlhWOanwR54KXBsimzpGokTHK7dB++09WCdteF9M5Eg0qe3zbq8PSsCC7C7cK6ICqcEu FNBxO2NMp0QrCYfL6msAI4EYTG4L4pLGU6sd7ZaNnuR7Jscb2hIFDD/rjPIEK04vM+kBKO72M69 7z3zmwBeh44pmvWsdGE0xLLJ8zhYZ0uoke9W9WJpKwo9w== X-Received: by 127.0.0.2 with SMTP id UydPYY7687511xI6uV5sWquJ; Wed, 13 Mar 2024 10:41:36 -0700 X-Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by mx.groups.io with SMTP id smtpd.web10.2303.1710351696142809401 for ; Wed, 13 Mar 2024 10:41:36 -0700 X-Received: from [10.137.194.171] (unknown [131.107.159.43]) by linux.microsoft.com (Postfix) with ESMTPSA id 4D60420B74C0; Wed, 13 Mar 2024 10:41:35 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 4D60420B74C0 Message-ID: Date: Wed, 13 Mar 2024 10:41:35 -0700 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [edk2-devel][PATCH v2 1/1] MdeModulePkg: Remove ArmPkg Dependency From: "Oliver Smith-Denny" To: devel@edk2.groups.io, Ray Ni , Michael Kinney Cc: Leif Lindholm , Ard Biesheuvel , Sami Mujawar , Liming Gao , Sean Brogan , Ard Biesheuvel Reply-To: devel@edk2.groups.io,osde@linux.microsoft.com References: <17B3DE09466BEB48.21218@groups.io> In-Reply-To: <17B3DE09466BEB48.21218@groups.io> Precedence: Bulk List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Resent-Date: Wed, 13 Mar 2024 10:41:36 -0700 List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: X-Gm-Message-State: dpfKy6p13Qik4C9XmDLmhAyLx7686176AA= Content-Language: en-US Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: quoted-printable X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20240206 header.b=zlOsdhOS; dmarc=fail reason="SPF not aligned (relaxed), DKIM not aligned (relaxed)" header.from=linux.microsoft.com (policy=none); spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce@groups.io Hi Ray and Mike, Can you help get this patch reviewed and merged? See my other email, this one has been languishing as well. Thanks, Oliver On 2/14/2024 3:20 PM, Oliver Smith-Denny wrote: > 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. >=20 > Github PR: https://github.com/tianocore/edk2/pull/5361 > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3D3651 >=20 > Cc: Leif Lindholm > Cc: Ard Biesheuvel > Cc: Sami Mujawar > Cc: Liming Gao >=20 > Signed-off-by: Oliver Smith-Denny > Reviewed-by: Sean Brogan > Acked-by: Ard Biesheuvel > --- >=20 > Notes: > v2: > - Add bugzilla link to commit message > - Add Sean's RB and Ard's ack >=20 > MdeModulePkg/MdeModulePkg.dsc | 2 -- > MdeModulePkg/MdeModulePkg.ci.yaml | 3 +-- > 2 files changed, 1 insertion(+), 4 deletions(-) >=20 > diff --git a/MdeModulePkg/MdeModulePkg.dsc b/MdeModulePkg/MdeModulePkg.ds= c > index 6b3052ff4614..c0f1df3bfbca 100644 > --- a/MdeModulePkg/MdeModulePkg.dsc > +++ b/MdeModulePkg/MdeModulePkg.dsc > @@ -179,8 +179,6 @@ [LibraryClasses.common.MM_STANDALONE] > MemLib|StandaloneMmPkg/Library/StandaloneMmMemLib/StandaloneMmMemLib.= inf > =20 > [LibraryClasses.ARM, LibraryClasses.AARCH64] > - ArmLib|ArmPkg/Library/ArmLib/ArmBaseLib.inf > - ArmMmuLib|ArmPkg/Library/ArmMmuLib/ArmMmuBaseLib.inf > LockBoxLib|MdeModulePkg/Library/LockBoxNullLib/LockBoxNullLib.inf > =20 > # > diff --git a/MdeModulePkg/MdeModulePkg.ci.yaml b/MdeModulePkg/MdeModulePk= g.ci.yaml > index 669f54e33c0d..a3de60a12c9e 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":[ -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#116720): https://edk2.groups.io/g/devel/message/116720 Mute This Topic: https://groups.io/mt/104363773/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-