From mboxrd@z Thu Jan 1 00:00:00 1970 From: Abel Vesa Date: Mon, 4 Feb 2019 15:15:41 +0000 Subject: [U-Boot] [PATCH v4 00/20] mx6sabre: Add DM and SPL FIT support In-Reply-To: <20190204131923.GV30838@bill-the-cat> References: <1549039198-16674-1-git-send-email-abel.vesa@nxp.com> <20190204095535.zvpcvlvfi4uvd2mo@fsr-ub1664-175> <20190204131923.GV30838@bill-the-cat> Message-ID: <20190204151541.ntboob335c4rlsrp@fsr-ub1664-175> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 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. > -- > Tom