From mboxrd@z Thu Jan  1 00:00:00 1970
From: Peter Robinson <pbrobinson@gmail.com>
Date: Mon, 1 Mar 2021 20:21:56 +0000
Subject: [PATCH V2] ARM: imx: Include u-boot.img in u-boot-with-spl.imx if
 OF_SEPARATE=y
In-Reply-To: <20210301194203.GH10169@bill-the-cat>
References: <20210201200725.49959-1-marex@denx.de>
 <4d8a1d9c-ed5e-06c6-5cd9-f24a450837d8@denx.de>
 <08057cd5-47fa-33f7-51d0-4583a32020e5@denx.de>
 <baedff14-0b84-be5c-255a-cc07af56d596@denx.de>
 <2f08deff-d957-5934-c69f-f314830f396f@denx.de>
 <20210301175657.GF10169@bill-the-cat>
 <CALeDE9NkM48Ev3ZYAK255vtj8o37ftRLAFWDtjCDC3GQBNrBBA@mail.gmail.com>
 <20210301194203.GH10169@bill-the-cat>
Message-ID: <CALeDE9OXRFH+jdEr31uRbg4QTo-Uo2+NF61oUkLiKCWG0ng84w@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Mon, Mar 1, 2021 at 7:42 PM Tom Rini <trini@konsulko.com> wrote:
>
> On Mon, Mar 01, 2021 at 07:28:17PM +0000, Peter Robinson wrote:
> > On Mon, Mar 1, 2021 at 5:57 PM Tom Rini <trini@konsulko.com> wrote:
> > >
> > > 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:
> >
> > FYI I am working to update the udoo and the udoo_neo so they're not
> > removed as we support those boards in Fedora, I've still working my
> > way through it with my schedule.
>
> Please find time soon.  I'm spelling this out more in my -rc email
> (which has been delayed in that I expect an imx PR and so I want to get
> that, then release) but I will be dropping stuff post v2021.04 release.

I was planning on having patches in time for the next merge window.