From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id BBFA981E9E for ; Fri, 20 Jan 2017 00:04:17 -0800 (PST) Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by orsmga105.jf.intel.com with ESMTP; 20 Jan 2017 00:04:17 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,257,1477983600"; d="scan'208";a="215592045" Received: from fmsmsx106.amr.corp.intel.com ([10.18.124.204]) by fmsmga004.fm.intel.com with ESMTP; 20 Jan 2017 00:04:17 -0800 Received: from fmsmsx155.amr.corp.intel.com (10.18.116.71) by FMSMSX106.amr.corp.intel.com (10.18.124.204) with Microsoft SMTP Server (TLS) id 14.3.248.2; Fri, 20 Jan 2017 00:04:17 -0800 Received: from shsmsx104.ccr.corp.intel.com (10.239.4.70) by FMSMSX155.amr.corp.intel.com (10.18.116.71) with Microsoft SMTP Server (TLS) id 14.3.248.2; Fri, 20 Jan 2017 00:04:16 -0800 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.88]) by SHSMSX104.ccr.corp.intel.com ([10.239.4.70]) with mapi id 14.03.0248.002; Fri, 20 Jan 2017 16:04:14 +0800 From: "Zeng, Star" To: Laszlo Ersek , "edk2-devel@lists.01.org" , "Kinney, Michael D" , "Steele, Kelly" CC: "Gao, Liming" , Ard Biesheuvel , "Fan, Jeff" , "Tian, Feng" , "Ni, Ruiyu" , "Ma, Maurice" , "Agyeman, Prince" , "Leif Lindholm" , "Zeng, Star" Thread-Topic: [edk2] [PATCH 0/7] Make the event empty function in UefiLib public Thread-Index: AQHScJCUfVTqwqc38EmF+BzSqozDFqE/m5JA//97PwCAAe4uYA== Date: Fri, 20 Jan 2017 08:04:14 +0000 Message-ID: <0C09AFA07DD0434D9E2A0C6AEB0483103B818836@shsmsx102.ccr.corp.intel.com> References: <1484636920-180432-1-git-send-email-star.zeng@intel.com> <0C09AFA07DD0434D9E2A0C6AEB0483103B818420@shsmsx102.ccr.corp.intel.com> <03db0f4d-610f-7cbf-76d1-a85938833c3d@redhat.com> In-Reply-To: <03db0f4d-610f-7cbf-76d1-a85938833c3d@redhat.com> Accept-Language: zh-CN, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: [PATCH 0/7] Make the event empty function in UefiLib public X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 20 Jan 2017 08:04:17 -0000 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Hi Laszlo, Np, just Cc you in case you have interest to it since you created the usefu= l EfiEventGroupSignal(). :) Hi All, I have pushed the patches at https://github.com/tianocore/edk2/compare/058c= d03b5857...9c8fe63c15d6 for MdePkg, IntelFrameworkPkg, MdeModulePkg, IntelF= rameworkModulePkg, CorebootPayloadPkg and ArmPlatformPkg with RB received. And I will keep QuarkPlatformPkg as is before I get any RB. Thanks, Star -----Original Message----- From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Lasz= lo Ersek Sent: Thursday, January 19, 2017 6:30 PM To: Zeng, Star ; edk2-devel@lists.01.org ; Kinney, Michael D ; Fan, Jeff ; Tian, Feng ; Ni, Ruiyu ; Ma, Maurice ; Agyeman, Prince ; Steele, Kelly ; Leif Lindholm Cc: Gao, Liming ; Ard Biesheuvel Subject: Re: [edk2] [PATCH 0/7] Make the event empty function in UefiLib pu= blic Hi Star, On 01/19/17 11:27, Zeng, Star wrote: > Hi, >=20 > Please kindly help take a review to the patches. :) >=20 > So far, I have got the RB from Liming and Ard to MdePkg and ArmPlatformPk= g. I didn't ignore the series, it's just that I don't co-maintain any of the p= ackages being modified. Thanks Laszlo > -----Original Message----- > From: Zeng, Star > Sent: Tuesday, January 17, 2017 3:09 PM > To: edk2-devel@lists.01.org > Cc: Zeng, Star ; Gao, Liming=20 > ; Kinney, Michael D=20 > ; Fan, Jeff ; Tian,=20 > Feng ; Ni, Ruiyu ; Ma,=20 > Maurice ; Agyeman, Prince=20 > ; Steele, Kelly ;=20 > Leif Lindholm ; Ard Biesheuvel=20 > ; Laszlo Ersek > Subject: [PATCH 0/7] Make the event empty function in UefiLib public >=20 > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D298 >=20 > We found there are many event empty functions in EDK2 code base, these pa= tches are to make the event empty function in UefiLib public, then this eve= nt empty function can be used to remove the duplication in drivers and othe= r libraries. >=20 > In CorebootPayloadPkg/QuarkPlatformPkg/ArmPlatformPkg, EfiEventGroupSigna= l from UefiLib can be used to remove the duplication of event empty functio= n. >=20 > Cc: Liming Gao > Cc: Michael Kinney > Cc: Jeff Fan > Cc: Feng Tian > Cc: Ruiyu Ni > Cc: Maurice Ma > Cc: Prince Agyeman > Cc: Kelly Steele > Cc: Leif Lindholm > Cc: Ard Biesheuvel > Cc: Laszlo Ersek >=20 > 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 >=20 > .../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 ++---------- .../Unive= rsal/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(-) >=20 > -- > 2.7.0.windows.1 >=20 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel