[meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to u-boot for Zynq

[Nathan Rossi]

On 27 April 2017 at 02:41, Manjukumar Harthikote Matha
<manjukumar.harthikote-matha at xilinx.com> wrote:
>
>
>> -----Original Message-----
>> From: meta-xilinx-bounces at yoctoproject.org [mailto:meta-xilinx-
>> bounces at yoctoproject.org] On Behalf Of Nathan Rossi
>> Sent: Wednesday, April 26, 2017 4:57 AM
>> To: meta-xilinx at lists.yoctoproject.org
>> Subject: [meta-xilinx] [PATCH] machine-xilinx-default.inc: Default to u-boot for Zynq
>>
>> Upstream U-Boot provides an almost complete environment for the majority of Zynq
>> targets and specifically covers all the boot functionality required for the boards in
>> the meta-xilinx layer. As such default to the mainline version of U-Boot.
>>
>> For users that require or prefer to use u-boot-xlnx this can be selected on a per-
>> machine basis using:
>>
>>   PREFERRED_PROVIDER_virtual/bootloader = "u-boot-xlnx"
>>
>> Signed-off-by: Nathan Rossi <nathan at nathanrossi.com>
>> ---
>>  conf/machine/include/machine-xilinx-default.inc | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/conf/machine/include/machine-xilinx-default.inc
>> b/conf/machine/include/machine-xilinx-default.inc
>> index 13e4df5746..f9e7e3a33f 100644
>> --- a/conf/machine/include/machine-xilinx-default.inc
>> +++ b/conf/machine/include/machine-xilinx-default.inc
>> @@ -19,6 +19,7 @@ PREFERRED_VERSION_linux-xlnx ?= "4.6-xilinx-v2016.4%"
>>
>>  # U-Boot Configuration
>>  XILINX_DEFAULT_UBOOT := "u-boot-xlnx"
>> +XILINX_DEFAULT_UBOOT_zynq := "u-boot"
>>  XILINX_DEFAULT_UBOOT_zynqmp := "u-boot"
>
> Why have any preferred_provider? Distro can provide these settings.
> For ex: meta-petalinux can provide u-boot-xlnx, oe-core can provide u-boot
>
> Any thoughts?

Unfortunately not picking a provider for "virtual/bootloader" will
result in bitbake attempting to build all providers (since
EXTRA_IMAGEDEPENDS is depending on virtual/bootloader), as bitbake has
no idea which one is desired (and they are all compatible).

--
NOTE: multiple providers are available for virtual/bootloader
(u-boot-xlnx, u-boot-xlnx-dev, u-boot)
NOTE: consider defining a PREFERRED_PROVIDER entry to match virtual/bootloader
...
ERROR: Multiple .bb files are due to be built which each provide
virtual/bootloader
...
<various failures due to overlapping files/etc.>
--

Also note, meta-petalinux or other layers should already be able to
override this default (by setting with ??=) either in a distro conf or
in a machine conf e.g. zynq-generic (for meta-petalinux).

Regards,
Nathan