There's some style issues with spaces missing before opening parenthesis. Those 
can be fixed before submission this time.

Reviewed-by: Michael Kubacki <[email protected]>

> -----Original Message-----
> From: Wei, David Y <[email protected]>
> Sent: Monday, September 16, 2019 3:39 PM
> To: [email protected]
> Cc: Wu, Hao A <[email protected]>; Gao, Liming <[email protected]>;
> Sinha, Ankit <[email protected]>; Agyeman, Prince
> <[email protected]>; Kubacki, Michael A
> <[email protected]>; Desimone, Nathaniel L
> <[email protected]>; Kinney, Michael D
> <[email protected]>
> Subject: [edk2-platforms PATCH v2] SimicsOpenBoardPkg: Always set the
> boot priority by default
> 
> When running networks of multiple machines, Simics assigns different disk
> IDs to each disk created.this change the boot priority and can't boot from
> SATA HDD directly.
> Clear boot priority in NVRAM can fix this issue.
> 
> Cc: Hao Wu <[email protected]>
> Cc: Liming Gao <[email protected]>
> Cc: Ankit Sinha <[email protected]>
> Cc: Agyeman Prince <[email protected]>
> Cc: Kubacki Michael A <[email protected]>
> Cc: Nate DeSimone <[email protected]>
> Cc: Michael D Kinney <[email protected]>
> 
> Signed-off-by: David Wei <[email protected]>
> ---
>  .../Library/PlatformBootManagerLib/BdsPlatform.c           | 14 +++++++++++-
> --
>  1 file changed, 11 insertions(+), 3 deletions(-)
> 
> diff --git
> a/Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/Bds
> Platform.c
> b/Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/Bd
> sPlatform.c
> index 953a4a6c15..6644ce124e 100644
> ---
> a/Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/Bds
> Platform.c
> +++
> b/Platform/Intel/SimicsOpenBoardPkg/Library/PlatformBootManagerLib/B
> +++ dsPlatform.c
> @@ -338,10 +338,18 @@ PlatformBootManagerBeforeConsole (
>    VOID
>    )
>  {
> -//  EFI_HANDLE    Handle;
> -//  EFI_STATUS    Status;
> +  EFI_BOOT_MANAGER_LOAD_OPTION  *NvBootOptions;
> +  UINTN                         NvBootOptionCount;
> +  UINTN                         Index;
> +  EFI_STATUS    Status;
> +
> +  DEBUG((DEBUG_INFO, "PlatformBootManagerBeforeConsole\n"));
> +
> +  NvBootOptions = EfiBootManagerGetLoadOptions(&NvBootOptionCount,
> + LoadOptionTypeBoot);  for (Index = 0; Index < NvBootOptionCount;
> Index++) {
> +    Status =
> +
> EfiBootManagerDeleteLoadOptionVariable(NvBootOptions[Index].OptionNu
> mb
> + er, LoadOptionTypeBoot);  }
> 
> -  DEBUG ((EFI_D_INFO, "PlatformBootManagerBeforeConsole\n"));
>    InstallDevicePathCallback ();
> 
>    VisitAllInstancesOfProtocol (&gEfiPciRootBridgeIoProtocolGuid,
> --
> 2.16.2.windows.1


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#47549): https://edk2.groups.io/g/devel/message/47549
Mute This Topic: https://groups.io/mt/34170508/21656
Group Owner: [email protected]
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to