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>,
	Liming Gao <liming.gao@intel.com>,
	Michael Kinney <michael.d.kinney@intel.com>,
	Jeff Fan <jeff.fan@intel.com>, Feng Tian <feng.tian@intel.com>,
	Ruiyu Ni <ruiyu.ni@intel.com>, Maurice Ma <maurice.ma@intel.com>,
	Prince Agyeman <prince.agyeman@intel.com>,
	Kelly Steele <kelly.steele@intel.com>,
	Leif Lindholm <leif.lindholm@linaro.org>,
	Ard Biesheuvel <ard.biesheuvel@linaro.org>,
	Laszlo Ersek <lersek@redhat.com>
Subject: [PATCH 0/7] Make the event empty function in UefiLib public
Date: Tue, 17 Jan 2017 15:08:33 +0800	[thread overview]
Message-ID: <1484636920-180432-1-git-send-email-star.zeng@intel.com> (raw)

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=298

We found there are many event empty functions in EDK2 code base,
these patches are to make the event empty function in UefiLib public,
then this event empty function can be used to remove
the duplication in drivers and other libraries.

In CorebootPayloadPkg/QuarkPlatformPkg/ArmPlatformPkg, EfiEventGroupSignal
from UefiLib can be used to remove the duplication of event empty function.

Cc: Liming Gao <liming.gao@intel.com>
Cc: Michael Kinney <michael.d.kinney@intel.com>
Cc: Jeff Fan <jeff.fan@intel.com>
Cc: Feng Tian <feng.tian@intel.com>
Cc: Ruiyu Ni <ruiyu.ni@intel.com>
Cc: Maurice Ma <maurice.ma@intel.com>
Cc: Prince Agyeman <prince.agyeman@intel.com>
Cc: Kelly Steele <kelly.steele@intel.com>
Cc: Leif Lindholm <leif.lindholm@linaro.org>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Laszlo Ersek <lersek@redhat.com>

Star Zeng (7):
  MdePkg UefiLib: Make the event empty function public
  IntelFrameworkPkg UefiLib: Make the event empty function public
  MdeModulePkg: Use EfiEventEmptyFunction from UefiLib
  IntelFrameworkModulePkg: Use EfiEventEmptyFunction from UefiLib
  CorebootPayloadPkg: Use EfiEventGroupSignal from UefiLib
  QuarkPlatformPkg: Use EfiEventGroupSignal from UefiLib
  ArmPlatformPkg: Use EfiEventGroupSignal from UefiLib

 .../Library/PlatformIntelBdsLib/IntelBdsPlatform.c | 36 ++-----------------
 .../Library/PlatformIntelBdsLib/IntelBdsPlatform.h |  3 +-
 .../PlatformIntelBdsLib/PlatformIntelBdsLib.inf    |  3 +-
 .../PlatformBootManagerLib/PlatformBootManager.c   | 25 ++-----------
 .../Universal/BdsDxe/BdsEntry.c                    | 20 +----------
 .../Library/FrameworkUefiLib/UefiLib.c             | 42 +++++++++++-----------
 .../Library/FrameworkUefiLib/UefiLibInternal.h     | 20 +----------
 .../Library/FrameworkUefiLib/UefiNotTiano.c        |  4 +--
 MdeModulePkg/Core/Dxe/Dispatcher/Dispatcher.c      | 23 ++----------
 MdeModulePkg/Core/Dxe/DxeMain.h                    | 18 +---------
 MdeModulePkg/Core/Dxe/Event/Event.c                |  4 +--
 MdeModulePkg/Library/UefiBootManagerLib/BmHotkey.c | 19 ++--------
 .../Universal/Console/ConSplitterDxe/ConSplitter.c | 22 ++----------
 .../Universal/Console/ConSplitterDxe/ConSplitter.h | 18 +---------
 .../Universal/DriverSampleDxe/DriverSample.c       | 20 ++---------
 .../Universal/SetupBrowserDxe/Presentation.c       | 20 ++---------
 MdePkg/Include/Library/UefiLib.h                   | 18 +++++++++-
 MdePkg/Library/UefiLib/UefiLib.c                   | 42 +++++++++++-----------
 MdePkg/Library/UefiLib/UefiLibInternal.h           | 20 +----------
 MdePkg/Library/UefiLib/UefiNotTiano.c              | 10 +++---
 .../PlatformBootManagerLib/PlatformBootManager.c   | 25 ++-----------
 21 files changed, 91 insertions(+), 321 deletions(-)

-- 
2.7.0.windows.1



             reply	other threads:[~2017-01-17  7:08 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-01-17  7:08 Star Zeng [this message]
2017-01-17  7:08 ` [PATCH 1/7] MdePkg UefiLib: Make the event empty function public Star Zeng
2017-01-18  7:19   ` Gao, Liming
2017-01-17  7:08 ` [PATCH 2/7] IntelFrameworkPkg " Star Zeng
2017-01-20  4:42   ` Fan, Jeff
2017-01-17  7:08 ` [PATCH 3/7] MdeModulePkg: Use EfiEventEmptyFunction from UefiLib Star Zeng
2017-01-17  7:08 ` [PATCH 4/7] IntelFrameworkModulePkg: " Star Zeng
2017-01-20  4:42   ` Fan, Jeff
2017-01-17  7:08 ` [PATCH 5/7] CorebootPayloadPkg: Use EfiEventGroupSignal " Star Zeng
2017-01-20  4:12   ` Ma, Maurice
2017-01-17  7:08 ` [PATCH 6/7] QuarkPlatformPkg: " Star Zeng
2017-01-17  7:08 ` [PATCH 7/7] ArmPlatformPkg: " Star Zeng
2017-01-17  9:33   ` Ard Biesheuvel
2017-01-19 10:27 ` [PATCH 0/7] Make the event empty function in UefiLib public Zeng, Star
2017-01-19 10:30   ` Laszlo Ersek
2017-01-20  0:42     ` Tian, Feng
2017-01-20  8:04     ` 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=1484636920-180432-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