From mboxrd@z Thu Jan  1 00:00:00 1970
From: Albert ARIBAUD <albert.u.boot@aribaud.net>
Date: Sat, 10 Nov 2012 15:30:04 +0100
Subject: [U-Boot] [PATCH v2 0/2] Factorize ARM startup code as mush as
 possible.
In-Reply-To: <20121108142028.GA5738@Hardy>
References: <1352001421-25913-1-git-send-email-albert.u.boot@aribaud.net>
	<1352028725-26683-1-git-send-email-albert.u.boot@aribaud.net>
	<20121104124312.0f627204@lilith>
	<20121104173832.GA32605@bill-the-cat> <20121105073925.GA8384@Hardy>
	<20121108142028.GA5738@Hardy>
Message-ID: <20121110153004.7fd0f923@lilith>
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

Hi Sughosh,

On Thu, 8 Nov 2012 19:50:28 +0530, Sughosh Ganu
<urwithsughosh@gmail.com> wrote:

> hi Albert,
> 
> On Mon Nov 05, 2012 at 01:09:25PM +0530, Sughosh Ganu wrote:
> > On Sun Nov 04, 2012 at 10:38:32AM -0700, Tom Rini wrote:
> > > On Sun, Nov 04, 2012 at 12:43:12PM +0100, Albert ARIBAUD wrote:
> > > 
> > > > Hi Tom,
> > > > 
> > > > On Sun,  4 Nov 2012 12:32:03 +0100, Albert ARIBAUD
> > > > <albert.u.boot@aribaud.net> wrote:
> > > > 
> > > > > The goal of this series is to scrub the start.S files
> > > > > which have proliferated across arch/arm and eliminate
> > > > > code redundancy.
> > > > 
> > > > I know this came a bit late in early nov 4th, but I really would like
> > > > it to be considered for 2013.01. Would you agree to make an exception
> > > > for it? Thanks in advance.
> > > 
> > > If you can collect a diverse set of Tested-by's, yes.
> > 
> > I have not gone through the patch, but will test it in a day or two on
> > the hawkboard, and report the findings.
> > 
> > The hawkboard comes with the arm926ejs core, so that part of the code
> > would be tested.
> 
> I tried the 1st patch of the series, and with that u-boot does not
> come up on the board. It is also printing out some random values for
> the dram and nand sizes.
> 
> The patch was applied on top of commit 1cc619be8b7. Also, with the
> mentioned commit, u-boot boots up fine on the board. Also to be noted
> is that the spl image compiled with these changes is booting up fine,
> loading the main u-boot image, and jumping to it -- the issue is with
> booting the main u-boot image.

Thanks Sughosh. Can you build an U-Boot with the following defined
in the hawkboard.h config file?

#define DEBUG
#if defined(CONFIG_SPL_BUILD) && ! defined (__ASSEMBLY__)
static inline int printf(const char *fmt, ...)
{
        return 0;
}
#endif

Note: only the #define DEBUG matter to me, but with it alone, SPL build
fails due to some code now requiring printf(). This is why I add a
dummy printf definition for C code during SPL build -- ASM code does not
need printf() and actually chokes on the definition, hence the
condition on __ASSEMBLY__.

This debug U-Boot should print a lot more info. Can you please try it
and copy/paste its output here? Thanks in advance.

Amicalement,
-- 
Albert.