public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Mark Jackson <mpfj-list@newflow.co.uk>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.
Date: Mon, 17 Jun 2013 17:10:28 +0100	[thread overview]
Message-ID: <51BF34F4.7060106@newflow.co.uk> (raw)
In-Reply-To: <51BF32D9.1090606@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.

  reply	other threads:[~2013-06-17 16:10 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-13 18:28 [U-Boot] [PATCH v2 1/7] am33xx/omap3: Clean up gpmc_init slightly Tom Rini
2013-05-13 18:28 ` [U-Boot] [PATCH v2 2/7] am335x_evm: Drop useless CONFIG_ENV_IS_NOWHERE Tom Rini
2013-05-13 18:28 ` [U-Boot] [PATCH v2 3/7] am335x_evm: Update SPI_BOOT support, add MTDPARTS info Tom Rini
2013-05-15 20:41   ` Peter Korsgaard
2013-05-13 18:28 ` [U-Boot] [PATCH v2 4/7] am335x_evm: Only set CONFIG_NAND when !CONFIG_SPI_BOOT Tom Rini
2013-05-15 20:42   ` Peter Korsgaard
2013-05-13 18:28 ` [U-Boot] [PATCH v2 5/7] am335x_evm: Rework board_is_foo() checks Tom Rini
2013-05-15 20:43   ` Peter Korsgaard
2013-05-13 18:28 ` [U-Boot] [PATCH v2 6/7] am335x_evm: Add support for the NOR module on the memory cape Tom Rini
2013-05-16 14:32   ` Peter Korsgaard
2013-05-16 14:46     ` Tom Rini
2013-05-16 18:54       ` Peter Korsgaard
2013-05-16 19:27         ` Tom Rini
2013-05-16 19:32           ` Peter Korsgaard
2013-05-13 18:28 ` [U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR Tom Rini
2013-05-16 14:36   ` Peter Korsgaard
2013-05-16 14:49     ` Tom Rini
2013-05-16 18:55       ` Peter Korsgaard
2013-05-16 19:30         ` Tom Rini
2013-05-16 19:32           ` Peter Korsgaard
2013-06-17 14:43   ` Mark Jackson
2013-06-17 14:49     ` Tom Rini
2013-06-17 14:59       ` Mark Jackson
2013-06-17 15:01         ` Tom Rini
2013-06-17 15:23           ` Mark Jackson
2013-06-17 15:23         ` Mark Jackson
2013-06-17 15:29           ` Tom Rini
2013-06-17 16:01           ` Stefan Roese
2013-06-17 16:10             ` Mark Jackson [this message]
2013-06-17 17:04               ` Kipisz, Steven
2013-06-17 18:38                 ` Mark Jackson
2013-06-18 12:11     ` Mark Jackson
2013-07-11 13:06       ` Mark Jackson
2013-07-11 13:28         ` Tom Rini
2013-07-11 13:45           ` Mark Jackson
2013-07-11 14:34             ` Albert ARIBAUD
2013-07-11 15:54               ` Tom Rini
2013-07-11 16:08                 ` Albert ARIBAUD
2013-07-11 16:14                   ` Albert ARIBAUD
2013-07-11 16:17                     ` Tom Rini
2013-07-17 12:58         ` Mark Jackson
2013-07-17 13:32           ` Tom Rini
2013-05-15 20:39 ` [U-Boot] [PATCH v2 1/7] am33xx/omap3: Clean up gpmc_init slightly Peter Korsgaard

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=51BF34F4.7060106@newflow.co.uk \
    --to=mpfj-list@newflow.co.uk \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox