From nobody Thu May 16 05:41:32 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) client-ip=66.175.222.108; envelope-from=bounce+27952+74076+1787277+3901457@groups.io; helo=mail02.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+74076+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=intel.com ARC-Seal: i=1; a=rsa-sha256; t=1618367675; cv=none; d=zohomail.com; s=zohoarc; b=RBFuh2A5ylXNOf4AkasdxR9E9CmYBo3ZWKS5GwjP/oKjU/ejtkwdFLSuDLzK5BzoHe7Ebavv8W2xfIfEQlzDl8uizASK0CUbqPVd+p593NDNG8v1N2p9/GnkTU1edccUeqlyajJRXUWIGvB2TN/e1xjRb8jZv6Ttr4Np5ogGkHY= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1618367675; h=Content-Transfer-Encoding:Cc:Date:From:List-Subscribe:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:Sender:Subject:To; bh=wd2QFhQis98GSfINOscqvWj1DXyTCBrdtrWDi1o4kIc=; b=P+dGKOi6dki/kIIDZR4D+dkFr+ItHWfi8sjmACtvLERsOOpZyyu9Do0FMjSWBT/gYVHxcVeuidobiToWIKK8J4ie9sKd6iJttp7fI4EGt6tHcLbq1gAnu6UIcA5pQI4ntW0Oyr32N5FSrYw/slYxXJIATMBd3NG8NU1O7J3brJo= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of groups.io designates 66.175.222.108 as permitted sender) smtp.mailfrom=bounce+27952+74076+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) header.from= Received: from mail02.groups.io (mail02.groups.io [66.175.222.108]) by mx.zohomail.com with SMTPS id 1618367675351420.261596448953; Tue, 13 Apr 2021 19:34:35 -0700 (PDT) Return-Path: X-Received: by 127.0.0.2 with SMTP id nvOSYY1788612x0UwMRkP3Ol; Tue, 13 Apr 2021 19:34:31 -0700 X-Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by mx.groups.io with SMTP id smtpd.web09.7579.1618367670644399683 for ; Tue, 13 Apr 2021 19:34:30 -0700 IronPort-SDR: 0OZfOJM7irUDKaE+B+W3p2lr4/3J30rPjdKfhmyQgtu/nV2MbPwnMWXSsPeOppGWKTNn/BYFc6 1v1TNctNXrYg== X-IronPort-AV: E=McAfee;i="6200,9189,9953"; a="191363303" X-IronPort-AV: E=Sophos;i="5.82,221,1613462400"; d="scan'208";a="191363303" X-Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 Apr 2021 19:34:29 -0700 IronPort-SDR: KsI16nYwY0khPrYd7uInUYMsRfmMevhR3o73SJ+wjNc2aq1WTp9Y78SS84W71nCqBqPCFq9luq PYEbX3wZCOtA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.82,221,1613462400"; d="scan'208";a="399000911" X-Received: from shwdeopenlab102.ccr.corp.intel.com ([10.239.183.74]) by orsmga002.jf.intel.com with ESMTP; 13 Apr 2021 19:34:28 -0700 From: "Jason Lou" To: devel@edk2.groups.io Cc: Jason , Ray Ni Subject: [edk2-devel] [PATCH v1] IntelFsp2WrapperPkg: Remove microcode PCDs Date: Wed, 14 Apr 2021 10:34:08 +0800 Message-Id: <20210414023408.15262-1-yun.lou@intel.com> MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,yun.lou@intel.com X-Gm-Message-State: Mr8eJX7Z93pSmyIE9ZMOF8cwx1787277AA= Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1618367671; bh=G/RbC/ih75ezA12MjdSZuIgTQRe0ilUZ3lWzN558mPM=; h=Cc:Date:From:Reply-To:Subject:To; b=A+6r0s6XlwcNqf8Vj3JOUmVTz/xMOoiChbfhpIq90gy1VAkvZTGmBVRpKbrWvJpxvw2 qkK9VHR6fILLnCHdhACbgFCg8+rtEwhMQC7bImf9eaxyo3rRM8cssBdT5nhE/jOfDkdet t2YW+iq/vtcYO4aN42TsnC1NO5JXngA4+dg= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D3334 IntelFsp2WrapperPkg defines following PCDs: PcdCpuMicrocodePatchAddress PcdCpuMicrocodePatchRegionSize PcdFlashMicrocodeOffset But the meanings of PcdCpuMicrocodePatchAddress and PcdCpuMicrocodePatchRegionSize are different from the ones that have The same name in UefiCpuPkg. To avoid confusion, remove the three PCDs defined in IntelFsp2WrapperPkg. Signed-off-by: Jason Lou Cc: Ray Ni Reviewed-by: Chasel Chiu --- IntelFsp2WrapperPkg/Library/SecFspWrapperPlatformSecLibSample/SecRamInitDa= ta.c | 6 +++--- IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec = | 8 +------- IntelFsp2WrapperPkg/Library/SecFspWrapperPlatformSecLibSample/SecFspWrappe= rPlatformSecLibSample.inf | 7 +++---- 3 files changed, 7 insertions(+), 14 deletions(-) diff --git a/IntelFsp2WrapperPkg/Library/SecFspWrapperPlatformSecLibSample/= SecRamInitData.c b/IntelFsp2WrapperPkg/Library/SecFspWrapperPlatformSecLibS= ample/SecRamInitData.c index 96b47e23da..e57b5b57be 100644 --- a/IntelFsp2WrapperPkg/Library/SecFspWrapperPlatformSecLibSample/SecRamI= nitData.c +++ b/IntelFsp2WrapperPkg/Library/SecFspWrapperPlatformSecLibSample/SecRamI= nitData.c @@ -1,7 +1,7 @@ /** @file Sample to provide TempRamInitParams data. =20 - Copyright (c) 2014 - 2020, Intel Corporation. All rights reserved.
+ Copyright (c) 2014 - 2021, Intel Corporation. All rights reserved.
SPDX-License-Identifier: BSD-2-Clause-Patent =20 **/ @@ -52,8 +52,8 @@ GLOBAL_REMOVE_IF_UNREFERENCED CONST FSPT_UPD_CORE_DATA Fs= ptUpdDataPtr =3D { } }, { - ((UINT32)FixedPcdGet64 (PcdCpuMicrocodePatchAddress) + FixedPcdGet32 (= PcdFlashMicrocodeOffset)), - ((UINT32)FixedPcdGet64 (PcdCpuMicrocodePatchRegionSize) - FixedPcdGet3= 2 (PcdFlashMicrocodeOffset)), + FixedPcdGet32 (PcdCpuMicrocodePatchAddress), + FixedPcdGet32 (PcdCpuMicrocodePatchRegionSize), FixedPcdGet32 (PcdFlashCodeCacheAddress), FixedPcdGet32 (PcdFlashCodeCacheSize), } diff --git a/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec b/IntelFsp2Wrapper= Pkg/IntelFsp2WrapperPkg.dec index 6852bf1271..a3b9363779 100644 --- a/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec +++ b/IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dec @@ -1,7 +1,7 @@ ## @file # Provides drivers and definitions to support fsp in EDKII bios. # -# Copyright (c) 2014 - 2020, Intel Corporation. All rights reserved.
+# Copyright (c) 2014 - 2021, Intel Corporation. All rights reserved.
# SPDX-License-Identifier: BSD-2-Clause-Patent # ## @@ -56,12 +56,6 @@ ## Provides the size of the BIOS Flash Device. gIntelFsp2WrapperTokenSpaceGuid.PcdFlashCodeCacheSize|0x00200000|UINT32|= 0x10000002 =20 - ## Indicates the base address of the first Microcode Patch in the Microc= ode Region - gIntelFsp2WrapperTokenSpaceGuid.PcdCpuMicrocodePatchAddress|0x0|UINT64|0= x10000005 - gIntelFsp2WrapperTokenSpaceGuid.PcdCpuMicrocodePatchRegionSize|0x0|UINT6= 4|0x10000006 - ## Indicates the offset of the Cpu Microcode. - gIntelFsp2WrapperTokenSpaceGuid.PcdFlashMicrocodeOffset|0x90|UINT32|0x10= 000007 - ## Indicate the PEI memory size platform want to report gIntelFsp2WrapperTokenSpaceGuid.PcdPeiMinMemSize|0x1800000|UINT32|0x4000= 0004 ## Indicate the PEI memory size platform want to report diff --git a/IntelFsp2WrapperPkg/Library/SecFspWrapperPlatformSecLibSample/= SecFspWrapperPlatformSecLibSample.inf b/IntelFsp2WrapperPkg/Library/SecFspW= rapperPlatformSecLibSample/SecFspWrapperPlatformSecLibSample.inf index d7f8301bef..027b127724 100644 --- a/IntelFsp2WrapperPkg/Library/SecFspWrapperPlatformSecLibSample/SecFspW= rapperPlatformSecLibSample.inf +++ b/IntelFsp2WrapperPkg/Library/SecFspWrapperPlatformSecLibSample/SecFspW= rapperPlatformSecLibSample.inf @@ -1,7 +1,7 @@ ## @file # Sample to provide FSP wrapper platform sec related function. # -# Copyright (c) 2014 - 2016, Intel Corporation. All rights reserved.
+# Copyright (c) 2014 - 2021, Intel Corporation. All rights reserved.
# # SPDX-License-Identifier: BSD-2-Clause-Patent # @@ -76,8 +76,7 @@ gIntelFsp2WrapperTokenSpaceGuid.PcdFspmBaseAddress ## CONSU= MES =20 [FixedPcd] - gIntelFsp2WrapperTokenSpaceGuid.PcdCpuMicrocodePatchAddress ## CONSU= MES - gIntelFsp2WrapperTokenSpaceGuid.PcdCpuMicrocodePatchRegionSize ## CONSU= MES - gIntelFsp2WrapperTokenSpaceGuid.PcdFlashMicrocodeOffset ## CONSU= MES + gUefiCpuPkgTokenSpaceGuid.PcdCpuMicrocodePatchAddress ## CONSU= MES + gUefiCpuPkgTokenSpaceGuid.PcdCpuMicrocodePatchRegionSize ## CONSU= MES gIntelFsp2WrapperTokenSpaceGuid.PcdFlashCodeCacheAddress ## CONSU= MES gIntelFsp2WrapperTokenSpaceGuid.PcdFlashCodeCacheSize ## CONSU= MES --=20 2.28.0.windows.1 -=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 (#74076): https://edk2.groups.io/g/devel/message/74076 Mute This Topic: https://groups.io/mt/82082551/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-