From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id B0BC31A1E31 for ; Thu, 20 Oct 2016 17:46:08 -0700 (PDT) Received: from orsmga005.jf.intel.com ([10.7.209.41]) by fmsmga105.fm.intel.com with ESMTP; 20 Oct 2016 17:46:08 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.31,521,1473145200"; d="scan'208";a="22005972" Received: from sprasa6-mobl.gar.corp.intel.com ([10.252.80.214]) by orsmga005.jf.intel.com with ESMTP; 20 Oct 2016 17:46:06 -0700 From: Satya Yarlagadda To: edk2-devel@lists.01.org Cc: Maurice Ma , Jiewen Yao , Giri P Mudusuru Date: Fri, 21 Oct 2016 06:15:55 +0530 Message-Id: <20161021004555.15076-1-satya.p.yarlagadda@intel.com> X-Mailer: git-send-email 2.10.0.windows.1 Subject: [PATCH] IntelFsp2Pkg: Removing FSP Data Table X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 21 Oct 2016 00:46:08 -0000 FSP Base address is part of FSP_INFO_HEADER and we don't need additional table to report the base address of FSP components. we expect Boot loaders to parse the FSP INFO Header to get the base address. Also, this the FSP_DATA_TABLE is not defined in the FSP EAS and we don't see a need for it. Cc: Maurice Ma Cc: Jiewen Yao Cc: Giri P Mudusuru Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Satya Yarlagadda --- IntelFsp2Pkg/Include/FspDataTable.h | 32 ---------------------- .../Library/BaseFspCommonLib/FspCommonLib.c | 1 - 2 files changed, 33 deletions(-) delete mode 100644 IntelFsp2Pkg/Include/FspDataTable.h diff --git a/IntelFsp2Pkg/Include/FspDataTable.h b/IntelFsp2Pkg/Include/FspDataTable.h deleted file mode 100644 index 3c79f34..0000000 --- a/IntelFsp2Pkg/Include/FspDataTable.h +++ /dev/null @@ -1,32 +0,0 @@ -/** @file - The header file of FSP data table - - Copyright (c) 2016, Intel Corporation. All rights reserved.
- This program and the accompanying materials - are licensed and made available under the terms and conditions of the BSD License - which accompanies this distribution. The full text of the license may be found at - http://opensource.org/licenses/bsd-license.php. - - THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, - WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. - -**/ - -#ifndef _FSP_DATA_TABLE_H_ -#define _FSP_DATA_TABLE_H_ - -#pragma pack(1) - -#define FSP_DATA_SIGNATURE SIGNATURE_32 ('F', 'S', 'P', 'D') - -typedef struct { - UINT32 Signature; - UINT32 Length; - UINT32 FsptBase; - UINT32 FspmBase; - UINT32 FspsBase; -} FSP_DATA_TABLE; - -#pragma pack() - -#endif diff --git a/IntelFsp2Pkg/Library/BaseFspCommonLib/FspCommonLib.c b/IntelFsp2Pkg/Library/BaseFspCommonLib/FspCommonLib.c index a9c8da6..2f2981c 100644 --- a/IntelFsp2Pkg/Library/BaseFspCommonLib/FspCommonLib.c +++ b/IntelFsp2Pkg/Library/BaseFspCommonLib/FspCommonLib.c @@ -17,7 +17,6 @@ #include #include #include -#include #include #pragma pack(1) -- 2.10.0.windows.1