Re: [edk2] [Patch] Document: Update Inf spec to remove EDK related contents
Hi Liming, I'll update patch for 1) and 2) For 3) This patch removed "and are valid for EDK-II modules only." Because I think there is only EDK-II now, we don't need to point out this section is for EDKII ONLY here. Thanks, Bob -Original Message- From: Gao, Liming Sent: Wednesday, March 6, 2019 4:02 PM To: Feng, Bob C ; edk2-devel@lists.01.org Cc: Carsey, Jaben Subject: RE: [edk2] [Patch] Document: Update Inf spec to remove EDK related contents Bob: I have some comments. 1) Please update the commit message. The change is to remove EDK and IPF related contents. 2) The below change should make sure the updated one keep the same alignment. -||- +|| 3) Below content is correct. It is for EDKII. No need to change it. -These sections are used for specifying PCD information and are valid for EDK -II modules only. The entries for these sections are looked up from the package -declaration files (DEC) for generating the `AutoGen.c` and `AutoGen.h` files. +These sections are used for specifying PCD information. The entries for these +sections are looked up from the packagedeclaration files (DEC) for generating +the `AutoGen.c` and `AutoGen.h` files. Thanks Liming > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Feng, > Bob C > Sent: Monday, March 4, 2019 5:52 AM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Gao, Liming > Subject: [edk2] [Patch] Document: Update Inf spec to remove EDK related > contents > > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1453 > > Remove EDK related contents > > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Bob Feng > Cc: Liming Gao > Cc: Jaben Carsey > --- > 1_introduction/11_overview.md | 2 +- > 2_inf_overview/210_[ppis]_section.md | 1 - > 2_inf_overview/211_[guids]_section.md | 1 - > .../212_[libraryclasses]_section.md | 1 - > 2_inf_overview/213_[packages]_section.md | 1 - > 2_inf_overview/214_pcd_sections.md| 14 +- > 2_inf_overview/215_[depex]_section.md | 4 - > 2_inf_overview/21_processing_overview.md | 19 +- > .../22_information_file_general_rules.md | 30 +- > 2_inf_overview/24_[defines]_section.md| 4 +- > 2_inf_overview/25_[sources]_section.md| 4 - > 2_inf_overview/26_[buildoptions]_section.md | 9 +- > 2_inf_overview/27_[binaries]_section.md | 5 - > 2_inf_overview/29_[protocols]_section.md | 1 - > 2_inf_overview/README.md | 11 - > .../314_[depex]_sections.md | 6 +- > .../315_[binaries]_section.md | 5 - > .../32_component_inf_definition.md| 4 +- > .../34_[defines]_section.md | 6 +- > .../35_[buildoptions]_sections.md | 5 +- > .../39_[sources]_sections.md | 7 +- > 3_edk_ii_inf_file_format/README.md| 4 - > ...ndix_a_build_changes_and_customizations.md | 0 > .../README.md | 40 -- > .../a1_design_discussion.md | 317 > .../a2_edk_file_specification.md | 461 -- > ...ndix_c_symbols.md => appendix_b_symbols.md | 0 > ...d => appendix_c_sample_driver_inf_files.md | 0 > ... => appendix_d_sample_library_inf_files.md | 0 > ...d => appendix_e_sample_binary_inf_files.md | 0 > ...ule_types.md => appendix_f_module_types.md | 0 > 31 files changed, 30 insertions(+), 932 deletions(-) > rename appendix_b_build_changes_and_customizations.md => > appendix_a_build_changes_and_customizations.md (100%) > delete mode 100644 appendix_a_edk_inf_file_specification/README.md > delete mode 100644 > appendix_a_edk_inf_file_specification/a1_design_discussion.md > delete mode 100644 > appendix_a_edk_inf_file_specification/a2_edk_file_specification.md > rename appendix_c_symbols.md => appendix_b_symbols.md (100%) > rename appendix_d_sample_driver_inf_files.md => > appendix_c_sample_driver_inf_files.md (100%) > rename appendix_e_sample_library_inf_files.md => > appendix_d_sample_library_inf_files.md (100%) > rename appendix_f_sample_binary_inf_files.md => > appendix_e_sample_binary_inf_files.md (100%) > rename appendix_g_module_types.md => appendix_f_module_types.md (100%) > > diff --git a/1_introduction/11_overview.md b/1_introduction/11_overview.md > index 9239080..7349614 100644 > --- a/1_introduction/11_overview.md > +++ b/1_introduction/11_overview.md > @@ -40,11 +40,11 @@ Backward compatibility with the existing INF file > formats. Changes m
Re: [edk2] [Patch] Document: Update Inf spec to remove EDK related contents
Bob: I have some comments. 1) Please update the commit message. The change is to remove EDK and IPF related contents. 2) The below change should make sure the updated one keep the same alignment. -||- +|| 3) Below content is correct. It is for EDKII. No need to change it. -These sections are used for specifying PCD information and are valid for EDK -II modules only. The entries for these sections are looked up from the package -declaration files (DEC) for generating the `AutoGen.c` and `AutoGen.h` files. +These sections are used for specifying PCD information. The entries for these +sections are looked up from the packagedeclaration files (DEC) for generating +the `AutoGen.c` and `AutoGen.h` files. Thanks Liming > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Feng, > Bob C > Sent: Monday, March 4, 2019 5:52 AM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Gao, Liming > Subject: [edk2] [Patch] Document: Update Inf spec to remove EDK related > contents > > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1453 > > Remove EDK related contents > > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Bob Feng > Cc: Liming Gao > Cc: Jaben Carsey > --- > 1_introduction/11_overview.md | 2 +- > 2_inf_overview/210_[ppis]_section.md | 1 - > 2_inf_overview/211_[guids]_section.md | 1 - > .../212_[libraryclasses]_section.md | 1 - > 2_inf_overview/213_[packages]_section.md | 1 - > 2_inf_overview/214_pcd_sections.md| 14 +- > 2_inf_overview/215_[depex]_section.md | 4 - > 2_inf_overview/21_processing_overview.md | 19 +- > .../22_information_file_general_rules.md | 30 +- > 2_inf_overview/24_[defines]_section.md| 4 +- > 2_inf_overview/25_[sources]_section.md| 4 - > 2_inf_overview/26_[buildoptions]_section.md | 9 +- > 2_inf_overview/27_[binaries]_section.md | 5 - > 2_inf_overview/29_[protocols]_section.md | 1 - > 2_inf_overview/README.md | 11 - > .../314_[depex]_sections.md | 6 +- > .../315_[binaries]_section.md | 5 - > .../32_component_inf_definition.md| 4 +- > .../34_[defines]_section.md | 6 +- > .../35_[buildoptions]_sections.md | 5 +- > .../39_[sources]_sections.md | 7 +- > 3_edk_ii_inf_file_format/README.md| 4 - > ...ndix_a_build_changes_and_customizations.md | 0 > .../README.md | 40 -- > .../a1_design_discussion.md | 317 > .../a2_edk_file_specification.md | 461 -- > ...ndix_c_symbols.md => appendix_b_symbols.md | 0 > ...d => appendix_c_sample_driver_inf_files.md | 0 > ... => appendix_d_sample_library_inf_files.md | 0 > ...d => appendix_e_sample_binary_inf_files.md | 0 > ...ule_types.md => appendix_f_module_types.md | 0 > 31 files changed, 30 insertions(+), 932 deletions(-) > rename appendix_b_build_changes_and_customizations.md => > appendix_a_build_changes_and_customizations.md (100%) > delete mode 100644 appendix_a_edk_inf_file_specification/README.md > delete mode 100644 > appendix_a_edk_inf_file_specification/a1_design_discussion.md > delete mode 100644 > appendix_a_edk_inf_file_specification/a2_edk_file_specification.md > rename appendix_c_symbols.md => appendix_b_symbols.md (100%) > rename appendix_d_sample_driver_inf_files.md => > appendix_c_sample_driver_inf_files.md (100%) > rename appendix_e_sample_library_inf_files.md => > appendix_d_sample_library_inf_files.md (100%) > rename appendix_f_sample_binary_inf_files.md => > appendix_e_sample_binary_inf_files.md (100%) > rename appendix_g_module_types.md => appendix_f_module_types.md (100%) > > diff --git a/1_introduction/11_overview.md b/1_introduction/11_overview.md > index 9239080..7349614 100644 > --- a/1_introduction/11_overview.md > +++ b/1_introduction/11_overview.md > @@ -40,11 +40,11 @@ Backward compatibility with the existing INF file > formats. Changes made to this > specification must not require changes to existing INF files. > > **Simplified platform build and configuration** > > Simplify the build setup and configuration for a given platform. The process > of > -adding EDK and EDK II firmware components to a firmware volume on any given > +adding EDK II firmware components to a firmware volume on any given > platform was also simplified. > > **Distributing Modules** > > Enable eas
[edk2] [Patch] Document: Update Inf spec to remove EDK related contents
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=1453 Remove EDK related contents Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Bob Feng Cc: Liming Gao Cc: Jaben Carsey --- 1_introduction/11_overview.md | 2 +- 2_inf_overview/210_[ppis]_section.md | 1 - 2_inf_overview/211_[guids]_section.md | 1 - .../212_[libraryclasses]_section.md | 1 - 2_inf_overview/213_[packages]_section.md | 1 - 2_inf_overview/214_pcd_sections.md| 14 +- 2_inf_overview/215_[depex]_section.md | 4 - 2_inf_overview/21_processing_overview.md | 19 +- .../22_information_file_general_rules.md | 30 +- 2_inf_overview/24_[defines]_section.md| 4 +- 2_inf_overview/25_[sources]_section.md| 4 - 2_inf_overview/26_[buildoptions]_section.md | 9 +- 2_inf_overview/27_[binaries]_section.md | 5 - 2_inf_overview/29_[protocols]_section.md | 1 - 2_inf_overview/README.md | 11 - .../314_[depex]_sections.md | 6 +- .../315_[binaries]_section.md | 5 - .../32_component_inf_definition.md| 4 +- .../34_[defines]_section.md | 6 +- .../35_[buildoptions]_sections.md | 5 +- .../39_[sources]_sections.md | 7 +- 3_edk_ii_inf_file_format/README.md| 4 - ...ndix_a_build_changes_and_customizations.md | 0 .../README.md | 40 -- .../a1_design_discussion.md | 317 .../a2_edk_file_specification.md | 461 -- ...ndix_c_symbols.md => appendix_b_symbols.md | 0 ...d => appendix_c_sample_driver_inf_files.md | 0 ... => appendix_d_sample_library_inf_files.md | 0 ...d => appendix_e_sample_binary_inf_files.md | 0 ...ule_types.md => appendix_f_module_types.md | 0 31 files changed, 30 insertions(+), 932 deletions(-) rename appendix_b_build_changes_and_customizations.md => appendix_a_build_changes_and_customizations.md (100%) delete mode 100644 appendix_a_edk_inf_file_specification/README.md delete mode 100644 appendix_a_edk_inf_file_specification/a1_design_discussion.md delete mode 100644 appendix_a_edk_inf_file_specification/a2_edk_file_specification.md rename appendix_c_symbols.md => appendix_b_symbols.md (100%) rename appendix_d_sample_driver_inf_files.md => appendix_c_sample_driver_inf_files.md (100%) rename appendix_e_sample_library_inf_files.md => appendix_d_sample_library_inf_files.md (100%) rename appendix_f_sample_binary_inf_files.md => appendix_e_sample_binary_inf_files.md (100%) rename appendix_g_module_types.md => appendix_f_module_types.md (100%) diff --git a/1_introduction/11_overview.md b/1_introduction/11_overview.md index 9239080..7349614 100644 --- a/1_introduction/11_overview.md +++ b/1_introduction/11_overview.md @@ -40,11 +40,11 @@ Backward compatibility with the existing INF file formats. Changes made to this specification must not require changes to existing INF files. **Simplified platform build and configuration** Simplify the build setup and configuration for a given platform. The process of -adding EDK and EDK II firmware components to a firmware volume on any given +adding EDK II firmware components to a firmware volume on any given platform was also simplified. **Distributing Modules** Enable easy distribution of modules, both in source and binary form. Individual diff --git a/2_inf_overview/210_[ppis]_section.md b/2_inf_overview/210_[ppis]_section.md index bbdd2a4..1ab84eb 100644 --- a/2_inf_overview/210_[ppis]_section.md +++ b/2_inf_overview/210_[ppis]_section.md @@ -46,11 +46,10 @@ This section uses one of the following section definitions: ```ini [Ppis] [Ppis.common] [Ppis.IA32] [Ppis.X64] -[Ppis.IPF] [Ppis.EBC] ``` The formats for entries in this section is: diff --git a/2_inf_overview/211_[guids]_section.md b/2_inf_overview/211_[guids]_section.md index 7f53f6c..4176310 100644 --- a/2_inf_overview/211_[guids]_section.md +++ b/2_inf_overview/211_[guids]_section.md @@ -46,11 +46,10 @@ This section uses one of the following section definitions: ```ini [Guids] [Guids.common] [Guids.IA32] [Guids.X64] -[Guids.IPF] [Guids.EBC] ``` The formats for entries in this section is: diff --git a/2_inf_overview/212_[libraryclasses]_section.md b/2_inf_overview/212_[libraryclasses]_section.md index 8d2fd06..8a77463 100644 --- a/2_inf_overview/212_[libraryclasses]_section.md +++ b/2_inf_overview/212_[libraryclasses]_section.md @@ -46,11 +46,10 @@ This section uses one of the following section definitions: ```ini [LibraryClasses] [LibraryClasses.common] [LibraryClasses.IA32] [LibraryClasses.X64] -[LibraryClasses.IPF] [LibraryClasses.EBC] ``` The format for entries in this section is: diff --git a/2_inf_overview/213_[packages]_section.md b/2_inf_overview/213_[packages]_section.md ind
