[U-Boot] pull request for u-boot-tegra/master

Tom Warren
2012-08-07T16:03:07Z

Albert,

> -----Original Message-----
> From: Albert ARIBAUD [mailto:albert.u.boot at aribaud.net]
> Sent: Monday, August 06, 2012 11:53 PM
> To: Allen Martin
> Cc: Tom Warren; Tom Warren; Simon at theia.denx.de; u-boot at lists.denx.de;
> Stephen Warren
> Subject: Re: [U-Boot] pull request for u-boot-tegra/master
>
> Hi Allen,
>
> On Mon, 6 Aug 2012 15:46:22 -0700, Allen Martin <amartin@nvidia.com> wrote:
> > > > -----Original Message-----
> > > > From: Albert ARIBAUD [mailto:albert.u.boot at aribaud.net]
> > > > Sent: Sunday, August 05, 2012 1:01 PM
> > > > To: Albert ARIBAUD
> > > > Cc: Tom Warren; Simon at theia.denx.de; u-boot at lists.denx.de; Stephen
> > > > Warren; Tom Warren
> > > > Subject: Re: [U-Boot] pull request for u-boot-tegra/master
> > > >
> > > > Hi Albert,
> > > >
> > > > On Sun, 5 Aug 2012 21:46:21 +0200, Albert ARIBAUD
> > > > <albert.u.boot@aribaud.net> wrote:
> > > > > Hi Tom,
> > > > >
> > > > > On Fri, 3 Aug 2012 16:35:40 -0700, Tom Warren
> > > > > <twarren.nvidia@gmail.com>
> > > > wrote:
> > > > > > Albert,
> > > > > >
> > > > > > Has this been pulled into ARM master? I don't think I saw a
> > > > > > response from you, but I might have missed it.
> > > > >
> > > > > Sorry Tom, but I have been unavailable for some time and
> > > > > especially on jul 30 and 31, so no, it has not been pulled in.
> > > > > :(
> > > > >
> > > > > I am pulling it in right now, but I have sparse access to
> > > > > Internet right
> > > > now.
> > > >
> > > > Seems like f414d013c5e129e66c04ff699bd60b34894d3d38 breaks a lot
> > > > of builds with the stock Ubuntu linaro toolchain, e.g.
> > > >
> > > > ------------------------------------------------------------------
> > > > ----------
> > > > ----------
> > > >
> > > > $ ./MAKEALL jornada
> > > > Configuring for jornada board...
> > > > make: *** [jornada_config] Error 1
> > > > arm-linux-gnueabi-size: './u-boot': No such file
> > > > make: *** No rule to make target `include/autoconf.mk', needed by
> `depend'.
> > > > Stop.
> > > > make: *** Waiting for unfinished jobs....
> > > >
> >
> > I think I found the problem. For some reason I don't fully understand
> > when a board doesn't have ${soc} defined the following code in mkconfig:
> >
> > ( echo "ARCH = ${arch}"
> > if [ ! -z "$spl_cpu" ] ; then
> > echo 'ifeq ($(CONFIG_SPL_BUILD),y)'
> > echo "CPU = ${spl_cpu}"
> > echo "else"
> > echo "CPU = ${cpu}"
> > echo "endif"
> > else
> > echo "CPU = ${cpu}"
> > fi
> > echo "BOARD = ${board}"
> >
> > [ "${vendor}" ] && echo "VENDOR = ${vendor}"
> > [ "${soc}" ] && echo "SOC = ${soc}" ) > config.mk
> >
> > causes the subshell to return a nonzero exit status. I think the
> > behavior may change depending on what shell you have /bin/sh pointing
> > to. Can you try out the following fix to verify? I'll put together a
> > proper patch in the mean time
> >
> > diff --git a/mkconfig b/mkconfig
> > index 07031c6..d3363c6 100755
> > --- a/mkconfig
> > +++ b/mkconfig
> > @@ -144,7 +144,8 @@ fi
> > echo "BOARD = ${board}"
> >
> > [ "${vendor}" ] && echo "VENDOR = ${vendor}"
> > - [ "${soc}" ] && echo "SOC = ${soc}" ) > config.mk
> > + [ "${soc}" ] && echo "SOC = ${soc}"
> > + exit 0 ) > config.mk
> >
> > # Assign board directory to BOARDIR variable if [ -z "${vendor}" ] ;
> > then
>
> Seems to work with this fix applied on top of the current tegra/master.
>
> Tom, can you fix the branch so that the original commit (f414d013...)
> contains the fix above, rather than just adding the fix on top of the
> branch? That'll help any bisecting through this branch's commits.

Will do. I'll send a new pull request when I've fixed & uploaded a new u-boot-tegra/master.

Thanks,

Tom
>
> Also, Allen:
>
> > ----------------------------------------------------------------------
> > ------------- This email message is for the sole use of the intended
> > recipient(s) and may contain confidential information. Any
> > unauthorized review, use, disclosure or distribution is prohibited.
> > If you are not the intended recipient, please contact the sender by
> > reply email and destroy all copies of the original message.
> > ----------------------------------------------------------------------
> > -------------
>
> Can this be removed from future e-mails to the list?
>
> Amicalement,
> --
> Albert.
--
nvpublic