[edk2-devel] [PATCH v3 0/5] Add DynamicPlatRepoLib

PierreGondois posted 5 patches 2 years, 4 months ago
Failed in applying to current master (apply log)
DynamicTablesPkg/DynamicTablesPkg.dec         |   3 +
DynamicTablesPkg/DynamicTablesPkg.dsc         |   1 +
.../Include/Library/DynamicPlatRepoLib.h      | 114 ++++
.../DynamicPlatRepoLib/CmObjectTokenFixer.c   | 165 ++++++
.../DynamicPlatRepoLib/CmObjectTokenFixer.h   |  52 ++
.../DynamicPlatRepoLib/DynamicPlatRepo.c      | 521 ++++++++++++++++++
.../DynamicPlatRepoInternal.h                 |  78 +++
.../DynamicPlatRepoLib/DynamicPlatRepoLib.inf |  33 ++
.../DynamicPlatRepoLib/TokenGenerator.c       |  29 +
.../DynamicPlatRepoLib/TokenGenerator.h       |  26 +
.../Common/DynamicPlatRepoLib/TokenMapper.c   | 224 ++++++++
.../Common/DynamicPlatRepoLib/TokenMapper.h   | 123 +++++
12 files changed, 1369 insertions(+)
create mode 100644 DynamicTablesPkg/Include/Library/DynamicPlatRepoLib.h
create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/CmObjectTokenFixer.c
create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/CmObjectTokenFixer.h
create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/DynamicPlatRepo.c
create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/DynamicPlatRepoInternal.h
create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/DynamicPlatRepoLib.inf
create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/TokenGenerator.c
create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/TokenGenerator.h
create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/TokenMapper.c
create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/TokenMapper.h
[edk2-devel] [PATCH v3 0/5] Add DynamicPlatRepoLib
Posted by PierreGondois 2 years, 4 months ago
From: Pierre Gondois <Pierre.Gondois@arm.com>

v3:
- Run uncrustify and apply the required formatting. [Pierre]
- No other change with v2. [Pierre]
v2:
- Various small fixes (change status returned, reset values when
  freeing memory, ...). [Sami]

Ref:https://bugzilla.tianocore.org/show_bug.cgi?id=3743

The DynamicPlatRepoLib is library allowing to receive and then
give Configuration Manager (CM) objects.
After being initialized, a dynamic platform repository can receive
CMObjects. The library generates a token to uniquely identify the
CMObject in the repository. The dynamic platform repository must
then be 'finalized' before receiving CMObject queries.

This library is complementary to the 'static' definition of
CMObjects (i.e. as C structs). It is particularly useful when
using a HwInfoParserLib that dynamically creates CMObjects
(from a device tree for instance).

The changes can be seen at: https://github.com/PierreARM/edk2/tree/1788_Add_Dynamic_Plarform_Repository_Lib_v3

Pierre Gondois (5):
  DynamicTablesPkg: Definition for DynamicPlatRepoLib interface
  DynamicTablesPkg: DynamicPlatRepo: Add TokenGenerator
  DynamicTablesPkg: DynamicPlatRepo: Add TokenFixer
  DynamicTablesPkg: DynamicPlatRepo: Add TokenMapper
  DynamicTablesPkg: Add DynamicPlatRepo library

 DynamicTablesPkg/DynamicTablesPkg.dec         |   3 +
 DynamicTablesPkg/DynamicTablesPkg.dsc         |   1 +
 .../Include/Library/DynamicPlatRepoLib.h      | 114 ++++
 .../DynamicPlatRepoLib/CmObjectTokenFixer.c   | 165 ++++++
 .../DynamicPlatRepoLib/CmObjectTokenFixer.h   |  52 ++
 .../DynamicPlatRepoLib/DynamicPlatRepo.c      | 521 ++++++++++++++++++
 .../DynamicPlatRepoInternal.h                 |  78 +++
 .../DynamicPlatRepoLib/DynamicPlatRepoLib.inf |  33 ++
 .../DynamicPlatRepoLib/TokenGenerator.c       |  29 +
 .../DynamicPlatRepoLib/TokenGenerator.h       |  26 +
 .../Common/DynamicPlatRepoLib/TokenMapper.c   | 224 ++++++++
 .../Common/DynamicPlatRepoLib/TokenMapper.h   | 123 +++++
 12 files changed, 1369 insertions(+)
 create mode 100644 DynamicTablesPkg/Include/Library/DynamicPlatRepoLib.h
 create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/CmObjectTokenFixer.c
 create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/CmObjectTokenFixer.h
 create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/DynamicPlatRepo.c
 create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/DynamicPlatRepoInternal.h
 create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/DynamicPlatRepoLib.inf
 create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/TokenGenerator.c
 create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/TokenGenerator.h
 create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/TokenMapper.c
 create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/TokenMapper.h

--
2.25.1


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


Re: [edk2-devel] [PATCH v3 0/5] Add DynamicPlatRepoLib
Posted by Sami Mujawar 2 years, 4 months ago
Hi Pierre,

Thank you for this patch series.

Reviewed-by: Sami Mujawar <sami.mujawar@arm.com>

Regards,

Sami Mujawar


On 09/12/2021 09:32 AM, Pierre.Gondois@arm.com wrote:
> From: Pierre Gondois <Pierre.Gondois@arm.com>
>
> v3:
> - Run uncrustify and apply the required formatting. [Pierre]
> - No other change with v2. [Pierre]
> v2:
> - Various small fixes (change status returned, reset values when
>    freeing memory, ...). [Sami]
>
> Ref:https://bugzilla.tianocore.org/show_bug.cgi?id=3743
>
> The DynamicPlatRepoLib is library allowing to receive and then
> give Configuration Manager (CM) objects.
> After being initialized, a dynamic platform repository can receive
> CMObjects. The library generates a token to uniquely identify the
> CMObject in the repository. The dynamic platform repository must
> then be 'finalized' before receiving CMObject queries.
>
> This library is complementary to the 'static' definition of
> CMObjects (i.e. as C structs). It is particularly useful when
> using a HwInfoParserLib that dynamically creates CMObjects
> (from a device tree for instance).
>
> The changes can be seen at: https://github.com/PierreARM/edk2/tree/1788_Add_Dynamic_Plarform_Repository_Lib_v3
>
> Pierre Gondois (5):
>    DynamicTablesPkg: Definition for DynamicPlatRepoLib interface
>    DynamicTablesPkg: DynamicPlatRepo: Add TokenGenerator
>    DynamicTablesPkg: DynamicPlatRepo: Add TokenFixer
>    DynamicTablesPkg: DynamicPlatRepo: Add TokenMapper
>    DynamicTablesPkg: Add DynamicPlatRepo library
>
>   DynamicTablesPkg/DynamicTablesPkg.dec         |   3 +
>   DynamicTablesPkg/DynamicTablesPkg.dsc         |   1 +
>   .../Include/Library/DynamicPlatRepoLib.h      | 114 ++++
>   .../DynamicPlatRepoLib/CmObjectTokenFixer.c   | 165 ++++++
>   .../DynamicPlatRepoLib/CmObjectTokenFixer.h   |  52 ++
>   .../DynamicPlatRepoLib/DynamicPlatRepo.c      | 521 ++++++++++++++++++
>   .../DynamicPlatRepoInternal.h                 |  78 +++
>   .../DynamicPlatRepoLib/DynamicPlatRepoLib.inf |  33 ++
>   .../DynamicPlatRepoLib/TokenGenerator.c       |  29 +
>   .../DynamicPlatRepoLib/TokenGenerator.h       |  26 +
>   .../Common/DynamicPlatRepoLib/TokenMapper.c   | 224 ++++++++
>   .../Common/DynamicPlatRepoLib/TokenMapper.h   | 123 +++++
>   12 files changed, 1369 insertions(+)
>   create mode 100644 DynamicTablesPkg/Include/Library/DynamicPlatRepoLib.h
>   create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/CmObjectTokenFixer.c
>   create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/CmObjectTokenFixer.h
>   create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/DynamicPlatRepo.c
>   create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/DynamicPlatRepoInternal.h
>   create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/DynamicPlatRepoLib.inf
>   create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/TokenGenerator.c
>   create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/TokenGenerator.h
>   create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/TokenMapper.c
>   create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/TokenMapper.h
>
> --
> 2.25.1



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


Re: [edk2-devel] [PATCH v3 0/5] Add DynamicPlatRepoLib
Posted by Sami Mujawar 2 years, 4 months ago
Pushed as 9006967c8d24..38f6d78c3b62

Regards,

Sami Mujawar


On 09/12/2021 09:32 AM, Pierre.Gondois@arm.com wrote:
> From: Pierre Gondois <Pierre.Gondois@arm.com>
>
> v3:
> - Run uncrustify and apply the required formatting. [Pierre]
> - No other change with v2. [Pierre]
> v2:
> - Various small fixes (change status returned, reset values when
>    freeing memory, ...). [Sami]
>
> Ref:https://bugzilla.tianocore.org/show_bug.cgi?id=3743
>
> The DynamicPlatRepoLib is library allowing to receive and then
> give Configuration Manager (CM) objects.
> After being initialized, a dynamic platform repository can receive
> CMObjects. The library generates a token to uniquely identify the
> CMObject in the repository. The dynamic platform repository must
> then be 'finalized' before receiving CMObject queries.
>
> This library is complementary to the 'static' definition of
> CMObjects (i.e. as C structs). It is particularly useful when
> using a HwInfoParserLib that dynamically creates CMObjects
> (from a device tree for instance).
>
> The changes can be seen at: https://github.com/PierreARM/edk2/tree/1788_Add_Dynamic_Plarform_Repository_Lib_v3
>
> Pierre Gondois (5):
>    DynamicTablesPkg: Definition for DynamicPlatRepoLib interface
>    DynamicTablesPkg: DynamicPlatRepo: Add TokenGenerator
>    DynamicTablesPkg: DynamicPlatRepo: Add TokenFixer
>    DynamicTablesPkg: DynamicPlatRepo: Add TokenMapper
>    DynamicTablesPkg: Add DynamicPlatRepo library
>
>   DynamicTablesPkg/DynamicTablesPkg.dec         |   3 +
>   DynamicTablesPkg/DynamicTablesPkg.dsc         |   1 +
>   .../Include/Library/DynamicPlatRepoLib.h      | 114 ++++
>   .../DynamicPlatRepoLib/CmObjectTokenFixer.c   | 165 ++++++
>   .../DynamicPlatRepoLib/CmObjectTokenFixer.h   |  52 ++
>   .../DynamicPlatRepoLib/DynamicPlatRepo.c      | 521 ++++++++++++++++++
>   .../DynamicPlatRepoInternal.h                 |  78 +++
>   .../DynamicPlatRepoLib/DynamicPlatRepoLib.inf |  33 ++
>   .../DynamicPlatRepoLib/TokenGenerator.c       |  29 +
>   .../DynamicPlatRepoLib/TokenGenerator.h       |  26 +
>   .../Common/DynamicPlatRepoLib/TokenMapper.c   | 224 ++++++++
>   .../Common/DynamicPlatRepoLib/TokenMapper.h   | 123 +++++
>   12 files changed, 1369 insertions(+)
>   create mode 100644 DynamicTablesPkg/Include/Library/DynamicPlatRepoLib.h
>   create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/CmObjectTokenFixer.c
>   create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/CmObjectTokenFixer.h
>   create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/DynamicPlatRepo.c
>   create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/DynamicPlatRepoInternal.h
>   create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/DynamicPlatRepoLib.inf
>   create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/TokenGenerator.c
>   create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/TokenGenerator.h
>   create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/TokenMapper.c
>   create mode 100644 DynamicTablesPkg/Library/Common/DynamicPlatRepoLib/TokenMapper.h
>
> --
> 2.25.1

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 (#84828): https://edk2.groups.io/g/devel/message/84828
Mute This Topic: https://groups.io/mt/87608923/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-