From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by mx.groups.io with SMTP id smtpd.web10.2706.1658351163364644772 for ; Wed, 20 Jul 2022 14:06:04 -0700 Authentication-Results: mx.groups.io; dkim=fail reason="unable to parse pub key" header.i=@intel.com header.s=intel header.b=kJaaIFUX; spf=pass (domain: intel.com, ip: 192.55.52.43, mailfrom: chasel.chiu@intel.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1658351163; x=1689887163; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=BxsEIiNstrK2U+yuoYSFY7UAKqPoNt/OCL3EZEDyV90=; b=kJaaIFUXRMXqC8RQOGC2hcJAlWU6uZGrfB8w5TJMovc4jbnUctg2Kt5J n9Qu6UcV5u+HHccSFw7TREXHXCsrhpyVBiHPsgrKJS6H2Vv4ttzL3K4lu RmjvBZ5wECnsUmPbrHxOaPq9j3Yw6UmonRqk3B1cZyzw5tWnCfrrTNtXW mFmZxspYeeDSB1vR0iQyZbt0/YerfoMRNtpjxw6SffVnW7rjSSuWli4Oi 7MYmKuB9EPJVjbvHIFF4tsdXnPunk888o21FiQcn78/SxjeVNcEJ0X1TL n+bzWj2IaNRy3GnbKRTWX7rG28l7mlx0I29KRNMCeKWBzx6tBPp+jDlNV Q==; X-IronPort-AV: E=McAfee;i="6400,9594,10414"; a="373187033" X-IronPort-AV: E=Sophos;i="5.92,287,1650956400"; d="scan'208";a="373187033" Received: from orsmga008.jf.intel.com ([10.7.209.65]) by fmsmga105.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 20 Jul 2022 14:05:35 -0700 X-IronPort-AV: E=Sophos;i="5.92,287,1650956400"; d="scan'208";a="625827028" Received: from cchiu4-mobl.gar.corp.intel.com ([10.255.228.191]) by orsmga008-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 20 Jul 2022 14:05:35 -0700 From: "Chiu, Chasel" To: devel@edk2.groups.io Cc: Chasel Chiu , Nate DeSimone , Star Zeng Subject: [PATCH v2] IntelFsp2Pkg: Add FSPI_ARCH_UPD. Date: Wed, 20 Jul 2022 14:05:24 -0700 Message-Id: <20220720210524.3855-1-chasel.chiu@intel.com> X-Mailer: git-send-email 2.35.0.windows.1 MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D3993 Adding the missing FSPI_ARCH_UPD, FSP_GLOBAL_DATA_VERSION bumpping up, and some comments for clarification. Cc: Nate DeSimone Cc: Star Zeng Signed-off-by: Chasel Chiu --- IntelFsp2Pkg/Include/FspEas/FspApi.h | 71 ++++++++++++++++++++++++++++++++= ++++++++++++++++++++++++++++++++------- IntelFsp2Pkg/Include/FspGlobalData.h | 2 +- 2 files changed, 65 insertions(+), 8 deletions(-) diff --git a/IntelFsp2Pkg/Include/FspEas/FspApi.h b/IntelFsp2Pkg/Include/Fs= pEas/FspApi.h index bf46f13f73..3f368574e8 100644 --- a/IntelFsp2Pkg/Include/FspEas/FspApi.h +++ b/IntelFsp2Pkg/Include/FspEas/FspApi.h @@ -1,6 +1,6 @@ /** @file=0D Intel FSP API definition from Intel Firmware Support Package External=0D - Architecture Specification v2.0 - v2.2=0D + Architecture Specification v2.0 and above.=0D =0D Copyright (c) 2014 - 2022, Intel Corporation. All rights reserved.
=0D SPDX-License-Identifier: BSD-2-Clause-Patent=0D @@ -100,13 +100,14 @@ typedef struct { /// "XXXXXX_T" for FSP-T=0D /// "XXXXXX_M" for FSP-M=0D /// "XXXXXX_S" for FSP-S=0D + /// "XXXXXX_I" for FSP-I=0D /// Where XXXXXX is an unique signature=0D ///=0D UINT64 Signature;=0D ///=0D /// Revision of the Data structure.=0D - /// For FSP spec 2.0/2.1 value is 1.=0D - /// For FSP spec 2.2 value is 2.=0D + /// For FSP spec 2.0/2.1, this value is 1 and only FSPM_UPD having ARC= H_UPD.=0D + /// For FSP spec 2.2 and above, this value is 2 and ARCH_UPD present i= n all UPD structures.=0D ///=0D UINT8 Revision;=0D UINT8 Reserved[23];=0D @@ -134,7 +135,7 @@ typedef struct { } FSPT_ARCH_UPD;=0D =0D ///=0D -/// FSPT_ARCH2_UPD Configuration.=0D +/// FSPT_ARCH2_UPD Configuration for FSP 2.4 and above.=0D ///=0D typedef struct {=0D ///=0D @@ -196,7 +197,7 @@ typedef struct { } FSPM_ARCH_UPD;=0D =0D ///=0D -/// FSPM_ARCH2_UPD Configuration.=0D +/// FSPM_ARCH2_UPD Configuration for FSP 2.4 and above.=0D ///=0D typedef struct {=0D ///=0D @@ -209,6 +210,13 @@ typedef struct { ///=0D UINT32 Length;=0D ///=0D + /// Pointer to the non-volatile storage (NVS) data buffer.=0D + /// If it is NULL it indicates the NVS data is not available.=0D + /// This value is deprecated starting with v2.4 of the FSP specification= ,=0D + /// and will be removed in an upcoming version of the FSP specification.= =0D + ///=0D + EFI_PHYSICAL_ADDRESS NvsBufferPtr;=0D + ///=0D /// Pointer to the temporary stack base address to be=0D /// consumed inside FspMemoryInit() API.=0D ///=0D @@ -232,7 +240,7 @@ typedef struct { /// This value is only valid if Revision is >=3D 2.=0D ///=0D EFI_PHYSICAL_ADDRESS FspEventHandler;=0D - UINT8 Reserved1[24];=0D + UINT8 Reserved1[16];=0D } FSPM_ARCH2_UPD;=0D =0D ///=0D @@ -265,7 +273,7 @@ typedef struct { } FSPS_ARCH_UPD;=0D =0D ///=0D -/// FSPS_ARCH2_UPD Configuration.=0D +/// FSPS_ARCH2_UPD Configuration for FSP 2.4 and above.=0D ///=0D typedef struct {=0D ///=0D @@ -285,6 +293,40 @@ typedef struct { UINT8 Reserved1[16];=0D } FSPS_ARCH2_UPD;=0D =0D +///=0D +/// FSPI_ARCH_UPD Configuration.=0D +///=0D +typedef struct {=0D + ///=0D + /// Revision of the structure is 1 for this version of the specification= .=0D + ///=0D + UINT8 Revision;=0D + UINT8 Reserved[3];=0D + ///=0D + /// Length of the structure in bytes. The current value for this field i= s 32.=0D + ///=0D + UINT32 Length;=0D + ///=0D + /// The physical memory-mapped base address of the bootloader SMM firmwa= re volume (FV).=0D + ///=0D + EFI_PHYSICAL_ADDRESS BootloaderSmmFvBaseAddress;=0D + ///=0D + /// The length in bytes of the bootloader SMM firmware volume (FV).=0D + ///=0D + UINT64 BootloaderSmmFvLength;=0D + ///=0D + /// The physical memory-mapped base address of the bootloader SMM FV con= text data.=0D + /// This data is provided to bootloader SMM drivers through a HOB by the= FSP MM Foundation.=0D + ///=0D + EFI_PHYSICAL_ADDRESS BootloaderSmmFvContextData;=0D + ///=0D + /// The length in bytes of the bootloader SMM FV context data.=0D + /// This data is provided to bootloader SMM drivers through a HOB by the= FSP MM Foundation.=0D + ///=0D + UINT16 BootloaderSmmFvContextDataLength;=0D + UINT8 Reserved1[24];=0D +} FSPI_ARCH_UPD;=0D +=0D ///=0D /// FSPT_UPD_COMMON Configuration.=0D ///=0D @@ -393,6 +435,21 @@ typedef struct { FSPS_ARCH2_UPD FspsArchUpd;=0D } FSPS_UPD_COMMON_FSP24;=0D =0D +///=0D +/// FSPI_UPD_COMMON Configuration.=0D +///=0D +typedef struct {=0D + ///=0D + /// FSP_UPD_HEADER Configuration.=0D + ///=0D + FSP_UPD_HEADER FspUpdHeader;=0D +=0D + ///=0D + /// FSPI_ARCH_UPD Configuration.=0D + ///=0D + FSPI_ARCH_UPD FspiArchUpd;=0D +} FSPI_UPD_COMMON;=0D +=0D ///=0D /// Enumeration of FSP_INIT_PHASE for NOTIFY_PHASE.=0D ///=0D diff --git a/IntelFsp2Pkg/Include/FspGlobalData.h b/IntelFsp2Pkg/Include/Fs= pGlobalData.h index 697b20ed4c..cf94f7b6a5 100644 --- a/IntelFsp2Pkg/Include/FspGlobalData.h +++ b/IntelFsp2Pkg/Include/FspGlobalData.h @@ -12,7 +12,7 @@ =0D #define FSP_IN_API_MODE 0=0D #define FSP_IN_DISPATCH_MODE 1=0D -#define FSP_GLOBAL_DATA_VERSION 1=0D +#define FSP_GLOBAL_DATA_VERSION 0x2=0D =0D #pragma pack(1)=0D =0D --=20 2.35.0.windows.1