With update, Reviewed-by: Star Zeng <[email protected]>.

-----Original Message-----
From: Gao, Liming 
Sent: Monday, June 4, 2018 2:39 PM
To: Zeng, Star <[email protected]>; [email protected]
Cc: Ni, Ruiyu <[email protected]>
Subject: RE: [edk2] [Patch] MdeModulePkg BootManagerMenuApp: Update usage info 
for BootLogo protocol

Good catch. I will update the commit message.

>-----Original Message-----
>From: Zeng, Star
>Sent: Monday, June 04, 2018 2:37 PM
>To: Gao, Liming <[email protected]>; [email protected]
>Cc: Ni, Ruiyu <[email protected]>; Zeng, Star <[email protected]>
>Subject: RE: [edk2] [Patch] MdeModulePkg BootManagerMenuApp: Update 
>usage info for BootLogo protocol
>
>Liming,
>
>Should " If it is installed " be " If it is not installed " in the commit 
>message?
>
>
>Thanks,
>Star
>-----Original Message-----
>From: edk2-devel [mailto:[email protected]] On Behalf Of 
>Liming Gao
>Sent: Tuesday, May 29, 2018 1:54 PM
>To: [email protected]
>Cc: Ni, Ruiyu <[email protected]>
>Subject: [edk2] [Patch] MdeModulePkg BootManagerMenuApp: Update usage 
>info for BootLogo protocol
>
>BootLogo protocol is not always required. If it is installed, 
>BootManagerMenuApp can work.
>
>Contributed-under: TianoCore Contribution Agreement 1.1
>Signed-off-by: Liming Gao <[email protected]>
>Cc: Ruiyu Ni <[email protected]>
>---
>
>MdeModulePkg/Application/BootManagerMenuApp/BootManagerMenuAp
>p.inf | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git
>a/MdeModulePkg/Application/BootManagerMenuApp/BootManagerMenuA
>pp.inf
>b/MdeModulePkg/Application/BootManagerMenuApp/BootManagerMenuA
>pp.inf
>index dd60ef4..392a970 100644
>---
>a/MdeModulePkg/Application/BootManagerMenuApp/BootManagerMenuA
>pp.inf
>+++
>b/MdeModulePkg/Application/BootManagerMenuApp/BootManagerMenuA
>pp.inf
>@@ -4,7 +4,7 @@
> #  The application pops up a menu showing all the boot options 
>referenced by #  BootOrder NV variable and user can choose to boot from one of 
>them.
> #
>-#  Copyright (c) 2011 - 2015, 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 @@ -51,7 +51,7 @@  [Guids]
>
> [Protocols]
>-  gEfiBootLogoProtocolGuid                      ## CONSUMES
>+  gEfiBootLogoProtocolGuid                      ## SOMETIMES_CONSUMES
>   gEfiLoadedImageDevicePathProtocolGuid         ## CONSUMES
>
> [Pcd]
>--
>2.8.0.windows.1
>
>_______________________________________________
>edk2-devel mailing list
>[email protected]
>https://lists.01.org/mailman/listinfo/edk2-devel
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to