[edk2-devel] [PATCH v2 00/10] Various DynamicTablesPkg modifications

PierreGondois posted 10 patches 2 years, 7 months ago
Failed in applying to current master (apply log)
DynamicTablesPkg/DynamicTables.dsc.inc        |   3 +-
DynamicTablesPkg/DynamicTablesPkg.ci.yaml     |  29 +
DynamicTablesPkg/DynamicTablesPkg.dec         |   4 +
DynamicTablesPkg/DynamicTablesPkg.dsc         |   1 +
.../Include/Library/AcpiHelperLib.h           |  93 +++
.../Include/Library/AmlLib/AmlLib.h           | 225 ++++--
.../Include/Library/TableHelperLib.h          |  49 +-
.../SsdtCmn600Generator.c                     |  14 +-
.../AcpiSsdtCmn600LibArm/SsdtCmn600LibArm.inf |   3 +-
.../SsdtSerialPortGenerator.c                 |   3 +-
.../SsdtSerialPortLibArm.inf                  |   4 +-
.../Library/Common/AcpiHelperLib/AcpiHelper.c | 210 ++++++
.../Common/AcpiHelperLib/AcpiHelperLib.inf    |  25 +
.../Common/AmlLib/AmlDbgPrint/AmlDbgPrint.c   |  16 +-
.../Library/Common/AmlLib/AmlLib.inf          |   3 +-
.../Library/Common/AmlLib/Api/AmlApi.c        | 147 +++-
.../AmlLib/CodeGen/AmlResourceDataCodeGen.c   | 194 ++---
.../AmlLib/CodeGen/AmlResourceDataCodeGen.h   |  67 +-
.../SsdtSerialPortFixupLib.c                  |   6 +-
.../SsdtSerialPortFixupLib.inf                |   1 +
.../ConfigurationManagerObjectParser.c        | 678 ++++++++++++++++++
.../ConfigurationManagerObjectParser.h        |  73 ++
.../Common/TableHelperLib/TableHelper.c       |  96 ---
.../Common/TableHelperLib/TableHelperLib.inf  |  13 +-
24 files changed, 1579 insertions(+), 378 deletions(-)
create mode 100644 DynamicTablesPkg/Include/Library/AcpiHelperLib.h
create mode 100644 DynamicTablesPkg/Library/Common/AcpiHelperLib/AcpiHelper.c
create mode 100644 DynamicTablesPkg/Library/Common/AcpiHelperLib/AcpiHelperLib.inf
create mode 100644 DynamicTablesPkg/Library/Common/TableHelperLib/ConfigurationManagerObjectParser.c
create mode 100644 DynamicTablesPkg/Library/Common/TableHelperLib/ConfigurationManagerObjectParser.h
[edk2-devel] [PATCH v2 00/10] Various DynamicTablesPkg modifications
Posted by PierreGondois 2 years, 7 months ago
From: Pierre Gondois <Pierre.Gondois@arm.com>

This patch-set aggregates various modifications in the
DynamicTablesPkg:
 - Extract an AcpiTableHelperLib from TableHelperLib to remove
   the dependency of some utility functions over configuration
   manager definitions
 - Add a HexFromAscii() function
 - Add a AmlGetEisaIdFromString() function
 - Add a configuration manager object parser
 - Use %a instead of %s in when printing AmlLib
 - Update the .ci.yaml once to prepare for other incoming patches
 - Modify the generic prototype of the AmlResourceDataCodeGen
   functions. This also means deprecating some functions.

The modifications can be seen at: https://github.com/PierreARM/edk2/tree/1718_Various_DynamicTablesPkg_modifications_v2

v2:
- Corrections in commit messages. [Sami]
- Document returned error code for some functions. [Sami]
- Correct wrong field name in Configuration Manager Object parser.
  [Joey]

Pierre Gondois (9):
  DynamicTablesPkg: Extract AcpiHelperLib from TableHelperLib
  DynamicTablesPkg: Update TableHelperLib.inf
  DynamicTablesPkg: Rename single char input parameter
  DynamicTablesPkg: Add HexFromAscii() to AcpiHelperLib
  DynamicTablesPkg: Add AmlGetEisaIdFromString() to AcpiHelperLib
  DynamicTablesPkg: Use %a formatter in AmlDbgPrint
  DynamicTablesPkg: Update DynamicTablesPkg.ci.yaml
  DynamicTablesPkg: Deprecate Crs specific methods in AmlLib
  DynamicTablesPkg: Rework AmlResourceDataCodegen.c/h

Sami Mujawar (1):
  DynamicTablesPkg: Add Configuration Manager Object parser

 DynamicTablesPkg/DynamicTables.dsc.inc        |   3 +-
 DynamicTablesPkg/DynamicTablesPkg.ci.yaml     |  29 +
 DynamicTablesPkg/DynamicTablesPkg.dec         |   4 +
 DynamicTablesPkg/DynamicTablesPkg.dsc         |   1 +
 .../Include/Library/AcpiHelperLib.h           |  93 +++
 .../Include/Library/AmlLib/AmlLib.h           | 225 ++++--
 .../Include/Library/TableHelperLib.h          |  49 +-
 .../SsdtCmn600Generator.c                     |  14 +-
 .../AcpiSsdtCmn600LibArm/SsdtCmn600LibArm.inf |   3 +-
 .../SsdtSerialPortGenerator.c                 |   3 +-
 .../SsdtSerialPortLibArm.inf                  |   4 +-
 .../Library/Common/AcpiHelperLib/AcpiHelper.c | 210 ++++++
 .../Common/AcpiHelperLib/AcpiHelperLib.inf    |  25 +
 .../Common/AmlLib/AmlDbgPrint/AmlDbgPrint.c   |  16 +-
 .../Library/Common/AmlLib/AmlLib.inf          |   3 +-
 .../Library/Common/AmlLib/Api/AmlApi.c        | 147 +++-
 .../AmlLib/CodeGen/AmlResourceDataCodeGen.c   | 194 ++---
 .../AmlLib/CodeGen/AmlResourceDataCodeGen.h   |  67 +-
 .../SsdtSerialPortFixupLib.c                  |   6 +-
 .../SsdtSerialPortFixupLib.inf                |   1 +
 .../ConfigurationManagerObjectParser.c        | 678 ++++++++++++++++++
 .../ConfigurationManagerObjectParser.h        |  73 ++
 .../Common/TableHelperLib/TableHelper.c       |  96 ---
 .../Common/TableHelperLib/TableHelperLib.inf  |  13 +-
 24 files changed, 1579 insertions(+), 378 deletions(-)
 create mode 100644 DynamicTablesPkg/Include/Library/AcpiHelperLib.h
 create mode 100644 DynamicTablesPkg/Library/Common/AcpiHelperLib/AcpiHelper.c
 create mode 100644 DynamicTablesPkg/Library/Common/AcpiHelperLib/AcpiHelperLib.inf
 create mode 100644 DynamicTablesPkg/Library/Common/TableHelperLib/ConfigurationManagerObjectParser.c
 create mode 100644 DynamicTablesPkg/Library/Common/TableHelperLib/ConfigurationManagerObjectParser.h

-- 
2.17.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#81323): https://edk2.groups.io/g/devel/message/81323
Mute This Topic: https://groups.io/mt/85968533/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-


Re: [edk2-devel] [PATCH v2 00/10] Various DynamicTablesPkg modifications
Posted by Sami Mujawar 2 years, 7 months ago
Merged as 422e5d2f7f1a..22873f58c40c

Regards,

Sami Mujawar


On 30/09/2021 08:48 AM, Pierre.Gondois@arm.com wrote:
> From: Pierre Gondois <Pierre.Gondois@arm.com>
>
> This patch-set aggregates various modifications in the
> DynamicTablesPkg:
>   - Extract an AcpiTableHelperLib from TableHelperLib to remove
>     the dependency of some utility functions over configuration
>     manager definitions
>   - Add a HexFromAscii() function
>   - Add a AmlGetEisaIdFromString() function
>   - Add a configuration manager object parser
>   - Use %a instead of %s in when printing AmlLib
>   - Update the .ci.yaml once to prepare for other incoming patches
>   - Modify the generic prototype of the AmlResourceDataCodeGen
>     functions. This also means deprecating some functions.
>
> The modifications can be seen at: https://github.com/PierreARM/edk2/tree/1718_Various_DynamicTablesPkg_modifications_v2
>
> v2:
> - Corrections in commit messages. [Sami]
> - Document returned error code for some functions. [Sami]
> - Correct wrong field name in Configuration Manager Object parser.
>    [Joey]
>
> Pierre Gondois (9):
>    DynamicTablesPkg: Extract AcpiHelperLib from TableHelperLib
>    DynamicTablesPkg: Update TableHelperLib.inf
>    DynamicTablesPkg: Rename single char input parameter
>    DynamicTablesPkg: Add HexFromAscii() to AcpiHelperLib
>    DynamicTablesPkg: Add AmlGetEisaIdFromString() to AcpiHelperLib
>    DynamicTablesPkg: Use %a formatter in AmlDbgPrint
>    DynamicTablesPkg: Update DynamicTablesPkg.ci.yaml
>    DynamicTablesPkg: Deprecate Crs specific methods in AmlLib
>    DynamicTablesPkg: Rework AmlResourceDataCodegen.c/h
>
> Sami Mujawar (1):
>    DynamicTablesPkg: Add Configuration Manager Object parser
>
>   DynamicTablesPkg/DynamicTables.dsc.inc        |   3 +-
>   DynamicTablesPkg/DynamicTablesPkg.ci.yaml     |  29 +
>   DynamicTablesPkg/DynamicTablesPkg.dec         |   4 +
>   DynamicTablesPkg/DynamicTablesPkg.dsc         |   1 +
>   .../Include/Library/AcpiHelperLib.h           |  93 +++
>   .../Include/Library/AmlLib/AmlLib.h           | 225 ++++--
>   .../Include/Library/TableHelperLib.h          |  49 +-
>   .../SsdtCmn600Generator.c                     |  14 +-
>   .../AcpiSsdtCmn600LibArm/SsdtCmn600LibArm.inf |   3 +-
>   .../SsdtSerialPortGenerator.c                 |   3 +-
>   .../SsdtSerialPortLibArm.inf                  |   4 +-
>   .../Library/Common/AcpiHelperLib/AcpiHelper.c | 210 ++++++
>   .../Common/AcpiHelperLib/AcpiHelperLib.inf    |  25 +
>   .../Common/AmlLib/AmlDbgPrint/AmlDbgPrint.c   |  16 +-
>   .../Library/Common/AmlLib/AmlLib.inf          |   3 +-
>   .../Library/Common/AmlLib/Api/AmlApi.c        | 147 +++-
>   .../AmlLib/CodeGen/AmlResourceDataCodeGen.c   | 194 ++---
>   .../AmlLib/CodeGen/AmlResourceDataCodeGen.h   |  67 +-
>   .../SsdtSerialPortFixupLib.c                  |   6 +-
>   .../SsdtSerialPortFixupLib.inf                |   1 +
>   .../ConfigurationManagerObjectParser.c        | 678 ++++++++++++++++++
>   .../ConfigurationManagerObjectParser.h        |  73 ++
>   .../Common/TableHelperLib/TableHelper.c       |  96 ---
>   .../Common/TableHelperLib/TableHelperLib.inf  |  13 +-
>   24 files changed, 1579 insertions(+), 378 deletions(-)
>   create mode 100644 DynamicTablesPkg/Include/Library/AcpiHelperLib.h
>   create mode 100644 DynamicTablesPkg/Library/Common/AcpiHelperLib/AcpiHelper.c
>   create mode 100644 DynamicTablesPkg/Library/Common/AcpiHelperLib/AcpiHelperLib.inf
>   create mode 100644 DynamicTablesPkg/Library/Common/TableHelperLib/ConfigurationManagerObjectParser.c
>   create mode 100644 DynamicTablesPkg/Library/Common/TableHelperLib/ConfigurationManagerObjectParser.h
>

IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#81359): https://edk2.groups.io/g/devel/message/81359
Mute This Topic: https://groups.io/mt/85968533/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-