public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Heiko Schocher <hs@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] arm926ejs, suen3: fix relocation issues
Date: Tue, 19 Oct 2010 10:06:38 +0200	[thread overview]
Message-ID: <4CBD518E.5020701@denx.de> (raw)
In-Reply-To: <F766E4F80769BD478052FB6533FA745D19A6A461A7@SC-VEXCH4.marvell.com>

Helo Prafulla,

Prafulla Wadaskar wrote:
>> -----Original Message-----
>> From: Heiko Schocher [mailto:hs at denx.de] 
>> Sent: Monday, October 18, 2010 11:43 AM
>> To: u-boot at lists.denx.de
>> Cc: Heiko Schocher; Prafulla Wadaskar
>> Subject: [PATCH] arm926ejs, suen3: fix relocation issues
>>
> ...snip...
> 
>> diff --git a/include/configs/km_arm.h b/include/configs/km_arm.h
>> index faa0f50..9270296 100644
>> --- a/include/configs/km_arm.h
>> +++ b/include/configs/km_arm.h
>> @@ -182,6 +182,8 @@ int get_scl (void);
>>  
>>  /* additions for new relocation code, must be added to all boards */
>>  #define CONFIG_SYS_SDRAM_BASE		0x00000000
>> -#define CONFIG_SYS_INIT_SP_ADDR		(0x00000000 + 
>> 0x1000 - /* Fix this */ \
>> -					CONFIG_SYS_GBL_DATA_SIZE)
>> +/* Kirkwood has 2k of Security SRAM, use it for SP */
>> +#define CONFIG_SYS_INIT_SP_ADDR		0xC8012000
>> +/* Do early setups now in board_init_f() */
>> +#define CONFIG_BOARD_EARLY_INIT_F	1
> 
> Do you really need to 1 here ?

No, I fix this in v2.

> Also can you pls confirm that you are getting early console message as pointed by Eric Cooper
> http://lists.denx.de/pipermail/u-boot/2010-October/079545.html

Hmm.. don;t know if we can compare this, because the suen3 starts
from spi flash ... and I see this messages on the suen3 ... Maybe,
Eric (added to cc: ) should also try to do early things
(like kirkwood_mpp_conf()) in board_early_init_f() as I on the
suen3?

bye,
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

  reply	other threads:[~2010-10-19  8:06 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-10-18  6:13 [U-Boot] [PATCH] arm926ejs, suen3: fix relocation issues Heiko Schocher
2010-10-19  6:50 ` Prafulla Wadaskar
2010-10-19  8:06   ` Heiko Schocher [this message]
2010-10-19 10:43     ` Eric Cooper
2010-10-19 11:04       ` Prafulla Wadaskar
2010-10-20 10:21       ` Prafulla Wadaskar
2010-10-19  8:08 ` [U-Boot] [PATCH v2] " Heiko Schocher
2010-10-20  5:37   ` Prafulla Wadaskar
2010-10-20  9:51   ` Prafulla Wadaskar

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=4CBD518E.5020701@denx.de \
    --to=hs@denx.de \
    --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