Re: [Buildroot] [PATCH v1 1/3] configs/zynqmp_zcu102_defconfig: bump to xilinx-v2024.1

From: "Yann E. MORIN" <yann.morin.1998@free.fr>
To: Neal Frager <neal.frager@amd.com>
Cc: ibai.erkiaga-elorza@amd.com, luca.ceresoli@bootlin.com,
	brandon.maier@collins.com, thomas.petazzoni@bootlin.com,
	buildroot@buildroot.org, michal.simek@amd.com
Subject: Re: [Buildroot] [PATCH v1 1/3] configs/zynqmp_zcu102_defconfig: bump to xilinx-v2024.1
Date: Thu, 6 Jun 2024 17:25:11 +0200	[thread overview]
Message-ID: <ZmHU1-EPxRFiHZDl@landeda> (raw)
In-Reply-To: <20240603115212.184529-1-neal.frager@amd.com>

Neal, Al,

On 2024-06-03 12:52 +0100, Neal Frager via buildroot spake thusly:
> This patch bumps the zynqmp_zcu102_defconfig to xilinx-v2024.1 which includes
> the following updates:
> 
> - Linux v6.6.10
> - U-Boot v2024.01
> - ATF v2.10
> - PMUFW xilinx-v2024.1
> 
> Signed-off-by: Neal Frager <neal.frager@amd.com>

I've now applied this series of three patches to next, thanks.

However, while applying, I noticed that those boards do not have
BR2_DOWNLOAD_FORCE_CHECK_HASHES=y (so my post-apply hooks triggered).

I wanted to add that and point to the newly introduced
board/xilinx/patches, but that will not be possible.

Indeed, those defconfig use BR2_TARGET_UBOOT_ZYNQMP_PMUFW, and that is
added to the extra downloads, and we would need a hash for that file.

But the filename is unversioned, which means that we can't differentiate
between pmufw.elf from yesterday, the one from today, and the one from
tomorrow.

So, when arbitrary downloads of unversioned files are done in a
defconfig (or in a package enabled by tht defconfig), we will not be
able to force checking the hashes...

Of course, this is not an issue directly introduced by this series,
which is why I applied it.

Still, for this case, I wonder if that would not be better to intriduce
a new package (e.g. boot/xilinx-soc-prebuilt-firmware/ ) which would be
properly versioned, and on which uboot would depend when
BR2_TARGET_UBOOT_ZYNQMP=y.

Luca, do you remember the details why a package was not introduced bacj
when you added the ZYNQMP PMU support?

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot