public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: Dandan Bi <dandan.bi@intel.com>
To: edk2-devel@lists.01.org
Cc: Eric Dong <eric.dong@intel.com>, Liming Gao <liming.gao@intel.com>
Subject: [patch 1/3] MdeModulePkg/UiApp: Signal event when enter/exit setup menu
Date: Thu, 12 Apr 2018 09:31:36 +0800	[thread overview]
Message-ID: <20180412013138.113904-2-dandan.bi@intel.com> (raw)
In-Reply-To: <20180412013138.113904-1-dandan.bi@intel.com>

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>
---
 MdeModulePkg/Application/UiApp/FrontPage.c                    | 4 +++-
 MdeModulePkg/Application/UiApp/FrontPageCustomizedUiSupport.c | 3 ++-
 MdeModulePkg/Application/UiApp/UiApp.inf                      | 4 +++-
 MdeModulePkg/MdeModulePkg.dec                                 | 6 ++++++
 4 files changed, 14 insertions(+), 3 deletions(-)

diff --git a/MdeModulePkg/Application/UiApp/FrontPage.c b/MdeModulePkg/Application/UiApp/FrontPage.c
index adee67a8ac1..e5d2cb00c62 100644
--- a/MdeModulePkg/Application/UiApp/FrontPage.c
+++ b/MdeModulePkg/Application/UiApp/FrontPage.c
@@ -1,9 +1,9 @@
 /** @file
   FrontPage routines to handle the callbacks and browser calls
 
-Copyright (c) 2004 - 2017, Intel Corporation. All rights reserved.<BR>
+Copyright (c) 2004 - 2018, Intel Corporation. All rights reserved.<BR>
 This program and the accompanying materials
 are licensed and made available under the terms and conditions of the BSD License
 which accompanies this distribution.  The full text of the license may be found at
 http://opensource.org/licenses/bsd-license.php
 
@@ -992,10 +992,11 @@ UiEntry (
   REPORT_STATUS_CODE (
     EFI_PROGRESS_CODE,
     (EFI_SOFTWARE_DXE_BS_DRIVER | EFI_SW_PC_USER_SETUP)
     );
 
+  EfiEventGroupSignal(&gEdkiiSetupEnterGuid);
   //
   // Indicate if the connect all has been performed before.
   // If has not been performed before, do here.
   //
   if (!ConnectAllHappened) {
@@ -1024,10 +1025,11 @@ UiEntry (
   if (mLanguageString != NULL) {
     FreePool (mLanguageString);
     mLanguageString = NULL;
   }
 
+  EfiEventGroupSignal(&gEdkiiSetupExitGuid);
   //
   //Will leave browser, check any reset required change is applied? if yes, reset system
   //
   SetupResetReminder ();
 }
diff --git a/MdeModulePkg/Application/UiApp/FrontPageCustomizedUiSupport.c b/MdeModulePkg/Application/UiApp/FrontPageCustomizedUiSupport.c
index 17fc3db507d..2c5f9d24204 100644
--- a/MdeModulePkg/Application/UiApp/FrontPageCustomizedUiSupport.c
+++ b/MdeModulePkg/Application/UiApp/FrontPageCustomizedUiSupport.c
@@ -1,10 +1,10 @@
 /** @file
 
   This library class defines a set of interfaces to customize Ui module
 
-Copyright (c) 2016, Intel Corporation. All rights reserved.<BR>
+Copyright (c) 2016 - 2018, Intel Corporation. All rights reserved.<BR>
 This program and the accompanying materials are licensed and made available under
 the terms and conditions of the BSD License that accompanies this distribution.
 The full text of the license may be found at
 http://opensource.org/licenses/bsd-license.php.
 
@@ -232,10 +232,11 @@ UiSupportLibCallbackHandler (
 
     case FRONT_PAGE_KEY_RESET:
       //
       // Reset
       //
+      EfiEventGroupSignal(&gEdkiiSetupExitGuid);
       gRT->ResetSystem (EfiResetCold, EFI_SUCCESS, 0, NULL);
       *Status = EFI_UNSUPPORTED;
 
     default:
       break;
diff --git a/MdeModulePkg/Application/UiApp/UiApp.inf b/MdeModulePkg/Application/UiApp/UiApp.inf
index d144462ce46..c212387e5be 100644
--- a/MdeModulePkg/Application/UiApp/UiApp.inf
+++ b/MdeModulePkg/Application/UiApp/UiApp.inf
@@ -1,9 +1,9 @@
 ## @file
 #  UiApp module is driver for BDS phase.
 #
-#  Copyright (c) 2011 - 2016, Intel Corporation. All rights reserved.<BR>
+#  Copyright (c) 2011 - 2018, Intel Corporation. All rights reserved.<BR>
 #  This program and the accompanying materials
 #  are licensed and made available under the terms and conditions of the BSD License
 #  which accompanies this distribution.  The full text of the license may be found at
 #  http://opensource.org/licenses/bsd-license.php
 #  
@@ -62,10 +62,12 @@
   UefiBootManagerLib
   
 [Guids]
   gEfiIfrTianoGuid                              ## CONSUMES ## GUID (Extended IFR Guid Opcode)
   gEfiIfrFrontPageGuid                          ## CONSUMES ## GUID
+  gEdkiiSetupEnterGuid                          ## CONSUMES ## GUID
+  gEdkiiSetupExitGuid                           ## CONSUMES ## GUID
 
 [Protocols]
   gEfiSmbiosProtocolGuid                        ## CONSUMES
   gEfiHiiConfigAccessProtocolGuid               ## CONSUMES
 
diff --git a/MdeModulePkg/MdeModulePkg.dec b/MdeModulePkg/MdeModulePkg.dec
index cc397185f7b..e0bdf5ecdf3 100644
--- a/MdeModulePkg/MdeModulePkg.dec
+++ b/MdeModulePkg/MdeModulePkg.dec
@@ -407,10 +407,16 @@
   gEdkiiEndOfS3ResumeGuid = { 0x96f5296d, 0x05f7, 0x4f3c, {0x84, 0x67, 0xe4, 0x56, 0x89, 0x0e, 0x0c, 0xb5 } }
 
   ## Include/Guid/S3SmmInitDone.h
   gEdkiiS3SmmInitDoneGuid = { 0x8f9d4825, 0x797d, 0x48fc, { 0x84, 0x71, 0x84, 0x50, 0x25, 0x79, 0x2e, 0xf6 } }
 
+  ## Guid used to create/signal the event of entering Setup.
+  gEdkiiSetupEnterGuid  = { 0xcb9f5b38, 0x1729, 0x4bc0, { 0x87, 0xfe, 0xf7, 0x67, 0x92, 0x56, 0xb6, 0xc5 } }
+
+  ## Guid used to create/signal the event of exiting Setup.
+  gEdkiiSetupExitGuid   = { 0x2729f07f, 0x2f8b, 0x44b2, { 0xb8, 0xc1, 0xc9, 0x8e, 0xcf, 0xba, 0x8d, 0x3f } }
+
 [Ppis]
   ## Include/Ppi/AtaController.h
   gPeiAtaControllerPpiGuid       = { 0xa45e60d1, 0xc719, 0x44aa, { 0xb0, 0x7a, 0xaa, 0x77, 0x7f, 0x85, 0x90, 0x6d }}
 
   ## Include/Ppi/UsbHostController.h
-- 
2.14.3.windows.1



  reply	other threads:[~2018-04-12  1:32 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-04-12  1:31 [patch 0/3] MdeModulePkg/UiApp: Signal event when enter/exit setup menu Dandan Bi
2018-04-12  1:31 ` Dandan Bi [this message]
2018-04-12  1:31 ` [patch 2/3] MdeModulePkg/BMMUiLib: " Dandan Bi
2018-04-12  1:31 ` [patch 3/3] MdeModulePkg/BMUiLib: " Dandan Bi
2018-04-12  1:44 ` [patch 0/3] MdeModulePkg/UiApp: " Kinney, Michael D
2018-04-12  3:16   ` Bi, Dandan
2018-04-12  3:46     ` Bi, Dandan
2018-04-12  3:49       ` Tim Lewis
2018-04-12  4:53         ` Bi, Dandan
2018-04-12  5:10           ` Tim Lewis
2018-04-12  6:02       ` Zeng, Star
2018-04-12  6:07         ` Bi, Dandan

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=20180412013138.113904-2-dandan.bi@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