1. Patchew
  2. EDK2
  3. View series

[edk2] [Patch] FDF Spec: Per PI 1.6 to extend FFS alignment to 16M

Yonghong Zhu posted 1 patch 7 years, 3 months ago
Failed in applying to current master (apply log)
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(-)
[edk2] [Patch] FDF Spec: Per PI 1.6 to extend FFS alignment to 16M
Posted by Yonghong Zhu 7 years, 3 months ago 
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

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [Patch] FDF Spec: Per PI 1.6 to extend FFS alignment to 16M
Posted by Gao, Liming 7 years, 3 months ago 
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

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Patchew 2.1-devel-8c64711

© 2016 - 2024 Red Hat, Inc.