public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: Star Zeng <star.zeng@intel.com>
To: edk2-devel@lists.01.org
Cc: Star Zeng <star.zeng@intel.com>,
	Younas khan <pmdyounaskhan786@gmail.com>,
	Michael D Kinney <michael.d.kinney@intel.com>,
	Liming Gao <liming.gao@intel.com>,
	Jiewen Yao <jiewen.yao@intel.com>,
	Jian J Wang <jian.j.wang@intel.com>,
	Ruiyu Ni <ruiyu.ni@intel.com>, Dandan Bi <dandan.bi@intel.com>,
	Eric Dong <eric.dong@intel.com>, Laszlo Ersek <lersek@redhat.com>
Subject: [PATCH 0/6] Add new EfiFindAcpiTableBySignature() API
Date: Fri, 31 Aug 2018 19:29:13 +0800	[thread overview]
Message-ID: <1535714959-73472-1-git-send-email-star.zeng@intel.com> (raw)

The code for this patch series is also at
git@github.com:lzeng14/edk2.git branch FindAcpiTableBySignature_UefiLib

https://bugzilla.tianocore.org/show_bug.cgi?id=967
Request to add a library function for GetAcpiTable() in order
to get ACPI table using signature as input.

After evaluation, we found there are many duplicated code to
find ACPI table by signature in different modules.

This patch adds new EfiFindAcpiTableBySignature() API in UefiLib
for the request and removing the duplicated code.

Cc: Younas khan <pmdyounaskhan786@gmail.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Liming Gao <liming.gao@intel.com>
Cc: Jiewen Yao <jiewen.yao@intel.com>
Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Ruiyu Ni <ruiyu.ni@intel.com>
Cc: Dandan Bi <dandan.bi@intel.com>
Cc: Eric Dong <eric.dong@intel.com>
Cc: Laszlo Ersek <lersek@redhat.com>

Star Zeng (6):
  MdePkg UefiLib: Add new EfiFindAcpiTableBySignature() API
  IntelSiliconPkg IntelVTdDxe: Use new EfiFindAcpiTableBySignature()
  MdeModulePkg S3SaveStateDxe: Use new EfiFindAcpiTableBySignature()
  PcAtChipsetPkg PcRtc: Use new EfiFindAcpiTableBySignature()
  ShellPkg DpDynamicCommand: Use new EfiFindAcpiTableBySignature()
  UefiCpuPkg PiSmmCpuDxeSmm: Use new EfiFindAcpiTableBySignature()

 .../Feature/VTd/IntelVTdDxe/DmarAcpiTable.c        | 136 +------------
 .../Acpi/S3SaveStateDxe/AcpiS3ContextSave.c        | 208 +------------------
 .../Acpi/S3SaveStateDxe/S3SaveStateDxe.inf         |   3 +-
 MdePkg/Include/Library/UefiLib.h                   |  17 ++
 MdePkg/Library/UefiLib/Acpi.c                      | 226 +++++++++++++++++++++
 MdePkg/Library/UefiLib/UefiLib.inf                 |   3 +
 PcAtChipsetPkg/PcatRealTimeClockRuntimeDxe/PcRtc.c |  80 +-------
 ShellPkg/DynamicCommand/DpDynamicCommand/Dp.c      | 136 +------------
 ShellPkg/DynamicCommand/DpDynamicCommand/Dp.h      |   1 -
 ShellPkg/DynamicCommand/DpDynamicCommand/Dp.uni    |   1 -
 ShellPkg/DynamicCommand/DpDynamicCommand/DpApp.inf |   2 -
 .../DpDynamicCommand/DpDynamicCommand.inf          |   2 -
 UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.inf       |   4 +-
 UefiCpuPkg/PiSmmCpuDxeSmm/SmmProfile.c             |  84 +-------
 UefiCpuPkg/PiSmmCpuDxeSmm/SmmProfileInternal.h     |   3 +-
 15 files changed, 266 insertions(+), 640 deletions(-)
 create mode 100644 MdePkg/Library/UefiLib/Acpi.c

-- 
2.7.0.windows.1



             reply	other threads:[~2018-08-31 11:29 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-08-31 11:29 Star Zeng [this message]
2018-08-31 11:29 ` [PATCH 1/6] MdePkg UefiLib: Add new EfiFindAcpiTableBySignature() API Star Zeng
2018-08-31 11:57   ` Yao, Jiewen
2018-08-31 16:28     ` Ni, Ruiyu
2018-08-31 16:29     ` Ni, Ruiyu
2018-08-31 23:04       ` Yao, Jiewen
2018-09-03  3:26         ` Zeng, Star
2018-09-03  5:09           ` Ni, Ruiyu
2018-09-03  6:11             ` Zeng, Star
2018-09-03  6:14               ` Yao, Jiewen
2018-09-03  6:32                 ` Ni, Ruiyu
2018-09-05 10:02                   ` Zeng, Star
2018-08-31 11:29 ` [PATCH 2/6] IntelSiliconPkg IntelVTdDxe: Use new EfiFindAcpiTableBySignature() Star Zeng
2018-08-31 11:29 ` [PATCH 3/6] MdeModulePkg S3SaveStateDxe: " Star Zeng
2018-08-31 11:29 ` [PATCH 4/6] PcAtChipsetPkg PcRtc: " Star Zeng
2018-08-31 11:29 ` [PATCH 5/6] ShellPkg DpDynamicCommand: " Star Zeng
2018-08-31 11:29 ` [PATCH 6/6] UefiCpuPkg PiSmmCpuDxeSmm: " Star Zeng
2018-08-31 20:33   ` Laszlo Ersek
2018-09-03  3:28     ` Zeng, Star

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1535714959-73472-1-git-send-email-star.zeng@intel.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox