Re: [Buildroot] [PATCH v9 06/11] boot/zynqmp-firmware: new boot firmware

[Luca Ceresoli via buildroot <buildroot@buildroot.org>]

Hello Neal,

On Thu, 22 Feb 2024 16:35:21 +0000
"Frager, Neal" <neal.frager@amd.com> wrote:

> Hello everyone,
> 
> Just a friendly reminder.  Any updates?
> 
> > Now that the toolchain-bare-metal-buildroot has been applied, do you have any
> > feedback regarding the remainder of the patch set?  
> 
> > One open question I have is regarding the patches for zynqmp-firmware and
> > versal-firmware.  These patches are no longer neeeded since being applied to
> > release version xilinx_v2023.2.  I kept them with the idea that users may
> > still be using older versions of software, so the patches enable users to
> > work with versions as old as 2022.2 for zynqmp and 2023.1 for versal.  
> 
> > Do you agree with keeping the patches or do you think it is better not to
> > include them and to say that support begins starting with version
> > xilinx_v2023.2?  
> 
> What are your thoughts?
> 
> > create mode 120000 boot/zynqmp-firmware/xilinx_v2022.2/0001-pmufw-misc-Makefile-specify-sequential-Makefiles.patch
> > create mode 100644 boot/zynqmp-firmware/xilinx_v2023.1/0001-pmufw-misc-Makefile-specify-sequential-Makefiles.patch  

Apologies for the late reply...

I think we should not put any effort in encouraging the practice of
using old and perhaps not-maintained-anymore software that some
proprietary tools tend to enforce.

We do have a mainline pmufw that works without needing any patches
thank to your (very appreciated!) effort, so let's just support the
latest and greatest pmufw version and be happy.

Your patches are there in this series anyway, and will be, so whoever
wants to use an old version can do some research and find them.

So, I vote to not have these patches in Buildroot.

Luca

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