All of lore.kernel.org
 help / color / mirror / Atom feed
From: Helmut Raiger <helmut.raiger@hale.at>
To: u-boot@lists.denx.de
Subject: [U-Boot] SPL broken on i.mx31 platforms
Date: Wed, 13 Aug 2014 14:01:31 +0200	[thread overview]
Message-ID: <53EB539B.7050404@hale.at> (raw)
In-Reply-To: <CAM=E1R4NHZ=4=ycZBYj1cnhJnPRyPAmmJ3ONDHHmy0eK7wQwiQ@mail.gmail.com>

On 08/05/2014 02:32 PM, Magnus Lilja wrote:
> Hi Fabio,
>
> On 5 August 2014 14:28, Fabio Estevam <festevam@gmail.com> wrote:
>> Hi Magnus,
>>
>> On Mon, Aug 4, 2014 at 5:23 PM, Magnus Lilja <lilja.magnus@gmail.com> wrote:
>>
>>> I have now done some tests on i.MX31 PDK:
>>> * v2013.04 and v2014.04 works
>>> * v2014.07 and "tip of tree as of this mail" does not work
>>>
>>> To make it work (both v2014.07 and tip of tree) I only have to change
>>> the "ldr pc, _reset" to "b reset". I do not have to apply the nop-part
>>> of the patch.
>> Could you please submit this change as a formal patch to the list?
>> Please Cc Albert Aribaud on it.
> I would expect Helmut to create a formal patch then I can test that
> and add a Tested-by.
>
The problem is it does not work with only the 'b reset' change on my 
platform.
Should I provide a patch with the nops and the question marks around them?
It still could be a toolchain difference, mine is pretty old:

$ arm-angstrom-linux-gnueabi-gcc --version
arm-angstrom-linux-gnueabi-gcc (GCC) 4.7.2
Copyright (C) 2012 Free Software Foundation, Inc.

When I objdump the elf file I can see the very same code in 
cpu_init_crit() as in start.S,
whatever that might mean (objdump is from the same toolchain).

>>> On a different note I noticed that I received a warning that the PDK
>>> board does not use "generic board", I will try to fix this but I don't
>>> know how much work that is and when I can submit a patch.
>> Just add "#define CONFIG_SYS_GENERIC_BOARD" into the board config file
>> and give it a try.
>>
>> If it works, please send this patch as well (with Stefano Babic on Cc).
> That I can do.
>
> Regards, Magnus

I tested the CONFIG_SYS_GENERIC_BOARD on TT-01 and it works fine.

Helmut

--
Scanned by MailScanner.

  reply	other threads:[~2014-08-13 12:01 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-01 13:33 [U-Boot] SPL broken on i.mx31 platforms Helmut Raiger
2014-07-01 13:53 ` Fabio Estevam
2014-07-01 19:45   ` Albert ARIBAUD
2014-07-02  7:04     ` Helmut Raiger
2014-07-02 10:58       ` Albert ARIBAUD
2014-07-02 23:20       ` Benoît Thébaudeau
2014-07-03  8:19         ` Helmut Raiger
2014-07-03 13:35           ` Albert ARIBAUD
2014-07-03 20:58             ` Benoît Thébaudeau
2014-07-04  8:50               ` Albert ARIBAUD
2014-07-04 10:42                 ` Magnus Lilja
2014-07-04 21:42                   ` Benoît Thébaudeau
2014-07-04 21:29                 ` Benoît Thébaudeau
2014-07-03 20:58           ` Benoît Thébaudeau
2014-07-08  8:05             ` Helmut Raiger
2014-07-08 12:32               ` Helmut Raiger
2014-07-08 12:48                 ` Helmut Raiger
2014-07-09  8:22               ` [U-Boot] SPL broken on i.mx31 platforms, FIX? Helmut Raiger
2014-07-11  7:56               ` [U-Boot] SPL broken on i.mx31 platforms Magnus Lilja
2014-07-15  9:31                 ` Helmut Raiger
2014-08-04 20:23                   ` Magnus Lilja
2014-08-04 20:40                     ` Jeroen Hofstee
2014-08-05 12:28                     ` Fabio Estevam
2014-08-05 12:32                       ` Magnus Lilja
2014-08-13 12:01                         ` Helmut Raiger [this message]
2014-08-15 17:45                           ` Magnus Lilja
2014-08-19 20:55                             ` Benoît Thébaudeau
2014-08-19 22:01                               ` Fabio Estevam
2014-09-03  0:53                               ` Benoît Thébaudeau
2014-09-03  1:13                                 ` Tom Rini
2014-09-04 19:12                                   ` Magnus Lilja
2014-09-04 19:50                                     ` Benoît Thébaudeau
2014-09-04 19:59                                       ` Magnus Lilja
2014-07-02  8:11   ` Stefano Babic

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=53EB539B.7050404@hale.at \
    --to=helmut.raiger@hale.at \
    --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.