From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <giri.p.mudusuru@intel.com>
Received: from mga03.intel.com (mga03.intel.com [134.134.136.65])
 by ml01.01.org (Postfix) with ESMTP id 765431A1DFB
 for <edk2-devel@lists.01.org>; Thu,  4 Aug 2016 22:44:16 -0700 (PDT)
Received: from fmsmga003.fm.intel.com ([10.253.24.29])
 by orsmga103.jf.intel.com with ESMTP; 04 Aug 2016 22:44:16 -0700
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.28,472,1464678000"; d="scan'208";a="744487186"
Received: from orsmsx106.amr.corp.intel.com ([10.22.225.133])
 by FMSMGA003.fm.intel.com with ESMTP; 04 Aug 2016 22:44:16 -0700
Received: from orsmsx113.amr.corp.intel.com ([169.254.9.118]) by
 ORSMSX106.amr.corp.intel.com ([169.254.1.199]) with mapi id 14.03.0248.002;
 Thu, 4 Aug 2016 22:44:15 -0700
From: "Mudusuru, Giri P" <giri.p.mudusuru@intel.com>
To: "Ma, Maurice" <maurice.ma@intel.com>, "edk2-devel@lists.01.org"
 <edk2-devel@lists.01.org>
CC: "Yao, Jiewen" <jiewen.yao@intel.com>
Thread-Topic: [edk2] [PATCH] IntelFsp2Pkg: Only include required header
 files in FspEas.h
Thread-Index: AQHR7tC7zPy0zHjbvE2wVTq3CR2QcqA52v/g
Date: Fri, 5 Aug 2016 05:44:14 +0000
Message-ID: <4666AEFED60F8E4198B42BB01DCEABDF76EA08B7@ORSMSX113.amr.corp.intel.com>
References: <c6e7ebb4cc163a054cf4fb20c3e3156242be05ca.1470370785.git.maurice.ma@intel.com>
In-Reply-To: <c6e7ebb4cc163a054cf4fb20c3e3156242be05ca.1470370785.git.maurice.ma@intel.com>
Accept-Language: en-US, hi-IN
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
x-originating-ip: [10.22.254.138]
MIME-Version: 1.0
Subject: Re: [PATCH] IntelFsp2Pkg: Only include required header files in FspEas.h
X-BeenThere: edk2-devel@lists.01.org
X-Mailman-Version: 2.1.21
Precedence: list
List-Id: EDK II Development  <edk2-devel.lists.01.org>
List-Unsubscribe: <https://lists.01.org/mailman/options/edk2-devel>,
 <mailto:edk2-devel-request@lists.01.org?subject=unsubscribe>
List-Archive: <http://lists.01.org/pipermail/edk2-devel/>
List-Post: <mailto:edk2-devel@lists.01.org>
List-Help: <mailto:edk2-devel-request@lists.01.org?subject=help>
List-Subscribe: <https://lists.01.org/mailman/listinfo/edk2-devel>,
 <mailto:edk2-devel-request@lists.01.org?subject=subscribe>
X-List-Received-Date: Fri, 05 Aug 2016 05:44:16 -0000
Content-Language: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable

Reviewed-by: Giri P Mudusuru <giri.p.mudusuru@intel.com>=20

> -----Original Message-----
> From: Ma, Maurice
> Sent: Thursday, August 4, 2016 9:20 PM
> To: edk2-devel@lists.01.org
> Cc: Ma, Maurice <maurice.ma@intel.com>; Yao, Jiewen
> <jiewen.yao@intel.com>; Mudusuru, Giri P <giri.p.mudusuru@intel.com>
> Subject: [edk2] [PATCH] IntelFsp2Pkg: Only include required header files =
in
> FspEas.h
>=20
> Current FspEas.h file includes Uefi.h which also refers to lots of other
> UEFI header files not used by FSP consumer. It caused many unnecessary
> header file overhead for a bootloader that consumes FSP, such as coreboot=
.
> This change reduces the required header file number down to 3.
>=20
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Giri P Mudusuru <giri.p.mudusuru@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Maurice Ma <maurice.ma@intel.com>
> ---
>  IntelFsp2Pkg/Include/FspEas.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>=20
> diff --git a/IntelFsp2Pkg/Include/FspEas.h b/IntelFsp2Pkg/Include/FspEas.=
h
> index 79bb0b8e8efc..00098a392699 100644
> --- a/IntelFsp2Pkg/Include/FspEas.h
> +++ b/IntelFsp2Pkg/Include/FspEas.h
> @@ -16,7 +16,7 @@
>  #ifndef _FSP_EAS_H_
>  #define _FSP_EAS_H_
>=20
> -#include <Uefi.h>
> +#include <Uefi/UefiBaseType.h>
>  #include <Guid/GuidHobFspEas.h>
>  #include <Guid/FspHeaderFile.h>
>  #include <FspEas/FspApi.h>
> --
> 1.9.5.msysgit.0