Re: [U-Boot] VCMA9 build fails: ARM board references PPC4xx header

2010-10-06 Thread David Müller (ELSOFT AG)
Wolfgang Denk schrieb:
 The problem comes from board/mpl/common/memtst.c which contains an
 unconditional #include asm/ppc4xx-i2c.h which is obviously wrong
 when the file gets compiled for an ARM board as VCMA9.
 
 Can you please provide a fix?

The solution is to remove the include completely as it is not needed at all.

Dave
Signed-off-by: David Mueller d.muel...@elsoft.ch

diff --git a/board/mpl/common/memtst.c b/board/mpl/common/memtst.c
index 9c08065..8810302 100644
--- a/board/mpl/common/memtst.c
+++ b/board/mpl/common/memtst.c
@@ -48,7 +48,6 @@ int testdram (void)
 
 #include common.h
 #include asm/processor.h
-#include asm/ppc4xx-i2c.h
 
 DECLARE_GLOBAL_DATA_PTR;
 
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] VCMA9 build fails: ARM board references PPC4xx header

2010-10-06 Thread Peter Tyser
On Wed, 2010-10-06 at 09:38 +0200, David Müller (ELSOFT AG) wrote:
 Wolfgang Denk schrieb:
  The problem comes from board/mpl/common/memtst.c which contains an
  unconditional #include asm/ppc4xx-i2c.h which is obviously wrong
  when the file gets compiled for an ARM board as VCMA9.
  
  Can you please provide a fix?
 
 The solution is to remove the include completely as it is not needed at all.

Is there any reason to not remove the memtst.c file altogether as in
this patch:
http://lists.denx.de/pipermail/u-boot/2010-October/078649.html ?

Best,
Peter

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] VCMA9 build fails: ARM board references PPC4xx header

2010-10-05 Thread Wolfgang Denk
Dear David  Peter,

since Stefan's ppc4xx reorganization patches (commit b36df561:
ppc4xx: Move ppc4xx headers to powerpc include directory) the
building of the VCMA9 breaks with this error message:

../common/memtst.c:51:28: error: asm/ppc4xx-i2c.h: No such file or directory

The problem comes from board/mpl/common/memtst.c which contains an
unconditional #include asm/ppc4xx-i2c.h which is obviously wrong
when the file gets compiled for an ARM board as VCMA9.

Can you please provide a fix?

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: w...@denx.de
Everything should be made as simple as possible, but not simpler.
- Albert Einstein
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot