[Buildroot] [PATCH v1 1/1] configs/zynqmp_kria_kv260_defconfig: fix to u-boot.itb generation

[Buildroot] [PATCH v1 1/1] configs/zynqmp_kria_kv260_defconfig: fix to u-boot.itb generation

[Neal Frager]

This patch is a temporary fix for the kv260 u-boot.itb generation
until a proper fix has been implemented within u-boot.

This patch will be removed from buildroot once u-boot can build a
correct u-boot.itb for the kria kv260 starter kit.

Signed-off-by: Neal Frager <neal.frager@xilinx.com>
---
 board/zynqmp/kria/kv260/kv260.sh    | 9 +++++++++
 configs/zynqmp_kria_kv260_defconfig | 4 ++--
 2 files changed, 11 insertions(+), 2 deletions(-)
 create mode 100755 board/zynqmp/kria/kv260/kv260.sh

diff --git a/board/zynqmp/kria/kv260/kv260.sh b/board/zynqmp/kria/kv260/kv260.sh
new file mode 100755
index 0000000000..2febb9b087
--- /dev/null
+++ b/board/zynqmp/kria/kv260/kv260.sh
@@ -0,0 +1,9 @@
+#!/bin/sh
+
+# temporary work around for generating kv260 u-boot.itb
+# this will be removed when u-boot can build the kv260 u-boot.itb natively
+
+UBOOT_DIR=$4
+
+fdtoverlay -o ${UBOOT_DIR}/fit-dtb.blob -i ${UBOOT_DIR}/arch/arm/dts/zynqmp-smk-k26-revA.dtb ${UBOOT_DIR}/arch/arm/dts/zynqmp-sck-kv-g-revB.dtbo
+${UBOOT_DIR}/tools/mkimage -E -f ${UBOOT_DIR}/u-boot.its -B 0x8 ${BINARIES_DIR}/u-boot.itb
diff --git a/configs/zynqmp_kria_kv260_defconfig b/configs/zynqmp_kria_kv260_defconfig
index c1366b705c..d98ac89c63 100644
--- a/configs/zynqmp_kria_kv260_defconfig
+++ b/configs/zynqmp_kria_kv260_defconfig
@@ -1,8 +1,8 @@
 BR2_aarch64=y
 BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_5_15=y
-BR2_ROOTFS_POST_BUILD_SCRIPT="board/zynqmp/post-build.sh"
+BR2_ROOTFS_POST_BUILD_SCRIPT="board/zynqmp/post-build.sh board/zynqmp/kria/kv260/kv260.sh"
 BR2_ROOTFS_POST_IMAGE_SCRIPT="board/zynqmp/post-image.sh"
-BR2_ROOTFS_POST_SCRIPT_ARGS="ttyPS1,115200 mmcblk1p2"
+BR2_ROOTFS_POST_SCRIPT_ARGS="ttyPS1,115200 mmcblk1p2 ${UBOOT_DIR}"
 BR2_LINUX_KERNEL=y
 BR2_LINUX_KERNEL_CUSTOM_TARBALL=y
 BR2_LINUX_KERNEL_CUSTOM_TARBALL_LOCATION="$(call github,Xilinx,linux-xlnx,xlnx_rebase_v5.15_LTS_2022.1)/xlnx_rebase_v5.15_LTS_2022.1.tar.gz"
-- 
2.17.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH v1 1/1] configs/zynqmp_kria_kv260_defconfig: fix to u-boot.itb generation

[Luca Ceresoli via buildroot]

Hi Neal,

On Mon, 20 Jun 2022 05:08:43 -0600
Neal Frager <neal.frager@xilinx.com> wrote:

> This patch is a temporary fix for the kv260 u-boot.itb generation
> until a proper fix has been implemented within u-boot.
> 
> This patch will be removed from buildroot once u-boot can build a
> correct u-boot.itb for the kria kv260 starter kit.
> 
> Signed-off-by: Neal Frager <neal.frager@xilinx.com>

Can you provide a brief explanation of what the problem is and a link
to any page documenting what effort (if any) is ongoing to fix it at
its root?

Other than that, looks good.

-- 
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH v1 1/1] configs/zynqmp_kria_kv260_defconfig: fix to u-boot.itb generation

[Frager, Neal via buildroot]

Hi Michal,

> This patch is a temporary fix for the kv260 u-boot.itb generation 
> until a proper fix has been implemented within u-boot.
>
> This patch will be removed from buildroot once u-boot can build a 
> correct u-boot.itb for the kria kv260 starter kit.
>
> Signed-off-by: Neal Frager <neal.frager@xilinx.com>

> Can you provide a brief explanation of what the problem is and a link to any page documenting what effort (if any) is ongoing to fix it at its root?

> Other than that, looks good.

Could you comment on any effort to add the various kria starter kit carrier board overlays to be included natively in the u-boot.itb build, 
so no post-build steps are required?

@Luca, I will add more comments documenting the problem in v2 of the patch.  Thank you for the review.

Best regards,
Neal Frager
AMD


_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot