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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox