From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Mon, 4 Feb 2019 10:30:01 -0500 Subject: [U-Boot] [PATCH v4 00/20] mx6sabre: Add DM and SPL FIT support In-Reply-To: References: <1549039198-16674-1-git-send-email-abel.vesa@nxp.com> <20190204095535.zvpcvlvfi4uvd2mo@fsr-ub1664-175> <20190204131923.GV30838@bill-the-cat> <20190204151541.ntboob335c4rlsrp@fsr-ub1664-175> Message-ID: <20190204153001.GB30838@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Mon, Feb 04, 2019 at 04:21:37PM +0100, Simon Goldschmidt wrote: > Am Mo., 4. Feb. 2019, 16:16 hat Abel Vesa 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 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... Yes, I believe it is. I guess what I'm waiting / hoping for is someone to patch the top-level Makefile to call $(BOARD_SIZE_CHECK) on a number of make targets that look reasonable, throw it at travis and see what if anything fails (and if stuff fails, ask for help to see if it's right or investigate a bit more). More or less, if it's a final target for U-Boot, we should probably give BOARD_SIZE_CHECK a try. Most of the rules start with u-boot, but I bet the %.imx target should also do it, and then we can drop all of that from arch/arm/mach-imx/Makefile. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: