From: "Guo Dong" <guo.dong@intel.com>
To: "Liu, Zhiguang" <zhiguang.liu@intel.com>,
"devel@edk2.groups.io" <devel@edk2.groups.io>
Cc: "Ma, Maurice" <maurice.ma@intel.com>,
"You, Benjamin" <benjamin.you@intel.com>
Subject: Re: [PATCH] UefiPayloadPkg: Get platform specific logic from protocol for BDS driver
Date: Wed, 2 Jun 2021 05:07:18 +0000 [thread overview]
Message-ID: <BYAPR11MB362278AC1439CDCFDD98A4849E3D9@BYAPR11MB3622.namprd11.prod.outlook.com> (raw)
In-Reply-To: <20210526085244.1992-1-zhiguang.liu@intel.com>
It would be better if having function comments in the PlatformBootManagerOverride.h.
Reviewed-by: Guo Dong <guo.dong@intel.com>
> -----Original Message-----
> From: Liu, Zhiguang <zhiguang.liu@intel.com>
> Sent: Wednesday, May 26, 2021 1:53 AM
> To: devel@edk2.groups.io
> Cc: Ma, Maurice <maurice.ma@intel.com>; Dong, Guo
> <guo.dong@intel.com>; You, Benjamin <benjamin.you@intel.com>
> Subject: [PATCH] UefiPayloadPkg: Get platform specific logic from protocol
> for BDS driver
>
> Currently, BDS driver will link a PlatformBootManagerLib, which contains
> platform
> sepcific logic. This patch get the platform specific logic from a protocol, so that
> platform logic for Boot manager can be in another binary.
>
> Cc: Maurice Ma <maurice.ma@intel.com>
> Cc: Guo Dong <guo.dong@intel.com>
> Cc: Benjamin You <benjamin.you@intel.com>
>
> Signed-off-by: Zhiguang Liu <zhiguang.liu@intel.com>
> ---
> UefiPayloadPkg/Include/Protocol/PlatformBootManagerOverride.h |
> 44 ++++++++++++++++++++++++++++++++++++++++++++
> UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> | 25 ++++++++++++++++++++++++-
>
> UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.i
> nf | 3 ++-
> UefiPayloadPkg/UefiPayloadPkg.dec | 4 +++-
> 4 files changed, 73 insertions(+), 3 deletions(-)
>
> diff --git
> a/UefiPayloadPkg/Include/Protocol/PlatformBootManagerOverride.h
> b/UefiPayloadPkg/Include/Protocol/PlatformBootManagerOverride.h
> new file mode 100644
> index 0000000000..0641cc4218
> --- /dev/null
> +++ b/UefiPayloadPkg/Include/Protocol/PlatformBootManagerOverride.h
> @@ -0,0 +1,44 @@
> +/** @file
>
> + This file defines the Univeral Payload Platform BootManager Protocol.
>
> +
>
> + Copyright (c) 2021, Intel Corporation. All rights reserved.<BR>
>
> + SPDX-License-Identifier: BSD-2-Clause-Patent
>
> +**/
>
> +
>
> +#ifndef __PLATFORM_BOOT_MANAGER_OVERRIDE_H__
>
> +#define __PLATFORM_BOOT_MANAGER_OVERRIDE_H__
>
> +
>
> +typedef
>
> +VOID
>
> +(EFIAPI *PLD_PLATFORM_BOOT_MANAGER_OVERRIDE_BEFORE_CONSOLE)
> (
>
> + VOID
>
> + );
>
> +
>
> +typedef
>
> +VOID
>
> +(EFIAPI *PLD_PLATFORM_BOOT_MANAGER_OVERRIDE_AFTER_CONSOLE)
> (
>
> + VOID
>
> + );
>
> +
>
> +typedef
>
> +VOID
>
> +(EFIAPI *PLD_PLATFORM_BOOT_MANAGER_OVERRIDE_WAIT_CALLBACK)
> (
>
> + UINT16 TimeoutRemain
>
> + );
>
> +
>
> +typedef
>
> +VOID
>
> +(EFIAPI
> *PLD_PLATFORM_BOOT_MANAGER_OVERRIDE_UNABLE_TO_BOOT) (
>
> + VOID
>
> + );
>
> +
>
> +typedef struct {
>
> + PLD_PLATFORM_BOOT_MANAGER_OVERRIDE_BEFORE_CONSOLE
> BeforeConsole;
>
> + PLD_PLATFORM_BOOT_MANAGER_OVERRIDE_AFTER_CONSOLE
> AfterConsole;
>
> + PLD_PLATFORM_BOOT_MANAGER_OVERRIDE_WAIT_CALLBACK
> WaitCallback;
>
> + PLD_PLATFORM_BOOT_MANAGER_OVERRIDE_UNABLE_TO_BOOT
> UnableToBoot;
>
> +} PLD_PLATFORM_BOOT_MANAGER_OVERRIDE_PROTOCOL;
>
> +
>
> +extern GUID gPldPlatformBootManagerOverrideProtocolGuid;
>
> +
>
> +#endif
>
> diff --git
> a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> index 7fa3a048b7..525977c15e 100644
> ---
> a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> +++
> b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> @@ -2,13 +2,16 @@
> This file include all platform action which can be customized
>
> by IBV/OEM.
>
>
>
> -Copyright (c) 2015 - 2018, Intel Corporation. All rights reserved.<BR>
>
> +Copyright (c) 2015 - 2021, Intel Corporation. All rights reserved.<BR>
>
> SPDX-License-Identifier: BSD-2-Clause-Patent
>
>
>
> **/
>
>
>
> #include "PlatformBootManager.h"
>
> #include "PlatformConsole.h"
>
> +#include <Protocol/PlatformBootManagerOverride.h>
>
> +
>
> +PLD_PLATFORM_BOOT_MANAGER_OVERRIDE_PROTOCOL
> *mPldPlatformBootManagerOverrideInstance = NULL;
>
>
>
> VOID
>
> InstallReadyToLock (
>
> @@ -156,6 +159,16 @@ PlatformBootManagerBeforeConsole (
> EFI_INPUT_KEY F2;
>
> EFI_INPUT_KEY Down;
>
> EFI_BOOT_MANAGER_LOAD_OPTION BootOption;
>
> + EFI_STATUS Status;
>
> +
>
> + Status = gBS->LocateProtocol
> (&gPldPlatformBootManagerOverrideProtocolGuid, NULL, (VOID **)
> &mPldPlatformBootManagerOverrideInstance);
>
> + if (EFI_ERROR (Status)) {
>
> + mPldPlatformBootManagerOverrideInstance = NULL;
>
> + }
>
> + if (mPldPlatformBootManagerOverrideInstance != NULL){
>
> + mPldPlatformBootManagerOverrideInstance->BeforeConsole();
>
> + return;
>
> + }
>
>
>
> //
>
> // Register ENTER as CONTINUE key
>
> @@ -213,6 +226,10 @@ PlatformBootManagerAfterConsole (
> EFI_GRAPHICS_OUTPUT_BLT_PIXEL Black;
>
> EFI_GRAPHICS_OUTPUT_BLT_PIXEL White;
>
>
>
> + if (mPldPlatformBootManagerOverrideInstance != NULL){
>
> + mPldPlatformBootManagerOverrideInstance->AfterConsole();
>
> + return;
>
> + }
>
> Black.Blue = Black.Green = Black.Red = Black.Reserved = 0;
>
> White.Blue = White.Green = White.Red = White.Reserved = 0xFF;
>
>
>
> @@ -244,6 +261,9 @@ PlatformBootManagerWaitCallback (
> UINT16 TimeoutRemain
>
> )
>
> {
>
> + if (mPldPlatformBootManagerOverrideInstance != NULL){
>
> + mPldPlatformBootManagerOverrideInstance->WaitCallback
> (TimeoutRemain);
>
> + }
>
> return;
>
> }
>
>
>
> @@ -260,6 +280,9 @@ PlatformBootManagerUnableToBoot (
> VOID
>
> )
>
> {
>
> + if (mPldPlatformBootManagerOverrideInstance != NULL){
>
> + mPldPlatformBootManagerOverrideInstance->UnableToBoot();
>
> + }
>
> return;
>
> }
>
>
>
> diff --git
> a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLi
> b.inf
> b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLi
> b.inf
> index 1f5a0bcad0..14997c1183 100644
> ---
> a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLi
> b.inf
> +++
> b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLi
> b.inf
> @@ -1,7 +1,7 @@
> ## @file
>
> # Include all platform action which can be customized by IBV/OEM.
>
> #
>
> -# Copyright (c) 2012 - 2016, Intel Corporation. All rights reserved.<BR>
>
> +# Copyright (c) 2012 - 2021, Intel Corporation. All rights reserved.<BR>
>
> # SPDX-License-Identifier: BSD-2-Clause-Patent
>
> #
>
> ##
>
> @@ -57,6 +57,7 @@
> gEfiBootLogoProtocolGuid ## CONSUMES
>
> gEfiDxeSmmReadyToLockProtocolGuid
>
> gEfiSmmAccess2ProtocolGuid
>
> + gPldPlatformBootManagerOverrideProtocolGuid
>
>
>
> [Pcd]
>
> gEfiMdePkgTokenSpaceGuid.PcdPlatformBootTimeOut
>
> diff --git a/UefiPayloadPkg/UefiPayloadPkg.dec
> b/UefiPayloadPkg/UefiPayloadPkg.dec
> index 99cb3311a6..d9c5775e76 100644
> --- a/UefiPayloadPkg/UefiPayloadPkg.dec
> +++ b/UefiPayloadPkg/UefiPayloadPkg.dec
> @@ -3,7 +3,7 @@
> #
>
> # Provides drivers and definitions to create uefi payload for bootloaders.
>
> #
>
> -# Copyright (c) 2014 - 2020, Intel Corporation. All rights reserved.<BR>
>
> +# Copyright (c) 2014 - 2021, Intel Corporation. All rights reserved.<BR>
>
> # SPDX-License-Identifier: BSD-2-Clause-Patent
>
> #
>
> ##
>
> @@ -43,6 +43,8 @@
> #
>
> gPlatformGOPPolicyGuid = { 0xec2e931b, 0x3281, 0x48a5, { 0x81,
> 0x07, 0xdf, 0x8a, 0x8b, 0xed, 0x3c, 0x5d } }
>
>
>
> + gPldPlatformBootManagerOverrideProtocolGuid = { 0xdb3fc2df, 0x7376,
> 0x4a8d, { 0x82, 0xab, 0x91, 0x54, 0xa1, 0x36, 0xa6, 0x5a } }
>
> +
>
>
> ##########################################################
> ######################
>
> #
>
> # PCD Declarations section - list of all PCDs Declared by this Package
>
> --
> 2.30.0.windows.2
prev parent reply other threads:[~2021-06-02 5:07 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-05-26 8:52 [PATCH] UefiPayloadPkg: Get platform specific logic from protocol for BDS driver Zhiguang Liu
2021-06-02 5:07 ` Guo Dong [this message]
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=BYAPR11MB362278AC1439CDCFDD98A4849E3D9@BYAPR11MB3622.namprd11.prod.outlook.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