From mboxrd@z Thu Jan  1 00:00:00 1970
From: Mark Jackson <mpfj-list@newflow.co.uk>
Date: Mon, 17 Jun 2013 17:10:28 +0100
Subject: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from
 NOR.
In-Reply-To: <51BF32D9.1090606@denx.de>
References: <1368469719-22406-1-git-send-email-trini@ti.com>
	<1368469719-22406-7-git-send-email-trini@ti.com>
	<51BF2089.90605@newflow.co.uk> <51BF2203.20206@ti.com>
	<51BF244E.9020603@newflow.co.uk> <51BF29F5.90905@newflow.co.uk>
	<51BF32D9.1090606@denx.de>
Message-ID: <51BF34F4.7060106@newflow.co.uk>
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 17/06/13 17:01, Stefan Roese wrote:
> Hi Mark,
> 
> On 17.06.2013 17:23, Mark Jackson wrote:

<snip>

>> One question ... the SPL code has:-
>>
>> 	...
>> 	gd = &gdata;
>> 	...
>>
>> But there seems to be no similar assignment when in NOR boot mode.
>> I'm no expert in the internal workings of u-boot, so I thought I'd
>> check, just in case !?!
> 
> I just noticed you mentioning "gd" here and wanted to point you to a
> patch that fixed a gd-related boot issue for me on OMAP3:
> 
> [U-Boot] [PATCH 1/3] arm: spl: Fix SPL booting for OMAP3
> 
> (sorry, I can't find a link to this patch right now and I'm in a hurry)
> 
> Not sure if this has something to do with your problem (I don't have the
> time to dig into this mail thread today as I'm leaving right now).

I'll look at that thanks.

But I've taken a slightly different approach, and added the following
to the start of s_init() ...



void s_init(void)
{
	unsigned short *p = (unsigned short*)0x08000000;
	while (1)
	{
		int i;
		for (i = 0; i < (1 * 1024); i++)
		{
			unsigned short d = p[i];
			d++;
			*p = d;
		}
	}
	...

AFAICT, that should just spin on reading/writing the NOR device,
but I still get only an initial 350us burst on the CS0 line, and
then nothing.

So I can only assume that the boot code isn't getting as far as
s_init()@all !!

In start.S and low_levelinit.c, there are various uses of:-

#ifdef CONFIG_SPL_BUILD

Do any of these also need referencing with a CONFIG_NOR_BOOT ?

Mark J.