Re: [PATCH 0/7] Make the event empty function in UefiLib public

["Zeng, Star" <star.zeng@intel.com>]

Hi,

Please kindly help take a review to the patches. :)

So far, I have got the RB from Liming and Ard to MdePkg and ArmPlatformPkg.

Thanks,
Star
-----Original Message-----
From: Zeng, Star 
Sent: Tuesday, January 17, 2017 3:09 PM
To: edk2-devel@lists.01.org
Cc: Zeng, Star <star.zeng@intel.com>; Gao, Liming <liming.gao@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>; Fan, Jeff <jeff.fan@intel.com>; Tian, Feng <feng.tian@intel.com>; Ni, Ruiyu <ruiyu.ni@intel.com>; Ma, Maurice <maurice.ma@intel.com>; Agyeman, Prince <prince.agyeman@intel.com>; Steele, Kelly <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

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