From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Mon, 1 Mar 2021 12:56:57 -0500 Subject: [PATCH V2] ARM: imx: Include u-boot.img in u-boot-with-spl.imx if OF_SEPARATE=y In-Reply-To: <2f08deff-d957-5934-c69f-f314830f396f@denx.de> References: <20210201200725.49959-1-marex@denx.de> <4d8a1d9c-ed5e-06c6-5cd9-f24a450837d8@denx.de> <08057cd5-47fa-33f7-51d0-4583a32020e5@denx.de> <2f08deff-d957-5934-c69f-f314830f396f@denx.de> Message-ID: <20210301175657.GF10169@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, Mar 01, 2021 at 02:44:13PM +0100, Stefano Babic wrote: > On 01.03.21 14:40, Stefano Babic wrote: > > On 01.03.21 14:36, Marek Vasut wrote: > > > On 3/1/21 10:27 AM, Stefano Babic wrote: > > > > Hi Marek, > > > > > > Hi, > > > > > > [...] > > > > > > > > diff --git a/arch/arm/mach-imx/Makefile b/arch/arm/mach-imx/Makefile > > > > > index 1aa26a50ad8..e6b4654cd35 100644 > > > > > --- a/arch/arm/mach-imx/Makefile > > > > > +++ b/arch/arm/mach-imx/Makefile > > > > > @@ -202,10 +202,10 @@ append = cat $(filter-out $< $(PHONY), $^) >> $@ > > > > > ? quiet_cmd_pad_cat = CAT???? $@ > > > > > ? cmd_pad_cat = $(cmd_objcopy) && $(append) || rm -f $@ > > > > > -u-boot-with-spl.imx: SPL u-boot.uim FORCE > > > > > +u-boot-with-spl.imx: SPL $(if > > > > > $(CONFIG_OF_SEPARATE),u-boot.img,u-boot.uim) FORCE > > > > > ????? $(call if_changed,pad_cat) > > > > > -u-boot-with-nand-spl.imx: spl/u-boot-nand-spl.imx u-boot.uim FORCE > > > > > +u-boot-with-nand-spl.imx: spl/u-boot-nand-spl.imx $(if > > > > > $(CONFIG_OF_SEPARATE),u-boot.img,u-boot.uim) FORCE > > > > > ????? $(call if_changed,pad_cat) > > > > > ? quiet_cmd_u-boot-nand-spl_imx = GEN???? $@ > > > > > > > > > > > > > This patch breaks build for two boards, udoo and ot1200_spl. > > > > Strange enough, I get weird results with git bisect (but > > > > definetely this is the only patch I am applying that cause the > > > > issue). > > > > > > > > Could you take a look, please ? > > > > > > There is V3 already, does that work? > > > [PATCH V3] ARM: imx: Include u-boot.img in u-boot-with-spl.imx if > > > OF_SEPARATE=y > > > > I pick it up and let you know, thanks. > > > > Sorry, I answered to V2 thread, but I had already picked up V3, and this > causes the issue with udoo and ot1200_spl. Please note that ot1200 has today been ack'd for removal and udoo is on the removal list: https://patchwork.ozlabs.org/project/uboot/patch/20210221010634.21310-24-trini at konsulko.com/ -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 659 bytes Desc: not available URL: