From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from foss.arm.com (foss.arm.com [217.140.110.172])
 by mx.groups.io with SMTP id smtpd.web12.7813.1615811601497666037
 for <devel@edk2.groups.io>;
 Mon, 15 Mar 2021 05:33:21 -0700
Authentication-Results: mx.groups.io;
 dkim=missing; spf=pass (domain: arm.com, ip: 217.140.110.172, mailfrom: vijayenthiran.subramaniam@arm.com)
Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14])
	by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id E35D811D4
	for <devel@edk2.groups.io>; Mon, 15 Mar 2021 05:33:20 -0700 (PDT)
Received: from mail-wr1-f52.google.com (unknown [172.31.20.19])
	by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id BC4C83FA35
	for <devel@edk2.groups.io>; Mon, 15 Mar 2021 05:33:20 -0700 (PDT)
Received: by mail-wr1-f52.google.com with SMTP id t9so5598505wrn.11
        for <devel@edk2.groups.io>; Mon, 15 Mar 2021 05:33:20 -0700 (PDT)
X-Gm-Message-State: AOAM532ZzQ/k6/RmEpoHmzEAikFhS6zIG4h1l181p9tHDA9hDEmJ73aH
	bzVJmDwHpWckwhGaBxmH+sb3Njkgv8SNFpf1ZGo=
X-Google-Smtp-Source: ABdhPJxo/JGO4/PH8ZQT03qnXUQwGPzVUg13WGr61Zz0jfMGyVYgKpNYhENiAztfhjVG3a726Gyk3RFYdSrS1rvryz8=
X-Received: by 2002:a5d:4dd2:: with SMTP id f18mr27009723wru.366.1615811591575;
 Mon, 15 Mar 2021 05:33:11 -0700 (PDT)
MIME-Version: 1.0
References: <20210225163737.15727-1-Pierre.Gondois@arm.com> <20210225163737.15727-4-Pierre.Gondois@arm.com>
In-Reply-To: <20210225163737.15727-4-Pierre.Gondois@arm.com>
From: "Vijayenthiran Subramaniam" <vijayenthiran.subramaniam@arm.com>
Date: Mon, 15 Mar 2021 12:32:59 +0000
X-Gmail-Original-Message-ID: <CAC0BY-dkgHszHhBD8=_5+ppgxVESwdffWsGGqCQVefVUAJMUAw@mail.gmail.com>
Message-ID: <CAC0BY-dkgHszHhBD8=_5+ppgxVESwdffWsGGqCQVefVUAJMUAw@mail.gmail.com>
Subject: Re: [edk2-devel] [PATCH v1 3/3] ArmPlatformPkg: Remove package dependency in NorFlashStandaloneMm
To: devel@edk2.groups.io, pierre.gondois@arm.com
Cc: ardb+tianocore@kernel.org, leif@nuviainc.com, 
	Sami Mujawar <sami.mujawar@arm.com>, Thomas Abraham <thomas.abraham@arm.com>, 
	sayanta.pattanayak@arm.com, 
	Vijayenthiran Subramaniam <vijayenthiran.subramaniam@arm.com>
Content-Type: text/plain; charset="UTF-8"

Hi,

On Thu, Feb 25, 2021 at 4:38 PM PierreGondois <pierre.gondois@arm.com> wrote:
>
> From: Pierre Gondois <Pierre.Gondois@arm.com>
>
> The DependencyCheck available in .pytool detects an unnecessary
> dependency of the NorFlashStandaloneMm.inf module over the
> EmbeddedPkg package.
> This patch removes this dependency.
>
> Signed-off-by: Pierre Gondois <Pierre.Gondois@arm.com>

Tested this patch on Arm's reference design platform.
Tested-by: Vijayenthiran Subramaniam <vijayenthiran.subramaniam@arm.com>

> ---
> The changes can be seen at: https://github.com/PierreARM/edk2/tree/1641_CI_fixes_for_ArmPlatformPkg_v1
>
>  ArmPlatformPkg/Drivers/NorFlashDxe/NorFlashStandaloneMm.inf | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/ArmPlatformPkg/Drivers/NorFlashDxe/NorFlashStandaloneMm.inf b/ArmPlatformPkg/Drivers/NorFlashDxe/NorFlashStandaloneMm.inf
> index 56347a8bc7c1..b8dddd5cbec3 100644
> --- a/ArmPlatformPkg/Drivers/NorFlashDxe/NorFlashStandaloneMm.inf
> +++ b/ArmPlatformPkg/Drivers/NorFlashDxe/NorFlashStandaloneMm.inf
> @@ -29,7 +29,6 @@ [Packages]
>    MdeModulePkg/MdeModulePkg.dec
>    ArmPlatformPkg/ArmPlatformPkg.dec
>    EmbeddedPkg/EmbeddedPkg.dec
> -  StandaloneMmPkg/StandaloneMmPkg.dec
>
>  [LibraryClasses]
>    BaseLib
> --
> 2.17.1
>
>