All of lore.kernel.org
 help / color / mirror / Atom feed
From: Aneesh V <aneesh@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] Board-specific commands unintentionally linked into SPL?
Date: Thu, 26 Jul 2012 13:41:07 -0700	[thread overview]
Message-ID: <5011AB63.8040101@ti.com> (raw)
In-Reply-To: <CANuwHqrFyKGBRGwPL1Ra_j=KsNi-y9651J0XqsNUGykbE2XVRw@mail.gmail.com>

Hi Tyler,

On 07/26/2012 11:54 AM, Tyler Olmstead wrote:
> Hi Christian,
>
> On Thu, Jul 26, 2012 at 10:03 AM, Christian Riesch
> <christian.riesch@omicron.at>  wrote:
>>
>> [cc'd Prabhakar Lad, Tom Rini, and Scott Wood]
>>
>> Tyler,
>>
>> On Thu, Jul 26, 2012 at 5:37 PM, Tyler Olmstead
>> <tyler.j.olmstead@gmail.com>  wrote:
>>> Hi all,
>>>
>>> I have encountered some issues adding a board-specific command to the
>>> board file of a project I have been working on. Specifically, after
>>> adding a U-Boot shell command to my board file, I have been seeing
>>> link-stage failures when attempting to build SPL.
>>
>> It's hard to tell without having your code, but I think this problem
>> was already discussed in [1]. However I do not remember how Prabhakar
>> solved it in the end.
>
> Yes, I ran into this thread while debugging the problem, which
> ultimately lead me to my solution. From that same thread [1], Wolfgang
> Denk writes:
>
> <quote>
>>
>> *I want to add a command using U_BOOT_CMD in uboot, where SPL_BUILD is
>> enabled for example for da850evm in spl frame work how can i do that *
>
> This makes no sense. Commands can only be executed when we have full
> U-Boot running (actually even only after relocation).  You cannot run
> commands in the SPL.
> </quote>
>
> I understand of course why it makes no sense to have command support
> in the SPL. However, the crux of this problem is that U-Boot and SPL
> both link in the same board object file, so in that sense compile-time
> switches wont work. From later in [1], Scott Wood writes:

No that's not correct. For SPL we build the object files into a
different directory "spl/". Please see the below in spl/Makefile

# We want the final binaries in this directory
obj := $(OBJTREE)/spl/

Object files used for U-Boot and SPL are not the same. You can use
compile-time switches and it should work just fine.

-Aneesh

  reply	other threads:[~2012-07-26 20:41 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-26 15:37 [U-Boot] Board-specific commands unintentionally linked into SPL? Tyler Olmstead
2012-07-26 17:03 ` Christian Riesch
2012-07-26 18:54   ` Tyler Olmstead
2012-07-26 20:41     ` Aneesh V [this message]
2012-07-26 23:02       ` Christian Riesch
2012-08-02 23:16         ` Tyler Olmstead
2012-08-03 16:40           ` Daniel Schwierzeck
2012-08-03 19:23             ` Tom Rini
2012-07-27 10:34     ` Lad, Prabhakar

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=5011AB63.8040101@ti.com \
    --to=aneesh@ti.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.