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>
Subject: Re: [Patch] FDF Spec: Per PI 1.6 to extend FFS alignment to 16M
Date: Tue, 26 Sep 2017 03:38:05 +0000 [thread overview]
Message-ID: <4A89E2EF3DFEDB4C8BFDE51014F606A14E15D620@SHSMSX152.ccr.corp.intel.com> (raw)
In-Reply-To: <1506310930-13104-1-git-send-email-yonghong.zhu@intel.com>
Reviewed-by: Liming Gao <liming.gao@intel.com>
> -----Original Message-----
> From: Zhu, Yonghong
> Sent: Monday, September 25, 2017 11:42 AM
> 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: Per PI 1.6 to extend FFS alignment to 16M
>
> 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/25_[fv]_sections.md | 4 ++--
> 3_edk_ii_fdf_file_format/32_fdf_definition.md | 4 +++-
> README.md | 1 +
> 3 files changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/2_fdf_design_discussion/25_[fv]_sections.md b/2_fdf_design_discussion/25_[fv]_sections.md
> index 928455c..82755dd 100644
> --- a/2_fdf_design_discussion/25_[fv]_sections.md
> +++ b/2_fdf_design_discussion/25_[fv]_sections.md
> @@ -339,12 +339,12 @@ The `Fixed` and `Checksum` attributes are boolean flags, both default to
>
> The Alignment attribute requires the "= value".
>
> * `Fixed` - File can not be moved, default (not specified) is relocate-able.
>
> -* `Alignment` - Data (value is one of: 1, 2 4, 8, 16, 32, 64 128, 512, 1K, 2K,
> - 4K, 8K, 16K, 32K, 64K) byte aligned
> +* `Alignment` - Data (value is one of: 1, 2, 4, 8, 16, 32, 64, 128, 512, 1K, 2K,
> + 4K, 8K, 16K, 32K, 64K, 128K, 256K, 512K, 1M, 2M, 4M, 8M, 16M) byte aligned
>
> * `Checksum` - It is recommended that this be controlled on an entire FV basis
> not at the file level, however, we are including this attribute for
> completeness.
>
> diff --git a/3_edk_ii_fdf_file_format/32_fdf_definition.md b/3_edk_ii_fdf_file_format/32_fdf_definition.md
> index 67976b5..1379db4 100644
> --- a/3_edk_ii_fdf_file_format/32_fdf_definition.md
> +++ b/3_edk_ii_fdf_file_format/32_fdf_definition.md
> @@ -225,11 +225,13 @@ The following are common definitions used by multiple section types.
> {"4K"} {"8K"} {"16K"} {"32K"} {"64K"}
> {"128K"} {"256K"} {"512K"} {"1M"} {"2M"} {"4M"}
> {"8M"} {"16M"} {"32M"} {"64M"}
> {"128M"} {"256M"} {"512M"} {"1G"} {"2G"}
> <FfsAlignmentValues> ::= {"Auto"} {"8"} {"16"} {"32"} {"64"} {"128"}
> - {"512"} {"1K"} {"4K"} {"32K"} {"64K"}
> + {"512"} {"1K"} {"4K"} {"32K"} {"64K"} {"128K"}
> + {"256K"} {"512K"} {"1M"} {"2M"} {"4M"} {"8M"}
> + {"16M"}
> ```
>
> **********
> **Note:** When using the characters "|" or "||" in an expression, the
> expression must be encapsulated in open "(" and close ")" parenthesis.
> diff --git a/README.md b/README.md
> index 7ff8f36..71c92fc 100644
> --- a/README.md
> +++ b/README.md
> @@ -206,5 +206,6 @@ Copyright (c) 2006-2017, Intel Corporation. All rights reserved.
> | | [#142](https://bugzilla.tianocore.org/show_bug.cgi?id=142) Update EDK II FDF Specification to allow sections in
> any order | |
> | | [#478](https://bugzilla.tianocore.org/show_bug.cgi?id=478) FDF spec: extend the <FmpFileData> to support
> <FvStatements> and <FdStatenents> | |
> | | [#353](https://bugzilla.tianocore.org/show_bug.cgi?id=353) Build spec: Allow nested includes in DSC and FDF files
> | |
> | | [#520](https://bugzilla.tianocore.org/show_bug.cgi?id=520) FDF spec: Update Precedence of PCD Values
> | |
> | | [#585](https://bugzilla.tianocore.org/show_bug.cgi?id=585) FDF Spec: Update the FDF_SPECIFICATION version to
> 0x0001001B or 1.27 | |
> +| | Per PI 1.6 to extend FFS alignment to 16M
> | |
> --
> 2.6.1.windows.1
prev parent reply other threads:[~2017-09-26 3:35 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-25 3:42 [Patch] FDF Spec: Per PI 1.6 to extend FFS alignment to 16M Yonghong Zhu
2017-09-26 3:38 ` Gao, Liming [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4A89E2EF3DFEDB4C8BFDE51014F606A14E15D620@SHSMSX152.ccr.corp.intel.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox