1. Patchew
  2. EDK2
  3. View series

[edk2-devel] [PATCH 0/4] Trace Hub debug library support

victorx.hsu@intel.com posted 4 patches 1 year ago
Failed in applying to current master (apply log)
There is a newer version of this series
.gitmodules                                   |   5 +-
.pytool/CISettings.py                         |   2 +
Maintainers.txt                               |  18 +
.../Include/Guid/TraceHubDebugInfoHob.h       |  25 +
.../BaseTraceHubDebugSysTLib.c                | 251 ++++++
.../BaseTraceHubDebugSysTLib.inf              |  44 +
.../DxeSmmTraceHubDebugSysTLib.c              | 247 ++++++
.../DxeSmmTraceHubDebugSysTLib.inf            |  50 ++
.../InternalTraceHubApi.c                     |  82 ++
.../InternalTraceHubApi.h                     |  46 +
.../InternalTraceHubApiCommon.c               | 208 +++++
.../InternalTraceHubApiCommon.h               | 119 +++
.../PeiTraceHubDebugSysTLib.c                 | 290 +++++++
.../PeiTraceHubDebugSysTLib.inf               |  50 ++
.../Library/TraceHubDebugSysTLib/Readme.md    |  30 +
MdeModulePkg/MdeModulePkg.dec                 |  21 +
MdeModulePkg/MdeModulePkg.dsc                 |   3 +
MdeModulePkg/MdeModulePkg.uni                 |  18 +
MdePkg/Include/Library/MipiSysTLib.h          |  66 ++
MdePkg/Include/Library/TraceHubDebugSysTLib.h |  81 ++
MdePkg/Library/MipiSysTLib/GenMipiSystH.py    | 132 +++
MdePkg/Library/MipiSysTLib/MipiSysTLib.c      | 123 +++
MdePkg/Library/MipiSysTLib/MipiSysTLib.inf    |  55 ++
MdePkg/Library/MipiSysTLib/Platform.c         | 164 ++++
MdePkg/Library/MipiSysTLib/Platform.h         | 138 +++
MdePkg/Library/MipiSysTLib/Readme.md          |  25 +
MdePkg/Library/MipiSysTLib/mipi_syst.h        | 789 ++++++++++++++++++
MdePkg/Library/MipiSysTLib/mipisyst           |   1 +
.../TraceHubDebugSysTLibNull.c                |  76 ++
.../TraceHubDebugSysTLibNull.inf              |  29 +
MdePkg/MdePkg.ci.yaml                         |  15 +-
MdePkg/MdePkg.dec                             |   9 +
MdePkg/MdePkg.dsc                             |   2 +
ReadMe.rst                                    |   1 +
34 files changed, 3212 insertions(+), 3 deletions(-)
create mode 100644 MdeModulePkg/Include/Guid/TraceHubDebugInfoHob.h
create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/BaseTraceHubDebugSysTLib.c
create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/BaseTraceHubDebugSysTLib.inf
create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/DxeSmmTraceHubDebugSysTLib.c
create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/DxeSmmTraceHubDebugSysTLib.inf
create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApi.c
create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApi.h
create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApiCommon.c
create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApiCommon.h
create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/PeiTraceHubDebugSysTLib.c
create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/PeiTraceHubDebugSysTLib.inf
create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/Readme.md
create mode 100644 MdePkg/Include/Library/MipiSysTLib.h
create mode 100644 MdePkg/Include/Library/TraceHubDebugSysTLib.h
create mode 100644 MdePkg/Library/MipiSysTLib/GenMipiSystH.py
create mode 100644 MdePkg/Library/MipiSysTLib/MipiSysTLib.c
create mode 100644 MdePkg/Library/MipiSysTLib/MipiSysTLib.inf
create mode 100644 MdePkg/Library/MipiSysTLib/Platform.c
create mode 100644 MdePkg/Library/MipiSysTLib/Platform.h
create mode 100644 MdePkg/Library/MipiSysTLib/Readme.md
create mode 100644 MdePkg/Library/MipiSysTLib/mipi_syst.h
create mode 160000 MdePkg/Library/MipiSysTLib/mipisyst
create mode 100644 MdePkg/Library/TraceHubDebugSysTLibNull/TraceHubDebugSysTLibNull.c
create mode 100644 MdePkg/Library/TraceHubDebugSysTLibNull/TraceHubDebugSysTLibNull.inf
[edk2-devel] [PATCH 0/4] Trace Hub debug library support
Posted by victorx.hsu@intel.com 1 year ago 
From: VictorX Hsu <victorx.hsu@intel.com>

-	TraceHub UniversalPayload Spec: https://github.com/UniversalScalableFirmware/documentation/pull/52  (Industry Standard)
-	MipiSysTLib Wrapper: https://github.com/MIPI-Alliance/public-mipi-sys-t.git (Industry Standard)
-	TraceHubDebugLib: Intel(R) Trace Hub (TH) — The Linux Kernel documentation (Industry Standard)
    Developers Guide: https://cdrdv2-public.intel.com/671536/intel-trace-hub-developers-manual-2-1-2.pdf (Public Document)

VictorX Hsu (4):
  MdePkg: Add MipiSysTLib library
  MdeModulePkg: Add TraceHubDebugSysTLib library
  MdePkg: Add NULL library of TraceHubDebugSysTLib
  Maintainers.txt: Update reviewers and maintainers for Trace Hub dbg
    lib.

 .gitmodules                                   |   5 +-
 .pytool/CISettings.py                         |   2 +
 Maintainers.txt                               |  18 +
 .../Include/Guid/TraceHubDebugInfoHob.h       |  25 +
 .../BaseTraceHubDebugSysTLib.c                | 251 ++++++
 .../BaseTraceHubDebugSysTLib.inf              |  44 +
 .../DxeSmmTraceHubDebugSysTLib.c              | 247 ++++++
 .../DxeSmmTraceHubDebugSysTLib.inf            |  50 ++
 .../InternalTraceHubApi.c                     |  82 ++
 .../InternalTraceHubApi.h                     |  46 +
 .../InternalTraceHubApiCommon.c               | 208 +++++
 .../InternalTraceHubApiCommon.h               | 119 +++
 .../PeiTraceHubDebugSysTLib.c                 | 290 +++++++
 .../PeiTraceHubDebugSysTLib.inf               |  50 ++
 .../Library/TraceHubDebugSysTLib/Readme.md    |  30 +
 MdeModulePkg/MdeModulePkg.dec                 |  21 +
 MdeModulePkg/MdeModulePkg.dsc                 |   3 +
 MdeModulePkg/MdeModulePkg.uni                 |  18 +
 MdePkg/Include/Library/MipiSysTLib.h          |  66 ++
 MdePkg/Include/Library/TraceHubDebugSysTLib.h |  81 ++
 MdePkg/Library/MipiSysTLib/GenMipiSystH.py    | 132 +++
 MdePkg/Library/MipiSysTLib/MipiSysTLib.c      | 123 +++
 MdePkg/Library/MipiSysTLib/MipiSysTLib.inf    |  55 ++
 MdePkg/Library/MipiSysTLib/Platform.c         | 164 ++++
 MdePkg/Library/MipiSysTLib/Platform.h         | 138 +++
 MdePkg/Library/MipiSysTLib/Readme.md          |  25 +
 MdePkg/Library/MipiSysTLib/mipi_syst.h        | 789 ++++++++++++++++++
 MdePkg/Library/MipiSysTLib/mipisyst           |   1 +
 .../TraceHubDebugSysTLibNull.c                |  76 ++
 .../TraceHubDebugSysTLibNull.inf              |  29 +
 MdePkg/MdePkg.ci.yaml                         |  15 +-
 MdePkg/MdePkg.dec                             |   9 +
 MdePkg/MdePkg.dsc                             |   2 +
 ReadMe.rst                                    |   1 +
 34 files changed, 3212 insertions(+), 3 deletions(-)
 create mode 100644 MdeModulePkg/Include/Guid/TraceHubDebugInfoHob.h
 create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/BaseTraceHubDebugSysTLib.c
 create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/BaseTraceHubDebugSysTLib.inf
 create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/DxeSmmTraceHubDebugSysTLib.c
 create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/DxeSmmTraceHubDebugSysTLib.inf
 create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApi.c
 create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApi.h
 create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApiCommon.c
 create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApiCommon.h
 create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/PeiTraceHubDebugSysTLib.c
 create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/PeiTraceHubDebugSysTLib.inf
 create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/Readme.md
 create mode 100644 MdePkg/Include/Library/MipiSysTLib.h
 create mode 100644 MdePkg/Include/Library/TraceHubDebugSysTLib.h
 create mode 100644 MdePkg/Library/MipiSysTLib/GenMipiSystH.py
 create mode 100644 MdePkg/Library/MipiSysTLib/MipiSysTLib.c
 create mode 100644 MdePkg/Library/MipiSysTLib/MipiSysTLib.inf
 create mode 100644 MdePkg/Library/MipiSysTLib/Platform.c
 create mode 100644 MdePkg/Library/MipiSysTLib/Platform.h
 create mode 100644 MdePkg/Library/MipiSysTLib/Readme.md
 create mode 100644 MdePkg/Library/MipiSysTLib/mipi_syst.h
 create mode 160000 MdePkg/Library/MipiSysTLib/mipisyst
 create mode 100644 MdePkg/Library/TraceHubDebugSysTLibNull/TraceHubDebugSysTLibNull.c
 create mode 100644 MdePkg/Library/TraceHubDebugSysTLibNull/TraceHubDebugSysTLibNull.inf

-- 
2.40.0.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#103180): https://edk2.groups.io/g/devel/message/103180
Mute This Topic: https://groups.io/mt/98351445/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH 0/4] Trace Hub debug library support
Posted by Guo, Gua 1 year ago 
+ Related folk

I want to let the PR for related stakeholder review it.

And have any open ?
https://github.com/tianocore/edk2/pull/3901


Thanks,
Gua

-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of victorx.hsu@intel.com
Sent: Tuesday, April 18, 2023 7:05 PM
To: devel@edk2.groups.io
Cc: Hsu, VictorX <victorx.hsu@intel.com>
Subject: [edk2-devel] [PATCH 0/4] Trace Hub debug library support

From: VictorX Hsu <victorx.hsu@intel.com>

-	TraceHub UniversalPayload Spec: https://github.com/UniversalScalableFirmware/documentation/pull/52  (Industry Standard)
-	MipiSysTLib Wrapper: https://github.com/MIPI-Alliance/public-mipi-sys-t.git (Industry Standard)
-	TraceHubDebugLib: Intel(R) Trace Hub (TH) — The Linux Kernel documentation (Industry Standard)
    Developers Guide: https://cdrdv2-public.intel.com/671536/intel-trace-hub-developers-manual-2-1-2.pdf (Public Document)

VictorX Hsu (4):
  MdePkg: Add MipiSysTLib library
  MdeModulePkg: Add TraceHubDebugSysTLib library
  MdePkg: Add NULL library of TraceHubDebugSysTLib
  Maintainers.txt: Update reviewers and maintainers for Trace Hub dbg
    lib.

 .gitmodules                                   |   5 +-
 .pytool/CISettings.py                         |   2 +
 Maintainers.txt                               |  18 +
 .../Include/Guid/TraceHubDebugInfoHob.h       |  25 +
 .../BaseTraceHubDebugSysTLib.c                | 251 ++++++
 .../BaseTraceHubDebugSysTLib.inf              |  44 +
 .../DxeSmmTraceHubDebugSysTLib.c              | 247 ++++++
 .../DxeSmmTraceHubDebugSysTLib.inf            |  50 ++
 .../InternalTraceHubApi.c                     |  82 ++
 .../InternalTraceHubApi.h                     |  46 +
 .../InternalTraceHubApiCommon.c               | 208 +++++
 .../InternalTraceHubApiCommon.h               | 119 +++
 .../PeiTraceHubDebugSysTLib.c                 | 290 +++++++
 .../PeiTraceHubDebugSysTLib.inf               |  50 ++
 .../Library/TraceHubDebugSysTLib/Readme.md    |  30 +
 MdeModulePkg/MdeModulePkg.dec                 |  21 +
 MdeModulePkg/MdeModulePkg.dsc                 |   3 +
 MdeModulePkg/MdeModulePkg.uni                 |  18 +
 MdePkg/Include/Library/MipiSysTLib.h          |  66 ++
 MdePkg/Include/Library/TraceHubDebugSysTLib.h |  81 ++
 MdePkg/Library/MipiSysTLib/GenMipiSystH.py    | 132 +++
 MdePkg/Library/MipiSysTLib/MipiSysTLib.c      | 123 +++
 MdePkg/Library/MipiSysTLib/MipiSysTLib.inf    |  55 ++
 MdePkg/Library/MipiSysTLib/Platform.c         | 164 ++++
 MdePkg/Library/MipiSysTLib/Platform.h         | 138 +++
 MdePkg/Library/MipiSysTLib/Readme.md          |  25 +
 MdePkg/Library/MipiSysTLib/mipi_syst.h        | 789 ++++++++++++++++++
 MdePkg/Library/MipiSysTLib/mipisyst           |   1 +
 .../TraceHubDebugSysTLibNull.c                |  76 ++
 .../TraceHubDebugSysTLibNull.inf              |  29 +
 MdePkg/MdePkg.ci.yaml                         |  15 +-
 MdePkg/MdePkg.dec                             |   9 +
 MdePkg/MdePkg.dsc                             |   2 +
 ReadMe.rst                                    |   1 +
 34 files changed, 3212 insertions(+), 3 deletions(-)  create mode 100644 MdeModulePkg/Include/Guid/TraceHubDebugInfoHob.h
 create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/BaseTraceHubDebugSysTLib.c
 create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/BaseTraceHubDebugSysTLib.inf
 create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/DxeSmmTraceHubDebugSysTLib.c
 create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/DxeSmmTraceHubDebugSysTLib.inf
 create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApi.c
 create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApi.h
 create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApiCommon.c
 create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApiCommon.h
 create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/PeiTraceHubDebugSysTLib.c
 create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/PeiTraceHubDebugSysTLib.inf
 create mode 100644 MdeModulePkg/Library/TraceHubDebugSysTLib/Readme.md
 create mode 100644 MdePkg/Include/Library/MipiSysTLib.h
 create mode 100644 MdePkg/Include/Library/TraceHubDebugSysTLib.h
 create mode 100644 MdePkg/Library/MipiSysTLib/GenMipiSystH.py
 create mode 100644 MdePkg/Library/MipiSysTLib/MipiSysTLib.c
 create mode 100644 MdePkg/Library/MipiSysTLib/MipiSysTLib.inf
 create mode 100644 MdePkg/Library/MipiSysTLib/Platform.c
 create mode 100644 MdePkg/Library/MipiSysTLib/Platform.h
 create mode 100644 MdePkg/Library/MipiSysTLib/Readme.md
 create mode 100644 MdePkg/Library/MipiSysTLib/mipi_syst.h
 create mode 160000 MdePkg/Library/MipiSysTLib/mipisyst
 create mode 100644 MdePkg/Library/TraceHubDebugSysTLibNull/TraceHubDebugSysTLibNull.c
 create mode 100644 MdePkg/Library/TraceHubDebugSysTLibNull/TraceHubDebugSysTLibNull.inf

--
2.40.0.windows.1








-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#103333): https://edk2.groups.io/g/devel/message/103333
Mute This Topic: https://groups.io/mt/98351445/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH 0/4] Trace Hub debug library support
Posted by Michael D Kinney 1 year ago 
Gua,

I see this PR says it passes CI but there are merge conflicts.

Can you please update the series to address the conflicts.

Thanks,

Mike

> -----Original Message-----
> From: Guo, Gua <gua.guo@intel.com>
> Sent: Thursday, April 20, 2023 7:33 PM
> To: devel@edk2.groups.io; Hsu, VictorX <victorx.hsu@intel.com>
> Cc: Chiu, Chasel <chasel.chiu@intel.com>; Kinney, Michael D
> <michael.d.kinney@intel.com>; Prakashan, Krishnadas Veliyathuparambil
> <krishnadas.veliyathuparambil.prakashan@intel.com>; K N, Karthik
> <karthik.k.n@intel.com>; Chan, Laura <laura.chan@intel.com>
> Subject: RE: [edk2-devel] [PATCH 0/4] Trace Hub debug library support
> 
> + Related folk
> 
> I want to let the PR for related stakeholder review it.
> 
> And have any open ?
> https://github.com/tianocore/edk2/pull/3901
> 
> 
> Thanks,
> Gua
> 
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> victorx.hsu@intel.com
> Sent: Tuesday, April 18, 2023 7:05 PM
> To: devel@edk2.groups.io
> Cc: Hsu, VictorX <victorx.hsu@intel.com>
> Subject: [edk2-devel] [PATCH 0/4] Trace Hub debug library support
> 
> From: VictorX Hsu <victorx.hsu@intel.com>
> 
> -	TraceHub UniversalPayload Spec:
> https://github.com/UniversalScalableFirmware/documentation/pull/52
> (Industry Standard)
> -	MipiSysTLib Wrapper: https://github.com/MIPI-Alliance/public-mipi-
> sys-t.git (Industry Standard)
> -	TraceHubDebugLib: Intel(R) Trace Hub (TH) — The Linux Kernel
> documentation (Industry Standard)
>     Developers Guide: https://cdrdv2-public.intel.com/671536/intel-trace-hub-
> developers-manual-2-1-2.pdf (Public Document)
> 
> VictorX Hsu (4):
>   MdePkg: Add MipiSysTLib library
>   MdeModulePkg: Add TraceHubDebugSysTLib library
>   MdePkg: Add NULL library of TraceHubDebugSysTLib
>   Maintainers.txt: Update reviewers and maintainers for Trace Hub dbg
>     lib.
> 
>  .gitmodules                                   |   5 +-
>  .pytool/CISettings.py                         |   2 +
>  Maintainers.txt                               |  18 +
>  .../Include/Guid/TraceHubDebugInfoHob.h       |  25 +
>  .../BaseTraceHubDebugSysTLib.c                | 251 ++++++
>  .../BaseTraceHubDebugSysTLib.inf              |  44 +
>  .../DxeSmmTraceHubDebugSysTLib.c              | 247 ++++++
>  .../DxeSmmTraceHubDebugSysTLib.inf            |  50 ++
>  .../InternalTraceHubApi.c                     |  82 ++
>  .../InternalTraceHubApi.h                     |  46 +
>  .../InternalTraceHubApiCommon.c               | 208 +++++
>  .../InternalTraceHubApiCommon.h               | 119 +++
>  .../PeiTraceHubDebugSysTLib.c                 | 290 +++++++
>  .../PeiTraceHubDebugSysTLib.inf               |  50 ++
>  .../Library/TraceHubDebugSysTLib/Readme.md    |  30 +
>  MdeModulePkg/MdeModulePkg.dec                 |  21 +
>  MdeModulePkg/MdeModulePkg.dsc                 |   3 +
>  MdeModulePkg/MdeModulePkg.uni                 |  18 +
>  MdePkg/Include/Library/MipiSysTLib.h          |  66 ++
>  MdePkg/Include/Library/TraceHubDebugSysTLib.h |  81 ++
>  MdePkg/Library/MipiSysTLib/GenMipiSystH.py    | 132 +++
>  MdePkg/Library/MipiSysTLib/MipiSysTLib.c      | 123 +++
>  MdePkg/Library/MipiSysTLib/MipiSysTLib.inf    |  55 ++
>  MdePkg/Library/MipiSysTLib/Platform.c         | 164 ++++
>  MdePkg/Library/MipiSysTLib/Platform.h         | 138 +++
>  MdePkg/Library/MipiSysTLib/Readme.md          |  25 +
>  MdePkg/Library/MipiSysTLib/mipi_syst.h        | 789 ++++++++++++++++++
>  MdePkg/Library/MipiSysTLib/mipisyst           |   1 +
>  .../TraceHubDebugSysTLibNull.c                |  76 ++
>  .../TraceHubDebugSysTLibNull.inf              |  29 +
>  MdePkg/MdePkg.ci.yaml                         |  15 +-
>  MdePkg/MdePkg.dec                             |   9 +
>  MdePkg/MdePkg.dsc                             |   2 +
>  ReadMe.rst                                    |   1 +
>  34 files changed, 3212 insertions(+), 3 deletions(-)  create mode 100644
> MdeModulePkg/Include/Guid/TraceHubDebugInfoHob.h
>  create mode 100644
> MdeModulePkg/Library/TraceHubDebugSysTLib/BaseTraceHubDebugSysTLib.c
>  create mode 100644
> MdeModulePkg/Library/TraceHubDebugSysTLib/BaseTraceHubDebugSysTLib.i
> nf
>  create mode 100644
> MdeModulePkg/Library/TraceHubDebugSysTLib/DxeSmmTraceHubDebugSysTL
> ib.c
>  create mode 100644
> MdeModulePkg/Library/TraceHubDebugSysTLib/DxeSmmTraceHubDebugSysTL
> ib.inf
>  create mode 100644
> MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApi.c
>  create mode 100644
> MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApi.h
>  create mode 100644
> MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApiCommon
> .c
>  create mode 100644
> MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApiCommon
> .h
>  create mode 100644
> MdeModulePkg/Library/TraceHubDebugSysTLib/PeiTraceHubDebugSysTLib.c
>  create mode 100644
> MdeModulePkg/Library/TraceHubDebugSysTLib/PeiTraceHubDebugSysTLib.inf
>  create mode 100644
> MdeModulePkg/Library/TraceHubDebugSysTLib/Readme.md
>  create mode 100644 MdePkg/Include/Library/MipiSysTLib.h
>  create mode 100644 MdePkg/Include/Library/TraceHubDebugSysTLib.h
>  create mode 100644 MdePkg/Library/MipiSysTLib/GenMipiSystH.py
>  create mode 100644 MdePkg/Library/MipiSysTLib/MipiSysTLib.c
>  create mode 100644 MdePkg/Library/MipiSysTLib/MipiSysTLib.inf
>  create mode 100644 MdePkg/Library/MipiSysTLib/Platform.c
>  create mode 100644 MdePkg/Library/MipiSysTLib/Platform.h
>  create mode 100644 MdePkg/Library/MipiSysTLib/Readme.md
>  create mode 100644 MdePkg/Library/MipiSysTLib/mipi_syst.h
>  create mode 160000 MdePkg/Library/MipiSysTLib/mipisyst
>  create mode 100644
> MdePkg/Library/TraceHubDebugSysTLibNull/TraceHubDebugSysTLibNull.c
>  create mode 100644
> MdePkg/Library/TraceHubDebugSysTLibNull/TraceHubDebugSysTLibNull.inf
> 
> --
> 2.40.0.windows.1
> 
> 
> 
> 
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#104226): https://edk2.groups.io/g/devel/message/104226
Mute This Topic: https://groups.io/mt/98351445/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/leave/3901457/1787277/102458076/xyzzy [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH 0/4] Trace Hub debug library support
Posted by Hsu, VictorX 1 year ago 
Hi @Kinney, Michael D

All conflicts were fixed

thanks

-----Original Message-----
From: Kinney, Michael D <michael.d.kinney@intel.com> 
Sent: Monday, May 8, 2023 8:45 AM
To: Guo, Gua <gua.guo@intel.com>; devel@edk2.groups.io; Hsu, VictorX <victorx.hsu@intel.com>
Cc: Chiu, Chasel <chasel.chiu@intel.com>; Prakashan, Krishnadas Veliyathuparambil <krishnadas.veliyathuparambil.prakashan@intel.com>; K N, Karthik <karthik.k.n@intel.com>; Chan, Laura <laura.chan@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>
Subject: RE: [edk2-devel] [PATCH 0/4] Trace Hub debug library support

Gua,

I see this PR says it passes CI but there are merge conflicts.

Can you please update the series to address the conflicts.

Thanks,

Mike

> -----Original Message-----
> From: Guo, Gua <gua.guo@intel.com>
> Sent: Thursday, April 20, 2023 7:33 PM
> To: devel@edk2.groups.io; Hsu, VictorX <victorx.hsu@intel.com>
> Cc: Chiu, Chasel <chasel.chiu@intel.com>; Kinney, Michael D 
> <michael.d.kinney@intel.com>; Prakashan, Krishnadas Veliyathuparambil 
> <krishnadas.veliyathuparambil.prakashan@intel.com>; K N, Karthik 
> <karthik.k.n@intel.com>; Chan, Laura <laura.chan@intel.com>
> Subject: RE: [edk2-devel] [PATCH 0/4] Trace Hub debug library support
> 
> + Related folk
> 
> I want to let the PR for related stakeholder review it.
> 
> And have any open ?
> https://github.com/tianocore/edk2/pull/3901
> 
> 
> Thanks,
> Gua
> 
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of 
> victorx.hsu@intel.com
> Sent: Tuesday, April 18, 2023 7:05 PM
> To: devel@edk2.groups.io
> Cc: Hsu, VictorX <victorx.hsu@intel.com>
> Subject: [edk2-devel] [PATCH 0/4] Trace Hub debug library support
> 
> From: VictorX Hsu <victorx.hsu@intel.com>
> 
> -	TraceHub UniversalPayload Spec:
> https://github.com/UniversalScalableFirmware/documentation/pull/52
> (Industry Standard)
> -	MipiSysTLib Wrapper: https://github.com/MIPI-Alliance/public-mipi-
> sys-t.git (Industry Standard)
> -	TraceHubDebugLib: Intel(R) Trace Hub (TH) — The Linux Kernel
> documentation (Industry Standard)
>     Developers Guide: 
> https://cdrdv2-public.intel.com/671536/intel-trace-hub-
> developers-manual-2-1-2.pdf (Public Document)
> 
> VictorX Hsu (4):
>   MdePkg: Add MipiSysTLib library
>   MdeModulePkg: Add TraceHubDebugSysTLib library
>   MdePkg: Add NULL library of TraceHubDebugSysTLib
>   Maintainers.txt: Update reviewers and maintainers for Trace Hub dbg
>     lib.
> 
>  .gitmodules                                   |   5 +-
>  .pytool/CISettings.py                         |   2 +
>  Maintainers.txt                               |  18 +
>  .../Include/Guid/TraceHubDebugInfoHob.h       |  25 +
>  .../BaseTraceHubDebugSysTLib.c                | 251 ++++++
>  .../BaseTraceHubDebugSysTLib.inf              |  44 +
>  .../DxeSmmTraceHubDebugSysTLib.c              | 247 ++++++
>  .../DxeSmmTraceHubDebugSysTLib.inf            |  50 ++
>  .../InternalTraceHubApi.c                     |  82 ++
>  .../InternalTraceHubApi.h                     |  46 +
>  .../InternalTraceHubApiCommon.c               | 208 +++++
>  .../InternalTraceHubApiCommon.h               | 119 +++
>  .../PeiTraceHubDebugSysTLib.c                 | 290 +++++++
>  .../PeiTraceHubDebugSysTLib.inf               |  50 ++
>  .../Library/TraceHubDebugSysTLib/Readme.md    |  30 +
>  MdeModulePkg/MdeModulePkg.dec                 |  21 +
>  MdeModulePkg/MdeModulePkg.dsc                 |   3 +
>  MdeModulePkg/MdeModulePkg.uni                 |  18 +
>  MdePkg/Include/Library/MipiSysTLib.h          |  66 ++
>  MdePkg/Include/Library/TraceHubDebugSysTLib.h |  81 ++
>  MdePkg/Library/MipiSysTLib/GenMipiSystH.py    | 132 +++
>  MdePkg/Library/MipiSysTLib/MipiSysTLib.c      | 123 +++
>  MdePkg/Library/MipiSysTLib/MipiSysTLib.inf    |  55 ++
>  MdePkg/Library/MipiSysTLib/Platform.c         | 164 ++++
>  MdePkg/Library/MipiSysTLib/Platform.h         | 138 +++
>  MdePkg/Library/MipiSysTLib/Readme.md          |  25 +
>  MdePkg/Library/MipiSysTLib/mipi_syst.h        | 789 ++++++++++++++++++
>  MdePkg/Library/MipiSysTLib/mipisyst           |   1 +
>  .../TraceHubDebugSysTLibNull.c                |  76 ++
>  .../TraceHubDebugSysTLibNull.inf              |  29 +
>  MdePkg/MdePkg.ci.yaml                         |  15 +-
>  MdePkg/MdePkg.dec                             |   9 +
>  MdePkg/MdePkg.dsc                             |   2 +
>  ReadMe.rst                                    |   1 +
>  34 files changed, 3212 insertions(+), 3 deletions(-)  create mode 
> 100644 MdeModulePkg/Include/Guid/TraceHubDebugInfoHob.h
>  create mode 100644
> MdeModulePkg/Library/TraceHubDebugSysTLib/BaseTraceHubDebugSysTLib.c
>  create mode 100644
> MdeModulePkg/Library/TraceHubDebugSysTLib/BaseTraceHubDebugSysTLib.i
> nf
>  create mode 100644
> MdeModulePkg/Library/TraceHubDebugSysTLib/DxeSmmTraceHubDebugSysTL
> ib.c
>  create mode 100644
> MdeModulePkg/Library/TraceHubDebugSysTLib/DxeSmmTraceHubDebugSysTL
> ib.inf
>  create mode 100644
> MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApi.c
>  create mode 100644
> MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApi.h
>  create mode 100644
> MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApiCommon
> .c
>  create mode 100644
> MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApiCommon
> .h
>  create mode 100644
> MdeModulePkg/Library/TraceHubDebugSysTLib/PeiTraceHubDebugSysTLib.c
>  create mode 100644
> MdeModulePkg/Library/TraceHubDebugSysTLib/PeiTraceHubDebugSysTLib.inf
>  create mode 100644
> MdeModulePkg/Library/TraceHubDebugSysTLib/Readme.md
>  create mode 100644 MdePkg/Include/Library/MipiSysTLib.h
>  create mode 100644 MdePkg/Include/Library/TraceHubDebugSysTLib.h
>  create mode 100644 MdePkg/Library/MipiSysTLib/GenMipiSystH.py
>  create mode 100644 MdePkg/Library/MipiSysTLib/MipiSysTLib.c
>  create mode 100644 MdePkg/Library/MipiSysTLib/MipiSysTLib.inf
>  create mode 100644 MdePkg/Library/MipiSysTLib/Platform.c
>  create mode 100644 MdePkg/Library/MipiSysTLib/Platform.h
>  create mode 100644 MdePkg/Library/MipiSysTLib/Readme.md
>  create mode 100644 MdePkg/Library/MipiSysTLib/mipi_syst.h
>  create mode 160000 MdePkg/Library/MipiSysTLib/mipisyst
>  create mode 100644
> MdePkg/Library/TraceHubDebugSysTLibNull/TraceHubDebugSysTLibNull.c
>  create mode 100644
> MdePkg/Library/TraceHubDebugSysTLibNull/TraceHubDebugSysTLibNull.inf
> 
> --
> 2.40.0.windows.1
> 
> 
> 
> 
> 



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#104272): https://edk2.groups.io/g/devel/message/104272
Mute This Topic: https://groups.io/mt/98351445/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [PATCH 0/4] Trace Hub debug library support
Posted by Michael D Kinney 1 year ago 
Can you please send the updated email patches.  I could not get the previous version to apply.

Mike

> -----Original Message-----
> From: Hsu, VictorX <victorx.hsu@intel.com>
> Sent: Sunday, May 7, 2023 10:19 PM
> To: Kinney, Michael D <michael.d.kinney@intel.com>; Guo, Gua
> <gua.guo@intel.com>; devel@edk2.groups.io
> Cc: Chiu, Chasel <chasel.chiu@intel.com>; Prakashan, Krishnadas
> Veliyathuparambil <krishnadas.veliyathuparambil.prakashan@intel.com>; K
> N, Karthik <karthik.k.n@intel.com>; Chan, Laura <laura.chan@intel.com>
> Subject: RE: [edk2-devel] [PATCH 0/4] Trace Hub debug library support
> 
> Hi @Kinney, Michael D
> 
> All conflicts were fixed
> 
> thanks
> 
> -----Original Message-----
> From: Kinney, Michael D <michael.d.kinney@intel.com>
> Sent: Monday, May 8, 2023 8:45 AM
> To: Guo, Gua <gua.guo@intel.com>; devel@edk2.groups.io; Hsu, VictorX
> <victorx.hsu@intel.com>
> Cc: Chiu, Chasel <chasel.chiu@intel.com>; Prakashan, Krishnadas
> Veliyathuparambil <krishnadas.veliyathuparambil.prakashan@intel.com>; K
> N, Karthik <karthik.k.n@intel.com>; Chan, Laura <laura.chan@intel.com>;
> Kinney, Michael D <michael.d.kinney@intel.com>
> Subject: RE: [edk2-devel] [PATCH 0/4] Trace Hub debug library support
> 
> Gua,
> 
> I see this PR says it passes CI but there are merge conflicts.
> 
> Can you please update the series to address the conflicts.
> 
> Thanks,
> 
> Mike
> 
> > -----Original Message-----
> > From: Guo, Gua <gua.guo@intel.com>
> > Sent: Thursday, April 20, 2023 7:33 PM
> > To: devel@edk2.groups.io; Hsu, VictorX <victorx.hsu@intel.com>
> > Cc: Chiu, Chasel <chasel.chiu@intel.com>; Kinney, Michael D
> > <michael.d.kinney@intel.com>; Prakashan, Krishnadas Veliyathuparambil
> > <krishnadas.veliyathuparambil.prakashan@intel.com>; K N, Karthik
> > <karthik.k.n@intel.com>; Chan, Laura <laura.chan@intel.com>
> > Subject: RE: [edk2-devel] [PATCH 0/4] Trace Hub debug library support
> >
> > + Related folk
> >
> > I want to let the PR for related stakeholder review it.
> >
> > And have any open ?
> > https://github.com/tianocore/edk2/pull/3901
> >
> >
> > Thanks,
> > Gua
> >
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> > victorx.hsu@intel.com
> > Sent: Tuesday, April 18, 2023 7:05 PM
> > To: devel@edk2.groups.io
> > Cc: Hsu, VictorX <victorx.hsu@intel.com>
> > Subject: [edk2-devel] [PATCH 0/4] Trace Hub debug library support
> >
> > From: VictorX Hsu <victorx.hsu@intel.com>
> >
> > -	TraceHub UniversalPayload Spec:
> > https://github.com/UniversalScalableFirmware/documentation/pull/52
> > (Industry Standard)
> > -	MipiSysTLib Wrapper: https://github.com/MIPI-Alliance/public-mipi-
> > sys-t.git (Industry Standard)
> > -	TraceHubDebugLib: Intel(R) Trace Hub (TH) — The Linux Kernel
> > documentation (Industry Standard)
> >     Developers Guide:
> > https://cdrdv2-public.intel.com/671536/intel-trace-hub-
> > developers-manual-2-1-2.pdf (Public Document)
> >
> > VictorX Hsu (4):
> >   MdePkg: Add MipiSysTLib library
> >   MdeModulePkg: Add TraceHubDebugSysTLib library
> >   MdePkg: Add NULL library of TraceHubDebugSysTLib
> >   Maintainers.txt: Update reviewers and maintainers for Trace Hub dbg
> >     lib.
> >
> >  .gitmodules                                   |   5 +-
> >  .pytool/CISettings.py                         |   2 +
> >  Maintainers.txt                               |  18 +
> >  .../Include/Guid/TraceHubDebugInfoHob.h       |  25 +
> >  .../BaseTraceHubDebugSysTLib.c                | 251 ++++++
> >  .../BaseTraceHubDebugSysTLib.inf              |  44 +
> >  .../DxeSmmTraceHubDebugSysTLib.c              | 247 ++++++
> >  .../DxeSmmTraceHubDebugSysTLib.inf            |  50 ++
> >  .../InternalTraceHubApi.c                     |  82 ++
> >  .../InternalTraceHubApi.h                     |  46 +
> >  .../InternalTraceHubApiCommon.c               | 208 +++++
> >  .../InternalTraceHubApiCommon.h               | 119 +++
> >  .../PeiTraceHubDebugSysTLib.c                 | 290 +++++++
> >  .../PeiTraceHubDebugSysTLib.inf               |  50 ++
> >  .../Library/TraceHubDebugSysTLib/Readme.md    |  30 +
> >  MdeModulePkg/MdeModulePkg.dec                 |  21 +
> >  MdeModulePkg/MdeModulePkg.dsc                 |   3 +
> >  MdeModulePkg/MdeModulePkg.uni                 |  18 +
> >  MdePkg/Include/Library/MipiSysTLib.h          |  66 ++
> >  MdePkg/Include/Library/TraceHubDebugSysTLib.h |  81 ++
> >  MdePkg/Library/MipiSysTLib/GenMipiSystH.py    | 132 +++
> >  MdePkg/Library/MipiSysTLib/MipiSysTLib.c      | 123 +++
> >  MdePkg/Library/MipiSysTLib/MipiSysTLib.inf    |  55 ++
> >  MdePkg/Library/MipiSysTLib/Platform.c         | 164 ++++
> >  MdePkg/Library/MipiSysTLib/Platform.h         | 138 +++
> >  MdePkg/Library/MipiSysTLib/Readme.md          |  25 +
> >  MdePkg/Library/MipiSysTLib/mipi_syst.h        | 789 ++++++++++++++++++
> >  MdePkg/Library/MipiSysTLib/mipisyst           |   1 +
> >  .../TraceHubDebugSysTLibNull.c                |  76 ++
> >  .../TraceHubDebugSysTLibNull.inf              |  29 +
> >  MdePkg/MdePkg.ci.yaml                         |  15 +-
> >  MdePkg/MdePkg.dec                             |   9 +
> >  MdePkg/MdePkg.dsc                             |   2 +
> >  ReadMe.rst                                    |   1 +
> >  34 files changed, 3212 insertions(+), 3 deletions(-)  create mode
> > 100644 MdeModulePkg/Include/Guid/TraceHubDebugInfoHob.h
> >  create mode 100644
> >
> MdeModulePkg/Library/TraceHubDebugSysTLib/BaseTraceHubDebugSysTLib.c
> >  create mode 100644
> >
> MdeModulePkg/Library/TraceHubDebugSysTLib/BaseTraceHubDebugSysTLib.i
> > nf
> >  create mode 100644
> >
> MdeModulePkg/Library/TraceHubDebugSysTLib/DxeSmmTraceHubDebugSysTL
> > ib.c
> >  create mode 100644
> >
> MdeModulePkg/Library/TraceHubDebugSysTLib/DxeSmmTraceHubDebugSysTL
> > ib.inf
> >  create mode 100644
> > MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApi.c
> >  create mode 100644
> > MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApi.h
> >  create mode 100644
> >
> MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApiCommon
> > .c
> >  create mode 100644
> >
> MdeModulePkg/Library/TraceHubDebugSysTLib/InternalTraceHubApiCommon
> > .h
> >  create mode 100644
> > MdeModulePkg/Library/TraceHubDebugSysTLib/PeiTraceHubDebugSysTLib.c
> >  create mode 100644
> >
> MdeModulePkg/Library/TraceHubDebugSysTLib/PeiTraceHubDebugSysTLib.inf
> >  create mode 100644
> > MdeModulePkg/Library/TraceHubDebugSysTLib/Readme.md
> >  create mode 100644 MdePkg/Include/Library/MipiSysTLib.h
> >  create mode 100644 MdePkg/Include/Library/TraceHubDebugSysTLib.h
> >  create mode 100644 MdePkg/Library/MipiSysTLib/GenMipiSystH.py
> >  create mode 100644 MdePkg/Library/MipiSysTLib/MipiSysTLib.c
> >  create mode 100644 MdePkg/Library/MipiSysTLib/MipiSysTLib.inf
> >  create mode 100644 MdePkg/Library/MipiSysTLib/Platform.c
> >  create mode 100644 MdePkg/Library/MipiSysTLib/Platform.h
> >  create mode 100644 MdePkg/Library/MipiSysTLib/Readme.md
> >  create mode 100644 MdePkg/Library/MipiSysTLib/mipi_syst.h
> >  create mode 160000 MdePkg/Library/MipiSysTLib/mipisyst
> >  create mode 100644
> > MdePkg/Library/TraceHubDebugSysTLibNull/TraceHubDebugSysTLibNull.c
> >  create mode 100644
> > MdePkg/Library/TraceHubDebugSysTLibNull/TraceHubDebugSysTLibNull.inf
> >
> > --
> > 2.40.0.windows.1
> >
> >
> >
> > 
> >



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

Patchew 2.1-devel-0870f84

© 2016 - 2024 Red Hat, Inc.