From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=134.134.136.24; helo=mga09.intel.com; envelope-from=nathaniel.l.desimone@intel.com; receiver=edk2-devel@lists.01.org Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) (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 191C0225B0284 for ; Tue, 20 Mar 2018 15:01:04 -0700 (PDT) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by orsmga102.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 20 Mar 2018 15:07:34 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.48,337,1517904000"; d="scan'208";a="25636876" Received: from orsmsx108.amr.corp.intel.com ([10.22.240.6]) by fmsmga007.fm.intel.com with ESMTP; 20 Mar 2018 15:07:34 -0700 Received: from orsmsx115.amr.corp.intel.com (10.22.240.11) by ORSMSX108.amr.corp.intel.com (10.22.240.6) with Microsoft SMTP Server (TLS) id 14.3.319.2; Tue, 20 Mar 2018 15:07:33 -0700 Received: from orsmsx114.amr.corp.intel.com ([169.254.8.126]) by ORSMSX115.amr.corp.intel.com ([169.254.4.94]) with mapi id 14.03.0319.002; Tue, 20 Mar 2018 15:07:33 -0700 From: "Desimone, Nathaniel L" To: Sami Mujawar , "edk2-devel@lists.01.org" CC: "nd@arm.com" , "leif.lindholm@linaro.org" , "Stephanie.Hughes-Fitt@arm.com" Thread-Topic: [edk2] [staging/dynamictables PATCH 0/2] Dynamic Tables Framework core Thread-Index: AQHTv5WgcjJxjPPl7Eqc+5A2QrIwiKPZrjng Date: Tue, 20 Mar 2018 22:07:32 +0000 Message-ID: <02A34F284D1DA44BB705E61F7180EF0A980EFDBB@ORSMSX114.amr.corp.intel.com> References: <20180319151847.85204-1-sami.mujawar@arm.com> In-Reply-To: <20180319151847.85204-1-sami.mujawar@arm.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.0.0.116 dlp-reaction: no-action x-originating-ip: [10.22.254.138] MIME-Version: 1.0 Subject: Re: [staging/dynamictables PATCH 0/2] Dynamic Tables Framework core X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 20 Mar 2018 22:01:05 -0000 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Please don't use the name DynamicTableManagerDxe. What does it manage? Dyna= micTables? How does adding the word "Manager" provide any useful informatio= n for the reader? I recommend the name "DynamicAcpiTableDxe". It makes it c= lear this this code is specifically for generating ACPI tables at runtime (= as opposed to some sort of unnamed table), and it removes the unnecessary p= rose. -----Original Message----- From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Sami= Mujawar Sent: Monday, March 19, 2018 8:19 AM To: edk2-devel@lists.01.org Cc: nd@arm.com; leif.lindholm@linaro.org; Stephanie.Hughes-Fitt@arm.com Subject: [edk2] [staging/dynamictables PATCH 0/2] Dynamic Tables Framework = core The Dynamic Tables Framework is a prototyped as a solution for automaticall= y generating the firmware tables based on hardware description.=20 This patchset is the Dynamic Tables Framework core and implement the generi= c/standard modules for dynamically generating ACPI 6.2 tables for ARM platf= orm. The platform specific modules are in the devel-dynamictables branch in= the edk2-platforms repository at:=20 https://github.com/tianocore/edk2-platforms/tree/devel-dynamictables The first patch in this patchset 'MdePkg: SMMUv3 updates for IORT' is a precursor for the Dynamic Tables Framework and has been submitted inde= pendently to the edk2-devel mailing list where it is currently awaiting acc= eptance. The sources for this patchset can be seen at: https://github.com/samimujawar/edk2-staging/tree/187_dynamictables_v1 Sami Mujawar (2): MdePkg: SMMUv3 updates for IORT table definitions DynamicTablesPkg: Dynamic Tables Framework DynamicTablesPkg/Drivers/DynamicTableFactoryDxe/AcpiTableFactory/AcpiTable= Factory.c | 226 +++ DynamicTablesPkg/Drivers/DynamicTableFactoryDxe/DeviceTreeTableFactory/Dev= iceTreeTableFactory.c | 225 +++ DynamicTablesPkg/Drivers/DynamicTableFactoryDxe/DynamicTableFactory.h = | 125 ++ DynamicTablesPkg/Drivers/DynamicTableFactoryDxe/DynamicTableFactoryDxe.c = | 84 + DynamicTablesPkg/Drivers/DynamicTableFactoryDxe/DynamicTableFactoryDxe.inf= | 59 + DynamicTablesPkg/Drivers/DynamicTableFactoryDxe/SmbiosTableFactory/SmbiosT= ableFactory.c | 226 +++ DynamicTablesPkg/Drivers/DynamicTableManagerDxe/DynamicTableManagerDxe.c = | 533 +++++ DynamicTablesPkg/Drivers/DynamicTableManagerDxe/DynamicTableManagerDxe.inf= | 48 + DynamicTablesPkg/DynamicTables.dsc.inc = | 46 + DynamicTablesPkg/DynamicTables.fdf.inc = | 35 + DynamicTablesPkg/DynamicTablesPkg.dec = | 42 + DynamicTablesPkg/Include/AcpiTableGenerator.h = | 282 +++ DynamicTablesPkg/Include/ArmNameSpaceObjects.h = | 587 ++++++ DynamicTablesPkg/Include/ConfigurationManagerHelper.h = | 119 ++ DynamicTablesPkg/Include/ConfigurationManagerObject.h = | 176 ++ DynamicTablesPkg/Include/DeviceTreeTableGenerator.h = | 182 ++ DynamicTablesPkg/Include/Library/TableHelperLib.h = | 70 + DynamicTablesPkg/Include/Protocol/ConfigurationManagerProtocol.h = | 128 ++ DynamicTablesPkg/Include/Protocol/DynamicTableFactoryProtocol.h = | 140 ++ DynamicTablesPkg/Include/SmbiosTableGenerator.h = | 240 +++ DynamicTablesPkg/Include/StandardNameSpaceObjects.h = | 116 ++ DynamicTablesPkg/Include/TableGenerator.h = | 252 +++ DynamicTablesPkg/Library/Acpi/Arm/AcpiDbg2LibArm/AcpiDbg2LibArm.inf = | 47 + DynamicTablesPkg/Library/Acpi/Arm/AcpiDbg2LibArm/Dbg2Generator.c = | 440 +++++ DynamicTablesPkg/Library/Acpi/Arm/AcpiFadtLibArm/AcpiFadtLibArm.inf = | 41 + DynamicTablesPkg/Library/Acpi/Arm/AcpiFadtLibArm/FadtGenerator.c = | 666 +++++++ DynamicTablesPkg/Library/Acpi/Arm/AcpiGtdtLibArm/AcpiGtdtLibArm.inf = | 41 + DynamicTablesPkg/Library/Acpi/Arm/AcpiGtdtLibArm/GtdtGenerator.c = | 670 +++++++ DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/AcpiIortLibArm.inf = | 41 + DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.c = | 2046 ++++++++++++++++++++ DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGenerator.h = | 50 + DynamicTablesPkg/Library/Acpi/Arm/AcpiMadtLibArm/AcpiMadtLibArm.inf = | 41 + DynamicTablesPkg/Library/Acpi/Arm/AcpiMadtLibArm/MadtGenerator.c = | 717 +++++++ DynamicTablesPkg/Library/Acpi/Arm/AcpiMcfgLibArm/AcpiMcfgLibArm.inf = | 41 + DynamicTablesPkg/Library/Acpi/Arm/AcpiMcfgLibArm/McfgGenerator.c = | 342 ++++ DynamicTablesPkg/Library/Acpi/Arm/AcpiRawLibArm/AcpiRawLibArm.inf = | 41 + DynamicTablesPkg/Library/Acpi/Arm/AcpiRawLibArm/RawGenerator.c = | 142 ++ DynamicTablesPkg/Library/Acpi/Arm/AcpiSpcrLibArm/AcpiSpcrLibArm.inf = | 41 + DynamicTablesPkg/Library/Acpi/Arm/AcpiSpcrLibArm/SpcrGenerator.c = | 324 ++++ DynamicTablesPkg/Library/Common/TableHelperLib/TableHelper.c = | 164 ++ DynamicTablesPkg/Library/Common/TableHelperLib/TableHelperLib.inf = | 35 + MdePkg/Include/IndustryStandard/IoRemappingTable.h = | 11 +- 42 files changed, 9881 insertions(+), 1 deletion(-) create mode 100644 Dy= namicTablesPkg/Drivers/DynamicTableFactoryDxe/AcpiTableFactory/AcpiTableFac= tory.c create mode 100644 DynamicTablesPkg/Drivers/DynamicTableFactoryDxe/DeviceT= reeTableFactory/DeviceTreeTableFactory.c create mode 100644 DynamicTablesPkg/Drivers/DynamicTableFactoryDxe/Dynamic= TableFactory.h create mode 100644 DynamicTablesPkg/Drivers/DynamicTableFactoryDxe/Dynamic= TableFactoryDxe.c create mode 100644 DynamicTablesPkg/Drivers/DynamicTableFactoryDxe/Dynamic= TableFactoryDxe.inf create mode 100644 DynamicTablesPkg/Drivers/DynamicTableFactoryDxe/SmbiosT= ableFactory/SmbiosTableFactory.c create mode 100644 DynamicTablesPkg/Drivers/DynamicTableManagerDxe/Dynamic= TableManagerDxe.c create mode 100644 DynamicTablesPkg/Drivers/DynamicTableManagerDxe/Dynamic= TableManagerDxe.inf create mode 100644 DynamicTablesPkg/DynamicTables.dsc.inc create mode 100644 DynamicTablesPkg/DynamicTables.fdf.inc create mode 100644 DynamicTablesPkg/DynamicTablesPkg.dec create mode 100644 DynamicTablesPkg/Include/AcpiTableGenerator.h create mode 100644 DynamicTablesPkg/Include/ArmNameSpaceObjects.h create mode 100644 DynamicTablesPkg/Include/ConfigurationManagerHelper.h create mode 100644 DynamicTablesPkg/Include/ConfigurationManagerObject.h create mode 100644 DynamicTablesPkg/Include/DeviceTreeTableGenerator.h create mode 100644 DynamicTablesPkg/Include/Library/TableHelperLib.h create mode 100644 DynamicTablesPkg/Include/Protocol/ConfigurationManagerP= rotocol.h create mode 100644 DynamicTablesPkg/Include/Protocol/DynamicTableFactoryPr= otocol.h create mode 100644 DynamicTablesPkg/Include/SmbiosTableGenerator.h create mode 100644 DynamicTablesPkg/Include/StandardNameSpaceObjects.h create mode 100644 DynamicTablesPkg/Include/TableGenerator.h create mode 100644 DynamicTablesPkg/Library/Acpi/Arm/AcpiDbg2LibArm/AcpiDb= g2LibArm.inf create mode 100644 DynamicTablesPkg/Library/Acpi/Arm/AcpiDbg2LibArm/Dbg2Ge= nerator.c create mode 100644 DynamicTablesPkg/Library/Acpi/Arm/AcpiFadtLibArm/AcpiFa= dtLibArm.inf create mode 100644 DynamicTablesPkg/Library/Acpi/Arm/AcpiFadtLibArm/FadtGe= nerator.c create mode 100644 DynamicTablesPkg/Library/Acpi/Arm/AcpiGtdtLibArm/AcpiGt= dtLibArm.inf create mode 100644 DynamicTablesPkg/Library/Acpi/Arm/AcpiGtdtLibArm/GtdtGe= nerator.c create mode 100644 DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/AcpiIo= rtLibArm.inf create mode 100644 DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGe= nerator.c create mode 100644 DynamicTablesPkg/Library/Acpi/Arm/AcpiIortLibArm/IortGe= nerator.h create mode 100644 DynamicTablesPkg/Library/Acpi/Arm/AcpiMadtLibArm/AcpiMa= dtLibArm.inf create mode 100644 DynamicTablesPkg/Library/Acpi/Arm/AcpiMadtLibArm/MadtGe= nerator.c create mode 100644 DynamicTablesPkg/Library/Acpi/Arm/AcpiMcfgLibArm/AcpiMc= fgLibArm.inf create mode 100644 DynamicTablesPkg/Library/Acpi/Arm/AcpiMcfgLibArm/McfgGe= nerator.c create mode 100644 DynamicTablesPkg/Library/Acpi/Arm/AcpiRawLibArm/AcpiRaw= LibArm.inf create mode 100644 DynamicTablesPkg/Library/Acpi/Arm/AcpiRawLibArm/RawGene= rator.c create mode 100644 DynamicTablesPkg/Library/Acpi/Arm/AcpiSpcrLibArm/AcpiSp= crLibArm.inf create mode 100644 DynamicTablesPkg/Library/Acpi/Arm/AcpiSpcrLibArm/SpcrGe= nerator.c create mode 100644 DynamicTablesPkg/Library/Common/TableHelperLib/TableHel= per.c create mode 100644 DynamicTablesPkg/Library/Common/TableHelperLib/TableHel= perLib.inf -- 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)' _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel