From: Heiko Schocher <hs@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] arm: Don't use printf() in SPL builds
Date: Thu, 15 Mar 2012 06:29:54 +0100 [thread overview]
Message-ID: <4F617E52.7070504@denx.de> (raw)
In-Reply-To: <CAPnjgZ14AvAbddtvj1e2G_Zq0hvya82skTZntcy_qBar=sw=uw@mail.gmail.com>
Hello Simon,
Simon Glass wrote:
> Hi Heiko,
>
> On Wed, Mar 14, 2012 at 10:13 PM, Heiko Schocher <hs@denx.de> wrote:
>> Hello Simon,
>>
>> Simon Glass wrote:
>>> Hi Sughosh,
>>>
>>> On Sat, Mar 10, 2012 at 12:33 AM, Sughosh Ganu <urwithsughosh@gmail.com> wrote:
>>>> hi Simon,
>>>>
>>>> On Sat Mar 03, 2012 at 12:33:15PM -0800, Simon Glass wrote:
[...]
>>>> Can you please check if libcommon support is needed for the
>>>> cam_enc_4xx board -- your relocation patch series did not add
>>>> CONFIG_SPL_LIBCOMMON_SUPPORT to this board. This will affect my
>>>> patch to initialise gd and bd objects for hawkboard. Please let me
>>>> know, and i will spin a patch accordingly.
>>> Yes I think this board needs both of these:
>>>
>>> #define CONFIG_SPL_LIBCOMMON_SUPPORT
>>> #define CONFIG_SPL_LIBGENERIC_SUPPORT
>> I posted a patch, which adds CONFIG_SPL_LIBGENERIC_SUPPORT, see
>>
>> http://patchwork.ozlabs.org/patch/145280/
>>
>> for this board. Build/works fine without CONFIG_SPL_LIBCOMMON_SUPPORT
>> maybe I miss something?
>>
>> Builded with ELDK-4.2 and ELDK-5.1 based on commit:
>>
>> commit e37ae40e9dec9af417c19de72f76becebf160730
>> Author: Stephen Warren <swarren@nvidia.com>
>> Date: Tue Nov 1 06:28:21 2011 +0000
>>
>> image: Support FDTs already loaded at their load address
>
> Yes I am talking about building with the generic relocation series
> included, so a slightly different point. With that I get:
>
> $ make -j8 -s
> Generating include/generated/asm-offsets.h
> arch/arm/cpu/arm926ejs/davinci/libdavinci.o: In function `board_init_f':
> /home/sjg/trunk/src/third_party/u-boot/files/arch/arm/cpu/arm926ejs/davinci/spl.c:75:
> undefined reference to `relocate_code'
> make[1]: *** [/home/sjg/trunk/src/third_party/u-boot/files/spl/u-boot-spl]
> Error 1
> make: *** [spl/u-boot-spl.bin] Error 2
> make: *** Waiting for unfinished jobs....
Ah, Ok. Currently I have no access to the hw, so I could not test
this :-(
bye,
Heiko
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
next prev parent reply other threads:[~2012-03-15 5:29 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-03 20:33 [U-Boot] [PATCH] arm: Don't use printf() in SPL builds Simon Glass
2012-03-05 8:21 ` Stefan Roese
2012-03-10 8:33 ` Sughosh Ganu
2012-03-15 3:47 ` Simon Glass
2012-03-15 5:13 ` Heiko Schocher
2012-03-15 5:23 ` Simon Glass
2012-03-15 5:29 ` Heiko Schocher [this message]
2012-03-15 7:37 ` Christian Riesch
2012-03-15 16:23 ` Simon Glass
2012-03-16 18:20 ` Sughosh Ganu
2012-05-07 12:28 ` Stefan Roese
2012-05-07 22:51 ` Tom Rini
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=4F617E52.7070504@denx.de \
--to=hs@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.