[U-Boot] [PATCH] Exclude drivers/qe/fdt.c without CONFIG_QE

[U-Boot] [PATCH] Exclude drivers/qe/fdt.c without CONFIG_QE

[Sebastien Carlier]

Previously with archive libraries fdt.o was compiled and included in
qe.a and then discarded by the linker.  With partial linking this
results in unresolved symbols, which this commit fixes.

This commit also cleans up a now-useless conditional in fdt.c.

Signed-off-by: Sebastien Carlier <sebastien.carlier@gmail.com>
---
 drivers/qe/Makefile |    2 +-
 drivers/qe/fdt.c    |    2 --
 2 files changed, 1 insertions(+), 3 deletions(-)

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Exclude-drivers-qe-fdt.c-without-CONFIG_QE.patch
Type: text/x-patch
Size: 896 bytes
Desc: not available
Url : http://lists.denx.de/pipermail/u-boot/attachments/20101118/759eee5a/attachment.bin 

[U-Boot] [PATCH] Exclude drivers/qe/fdt.c without CONFIG_QE

[Wolfgang Denk]

Dear Sebastien Carlier,

In message <1290078406-23921-1-git-send-email-sebastien.carlier@gmail.com> you wrote:
> This is a multi-part message in MIME format.
> --------------1.7.3.2.161.g3089c
> Content-Type: text/plain; charset=UTF-8; format=fixed
> Content-Transfer-Encoding: 8bit
> 
> 
> Previously with archive libraries fdt.o was compiled and included in
> qe.a and then discarded by the linker.  With partial linking this
> results in unresolved symbols, which this commit fixes.
> 
> This commit also cleans up a now-useless conditional in fdt.c.
> 
> Signed-off-by: Sebastien Carlier <sebastien.carlier@gmail.com>
> ---
>  drivers/qe/Makefile |    2 +-
>  drivers/qe/fdt.c    |    2 --
>  2 files changed, 1 insertions(+), 3 deletions(-)

This patch fixes building for the following boards:

MPC8308RDB, MPC8313ERDB_33, MPC8313ERDB_66, MPC8313ERDB_NAND_33,
MPC8313ERDB_NAND_66, MPC8315ERDB, MPC8315ERDB_NAND, MPC8349EMDS,
MPC8349ITX, MPC8349ITXGP, MPC8349ITX_LOWBOOT, MPC837XEMDS,
MPC837XEMDS_HOST, MPC837XERDB, MVBLM7, SIMPC8313_LP, SIMPC8313_SP,
TQM834x, mpc8308_p1m, sbc8349, sbc8349_PCI_33, sbc8349_PCI_66,
ve8313, vme8349

Tested-by: Wolfgang Denk <wd@denx.de>



Applied, thanks.

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
Not one hundred percent efficient, of course ... but nothing ever is.
	-- Kirk, "Metamorphosis", stardate 3219.8

[U-Boot] [PATCH] Exclude drivers/qe/fdt.c without CONFIG_QE

[Kumar Gala]

On Nov 18, 2010, at 5:06 AM, Sebastien Carlier wrote:

> 
> Previously with archive libraries fdt.o was compiled and included in
> qe.a and then discarded by the linker.  With partial linking this
> results in unresolved symbols, which this commit fixes.
> 
> This commit also cleans up a now-useless conditional in fdt.c.
> 
> Signed-off-by: Sebastien Carlier <sebastien.carlier@gmail.com>
> ---
> drivers/qe/Makefile |    2 +-
> drivers/qe/fdt.c    |    2 --
> 2 files changed, 1 insertions(+), 3 deletions(-)

applied to 85xx

- k