Re: [staging/cadence-aarch64 PATCH v2 6/6] CadencePkg: Add .dsc, .fdf and .dec files for Cadence CSP platform.

[Leif Lindholm <leif.lindholm@linaro.org>]

On Thu, Jun 22, 2017 at 09:26:47AM +0000, Scott Telford wrote:
> > -----Original Message-----
> > From: Leif Lindholm [mailto:leif.lindholm@linaro.org]
> > Sent: 06 June 2017 12:37
> > To: Scott Telford <stelford@cadence.com>
> > Cc: edk2-devel@ml01.01.org; ard.biesheuvel@linaro.org;
> > graeme.gregory@linaro.org; afish@apple.com; michael.d.kinney@intel.com
> > Subject: Re: [staging/cadence-aarch64 PATCH v2 6/6] CadencePkg: Add .dsc,
> > .fdf and .dec files for Cadence CSP platform.
> 
> > > +
> > > +  #
> > > +  # Bds
> > > +  #
> > > +  MdeModulePkg/Universal/DevicePathDxe/DevicePathDxe.inf
> > > +  MdeModulePkg/Universal/DisplayEngineDxe/DisplayEngineDxe.inf
> > > +  MdeModulePkg/Universal/SetupBrowserDxe/SetupBrowserDxe.inf
> > > +  IntelFrameworkModulePkg/Universal/BdsDxe/BdsDxe.inf
> > 
> > Oh, and unless this causes issues for you, please substitute the above
> > line for
> > ---
> >   MdeModulePkg/Application/UiApp/UiApp.inf {
> >     <LibraryClasses>
> > 
> > NULL|MdeModulePkg/Library/DeviceManagerUiLib/DeviceManagerUiLib.inf
> >       NULL|MdeModulePkg/Library/BootManagerUiLib/BootManagerUiLib.inf
> > 
> > NULL|MdeModulePkg/Library/BootMaintenanceManagerUiLib/BootMainte
> > nanceManagerUiLib.inf
> >  }
> > ---
> > 
> > (The platform you cloned has not yet migrated over to the generic Bds.)
> 
> I tried that, but I just got some debug output saying the BDS was
> waiting for something. The Intel BDS drops me straight into the EFI
> Shell which is ideal for the purpose this platform exists for
> (i.e. developing and testing the Cadence PCIe library).

Sorry, I never gave proper feedback on the new revision of this file.

My concern remains - and there is nothing fundamental about UiApp that
makes it wait for timeouts. That is a simple configuration issue.
Meanwhile, the BdsDxe from IntelFrameworkModulePkg is not intended for
use in new platforms.

So can we look into resolving the timeout issue for you instead?
Is this a question of falling back to PXE boot when no Boot####
entries exist?

Regards,

Leif