[PATCH v9 14/14] treewide: Disable USE_SPL_FIT_GENERATOR by default

Tom Rini trini at konsulko.com
Sun Jan 8 17:24:38 CET 2023


On Sun, Jan 08, 2023 at 09:20:09AM -0700, Simon Glass wrote:
> Hi Tom,
> 
> On Sun, 8 Jan 2023 at 09:06, Tom Rini <trini at konsulko.com> wrote:
> >
> > On Sun, Jan 08, 2023 at 08:48:37AM -0700, Simon Glass wrote:
> > > Hi Tom,
> > >
> > > On Sun, 8 Jan 2023 at 06:41, Tom Rini <trini at konsulko.com> wrote:
> > > >
> > > > On Sat, Jan 07, 2023 at 02:07:21PM -0700, Simon Glass wrote:
> > > >
> > > > > This option is deprecated and only used by two boards. Enable it for just
> > > > > those two boards, so others don't accidentally enable it.
> > > > >
> > > > > Signed-off-by: Simon Glass <sjg at chromium.org>
> > > > [snip]
> > > > > diff --git a/boot/Kconfig b/boot/Kconfig
> > > > > index 55f06761ef8..7ab0dd14211 100644
> > > > > --- a/boot/Kconfig
> > > > > +++ b/boot/Kconfig
> > > > > @@ -282,12 +282,13 @@ config SPL_FIT_SOURCE
> > > > >  config USE_SPL_FIT_GENERATOR
> > > > >       bool "Use a script to generate the .its script"
> > > > >       depends on SPL_FIT
> > > > > -     default y if SPL_FIT && ARCH_ZYNQMP
> > > > > +     help
> > > > > +       This is deprecated. Please do not use it. Use binman instead.
> > > >
> > > > Lets remove the text around bool so it can't be enabled, and move to
> > > > select'ing it from the two boards that need it. Michal, Luca, what's
> > > > needed to move your two platforms
> > > > (avnet_ultrazedev_cc_v1_0_ultrazedev_som_v1_0 and xilinx_zynqmp_virt off
> > > > of this very legacy option, given that other xilinx platforms have
> > > > already migrated to binman ?
> > >
> > > That's a good idea, but these two boards do not have TARGET Kconfig
> > > options so it is not possible without adding some Kconfig specific to
> > > those boards, then defining it in the defconfig files.
> > >
> > > We already have the legacy warning.
> >
> > Yes, but I swear these are new legacy users as when we started trying to
> > kill off this option it was just a few i.MX platforms lagging behind.
> >
> > Maybe make ARCH_ZYNQMP select DEPRECATED, USE_SPL_FIT_GENERATOR depend
> > on DEPRECATED and add "DERECATED" to the end of the text line?  Or maybe
> > Michal or Luca will speak up soon and migrate these over quickly so we
> > can just nuke this.
> 
> Isn't that just more tortuous? I can disable SPL_LOAD_FIT on these two
> boards since they don't appear in CI. Then they can convert them when
> ready.
> 
> That way we can drop the option now, if that is your goal.

I thought xilinx_zynqmp_virt was in CI, but I see I'm mistaken. I still
don't want to break platforms outright, and since it's Sunday right now
afterall, we should let Michal and Luca a chance to catch up and chime
in. I hope it's either going to be a quick conversion or expose
something missing and needed in binman, as to why these still haven't
been converted.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230108/71310a03/attachment.sig>


More information about the U-Boot mailing list