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

[Tom Rini]

On Sun, Jan 08, 2023 at 12:36:15PM -0700, Simon Glass wrote:
> Hi Tom,
> 
> On Sun, 8 Jan 2023 at 09:24, Tom Rini <trini at konsulko.com> wrote:
> >
> > 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.
> >
> 
> OK let's hold off on this patch for now. It is just a clean-up anyway.

In light of the feedback, for this series, lets just make the generator
option depend on ARCH_ZYNQMP as well and leave it default y. This should
prevent any new users from showing up while the feedback Michal has
provided gets addressed and leave this series with one less blocker at
least.

-- 
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/20230109/84d87891/attachment.sig>