All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andrejs Cainikovs <AndrejsC@GlobalAutomationSystems.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] AT91RM9200 relocation
Date: Thu, 04 Sep 2008 00:47:06 +0300	[thread overview]
Message-ID: <48BF05DA.5020904@GlobalAutomationSystems.com> (raw)
In-Reply-To: <48BE47E9.20208@GlobalAutomationSystems.com>


Pardon, appearantly this stopped working somewhere between 1.3.2 and 1.3.4.
Now I am able to load only from ram. 1.3.2 works fine, though.

Regards,
Andrejs Cainikovs.


Andrejs Cainikovs wrote:
> Nicolas,
>
> I faced the similar problem before. Playing around with 
> CONFIG_SKIP_RELOCATE_UBOOT and others don't gived any results.. The only 
> result was that you should change define you mentioned every time when 
> compiling for flash or ram. This has fixed my problem:
>
> --- a/cpu/arm920t/start.S      2008-09-03 10:50:21.000000000 +0300
> +++ b/cpu/arm920t/start.S     2008-05-19 15:22:00.000000000 +0300
> @@ -261,11 +261,10 @@
>          * find a lowlevel_init.S in your board directory.
>          */
>         mov     ip, lr
> -#if    defined(CONFIG_AT91RM9200DK) || defined(CONFIG_AT91RM9200EK) || 
> defined(CONFIG_AT91RM9200DF)
> -
> -#else
> -       bl      lowlevel_init
> -#endif
> +       adr     r0, _start              /* r0 <- current position of 
> code   */
> +       ldr     r1, _TEXT_BASE          /* test if we run from flash or 
> RAM */
> +       cmp     r0, r1                  /* don't init during 
> debug          */
> +       blne    lowlevel_init
>         mov     lr, ip
>         mov     pc, lr
>  #endif /* CONFIG_SKIP_LOWLEVEL_INIT */
>
>
> Regards,
> Andrejs Cainikovs.
>
>
>
> N?colas Carneiro Lebedenco wrote:
>   
>> Hi,
>>
>> I'm porting u-boot to a custom board based on the at91rm9200-dk (only 
>> exception is that only parallel nor flash is available on my board)
>>
>> Soon I realized that I could load u-boot 1.1.4 from flash but not u-boot 
>> 1.3.4. It really bugged me until I noticed some differences in 
>> cpu/arm920t/start.S and came across this patch
>> "Fix regression introduced by a typo in 'Tidied other..." introduced by 
>> Guennadi Liakhovetski in the main branch on 2008-04-18 as follows:
>>
>> diff --git a/cpu/arm920t/start.S b/cpu/arm920t/start.S
>>  index ae86002..acc00ad 100644 (file)
>>  
>> --- a/cpu/arm920t/start.S
>> +++ b/cpu/arm920t/start.S
>> @@ -178,7 +178,7 @@ copyex:
>>         bl      cpu_init_crit
>>  #endif
>>  
>> -#ifdef CONFIG_AT91RM9200
>> +#ifndef        CONFIG_AT91RM9200
>>  
>>  #ifndef CONFIG_SKIP_RELOCATE_UBOOT
>>  relocate:                              /* relocate U-Boot to RAM           */
>> --
>>
>> The point is I can only get u-boot to load from nor flash if I remove the 
>> #ifndef condition. So what was the reason behind this patch after all. Is the 
>> relocation supposed to take place somewhere else? Maybe someone has already 
>> pointed that out and got it fixed in the main branch?
>>
>> Regards,
>>
>> N?colas

  reply	other threads:[~2008-09-03 21:47 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-09-01 18:55 [U-Boot] AT91RM9200 relocation Nícolas Carneiro Lebedenco
2008-09-03  8:16 ` Andrejs Cainikovs
2008-09-03 21:47   ` Andrejs Cainikovs [this message]
2008-09-03 22:12     ` Wolfgang Denk

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=48BF05DA.5020904@GlobalAutomationSystems.com \
    --to=andrejsc@globalautomationsystems.com \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.