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

[Scott Telford <stelford@cadence.com>]

Hi Leif, sorry for the late reply, I've been away. I had another go at switching to the generic BDS, and this time it seems to work. Revised patch incoming.

Regards,
Scott.

> -----Original Message-----
> From: Leif Lindholm [mailto:leif.lindholm@linaro.org]
> Sent: 29 June 2017 17:16
> 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.
> 
> EXTERNAL MAIL
> 
> 
> 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