[U-Boot] [PATCH v4 00/20] mx6sabre: Add DM and SPL FIT support

Simon Goldschmidt
2019-02-04T15:21:37Z

Am Mo., 4. Feb. 2019, 16:16 hat Abel Vesa <abel.vesa@nxp.com> geschrieben:

> On 19-02-04 08:19:23, Tom Rini wrote:
> > On Mon, Feb 04, 2019 at 09:03:43AM -0200, Fabio Estevam wrote:
> > > On Mon, Feb 4, 2019 at 7:55 AM Abel Vesa <abel.vesa@nxp.com> wrote:
> > >
> > > > If the SPL size (without the dtb appended) is larger then yes, the
> build fails.
> > > > Trouble is if the SPL (without the dtb appended) is, lets say, 63kB
> and
> > > > then the dtb is larger than 1kB. Then there is no mechanism in place
> to check that
> > > > and it will just fail to boot without giving any clues why. But this
> is a totally
> > > > unrelated problem from this patchset's point of view and I think it
> impacts all
> > > > the platforms that support SPL with DM.
> > >
> > > Yes, it is unrelated to this series and should be treated separately.
> >
> > I also agree. But, don't we have a mechanism for that? It sounds like
> > some additional targets need to call $(BOARD_SIZE_CHECK) at the end. Or
> > am I missing something?
> >
>
> Hmm, I believe that is true. I haven't looked deeper into it but it seems
> that's the thing the SPL is missing.

Haven't we talked about that just some days or weeks ago? That's exactly
what I am missing for socfpga. I keep getting broken SPL images when adding
functionality. And you don't even get an error message because the dtb is
broken and cannot provide the serial output mode...

Regards,
Simon