There's some style issues with spaces missing before opening parenthesis. Those can be fixed before submission this time.
Reviewed-by: Michael Kubacki <michael.a.kuba...@intel.com> > -----Original Message----- > From: Wei, David Y <david.y....@intel.com> > Sent: Monday, September 16, 2019 3:39 PM > To: devel@edk2.groups.io > Cc: Wu, Hao A <hao.a...@intel.com>; Gao, Liming <liming....@intel.com>; > Sinha, Ankit <ankit.si...@intel.com>; Agyeman, Prince > <prince.agye...@intel.com>; Kubacki, Michael A > <michael.a.kuba...@intel.com>; Desimone, Nathaniel L > <nathaniel.l.desim...@intel.com>; Kinney, Michael D > <michael.d.kin...@intel.com> > 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 <hao.a...@intel.com> > Cc: Liming Gao <liming....@intel.com> > Cc: Ankit Sinha <ankit.si...@intel.com> > Cc: Agyeman Prince <prince.agye...@intel.com> > Cc: Kubacki Michael A <michael.a.kuba...@intel.com> > Cc: Nate DeSimone <nathaniel.l.desim...@intel.com> > Cc: Michael D Kinney <michael.d.kin...@intel.com> > > Signed-off-by: David Wei <david.y....@intel.com> > --- > .../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: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-