I have reviewed the ShellPkg part. Did you get it reviewed in the MdeModulePkg? 
If yes, you can inform the maintainers of MdeModulePkg to raise the PR and 
merge it.

Thanks,
Zhichao

> -----Original Message-----
> From: Jeff Brasen <jbra...@nvidia.com>
> Sent: Tuesday, September 12, 2023 4:39 AM
> To: devel@edk2.groups.io
> Cc: Wang, Jian J <jian.j.w...@intel.com>; Gao, Liming
> <gaolim...@byosoft.com.cn>; Bi, Dandan <dandan...@intel.com>; Gao,
> Zhichao <zhichao....@intel.com>
> Subject: RE: [PATCH v2 0/2] dp command without ACPI
> 
> 
> Any additional thoughts/feedback on this patch series?
> 
> Thanks,
> Jeff
> 
> 
> > -----Original Message-----
> > From: Jeff Brasen <jbra...@nvidia.com>
> > Sent: Friday, June 30, 2023 11:30 AM
> > To: devel@edk2.groups.io
> > Cc: jian.j.w...@intel.com; gaolim...@byosoft.com.cn;
> > dandan...@intel.com; zhichao....@intel.com; Jeff Brasen
> > <jbra...@nvidia.com>
> > Subject: [PATCH v2 0/2] dp command without ACPI
> >
> > Systems that do not boot with ACPI (system that use device tree for
> > example) can not use the shell dp command. This patch adds this to the
> > configuration table so that dp command can get this without the FPDT table.
> >
> > I am open to other ways for this to be passed if desired (Installed
> > protocol, handler of the status code, etc) but wanted to post this to
> > at least get thoughts on this.
> >
> > Change Log
> > v2 - Fix missing cast for IA32 builds
> >
> > -Jeff
> >
> > Jeff Brasen (2):
> >   MdeModulePkg/DxeCorePerformanceLib: Install BPDT in config table
> >   ShellPkg/Dp: Allow dp command to work without ACPI
> >
> >  ShellPkg/DynamicCommand/DpDynamicCommand/DpApp.inf    |  1 +
> >  .../DpDynamicCommand/DpDynamicCommand.inf             |  1 +
> >  .../DxeCorePerformanceLib/DxeCorePerformanceLib.c     |  2 ++
> >  ShellPkg/DynamicCommand/DpDynamicCommand/Dp.c         | 11
> ++++++++-
> > --
> >  4 files changed, 12 insertions(+), 3 deletions(-)
> >
> > --
> > 2.25.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#108524): https://edk2.groups.io/g/devel/message/108524
Mute This Topic: https://groups.io/mt/99877761/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to