From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <liming.gao@intel.com>
Received-SPF: Pass (sender SPF authorized) identity=mailfrom;
 client-ip=134.134.136.20; helo=mga02.intel.com;
 envelope-from=liming.gao@intel.com; receiver=edk2-devel@lists.01.org 
Received: from mga02.intel.com (mga02.intel.com [134.134.136.20])
 (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 8CA4021FB7D53
 for <edk2-devel@lists.01.org>; Tue,  3 Apr 2018 01:35:08 -0700 (PDT)
X-Amp-Result: SKIPPED(no attachment in message)
X-Amp-File-Uploaded: False
Received: from orsmga004.jf.intel.com ([10.7.209.38])
 by orsmga101.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384;
 03 Apr 2018 01:35:08 -0700
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.48,400,1517904000"; d="scan'208";a="188223499"
Received: from fmsmsx107.amr.corp.intel.com ([10.18.124.205])
 by orsmga004.jf.intel.com with ESMTP; 03 Apr 2018 01:35:08 -0700
Received: from fmsmsx152.amr.corp.intel.com (10.18.125.5) by
 fmsmsx107.amr.corp.intel.com (10.18.124.205) with Microsoft SMTP Server (TLS)
 id 14.3.319.2; Tue, 3 Apr 2018 01:35:07 -0700
Received: from shsmsx102.ccr.corp.intel.com (10.239.4.154) by
 FMSMSX152.amr.corp.intel.com (10.18.125.5) with Microsoft SMTP Server (TLS)
 id 14.3.319.2; Tue, 3 Apr 2018 01:35:07 -0700
Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.226]) by
 shsmsx102.ccr.corp.intel.com ([169.254.2.80]) with mapi id 14.03.0319.002;
 Tue, 3 Apr 2018 16:35:05 +0800
From: "Gao, Liming" <liming.gao@intel.com>
To: "Zhu, Yonghong" <yonghong.zhu@intel.com>, "edk2-devel@lists.01.org"
 <edk2-devel@lists.01.org>
CC: "Kinney, Michael D" <michael.d.kinney@intel.com>, "Shaw, Kevin W"
 <kevin.w.shaw@intel.com>
Thread-Topic: [Patch] FDF spec: Update version to 1.28
Thread-Index: AQHTyxvteO0tyO+xmUm/3Hmli84z+qPutxdQ
Date: Tue, 3 Apr 2018 08:35:04 +0000
Message-ID: <4A89E2EF3DFEDB4C8BFDE51014F606A14E1F197C@SHSMSX104.ccr.corp.intel.com>
References: <1522739869-18508-1-git-send-email-yonghong.zhu@intel.com>
 <1522739869-18508-6-git-send-email-yonghong.zhu@intel.com>
In-Reply-To: <1522739869-18508-6-git-send-email-yonghong.zhu@intel.com>
Accept-Language: en-US
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
x-originating-ip: [10.239.127.40]
MIME-Version: 1.0
Subject: Re: [Patch] FDF spec: Update version to 1.28
X-BeenThere: edk2-devel@lists.01.org
X-Mailman-Version: 2.1.26
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: Tue, 03 Apr 2018 08:35:08 -0000
Content-Language: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable

Reviewed-by: Liming Gao <liming.gao@intel.com>

>-----Original Message-----
>From: Zhu, Yonghong
>Sent: Tuesday, April 03, 2018 3:18 PM
>To: edk2-devel@lists.01.org
>Cc: Gao, Liming <liming.gao@intel.com>; Kinney, Michael D
><michael.d.kinney@intel.com>; Shaw, Kevin W <kevin.w.shaw@intel.com>
>Subject: [Patch] FDF spec: Update version to 1.28
>
>Cc: Liming Gao <liming.gao@intel.com>
>Cc: Michael Kinney <michael.d.kinney@intel.com>
>Cc: Kevin W Shaw <kevin.w.shaw@intel.com>
>Contributed-under: TianoCore Contribution Agreement 1.1
>Signed-off-by: Yonghong Zhu <yonghong.zhu@intel.com>
>---
> 2_fdf_design_discussion/README.md                |  4 ++--
> 3_edk_ii_fdf_file_format/34_[defines]_section.md | 16 ++++++++--------
> README.md                                        |  5 +++--
> 3 files changed, 13 insertions(+), 12 deletions(-)
>
>diff --git a/2_fdf_design_discussion/README.md
>b/2_fdf_design_discussion/README.md
>index 3202f56..7d87e43 100644
>--- a/2_fdf_design_discussion/README.md
>+++ b/2_fdf_design_discussion/README.md
>@@ -1,9 +1,9 @@
> <!--- @file
>   2 FDF Design Discussion
>
>-  Copyright (c) 2006-2017, Intel Corporation. All rights reserved.<BR>
>+  Copyright (c) 2006-2018, Intel Corporation. All rights reserved.<BR>
>
>   Redistribution and use in source (original document form) and 'compiled=
'
>   forms (converted to PDF, epub, HTML and other formats) with or without
>   modification, are permitted provided that the following conditions are =
met:
>
>@@ -50,11 +50,11 @@ The EDK II Build generates UEFI and PI specification
>compliant binary images.
> The tools provided in the EDK and the EdkCompatibilityPkg module support
> earlier versions of the specifications.
>
> This revision of the specification adds support for multiple binary files=
 in
> an FV FILE RAW statement. FDF files that use this feature must use the ne=
w
>-`FDF_SPECIFICATION =3D 0x0001001B` in the `[Defines]` section. Older FDF =
files
>+`FDF_SPECIFICATION =3D 0x0001001C` in the `[Defines]` section. Older FDF =
files
> do not need to update the `FDF_SPECIFICATION` value.
>
> The EDK II build system has been updated to allow the setting of multiple
>paths
> that will be searched when attempting to resolve the location of EDK II
> packages. This new feature allows for more flexibility when designing a t=
ree
>diff --git a/3_edk_ii_fdf_file_format/34_[defines]_section.md
>b/3_edk_ii_fdf_file_format/34_[defines]_section.md
>index 5bbb890..83b0bbf 100644
>--- a/3_edk_ii_fdf_file_format/34_[defines]_section.md
>+++ b/3_edk_ii_fdf_file_format/34_[defines]_section.md
>@@ -1,9 +1,9 @@
> <!--- @file
>   3.4 [Defines] Section
>
>-  Copyright (c) 2006-2017, Intel Corporation. All rights reserved.<BR>
>+  Copyright (c) 2006-2018, Intel Corporation. All rights reserved.<BR>
>
>   Redistribution and use in source (original document form) and 'compiled=
'
>   forms (converted to PDF, epub, HTML and other formats) with or without
>   modification, are permitted provided that the following conditions are =
met:
>
>@@ -38,17 +38,17 @@ section following comment blocks at the beginning of
>the file.
>
> This section describes the defines section content in the FDF files. This=
 file
> can be created by a developer and is an input to the EDK II build tool pa=
rsing
> utilities. Elements may appear in any order within this section.
>
>-The code for this version of the FDF specification is "0x0001001B". New
>-versions of this specification must increment the minor (001B) portion of=
 the
>+The code for this version of the FDF specification is "0x0001001C". New
>+versions of this specification must increment the minor (001C) portion of=
 the
> specification code for backward-compatible changes, and increment the
>major
> specification number for non-backward-compatible changes.
>
> This revision of the specification adds new features. Any FDF file that u=
ses
>-these new features must use the value `0x0001001B` in the
>`FDF_SPECIFICATION`
>+these new features must use the value `0x0001001C` in the
>`FDF_SPECIFICATION`
> statement. Older FDF files that do not use these new feature do not need =
to
> update the value.
>
> Conditional statements may be used anywhere within this section.
>
>@@ -78,17 +78,17 @@ The version number for this flash definition; the valu=
e
>is not used by build
> tools, but the version element is provided for user tracking capabilities=
 that
> may be used by other user interface tools.
>
> **_FDF_SPECIFICATION_**
>
>-For this specification, the version value is 0x0001001B. Tools that proce=
ss
>+For this specification, the version value is 0x0001001C. Tools that proce=
ss
> this version of the FDF file can successfully process earlier versions of=
 the
> FDF files (this is a backward compatible update). If an FDF file with an
> earlier version of the `FDF_SPECIFICATION` is modified to use a feature
>added
>-in the 1.27 version of this specification must be updated to 0x0001001B.
>There
>+in the 1.28 version of this specification must be updated to 0x0001001C.
>There
> is no requirement to change existing entries if no other content changes.=
 This
>-value may also be specified as decimal value, such as 1.27.
>+value may also be specified as decimal value, such as 1.28.
>
> **_PcdNames_**
>
> PCDs defined in this section take precedence over PCD values specified in
>other
> meta-data files. Note that PCDs defined via the SET statements in sub-
>sections
>@@ -97,9 +97,9 @@ meta-data files.
>
> #### Example
>
> ```ini
> [Defines]
>-  FDF_SPECIFICATION                          =3D 0x0001001B
>+  FDF_SPECIFICATION                          =3D 0x0001001C
>   DEFINE BIG_STUFF                           =3D False
>   SET gEfiMyPlatformTokenSpaceGuid.MyUsbFlag =3D True
> ```
>diff --git a/README.md b/README.md
>index 4e15a73..b4a04eb 100644
>--- a/README.md
>+++ b/README.md
>@@ -1,9 +1,9 @@
> <!--- @file
>   README.md for EDK II Flash Description (FDF) File Specification
>
>-  Copyright (c) 2006-2017, Intel Corporation. All rights reserved.<BR>
>+  Copyright (c) 2006-2018, Intel Corporation. All rights reserved.<BR>
>
>   Redistribution and use in source (original document form) and 'compiled=
'
>   forms (converted to PDF, epub, HTML and other formats) with or without
>   modification, are permitted provided that the following conditions are =
met:
>
>@@ -192,11 +192,11 @@ Copyright (c) 2006-2017, Intel Corporation. All righ=
ts
>reserved.
> |            | WORKSPACE directory tree (refer to the TianoCore.org/ EDKI=
I
>website for additional instructions on setting up a development environmen=
t).
>|               |
> |            | Added new system environment variables,
>|               |
> |            | PACKAGES_PATH and EDK_TOOLS_BIN, used by the build system.
>|               |
> |            | Allow INF statements in FD regions.
>|               |
> |            | Clarified [UserExtensions] content in chapter 2 (to match
>implementation)                                                           =
                                       |               |
>-| 1.28       | Convert to GitBooks
>| June 2017     |
>+| 1.27       | Convert to GitBooks
>| June 2017     |
> |            | [#426](https://bugzilla.tianocore.org/show_bug.cgi?id=3D42=
6)
>IMAGE_TYPE_ID must be provided with value, FDF should mark it as required
>section                               |               |
> |            | [#373](https://bugzilla.tianocore.org/show_bug.cgi?id=3D37=
3)
>Conditional statement examples incorrect
>|               |
> |            | [#461](https://bugzilla.tianocore.org/show_bug.cgi?id=3D46=
1) FDF
>Spec: add a super script number for the <FmpFileData>
>|               |
> |            | [#249](https://bugzilla.tianocore.org/show_bug.cgi?id=3D24=
9) FDF
>spec miss '<UiFmpName>' definition                                        =
                                  |
>|
> |            | [#350](https://bugzilla.tianocore.org/show_bug.cgi?id=3D35=
0) [FDF
>Spec] Extend the macro usage in the !include statement
>|               |
>@@ -206,7 +206,8 @@ Copyright (c) 2006-2017, Intel Corporation. All rights
>reserved.
> |            | [#142](https://bugzilla.tianocore.org/show_bug.cgi?id=3D14=
2) Update
>EDK II FDF Specification to allow sections in any order
>|               |
> |            | [#478](https://bugzilla.tianocore.org/show_bug.cgi?id=3D47=
8) FDF
>spec: extend the <FmpFileData> to support <FvStatements> and
><FdStatenents>                                 |               |
> |            | [#353](https://bugzilla.tianocore.org/show_bug.cgi?id=3D35=
3) Build
>spec: Allow nested includes in DSC and FDF files                          =
                                |
>|
> |            | [#520](https://bugzilla.tianocore.org/show_bug.cgi?id=3D52=
0) FDF
>spec: Update Precedence of PCD Values                                     =
                                  |
>|
> |            | [#585](https://bugzilla.tianocore.org/show_bug.cgi?id=3D58=
5) FDF
>Spec: Update the FDF_SPECIFICATION version to 0x0001001B or 1.27
>|               |
>+| 1.28       | Update version to 1.28
>| March 2018    |
> |            | Per PI 1.6 to extend FFS alignment to 16M
>|               |
> |            | Per PI 1.6 to support FV extended header entry contain the=
 used size
>of FV                                                                     =
                            |               |
>--
>2.6.1.windows.1