All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefano Babic <sbabic@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC] ARM: mx31pdk: Use the new relocation scheme
Date: Thu, 13 Jan 2011 14:38:19 +0100	[thread overview]
Message-ID: <4D2F004B.9090204@denx.de> (raw)
In-Reply-To: <1294836544-24512-1-git-send-email-fabio.estevam@freescale.com>

On 01/12/2011 01:49 PM, Fabio Estevam wrote:
> Hi,

Hi Fabio,

> 
> I am trying to make the new relocation scheme to work on the mx31pdk board.
> 
> With this patch applied the mx31pdk build works, but not able to get a U-boot prompt yet.

I have tried to imagine some issues, but I cannot find a clear reason.
Building with your patch I checked that nand_spl is still less than 2KB
code, and that matches the NAND page, as required by i.MX31 - so it is
not a problem.

Have you tried to disable the relocation, just to check if the
relocation is an issue with CONFIG_SKIP_RELOCATE_UBOOT ? This is not a
solution, but only to check what happens.

> 
> I copied the nand_spl/board/karo/tx25/u-boot.lds to the mx31pdk one.
> 
> I am not certain whether the CONFIG_SYS_TEXT_BASE should be changed or not:
> 
> ifdef CONFIG_NAND_SPL
> CONFIG_SYS_TEXT_BASE = 0x87ec0000
> else
> CONFIG_SYS_TEXT_BASE = 0x87f00000
> endif

I think they are ok - both addresses are valid RAM addresses. The
relocation will move then the code at the end of the RAM. And as I
understood from i.MX31 manual, the NFC controller does not use the IRAM
at all, so the area pointed by your starting stack pointer should not be
overwritten.

> 
> If anyone has any comments/suggestions I will appreciate it.

No great idea, really. Probably you need to attach a JTAG debugger and
see what happens.

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================

  reply	other threads:[~2011-01-13 13:38 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-12 12:49 [U-Boot] [RFC] ARM: mx31pdk: Use the new relocation scheme Fabio Estevam
2011-01-13 13:38 ` Stefano Babic [this message]
2011-01-14 18:33   ` Fabio Estevam
2011-02-06 12:03     ` Magnus Lilja
2011-02-06 15:07       ` Albert ARIBAUD
2011-02-07 11:32       ` Fabio Estevam
2011-02-07 14:13         ` Stefano Babic
2011-02-07 19:48           ` Magnus Lilja
2011-02-08 17:09             ` Fabio Estevam
2011-02-08 17:50               ` stefano babic
2011-02-08 19:26                 ` Magnus Lilja
2011-02-08 20:18                   ` stefano babic
2011-02-08 20:50                     ` Albert ARIBAUD
2011-02-09 11:45                       ` Fabio Estevam
2011-02-11 10:51                       ` Aneesh V
2011-02-11 12:46                         ` Albert ARIBAUD
2011-02-11 13:49                           ` Aneesh V

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=4D2F004B.9090204@denx.de \
    --to=sbabic@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 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.