[v1] dp command without ACPI

[edk2-devel] [PATCH 0/2] dp command without ACPI

Posted by Jeff Brasen via groups.io 10 months, 1 week ago

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.

-Jeff

Jeff Brasen (2):
  MdeModulePkg/DxeCorePerformanceLib: Install BPDT in config table
  ShellPkg/Dp: Allow dp command to work without ACPI

 .../DxeCorePerformanceLib/DxeCorePerformanceLib.c     |  2 ++
 ShellPkg/DynamicCommand/DpDynamicCommand/Dp.c         | 11 ++++++++---
 ShellPkg/DynamicCommand/DpDynamicCommand/DpApp.inf    |  1 +
 .../DpDynamicCommand/DpDynamicCommand.inf             |  1 +
 4 files changed, 12 insertions(+), 3 deletions(-)

-- 
2.25.1



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

[edk2-devel] 回复: [PATCH 0/2] dp command without ACPI

Posted by gaoliming via groups.io 10 months, 1 week ago

Jeff:
  Performance has been stored as FPDT table. It can be directly mapped to
the configuration table. So, this solution is good to me. Reviewed-by:
Liming Gao <gaoliming@byosoft.com.cn>

Thanks
Liming
> -----邮件原件-----
> 发件人: Jeff Brasen <jbrasen@nvidia.com>
> 发送时间: 2023年6月30日 5:09
> 收件人: devel@edk2.groups.io
> 抄送: jian.j.wang@intel.com; gaoliming@byosoft.com.cn;
> dandan.bi@intel.com; zhichao.gao@intel.com; Jeff Brasen
> <jbrasen@nvidia.com>
> 主题: [PATCH 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.
> 
> -Jeff
> 
> Jeff Brasen (2):
>   MdeModulePkg/DxeCorePerformanceLib: Install BPDT in config table
>   ShellPkg/Dp: Allow dp command to work without ACPI
> 
>  .../DxeCorePerformanceLib/DxeCorePerformanceLib.c     |  2 ++
>  ShellPkg/DynamicCommand/DpDynamicCommand/Dp.c         | 11
> ++++++++---
>  ShellPkg/DynamicCommand/DpDynamicCommand/DpApp.inf    |  1 +
>  .../DpDynamicCommand/DpDynamicCommand.inf             |  1 +
>  4 files changed, 12 insertions(+), 3 deletions(-)
> 
> --
> 2.25.1





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