public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Daniel Hobi <daniel.hobi@schmid-telecom.ch>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support
Date: Mon, 15 Nov 2010 15:15:53 +0100	[thread overview]
Message-ID: <4CE14099.1070105@schmid-telecom.ch> (raw)
In-Reply-To: <20101109234346.GF17882@localhost>

Hi Eric,

On 10.11.2010 00:43, Eric Cooper wrote:
> I have been periodically rebasing my patches for the Seagate DockStar
> on master.  But ever since the elf_reloc changes were merged, I have
> been unable to get a working build.  First I saw symptoms similar to
> what Alexander Holler reported (failing during NAND initialization due
> to incorrect BSS relocation), but the latest arm926ejs ld script
> patches did not fix it.  The system would hang here:
> 
>     U-Boot 2010.12-rc1 (Nov 09 2010 - 17:52:38)
>     Seagate FreeAgent DockStar
> 
>     SoC:   Kirkwood 88F6281_A0
>     DRAM:  128 MiB
>     NAND:  
> 
> This was built with the CodeSourcery 2010q1 toolchain.  I also
> included Alexander's patch to double-check the relocation, and no
> error message gets printed.

Did you locally fix arch/arm/cpu/arm926ejs/kirkwood/timer.c to not
access uninitialized data (variables timestamp and lastdec) before
relocation?

Otherwise timer_init() will write into the relocation table which may
lead to your problem. If you change unrelated code, timer_init() will
overwrite some other relocation table entries and U-Boot seems to work.

@Albert: If your patch is applied without fixing the accesses to BSS
before relocation, some ARM ports will be broken even when compiling
with ELDK 4.2.

Best regards,
Daniel

  parent reply	other threads:[~2010-11-15 14:15 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-04 22:22 [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support Albert Aribaud
2010-11-04 22:22 ` [U-Boot] [PATCH V4 2/2] tx25: " Albert Aribaud
2010-11-04 22:27 ` [U-Boot] [PATCH V4 1/2] arm926ejs: " Albert ARIBAUD
2010-11-09 13:49   ` Albert ARIBAUD
2010-11-05  8:38 ` Reinhard Meyer
2010-11-05  9:04   ` Albert ARIBAUD
2010-11-09 18:24 ` Daniel Hobi
2010-11-09 18:47   ` Albert ARIBAUD
2010-11-09 18:55     ` Albert ARIBAUD
2010-11-09 19:30     ` Andreas Bießmann
2010-11-10 12:31     ` Daniel Hobi
2010-11-10 12:48       ` Albert ARIBAUD
2010-11-10 13:24         ` Daniel Hobi
2010-11-11  8:11           ` Albert ARIBAUD
2010-11-14 21:22             ` Wolfgang Denk
2010-11-15 11:01               ` Albert ARIBAUD
2010-11-15 11:09                 ` Andreas Bießmann
2010-11-15 11:43                   ` Sebastien Carlier
2010-11-16  7:38                     ` Andreas Bießmann
2010-11-15 11:13                 ` Wolfgang Denk
2010-11-15 11:49                   ` Albert ARIBAUD
2010-11-15 11:55                     ` Loïc Minier
2010-11-15 12:03                       ` Wolfgang Denk
2010-11-15 14:06                         ` Loïc Minier
2010-11-15 14:29                           ` Albert ARIBAUD
2010-11-15 15:09                             ` Loïc Minier
2010-11-16 13:42                               ` Peter Maydell
2010-11-16 13:54                                 ` [U-Boot] qemu-arm (was: [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support) Albert ARIBAUD
2010-11-16 14:49                                   ` Peter Maydell
2010-11-16 16:02                                     ` [U-Boot] qemu-arm Albert ARIBAUD
2010-11-15 12:00                     ` [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support Reinhard Meyer
2010-11-15 12:02                     ` Wolfgang Denk
2010-11-15 12:13                       ` Albert ARIBAUD
2010-11-15 12:16                     ` Bas Mevissen
2010-11-10 13:02       ` [U-Boot] Timer Statics (was: arm926ejs: fix linker file for newer ld support) Reinhard Meyer
2010-11-09 19:27   ` [U-Boot] [PATCH V4 1/2] arm926ejs: fix linker file for newer ld support Andreas Bießmann
2010-11-09 19:31     ` Albert ARIBAUD
2010-11-09 23:43 ` Eric Cooper
2010-11-10  7:53   ` Albert ARIBAUD
2010-11-10 14:20     ` Eric Cooper
2010-11-11  7:33       ` Albert ARIBAUD
2010-11-15 14:15   ` Daniel Hobi [this message]
2010-11-15 14:37     ` Albert ARIBAUD
2010-11-15 19:14     ` Eric Cooper

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=4CE14099.1070105@schmid-telecom.ch \
    --to=daniel.hobi@schmid-telecom.ch \
    --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