From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Babic Date: Thu, 25 Apr 2019 16:52:58 +0200 Subject: [U-Boot] [PATCH v2 5/5] board: toradex: add colibri imx8qxp 2gb wb it v1.0b module support In-Reply-To: References: <20190409152534.11691-1-marcel@ziswiler.com> <20190409152534.11691-6-marcel@ziswiler.com> <14d13549-73fe-fe36-f5e8-7648e823335a@denx.de> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Marcel, On 25/04/19 14:35, Marcel Ziswiler wrote: > Hi Stefano > > On Thu, 2019-04-25 at 12:48 +0200, Stefano Babic wrote: >> Hi Marcel, >> >> On 09/04/19 17:25, Marcel Ziswiler wrote: >>> From: Marcel Ziswiler >>> >>> This commit adds initial support for the Toradex Colibri iMX8QXP >>> 2GB WB >>> IT V1.0B module. Unlike the V1.0A early access samples exclusively >>> booting from SD card, they are now strapped to boot from eFuses >>> which >>> are factory fused to properly boot from their on-module eMMC. U- >>> Boot >>> supports either booting from the on-module eMMC or may be used for >>> recovery purpose using the universal update utility (uuu) aka >>> mfgtools >>> 3.0. >>> >>> Functionality wise the following is known to be working: >>> - eMMC and MMC/SD card >>> - Ethernet >>> - GPIOs >>> - I2C >>> >>> Unfortunately, there is no USB functionality for the i.MX 8QXP as >>> of >>> yet. >>> >>> Signed-off-by: Marcel Ziswiler >>> >>> --- >>> >> >> I merged the series and build locally (fine), but Travis complains >> and >> stops with error: >> >> +cc1: fatal error: opening output file spl/u-boot-spl.cfgout: No such >> file or directory >> +compilation terminated. >> >> Can you take a look at it ? > > Sure, looks like Peng's commit caceb739ea07 ("imx: build flash.bin for > i.MX8") takes SPL for granted while my patchset currently avoids it. It looks so, yes. > > BTW: I still don't believe SPL makes much sense on i.MX 8X given all > the other proprietary parts involved in booting. SPL makes more sense if it is possible to detect at runtime the HW and change the configuration - for i.MX6, this means RAMS detection, which boot device is booting, and so on. On i.MX8 there is a lot of proprietary parts - we lose the flexibility of SPL, and most features are lost (or must be provided by proprietary code). I agree that on this platform SPL makes less sense, and i.MX8 should be built independently if CONFIG_SPL is set (this is also for i.MX6 / MX5, there are boards without SPL and using the DCD image to set up the RAM controller). > Plus currently SPL > actually breaks the USB serial downloader aka recovery mode using the > universal update utility (uuu) aka mfgtools 3.0. > Regards, Stefano -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de =====================================================================