Re: [PATCH edk2-platforms 3/3] Platform/RaspberryPi: don't connect all devices on an ordinary boot

["Pete Batard" <pete@akeo.ie>]

On 2020.06.04 10:50, Ard Biesheuvel wrote:
> The BDS will connect device paths that are considered as boot options,
> so there is really no reason to always connect absolutely everything.
> So now that all the drivers have been updated to play nice in this
> case, remove the ConnectAll() call from the RPi BDS code.
> 
> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@arm.com>
> ---
>   Platform/RaspberryPi/Library/PlatformBootManagerLib/PlatformBm.c | 5 -----
>   1 file changed, 5 deletions(-)
> 
> diff --git a/Platform/RaspberryPi/Library/PlatformBootManagerLib/PlatformBm.c b/Platform/RaspberryPi/Library/PlatformBootManagerLib/PlatformBm.c
> index 2bd625ad7e7c..253614a646c1 100644
> --- a/Platform/RaspberryPi/Library/PlatformBootManagerLib/PlatformBm.c
> +++ b/Platform/RaspberryPi/Library/PlatformBootManagerLib/PlatformBm.c
> @@ -655,11 +655,6 @@ PlatformBootManagerAfterConsole (
>       Print (BOOT_PROMPT);
> 
>     }
> 
>   
> 
> -  //
> 
> -  // Connect the rest of the devices.
> 
> -  //
> 
> -  EfiBootManagerConnectAll ();
> 
> -
> 
>     Status = gBS->LocateProtocol (&gEsrtManagementProtocolGuid, NULL, (VOID**)&EsrtManagement);
> 
>     if (!EFI_ERROR (Status)) {
> 
>       EsrtManagement->SyncEsrtFmp ();
> 

Reviewed-by: Pete Batard <pete@akeo.ie>
For the series (after application of the planned edk2 commits to confirm 
that UEFI Shell is now added as an option in Boot Manager):
Tested-by: Pete Batard <pete@akeo.ie>