From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=192.55.52.151; helo=mga17.intel.com; envelope-from=liming.gao@intel.com; receiver=edk2-devel@lists.01.org Received: from mga17.intel.com (mga17.intel.com [192.55.52.151]) (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 3430B21128730 for ; Thu, 27 Sep 2018 08:43:04 -0700 (PDT) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga107.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 27 Sep 2018 08:43:03 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.54,311,1534834800"; d="scan'208";a="95415977" Received: from fmsmsx107.amr.corp.intel.com ([10.18.124.205]) by orsmga002.jf.intel.com with ESMTP; 27 Sep 2018 08:42:15 -0700 Received: from FMSMSX110.amr.corp.intel.com (10.18.116.10) by fmsmsx107.amr.corp.intel.com (10.18.124.205) with Microsoft SMTP Server (TLS) id 14.3.319.2; Thu, 27 Sep 2018 08:42:14 -0700 Received: from shsmsx152.ccr.corp.intel.com (10.239.6.52) by fmsmsx110.amr.corp.intel.com (10.18.116.10) with Microsoft SMTP Server (TLS) id 14.3.319.2; Thu, 27 Sep 2018 08:42:14 -0700 Received: from shsmsx104.ccr.corp.intel.com ([169.254.5.183]) by SHSMSX152.ccr.corp.intel.com ([169.254.6.37]) with mapi id 14.03.0319.002; Thu, 27 Sep 2018 23:42:12 +0800 From: "Gao, Liming" To: "Zhu, Yonghong" , "edk2-devel@lists.01.org" CC: "Kinney, Michael D" , "Shaw, Kevin W" Thread-Topic: [Patch] Build Spec: Add some clarification of Build report section Thread-Index: AQHUVK7MfQCBt9l8bkqr2HjyDX9U0aUER+rg Date: Thu, 27 Sep 2018 15:42:11 +0000 Message-ID: <4A89E2EF3DFEDB4C8BFDE51014F606A14E31C9EC@SHSMSX104.ccr.corp.intel.com> References: <1537865854-9408-1-git-send-email-yonghong.zhu@intel.com> In-Reply-To: <1537865854-9408-1-git-send-email-yonghong.zhu@intel.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ctpclassification: CTP_NT x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiZjQzMzNmMmUtMGUzOC00YjhkLWFhNmQtOWI3YTYxZWZlMGRmIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoiU29nYkpXTnR3NWd0M0FwcXcxQTFjVkJ3MWVEVXFIOSswZ2RYN3NYWFhNc1VjQWJmNjI2a0syOG9FTDE2M1dGNSJ9 dlp-product: dlpe-windows dlp-version: 11.0.400.15 dlp-reaction: no-action x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [Patch] Build Spec: Add some clarification of Build report section X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 27 Sep 2018 15:43:04 -0000 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Reviewed-by: Liming Gao > -----Original Message----- > From: Zhu, Yonghong > Sent: Tuesday, September 25, 2018 4:58 PM > To: edk2-devel@lists.01.org > Cc: Gao, Liming ; Kinney, Michael D ; Shaw, Kevin W > Subject: [Patch] Build Spec: Add some clarification of Build report secti= on >=20 > 1. Add the section header of Global PCD Section > 2. Add 'INF DEFAULT' in the module PCD section >=20 > Cc: Liming Gao > Cc: Michael Kinney > Cc: Kevin W Shaw > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Yonghong Zhu > --- > 13_build_reports/136_global_pcd_section.md | 13 +++++++++++++ > 13_build_reports/138_module_section.md | 8 +++++--- > 2 files changed, 18 insertions(+), 3 deletions(-) >=20 > diff --git a/13_build_reports/136_global_pcd_section.md b/13_build_report= s/136_global_pcd_section.md > index 2b7cf5f..0e3afa7 100644 > --- a/13_build_reports/136_global_pcd_section.md > +++ b/13_build_reports/136_global_pcd_section.md > @@ -45,10 +45,23 @@ gEfiNt32PkgTokenSpaceGuid > ``` >=20 > PCD values derived from expressions or other PCDs are not differentiated= in the > report. Only the final value is displayed. >=20 > +The section header is: > +``` > +>=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=3D=3D=3D< > +Platform Configuration Database Report > + *B - PCD override in the build option > + *P - Platform scoped PCD override in DSC file > + *F - Platform scoped PCD override in FDF file > + *M - Module scoped PCD override > +=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=3D=3D=3D=3D=3D > +.. (List of PCDs) > +<=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=3D=3D=3D> > +``` > + > Each global PCD item contains one or more lines: >=20 > ### 13.6.1 Required line >=20 > The first line is required: > diff --git a/13_build_reports/138_module_section.md b/13_build_reports/13= 8_module_section.md > index 52288d7..843ad34 100644 > --- a/13_build_reports/138_module_section.md > +++ b/13_build_reports/138_module_section.md > @@ -241,15 +241,17 @@ Each PCD may contain up to following lines: > ``` > *F PcdVpdSample : DYN-VPD (UINT32) =3D 0x1 (1) > 0x0001FFF > ``` >=20 > -3. The third and fourth lines are both option if the module's final > - `` is not equal to the PCD value in the PCD common section = in the > - DSC file and the PCD value in the DEC file respectively. > +3. The `DSC DEFAULT` `INF DEFAULT` and `DEC DEFAULT` are option if the m= odule's > + final `` is not equal to the PCD value in the PCD common se= ction in > + DSC file, the PCD value in the module INF file and the PCD value in t= he DEC > + file respectively. > ``` > DSC DEFAULT =3D > + INF DEFAULT =3D > DEC DEFAULT =3D > ``` > For example: > ``` > *M PcdDebugPrintErrorLevel : FIXED (UINT32) =3D 0x80000042 (2147483= 714) > -- > 2.6.1.windows.1