[U-Boot] [PATCH v5 10/15] Makefile: Query the SPL Fit Generator for its dependencies

[Tom Rini]

On Mon, Oct 07, 2019 at 11:43:07AM +0200, Jean-Jacques Hiblot wrote:
> 
> On 06/10/2019 19:12, Tom Rini wrote:
> > On Sun, Oct 06, 2019 at 04:57:29PM +0000, Auer, Lukas wrote:
> > > Hi Jean-Jacques,
> > > 
> > > On Fri, 2019-09-20 at 17:28 +0200, Jean-Jacques Hiblot wrote:
> > > > To reduce the complexity of the Makefile, let the generator tell what its
> > > > dependencies are. For this purpose use the "--deps" option.
> > > > 
> > > We recently added a generic FIT generator script for RISC-V, located at
> > > arch/riscv/lib/mkimage_fit_opensbi.sh . Can you add the --deps option
> > > to that script as well? It does not have any dependencies, so it can be
> > > handled the same way as in the i.MX and sunxi FIT generator scripts.
> > Can we please work on generalizing these scripts and either using
> > binman, updating binman as needed or coming up with a POSIX SH tool that
> > can do what needs doing?  Thanks!
> 
> Maybe it can be done. But we would need to know first what the needs are.
> 
> In may case, all I need is to bundle at build time a set of DTB overlays
> with u-boot.bin and some DTBs.
> 
> For this binman is overkill, using dtc is enough, though I guess binman can
> be used instead.

OK, and in this case we're talking about tools/k3_fit_atf.sh which is
based on board/sunxi/mksunxi_fit_atf.sh ?  That's two similar but not
quite just mergable POSIX SH tools.  If we end up with an intermediate
state of having tools/generators/ with a number of $(soc)_fit_atf.sh
scripts that'll start to make it clearer what is/isn't the same.  Maybe
we won't have a single tool, but if it's at least make it clearer where
things belong and we can get consistency.  Thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20191007/60ba5fd6/attachment.sig>