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

Michael D Kinney michael.d.kinney at intel.com
Mon May 8 05:21:52 UTC 2023


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 at intel.com>
> Sent: Sunday, May 7, 2023 10:19 PM
> To: Kinney, Michael D <michael.d.kinney at intel.com>; Guo, Gua
> <gua.guo at intel.com>; devel at edk2.groups.io
> Cc: Chiu, Chasel <chasel.chiu at intel.com>; Prakashan, Krishnadas
> Veliyathuparambil <krishnadas.veliyathuparambil.prakashan at intel.com>; K
> N, Karthik <karthik.k.n at intel.com>; Chan, Laura <laura.chan at 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 at intel.com>
> Sent: Monday, May 8, 2023 8:45 AM
> To: Guo, Gua <gua.guo at intel.com>; devel at edk2.groups.io; Hsu, VictorX
> <victorx.hsu at intel.com>
> Cc: Chiu, Chasel <chasel.chiu at intel.com>; Prakashan, Krishnadas
> Veliyathuparambil <krishnadas.veliyathuparambil.prakashan at intel.com>; K
> N, Karthik <karthik.k.n at intel.com>; Chan, Laura <laura.chan at intel.com>;
> Kinney, Michael D <michael.d.kinney at 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 at intel.com>
> > Sent: Thursday, April 20, 2023 7:33 PM
> > To: devel at edk2.groups.io; Hsu, VictorX <victorx.hsu at intel.com>
> > Cc: Chiu, Chasel <chasel.chiu at intel.com>; Kinney, Michael D
> > <michael.d.kinney at intel.com>; Prakashan, Krishnadas Veliyathuparambil
> > <krishnadas.veliyathuparambil.prakashan at intel.com>; K N, Karthik
> > <karthik.k.n at intel.com>; Chan, Laura <laura.chan at 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 at edk2.groups.io <devel at edk2.groups.io> On Behalf Of
> > victorx.hsu at intel.com
> > Sent: Tuesday, April 18, 2023 7:05 PM
> > To: devel at edk2.groups.io
> > Cc: Hsu, VictorX <victorx.hsu at intel.com>
> > Subject: [edk2-devel] [PATCH 0/4] Trace Hub debug library support
> >
> > From: VictorX Hsu <victorx.hsu at 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/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/leave/3943202/1813853/130120423/xyzzy [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-




More information about the edk2-devel-archive mailing list