[meta-xilinx] [PATCH 6/12] u-boot-xlnx: Add recipe for xilinx-v2017.1 release

[Nathan Rossi]

On 9 May 2017 at 03:40, 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: Sunday, May 07, 2017 7:07 AM
>> To: meta-xilinx at lists.yoctoproject.org
>> Subject: [meta-xilinx] [PATCH 6/12] u-boot-xlnx: Add recipe for xilinx-v2017.1
>> release
>>
>> Add a recipe for the u-boot-xlnx xilinx-v2017.1 release, this release is based on
>> v2017.01 of U-Boot. Also add an updated version of the patches.
>>
>> This patch also moves the FILESEXTRAPATHS prepend to the u-boot-xlnx.inc due to
>> common use. And a u-boot-xlnx/${XILINX_RELEASE_VERSION} subdir is added to the
>> list for version specific patches.
>>
>> Signed-off-by: Nathan Rossi <nathan at nathanrossi.com>
>> ---
>>  recipes-bsp/u-boot/u-boot-xlnx.inc                 |  4 +++
>>  ...xilinx_zynqmp.h-Auto-boot-in-JTAG-if-imag.patch | 39 ++++++++++++++++++++++
>>  recipes-bsp/u-boot/u-boot-xlnx_2016.4.bb           |  3 --
>>  recipes-bsp/u-boot/u-boot-xlnx_2017.1.bb           | 24 +++++++++++++
>>  4 files changed, 67 insertions(+), 3 deletions(-)  create mode 100644 recipes-bsp/u-
>> boot/u-boot-xlnx/v2017.1/arm-zynqmp-xilinx_zynqmp.h-Auto-boot-in-JTAG-if-
>> imag.patch
>>  create mode 100644 recipes-bsp/u-boot/u-boot-xlnx_2017.1.bb
>>
>> diff --git a/recipes-bsp/u-boot/u-boot-xlnx.inc b/recipes-bsp/u-boot/u-boot-xlnx.inc
>> index 09feb7f271..dd28a0530d 100644
>> --- a/recipes-bsp/u-boot/u-boot-xlnx.inc
>> +++ b/recipes-bsp/u-boot/u-boot-xlnx.inc
>> @@ -8,3 +8,7 @@ SRC_URI = "git://github.com/Xilinx/u-boot-
>> xlnx.git;protocol=https;branch=${UBRAN
>>  S = "${WORKDIR}/git"
>>  PACKAGE_ARCH = "${MACHINE_ARCH}"
>>
>> +FILESEXTRAPATHS_prepend := "${THISDIR}/u-boot:"
>> +FILESEXTRAPATHS_prepend := "${THISDIR}/u-boot-xlnx:"
>> +FILESEXTRAPATHS_prepend := "${@'${THISDIR}/u-boot-
>> xlnx/${XILINX_RELEASE_VERSION}:' if d.getVar('XILINX_RELEASE_VERSION') else ''}"
>> +
>> diff --git a/recipes-bsp/u-boot/u-boot-xlnx/v2017.1/arm-zynqmp-xilinx_zynqmp.h-
>> Auto-boot-in-JTAG-if-imag.patch b/recipes-bsp/u-boot/u-boot-xlnx/v2017.1/arm-
>> zynqmp-xilinx_zynqmp.h-Auto-boot-in-JTAG-if-imag.patch
>> new file mode 100644
>> index 0000000000..de7954a8a0
>> --- /dev/null
>> +++ b/recipes-bsp/u-boot/u-boot-xlnx/v2017.1/arm-zynqmp-xilinx_zynqmp.h-
>> +++ Auto-boot-in-JTAG-if-imag.patch
>> @@ -0,0 +1,39 @@
>> +From 05fc0a41a14dac435248aa4ea2afa44450f70d46 Mon Sep 17 00:00:00 2001
>> +From: Nathan Rossi <nathan at nathanrossi.com>
>> +Date: Fri, 4 Nov 2016 21:33:14 +1000
>> +Subject: [PATCH] arm: zynqmp: xilinx_zynqmp.h: Auto boot in JTAG if
>> +images in  memory
>> +
>> +Add a command that checks if the Kernel image (in aarch64 Image format)
>> +and optionally a rootfs is in memory and automatically boot these
>> +images if in JTAG boot mode.
>> +
>> +This allows for simpler boot automation in JTAG boot environments
>> +(including QEMU) where manual interaction would otherwise be required.
>> +
>> +Signed-off-by: Nathan Rossi <nathan at nathanrossi.com>
>> +Upstream-Status: Denied [Upstream prefers FIT instead]
>
> I think we should remove this patch from meta-xilinx. It should not be holding ground for Denied patches.
> If FIT is preferred from upstream, we should introduced producing FIT images for QEMU to work.
> FIT image support in OE-core is also available at this point.

I have not yet had a chance to look at setting up kernel-fitimage yet
(for ZynqMP or in general), but regardless a patch like this would
still need to be added to achieve the same goal just with FIT. I don't
have a huge problem keeping this patch around until fitimage is sorted
out.

Regards,
Nathan