On 07/13/15 18:36, Olivier Martin wrote:
> PcdDefaultBootType has been removed when the embedded
> Linux Loader has been removed from BdsLib.
> The boot arguments (defined by PcdDefaultBootArgument) are
> now always targetting EFI applications.
> 
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Olivier Martin <olivier.mar...@arm.com>
> Reviewed-by: Ronald Cron <ronald.c...@arm.com>
> ---
>  ArmVirtPkg/ArmVirtQemu.dsc | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/ArmVirtPkg/ArmVirtQemu.dsc b/ArmVirtPkg/ArmVirtQemu.dsc
> index fbc2b12..0d4f4b0 100644
> --- a/ArmVirtPkg/ArmVirtQemu.dsc
> +++ b/ArmVirtPkg/ArmVirtQemu.dsc
> @@ -133,7 +133,6 @@
>    gArmPlatformTokenSpaceGuid.PcdDefaultBootDescription|L"Linux (EFI stub) on 
> virtio31:hd0:part0"
>    
> gArmPlatformTokenSpaceGuid.PcdDefaultBootDevicePath|L"VenHw(837DCA9E-E874-4D82-B29A-23FE0E23D1E2,003E000A00000000)/HD(1,MBR,0x00000000,0x3F,0x19FC0)/Image"
>    gArmPlatformTokenSpaceGuid.PcdDefaultBootArgument|"root=/dev/vda2 
> console=ttyAMA0 earlycon uefi_debug"
> -  gArmPlatformTokenSpaceGuid.PcdDefaultBootType|0
>  
>    #
>    # Settings for ARM BDS -- use the serial console (ConIn & ConOut).
> 

The declaration of "gArmPlatformTokenSpaceGuid.PcdDefaultBootType" is
removed from "ArmPlatformPkg/ArmPlatformPkg.dec" in patch #5. Therefore,
if someone were to build the ArmVirtQemu.dsc platform right at patch #5,
it would break.

I'd like to request the following:
- First, please remove the code (driver, library, etc) that consumes
  PcdDefaultBootType. At this point the PCD becomes unused, code-wise,
  but the tree should build.
- Second, please remove the PCD assignments from the DSC files across
  the tree.
- Third, please drop the PCD declaration as last step.

Thank you
Laszlo

------------------------------------------------------------------------------
Don't Limit Your Business. Reach for the Cloud.
GigeNET's Cloud Solutions provide you with the tools and support that
you need to offload your IT needs and focus on growing your business.
Configured For All Businesses. Start Your Cloud Today.
https://www.gigenetcloud.com/
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to