[Buildroot] [PATCH 1/1] configs/imx8mqevk: bump TF-A, U-Boot and Linux Kernel version

[Buildroot] [PATCH 1/1] configs/imx8mqevk: bump TF-A, U-Boot and Linux Kernel version

[Julien Olivain]

The imx8mqevk_defconfig is failing to build the arm-trusted-firmware
package, with the error message:

    plat/imx/imx8m/hab.c: In function 'imx_hab_handler':
    plat/imx/imx8m/hab.c:65:57: error: array subscript 0 is outside array bounds of 'uint32_t[0]' {aka 'unsigned int[]'} [-Werror=array-bounds=]
       65 | #define HAB_RVT_CHECK_TARGET_ARM64      ((unsigned long)*(uint32_t *)(HAB_RVT_BASE + 0x18))
          |                                                         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

This is happening since commit dc0f721 "package/gcc: switch to 13.x
as default".

This commit fixes this build failure by updating the TF-A, U-Boot and
Linux Kernel. Note: this imx8mqevk_defconfig uses the upstream Kernel
(in contrast of the freescale_imx8mqevk_defconfig which is using the NXP
BSP Kernel version).

Also, this new uboot version needs gnutls and openssl, so the
corresponding options are added to the defconfig.

Fixes:
https://gitlab.com/buildroot.org/buildroot/-/jobs/7691701411

Signed-off-by: Julien Olivain <ju.o@free.fr>
---
Patch tested in:
https://gitlab.com/jolivain/buildroot/-/jobs/7715641622
---
 configs/imx8mqevk_defconfig | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/configs/imx8mqevk_defconfig b/configs/imx8mqevk_defconfig
index 943a479932..350b863ec9 100644
--- a/configs/imx8mqevk_defconfig
+++ b/configs/imx8mqevk_defconfig
@@ -1,12 +1,12 @@
 BR2_aarch64=y
 BR2_ARM_FPU_VFPV3=y
-BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_5_8=y
+BR2_PACKAGE_HOST_LINUX_HEADERS_CUSTOM_6_6=y
 BR2_TARGET_GENERIC_GETTY_PORT="ttymxc0"
 BR2_ROOTFS_POST_IMAGE_SCRIPT="board/freescale/common/imx/imx8-bootloader-prepare.sh board/freescale/common/imx/post-image.sh"
 BR2_ROOTFS_POST_SCRIPT_ARGS="${UBOOT_DIR}/arch/arm/dts/imx8mq-evk.dtb"
 BR2_LINUX_KERNEL=y
 BR2_LINUX_KERNEL_CUSTOM_VERSION=y
-BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="5.8.7"
+BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="6.6.48"
 BR2_LINUX_KERNEL_USE_ARCH_DEFAULT_CONFIG=y
 BR2_LINUX_KERNEL_DTS_SUPPORT=y
 BR2_LINUX_KERNEL_INTREE_DTS_NAME="freescale/imx8mq-evk"
@@ -19,15 +19,17 @@ BR2_TARGET_ROOTFS_EXT2_4=y
 BR2_TARGET_ROOTFS_EXT2_SIZE="120M"
 BR2_TARGET_ARM_TRUSTED_FIRMWARE=y
 BR2_TARGET_ARM_TRUSTED_FIRMWARE_CUSTOM_TARBALL=y
-BR2_TARGET_ARM_TRUSTED_FIRMWARE_CUSTOM_TARBALL_LOCATION="$(call github,nxp-imx,imx-atf,rel_imx_5.4.24_2.1.0)/imx-atf-rel_imx_5.4.24_2.1.0.tar.gz"
+BR2_TARGET_ARM_TRUSTED_FIRMWARE_CUSTOM_TARBALL_LOCATION="$(call github,nxp-imx,imx-atf,lf-6.6.23-2.0.0)/imx-atf-lf-6.6.23-2.0.0.tar.gz"
 BR2_TARGET_ARM_TRUSTED_FIRMWARE_PLATFORM="imx8mq"
 BR2_TARGET_ARM_TRUSTED_FIRMWARE_BL31=y
 BR2_TARGET_UBOOT=y
 BR2_TARGET_UBOOT_BUILD_SYSTEM_KCONFIG=y
 BR2_TARGET_UBOOT_CUSTOM_TARBALL=y
-BR2_TARGET_UBOOT_CUSTOM_TARBALL_LOCATION="$(call github,nxp-imx,uboot-imx,rel_imx_5.4.24_2.1.0)/uboot-imx-rel_imx_5.4.24_2.1.0.tar.gz"
+BR2_TARGET_UBOOT_CUSTOM_TARBALL_LOCATION="$(call github,nxp-imx,uboot-imx,lf-6.6.23-2.0.0)/uboot-imx-lf-6.6.23-2.0.0.tar.gz"
 BR2_TARGET_UBOOT_BOARD_DEFCONFIG="imx8mq_evk"
 BR2_TARGET_UBOOT_NEEDS_DTC=y
+BR2_TARGET_UBOOT_NEEDS_GNUTLS=y
+BR2_TARGET_UBOOT_NEEDS_OPENSSL=y
 BR2_TARGET_UBOOT_FORMAT_CUSTOM=y
 BR2_TARGET_UBOOT_FORMAT_CUSTOM_NAME="u-boot-nodtb.bin"
 BR2_TARGET_UBOOT_SPL=y
-- 
2.46.0

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

Re: [Buildroot] [PATCH 1/1] configs/imx8mqevk: bump TF-A, U-Boot and Linux Kernel version

[Fabio Estevam]

Hi Julien,

On Sun, Sep 1, 2024 at 4:44 PM Julien Olivain <ju.o@free.fr> wrote:
>
> The imx8mqevk_defconfig is failing to build the arm-trusted-firmware
> package, with the error message:
>
>     plat/imx/imx8m/hab.c: In function 'imx_hab_handler':
>     plat/imx/imx8m/hab.c:65:57: error: array subscript 0 is outside array bounds of 'uint32_t[0]' {aka 'unsigned int[]'} [-Werror=array-bounds=]
>        65 | #define HAB_RVT_CHECK_TARGET_ARM64      ((unsigned long)*(uint32_t *)(HAB_RVT_BASE + 0x18))
>           |                                                         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>
> This is happening since commit dc0f721 "package/gcc: switch to 13.x
> as default".
>
> This commit fixes this build failure by updating the TF-A, U-Boot and
> Linux Kernel. Note: this imx8mqevk_defconfig uses the upstream Kernel
> (in contrast of the freescale_imx8mqevk_defconfig which is using the NXP
> BSP Kernel version).
>
> Also, this new uboot version needs gnutls and openssl, so the
> corresponding options are added to the defconfig.
>
> Fixes:
> https://gitlab.com/buildroot.org/buildroot/-/jobs/7691701411
>
> Signed-off-by: Julien Olivain <ju.o@free.fr>

Thanks for the fix:

Reviewed-by: Fabio Estevam <festevam@gmail.com>

At some point, I will try to switch to U-Boot and TF-A mainline.
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

Re: [Buildroot] [PATCH 1/1] configs/imx8mqevk: bump TF-A, U-Boot and Linux Kernel version

[Thomas Petazzoni via buildroot]

On Sun,  1 Sep 2024 21:44:22 +0200
Julien Olivain <ju.o@free.fr> wrote:

> The imx8mqevk_defconfig is failing to build the arm-trusted-firmware
> package, with the error message:
> 
>     plat/imx/imx8m/hab.c: In function 'imx_hab_handler':
>     plat/imx/imx8m/hab.c:65:57: error: array subscript 0 is outside array bounds of 'uint32_t[0]' {aka 'unsigned int[]'} [-Werror=array-bounds=]
>        65 | #define HAB_RVT_CHECK_TARGET_ARM64      ((unsigned long)*(uint32_t *)(HAB_RVT_BASE + 0x18))
>           |                                                         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 
> This is happening since commit dc0f721 "package/gcc: switch to 13.x
> as default".
> 
> This commit fixes this build failure by updating the TF-A, U-Boot and
> Linux Kernel. Note: this imx8mqevk_defconfig uses the upstream Kernel
> (in contrast of the freescale_imx8mqevk_defconfig which is using the NXP
> BSP Kernel version).
> 
> Also, this new uboot version needs gnutls and openssl, so the
> corresponding options are added to the defconfig.
> 
> Fixes:
> https://gitlab.com/buildroot.org/buildroot/-/jobs/7691701411
> 
> Signed-off-by: Julien Olivain <ju.o@free.fr>
> ---
> Patch tested in:
> https://gitlab.com/jolivain/buildroot/-/jobs/7715641622
> ---
>  configs/imx8mqevk_defconfig | 10 ++++++----
>  1 file changed, 6 insertions(+), 4 deletions(-)

I have initially applied to next... and then realized it was fixing a
Gitlab CI issue, so I also applied to master. Thanks!

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