From nobody Sun Apr 28 12:15:33 2024
Delivered-To: importer@patchew.org
Authentication-Results: mx.zohomail.com;
	spf=none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain
 of lists.01.org)  smtp.mailfrom=edk2-devel-bounces@lists.01.org
Return-Path: <edk2-devel-bounces@lists.01.org>
Received: from ml01.01.org (ml01.01.org [198.145.21.10]) by mx.zohomail.com
	with SMTPS id 1522639783846561.4117412323529;
 Sun, 1 Apr 2018 20:29:43 -0700 (PDT)
Received: from [127.0.0.1] (localhost [IPv6:::1])
	by ml01.01.org (Postfix) with ESMTP id 7469121E0816B;
	Sun,  1 Apr 2018 20:29:42 -0700 (PDT)
Received: from mga14.intel.com (mga14.intel.com [192.55.52.115])
 (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 48074225F31E7
 for <edk2-devel@lists.01.org>; Sun,  1 Apr 2018 20:29:41 -0700 (PDT)
Received: from fmsmga001.fm.intel.com ([10.253.24.23])
 by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384;
 01 Apr 2018 20:29:40 -0700
Received: from shwdeopenpsi168.ccr.corp.intel.com ([10.239.158.129])
 by fmsmga001.fm.intel.com with ESMTP; 01 Apr 2018 20:29:39 -0700
X-Original-To: edk2-devel@lists.01.org
Received-SPF: none (zoho.com: 198.145.21.10 is neither permitted nor denied by
 domain of lists.01.org) client-ip=198.145.21.10;
 envelope-from=edk2-devel-bounces@lists.01.org; helo=ml01.01.org;
Received-SPF: Pass (sender SPF authorized) identity=mailfrom;
 client-ip=192.55.52.115; helo=mga14.intel.com;
 envelope-from=yonghong.zhu@intel.com; receiver=edk2-devel@lists.01.org
X-Amp-Result: SKIPPED(no attachment in message)
X-Amp-File-Uploaded: False
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.48,393,1517904000"; d="scan'208";a="43203328"
From: Yonghong Zhu <yonghong.zhu@intel.com>
To: edk2-devel@lists.01.org
Date: Mon,  2 Apr 2018 11:29:39 +0800
Message-Id: <1522639779-23096-1-git-send-email-yonghong.zhu@intel.com>
X-Mailer: git-send-email 2.6.1.windows.1
Subject: [edk2] [Patch] Build spec: Add some clarification and clean up for
 build report
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>
Cc: Michael Kinney <michael.d.kinney@intel.com>,
 Kevin W Shaw <kevin.w.shaw@intel.com>, Liming Gao <liming.gao@intel.com>
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Errors-To: edk2-devel-bounces@lists.01.org
Sender: "edk2-devel" <edk2-devel-bounces@lists.01.org>
X-ZohoMail: RSF_4  Z_629925259 SPT_0
Content-Type: text/plain; charset="utf-8"

Add some clarification and clean up for build report section.

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>
Reviewed-by: Liming Gao <liming.gao@intel.com>
---
 13_build_reports/134_platform_summary.md   | 44 +++++++++++---------------=
----
 13_build_reports/136_global_pcd_section.md |  7 +++--
 13_build_reports/138_module_section.md     | 11 +++++---
 3 files changed, 27 insertions(+), 35 deletions(-)

diff --git a/13_build_reports/134_platform_summary.md b/13_build_reports/13=
4_platform_summary.md
index 09b8db6..23964af 100644
--- a/13_build_reports/134_platform_summary.md
+++ b/13_build_reports/134_platform_summary.md
@@ -1,9 +1,9 @@
 <!--- @file
   13.4 Platform Summary
=20
-  Copyright (c) 2008-2017, Intel Corporation. All rights reserved.<BR>
+  Copyright (c) 2008-2018, Intel Corporation. All rights reserved.<BR>
=20
   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 m=
et:
=20
@@ -72,35 +72,14 @@ Report Contents:    PCD, LIBRARY, BUILD_FLAGS, DEPEX, F=
LASH, FIXED_ADDRESS
 **********
 **Note:** Platform Summary is always present and appears at the beginning =
of
 report.
 **********
=20
-If the DSC or FDF file contains conditional directive statements (`!if`,
-`!elseif`, `!ifdef` or `!ifndef`) or the value of PCD is not used by a mod=
ule is
-set in the DSC file (PCD Sections) or the FDF file (`SET` statements for
-example), the following sub-section is generated.
-
-```
-=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D<
-Conditional Directives used by the build system
-=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D
-```
-
-If the DSC or FDF file define values for PCDs that are not used by any mod=
ule
-and are not used in conditional directive statements, the following sub-se=
ction
-is generated.
-
-```
-=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D<
-PCDs not used by modules or in conditional directives
-=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D
-```
-
 ### 13.4.1 PCDs in Conditional Directives
=20
-If a PCD is used in a conditional directive statement, the PCD section is
-generated.
+If a PCD is used in a conditional directive statement in DSC or FDF file, =
this
+PCD section is generated. This is optional section.
=20
 PCD values derived from expressions or other PCDs are not differentiated i=
n the
 report. Only the final value is displayed.
=20
 The first line is required:
@@ -108,11 +87,15 @@ The first line is required:
 `[*P|*F|*B] <PcdCName>: <PcdType> (<DatumType>) =3D <PcdValue>`
=20
 * `*P` means the Pcd's value was obtained from the DSC file
 * `*F` means the PCD's value was obtained from the FDF file.
 * `*B` means the PCD's value set by a build option.
-**Note:** If the Pcd is a Structure PCD, <DatumType> is the Struct Name.
+* If no `*P`, `*F` or `*B` is shown, the PCD's value comes from DEC file. =
If the
+  value obtained from either a build option, the DSC or FDF is the same as=
 the
+  value in the DEC, then `*B`, `*P` or `*F` will not be shown in the repor=
t.
+
+**Note: ** If the Pcd is a Structure PCD, `<DatumType>` is the Struct Name.
=20
 Additional lines may be displayed showing default values when the value is=
 not a
 default value.
=20
 ### Example
@@ -132,12 +115,13 @@ gMyTokenSpaceGuid
 >=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D<
 ```
=20
 ### 13.4.2 PCDs not used
=20
-If a PCD is not used in a conditional directive statement or by a module, =
the
-not used PCD section is generated.
+If a PCD defined in DSC or FDF file, but the PCD is not used in a conditio=
nal
+directive statement and not used by any module, the not used PCD section is
+generated. This is optional section.
=20
 PCD values derived from expressions or other PCDs are not differentiated i=
n the
 report. Only the final value is displayed.
=20
 The first line is required:
@@ -145,11 +129,15 @@ The first line is required:
 `[*P|*F|*B] <PcdCName>: <PcdType> (<DatumType>) [(<SKUID>)][(<DefaultStore=
>)] =3D <PcdValue>`
=20
 * `*P` means the Pcd's value was obtained from the DSC file
 * `*F` means the PCD's value was obtained from the FDF file.
 * `*B` means the PCD's value set by a build option.
-**Note:** If the Pcd is a Structure PCD, <DatumType> is the Struct Name.
+* If no `*P`, `*F` or `*B` is shown, the PCD's value comes from DEC file. =
If the
+  value obtained from either a build option, the DSC or FDF is the same as=
 the
+  value in the DEC, then `*B`, `*P` or `*F` will not be shown in the repor=
t.
+
+**Note: ** If the Pcd is a Structure PCD, `<DatumType>` is the Struct Name.
=20
 Additional lines may be displayed showing default values when the value is=
 not a
 default value.
=20
 Since the PCDs in this section are not used by any module, the PCD value i=
s not
diff --git a/13_build_reports/136_global_pcd_section.md b/13_build_reports/=
136_global_pcd_section.md
index a9661b6..2a5b682 100644
--- a/13_build_reports/136_global_pcd_section.md
+++ b/13_build_reports/136_global_pcd_section.md
@@ -1,9 +1,9 @@
 <!--- @file
   13.6 Global PCD Section
=20
-  Copyright (c) 2008-2017, Intel Corporation. All rights reserved.<BR>
+  Copyright (c) 2008-2018, Intel Corporation. All rights reserved.<BR>
=20
   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 m=
et:
=20
@@ -60,11 +60,12 @@ The first line is required:
 * `*F` means the PCD's value was obtained from the FDF file.
 * `*B` means the PCD's value was obtained from a build option.
 * If no `*P`, `*F` or `*B` is shown, the PCD's value comes from DEC file. =
If the
   value obtained from either a build option, the DSC or FDF is the same as=
 the
   value in the DEC, then `*B`, `*P` or `*F` will not be shown in the repor=
t.
-**Note:** If the Pcd is a Structure PCD, <DatumType> is the Struct Name.
+
+**Note: ** If the Pcd is a Structure PCD, `<DatumType>` is the Struct Name.
=20
 #### Examples
=20
 ```
 *P PcdWinNtFirmwareVolume               : FIXED   (VOID*) =3D L"..\\Fv\\Nt=
32.fd"
@@ -109,11 +110,11 @@ gTokenSpaceGuid
 #### 13.6.2.2 Default (optional) line
=20
 The second optional line is present if the value from the DSC was overridd=
en
 by build option. It is formatted as follows:
=20
-`DSC DEFAULT =3D <Value in PCD SectionSection in DSC>`
+`DSC DEFAULT =3D <Value in PCD Section in DSC>`
=20
 The third optional line is present if the value from the DEC was overridde=
n.
 It is formatted as follows:
=20
 `DEC DEFAULT =3D <Value in DEC>`
diff --git a/13_build_reports/138_module_section.md b/13_build_reports/138_=
module_section.md
index fb2417d..88e9ac2 100644
--- a/13_build_reports/138_module_section.md
+++ b/13_build_reports/138_module_section.md
@@ -1,9 +1,9 @@
 <!--- @file
   13.8 Module Section
=20
-  Copyright (c) 2008-2017, Intel Corporation. All rights reserved.<BR>
+  Copyright (c) 2008-2018, Intel Corporation. All rights reserved.<BR>
=20
   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 m=
et:
=20
@@ -238,18 +238,21 @@ gEfiMdeModulePkgTokenSpaceGuid
 ```
=20
 Each PCD may contain up to four lines:
=20
 1. The first line is a mandatory line with the following format:
-   `[*P|*F|*B] <PcdCName>: <PcdType> (<DatumType>) [(<SKUID>)][(<DefaultSt=
ore>)] =3D <PcdValue>`
-   **Note:** If the Pcd is a Structure PCD, <DatumType> is the Struct Name.
+   `[*P|*F|*B|*M] <PcdCName>: <PcdType> (<DatumType>) [(<SKUID>)][(<Defaul=
tStore>)] =3D <PcdValue>`
+
+   **Note: ** If the Pcd is a Structure PCD, `<DatumType>` is the Struct N=
ame.
+
    - \*P means the Pcd's value is the platform default (listed in DSC PCD =
common
      section or inherited from Module INF file).
    - \*M means the PCD's value in module INF was obtained from the `[Compo=
nents]`
      section of the DSC file.
    - \*F means the PCD's value is override in FDF file.
-   - If no \*P or \*F, mean the PCD's value comes from DEC file.
+   - \*B means the PCD's value is override in build option.
+   - If no \*P or \*F or \*B or \*M, mean the PCD's value comes from DEC f=
ile.
    - For example:
      ```
      *P PcdWinNtFirmwareVolume : FIXED (VOID*) =3D L"..\\Fv\\Nt32.fd"
      ```
=20
--=20
2.6.1.windows.1

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