public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Albert ARIBAUD <albert.aribaud@free.fr>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH V2 3/3] Add support for the LaCie ED Mini V2 board
Date: Sun, 29 Nov 2009 12:22:31 +0100	[thread overview]
Message-ID: <4B125977.20704@free.fr> (raw)
In-Reply-To: <73173D32E9439E4ABB5151606C3E19E202F45D5585@SC-VEXCH1.marvell.com>

Prafulla Wadaskar a ?crit :

>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index d70a9d2..93e57f5 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -512,6 +512,10 @@ Unknown / orphaned boards:
>>  #	Board		CPU					
>> 	#
>>  
>> ##############################################################
>> ###########
>>  
>> +Albert ARIBAUD <albert.aribaud@free.fr>
>> +
>> +	edminiv2	ARM926EJS (Orion5x SoC)
>> +
> 
> That's good if you can move this entry above Albin Tonnerre's entry

Are you sure? Seems to me like this list is ordered by last, not first, 
name.

> How you are addressing DRAM configuration?
> I think you need lowlevel_init.

Yes. That will be in patch V3.

> ...snip...
>> +/*
>> + *  Environment variables configurations
>> + */
>> +#define CONFIG_ENV_IS_IN_FLASH		1
>> +#define CONFIG_ENV_SECT_SIZE		0x2000	/* 16K */
>> +#define CONFIG_ENV_SIZE			0x2000
>> +#define CONFIG_ENV_OFFSET		0x4000	/* env starts here */
> 
> I don't know how this is going to work, how much flash do you have?

51KB NOR located at fff88000 in the final mapping, and mirrored 
throughout f8000000-ffffffff at power-up. That's an MX29LV400CB, with a 
1x16kb,2x8kb,1x32kb,7x64kb region layout, of which U-boot utilizes the 7 
64K banks for code and the lowest 8K bank for environment.

> I hope it is mapped in the end since your reset address is 0xffff0000.

This *does* work, I step-debugged through the Orion init sequence. The 
fact is, there's ROM code in the Orion SoC apparently, at 0xffff0000, 
and which takes precedence over the boot CS. This ROM code checks the 
SoC device type and, if 5182, branches to 0xfff90000 (at this point a 
mirror of the boot CS).

> Where is the code to fetch flashed image from flash to DRAM at TEXT_BASE?

That's done by the start.S relocation code, subject to not defining 
CONFIG_SKIP_RELOCATION of course. Note that Orion has NOR flash, not NAND.

> I suggest you provide some more details about your board in the commit log

I will.

Amicalement,
-- 
Albert.

  reply	other threads:[~2009-11-29 11:22 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-11-14 23:03 [U-Boot] [PATCH V2 0/3] Support for Marvell Orion5x SoC and LaCie ED Mini V2 board Albert Aribaud
2009-11-14 23:03 ` [U-Boot] [PATCH V2 1/3] Initial support for Marvell Orion5x SoC Albert Aribaud
2009-11-14 23:03   ` [U-Boot] [PATCH V2 2/3] Add Orion5x support to 16550 serial device driver Albert Aribaud
2009-11-14 23:03     ` [U-Boot] [PATCH V2 3/3] Add support for the LaCie ED Mini V2 board Albert Aribaud
2009-11-16  9:05       ` Prafulla Wadaskar
2009-11-29 11:22         ` Albert ARIBAUD [this message]
2009-11-18 22:21       ` Wolfgang Denk
2009-11-19  7:51         ` Stefan Roese
2009-11-19 12:48           ` Albert ARIBAUD
2009-11-19 13:10             ` Stefan Roese
2009-11-28 22:21         ` Albert ARIBAUD
2009-11-29 12:56         ` Albert ARIBAUD
2009-12-05  0:35           ` Wolfgang Denk
2009-12-05  7:03             ` Albert ARIBAUD
2009-12-07 21:56               ` Wolfgang Denk
2009-12-11 15:51                 ` Albert ARIBAUD
2009-12-02  8:31         ` [U-Boot] Mixed 8/16 bits NOR FLASH support (was: [PATCH V2 3/3] Add support for the LaCie ED Mini V2 board) Albert ARIBAUD
2009-12-02  9:15           ` Stefan Roese
2009-12-02  9:38             ` Wolfgang Denk
2009-12-02 11:07               ` [U-Boot] Mixed 8/16 bits NOR FLASH support Albert ARIBAUD
2009-12-02 10:57             ` Albert ARIBAUD
2009-11-18 22:17   ` [U-Boot] [PATCH V2 1/3] Initial support for Marvell Orion5x SoC Wolfgang Denk
2009-11-18 22:51     ` Albert ARIBAUD
2009-11-29  8:55     ` Albert ARIBAUD
2009-12-05  0:37       ` Wolfgang Denk
2009-12-23  6:07         ` Prafulla Wadaskar
2009-11-29 10:34     ` Albert ARIBAUD

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=4B125977.20704@free.fr \
    --to=albert.aribaud@free.fr \
    --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