Re: [patch 0/3] MdeModulePkg/UiApp: Signal event when enter/exit setup menu

["Kinney, Michael D" <michael.d.kinney@intel.com>]

Dandan Bi,

Is it possible to move the enter/exit event signaling
into the FormBrowwer SendForm() function?  The current
patch set adds these signal calls in all the places that
SendForm() is called and if one of these is missed, 
a notification will be missed.

Mike

> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-
> bounces@lists.01.org] On Behalf Of Dandan Bi
> Sent: Wednesday, April 11, 2018 6:32 PM
> To: edk2-devel@lists.01.org
> Cc: Dong, Eric <eric.dong@intel.com>; Gao, Liming
> <liming.gao@intel.com>
> Subject: [edk2] [patch 0/3] MdeModulePkg/UiApp: Signal
> event when enter/exit setup menu
> 
> These changes are to support notify callbacks when
> enter/exit
> setup menu, since some driver may need to hook setup
> enter/exit
> points to do something.
> 
> We will signal setup enter/exit events for all setup
> menu
> enter/exit cases.Then the module which pay attention to
> these
> events can execute the callback.
> 
> Cc: Eric Dong <eric.dong@intel.com>
> Cc: Liming Gao <liming.gao@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Dandan Bi <dandan.bi@intel.com>
> 
> Dandan Bi (3):
>   MdeModulePkg/UiApp: Signal event when enter/exit setup
> menu
>   MdeModulePkg/BMMUiLib: Signal event when enter/exit
> setup menu
>   MdeModulePkg/BMUiLib: Signal event when enter/exit
> setup menu
> 
>  MdeModulePkg/Application/UiApp/FrontPage.c
> | 4 +++-
> 
> MdeModulePkg/Application/UiApp/FrontPageCustomizedUiSupp
> ort.c     | 3 ++-
>  MdeModulePkg/Application/UiApp/UiApp.inf
> | 4 +++-
> 
> .../BootMaintenanceManagerUiLib/BootMaintenanceManagerUi
> Lib.inf   | 4 +++-
> 
> MdeModulePkg/Library/BootMaintenanceManagerUiLib/BootOpt
> ion.c     | 8 +++++++-
>  MdeModulePkg/Library/BootManagerUiLib/BootManager.c
> | 7 ++++++-
> 
> MdeModulePkg/Library/BootManagerUiLib/BootManagerUiLib.i
> nf        | 4 +++-
>  MdeModulePkg/MdeModulePkg.dec
> | 6 ++++++
>  8 files changed, 33 insertions(+), 7 deletions(-)
> 
> --
> 2.14.3.windows.1
> 
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel