From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182])
 by mx.groups.io with SMTP id smtpd.web12.461.1628616834303027266
 for <devel@edk2.groups.io>;
 Tue, 10 Aug 2021 10:33:54 -0700
Authentication-Results: mx.groups.io;
 dkim=pass header.i=@linux.microsoft.com header.s=default header.b=d48thlkJ;
 spf=pass (domain: linux.microsoft.com, ip: 13.77.154.182, mailfrom: mikuback@linux.microsoft.com)
Received: from [10.124.238.202] (unknown [167.220.2.74])
	by linux.microsoft.com (Postfix) with ESMTPSA id B445720B36E0;
	Tue, 10 Aug 2021 10:33:53 -0700 (PDT)
DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com B445720B36E0
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.microsoft.com;
	s=default; t=1628616833;
	bh=cokX3i0PI7xhP9UjQIE61RaUHwgafStzJyX49GL8E2Q=;
	h=Subject:To:Cc:References:From:Date:In-Reply-To:From;
	b=d48thlkJtbgguah3pwXyo3bP8a2ozbY3wHNHdwRnhJ6tlvyQj+3Pom78iWtBXmSwN
	 yc48PHSdSTUFZ6v2o/Si11ZWGoA1oe3eHxUaCyVU+JCkYqqae7BoYuIX9+R5agcCga
	 gxKbLB1nRXsrNHEu7qepb1FPzItH5o7Z2VmQmMl4=
Subject: Re: [edk2-devel] [edk2-platforms] [PATCH V1] MinPlatformPkg: Cleanup PeiFspWrapperHobProcessLib dependencies
To: devel@edk2.groups.io, nathaniel.l.desimone@intel.com
Cc: Chasel Chiu <chasel.chiu@intel.com>,
 Michael Kubacki <Michael.Kubacki@microsoft.com>,
 Benjamin Doron <benjamin.doron00@gmail.com>
References: <20210810043714.26890-1-nathaniel.l.desimone@intel.com>
From: "Michael Kubacki" <mikuback@linux.microsoft.com>
Message-ID: <75fc1413-ae4a-cf46-e238-b4ab75811788@linux.microsoft.com>
Date: Tue, 10 Aug 2021 13:33:54 -0400
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101
 Thunderbird/78.12.0
MIME-Version: 1.0
In-Reply-To: <20210810043714.26890-1-nathaniel.l.desimone@intel.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit

Acked-by: Michael Kubacki <michael.kubacki@microsoft.com>

On 8/10/2021 12:37 AM, Nate DeSimone wrote:
> PeiFspWrapperHobProcessLib is currently set to depens on
> FspWrapperPlatformLib, but it does not use any of the
> functions implemented by that LibraryClass. This change
> removes that unneeded dependency.
> 
> Cc: Chasel Chiu <chasel.chiu@intel.com>
> Cc: Michael Kubacki <Michael.Kubacki@microsoft.com>
> Cc: Benjamin Doron <benjamin.doron00@gmail.com>
> Signed-off-by: Nate DeSimone <nathaniel.l.desimone@intel.com>
> ---
>   .../PeiFspWrapperHobProcessLib/PeiFspWrapperHobProcessLib.inf  | 3 +--
>   1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/Platform/Intel/MinPlatformPkg/FspWrapper/Library/PeiFspWrapperHobProcessLib/PeiFspWrapperHobProcessLib.inf b/Platform/Intel/MinPlatformPkg/FspWrapper/Library/PeiFspWrapperHobProcessLib/PeiFspWrapperHobProcessLib.inf
> index 64f3302959..b846e7af1d 100644
> --- a/Platform/Intel/MinPlatformPkg/FspWrapper/Library/PeiFspWrapperHobProcessLib/PeiFspWrapperHobProcessLib.inf
> +++ b/Platform/Intel/MinPlatformPkg/FspWrapper/Library/PeiFspWrapperHobProcessLib/PeiFspWrapperHobProcessLib.inf
> @@ -1,7 +1,7 @@
>   ## @file
>   #  Provide FSP wrapper hob process related function.
>   #
> -# Copyright (c) 2017 - 2019, Intel Corporation. All rights reserved.<BR>
> +# Copyright (c) 2017 - 2021, Intel Corporation. All rights reserved.<BR>
>   #
>   # SPDX-License-Identifier: BSD-2-Clause-Patent
>   #
> @@ -55,7 +55,6 @@
>     BaseMemoryLib
>     HobLib
>     DebugLib
> -  FspWrapperPlatformLib
>     PeiServicesLib
>     PeiServicesTablePointerLib
>   
>