public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot-Users] latest (1.3.1) doesn't compile: Makefile issue
@ 2008-01-23 19:15 Tales Toledo
  2008-01-23 20:16 ` Jon Loeliger
  0 siblings, 1 reply; 4+ messages in thread
From: Tales Toledo @ 2008-01-23 19:15 UTC (permalink / raw)
  To: u-boot

Hi

I saw a previous discussion but I think main reason why 1.3.1 doesn't
compile is different from mentioned there.
I had the same error trying to compile to EP88x with eldk 3.1 toolchains.

It seems that /common/Makefile conditional CONFIG_CMD_FLASH isn't working fine.
It should be set at previous configuration files like config.mk but it
is just set at header files.

I just change the line as follow I got it work.

#COBJS-$(CONFIG_CMD_FLASH) += cmd_flash.o
COBJS-y += cmd_flash.o

I'm not familiar with this new Makefile style since I have used old
u-boot versions.
Any help to make this work properly is appreciated.

Thx,
TT

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot-Users] latest (1.3.1) doesn't compile: Makefile issue
  2008-01-23 19:15 [U-Boot-Users] latest (1.3.1) doesn't compile: Makefile issue Tales Toledo
@ 2008-01-23 20:16 ` Jon Loeliger
  2008-01-24 18:40   ` Tales Toledo
  0 siblings, 1 reply; 4+ messages in thread
From: Jon Loeliger @ 2008-01-23 20:16 UTC (permalink / raw)
  To: u-boot

Tales Toledo wrote:
> Hi
> 
> I saw a previous discussion but I think main reason why 1.3.1 doesn't
> compile is different from mentioned there.
> I had the same error trying to compile to EP88x with eldk 3.1 toolchains.
> 
> It seems that /common/Makefile conditional CONFIG_CMD_FLASH isn't working fine.
> It should be set at previous configuration files like config.mk but it
> is just set at header files.
> 
> I just change the line as follow I got it work.
> 
> #COBJS-$(CONFIG_CMD_FLASH) += cmd_flash.o
> COBJS-y += cmd_flash.o
> 
> I'm not familiar with this new Makefile style since I have used old
> u-boot versions.
> Any help to make this work properly is appreciated.
> 

The line you commented-out is the _new_ U-Boot
Makefile style.  Expect much, much more of it!

If your port needs the cmd_flash file, then you
need to ensure that CONFIG_CMD_FLASH is #define'd
in your port config .h file.  Then, ensure that it
is also appearing in the autoconf.mk file after it
is generated too.

jdl

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot-Users] latest (1.3.1) doesn't compile: Makefile issue
  2008-01-23 20:16 ` Jon Loeliger
@ 2008-01-24 18:40   ` Tales Toledo
  2008-01-24 23:17     ` Wolfgang Denk
  0 siblings, 1 reply; 4+ messages in thread
From: Tales Toledo @ 2008-01-24 18:40 UTC (permalink / raw)
  To: u-boot

On Jan 23, 2008 6:16 PM, Jon Loeliger <jdl@freescale.com> wrote:
>
> Tales Toledo wrote:
> > Hi
> >
> > I saw a previous discussion but I think main reason why 1.3.1 doesn't
> > compile is different from mentioned there.
> > I had the same error trying to compile to EP88x with eldk 3.1 toolchains.
> >
> > It seems that /common/Makefile conditional CONFIG_CMD_FLASH isn't working fine.
> > It should be set at previous configuration files like config.mk but it
> > is just set at header files.
> >
> > I just change the line as follow I got it work.
> >
> > #COBJS-$(CONFIG_CMD_FLASH) += cmd_flash.o
> > COBJS-y += cmd_flash.o
> >
> > I'm not familiar with this new Makefile style since I have used old
> > u-boot versions.
> > Any help to make this work properly is appreciated.
> >
>
> The line you commented-out is the _new_ U-Boot
> Makefile style.  Expect much, much more of it!
>
> If your port needs the cmd_flash file, then you
> need to ensure that CONFIG_CMD_FLASH is #define'd
> in your port config .h file.

EP88x.h includes config_cmd_defaults.h and CONFIG_CMD_FLASH is defined there.

  Then, ensure that it
> is also appearing in the autoconf.mk file after it
> is generated too.

I didn't see any include/autoconf.mk file, only
include/autoconf.mk.dep and it was empty.

toledo at work:~/toledo/u-boot$ make all
Generating include/autoconf.mk
ppc_8xx-gcc: compilation of header file requested
for dir in tools examples ; do make -C $dir _depend ; done
make[1]: Entering directory `/home/toledo/u-boot/tools'
ln -s ../common/environment.c environment.c
ln -s ../lib_generic/crc32.c crc32.c
ln -s ../lib_generic/sha1.c sha1.c

tt
>
> jdl
>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot-Users] latest (1.3.1) doesn't compile: Makefile issue
  2008-01-24 18:40   ` Tales Toledo
@ 2008-01-24 23:17     ` Wolfgang Denk
  0 siblings, 0 replies; 4+ messages in thread
From: Wolfgang Denk @ 2008-01-24 23:17 UTC (permalink / raw)
  To: u-boot

In message <2308fb0f0801241040o2ef82592m63fef2cdca4c38dc@mail.gmail.com> you wrote:
>
> toledo at work:~/toledo/u-boot$ make all
> Generating include/autoconf.mk
> ppc_8xx-gcc: compilation of header file requested
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Please use recent code instead, or a more recent toolchain.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Heavier than air flying machines are impossible.
                    -- Lord Kelvin, President, Royal Society, c. 1895

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2008-01-24 23:17 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-01-23 19:15 [U-Boot-Users] latest (1.3.1) doesn't compile: Makefile issue Tales Toledo
2008-01-23 20:16 ` Jon Loeliger
2008-01-24 18:40   ` Tales Toledo
2008-01-24 23:17     ` Wolfgang Denk

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox