On Oct 23, 2011, at 4:53 AM, Wolfgang Denk wrote:

> Dear Kumar Gala,
> 
> In message <2a6300e8-a874-49ac-84cb-be681af1b...@freescale.com> you wrote:
>> 
>> On Aug 26, 2011, at 1:32 PM, York Sun wrote:
>> 
>>> Update MPC8349EMDS to use unified DDR driver instead of spd_sdram.c.
>>> The unified driver can initialize data using DDR controller. No need to
>>> use DMA if just to initialze for ECC.
>>> 
>>> Signed-off-by: York Sun <york...@freescale.com>
>>> Signed-off-by: Kim Phillips <kim.phill...@freescale.com>
>>> ---
>>> board/freescale/mpc8349emds/Makefile      |    1 +
>>> board/freescale/mpc8349emds/ddr.c         |  107 
>>> +++++++++++++++++++++++++++++
>>> board/freescale/mpc8349emds/mpc8349emds.c |   26 ++++---
>>> include/configs/MPC8349EMDS.h             |   16 ++++
>>> 4 files changed, 139 insertions(+), 11 deletions(-)
>>> create mode 100644 board/freescale/mpc8349emds/ddr.c
>> 
>> applied to 85xx 'next'
> 
> Did you test it???
> 
> 
> This patch breaks out-of-tree building of the MPC8349EMDS board:
> 
> + MAKEALL_LOGDIR=/work/wd/tmp-ppc-LOG
> + BUILD_DIR=/work/wd/tmp-ppc
> + ./MAKEALL MPC8349EMDS
> Configuring for MPC8349EMDS board...
> make[1]: *** No rule to make target 
> `/work/wd/tmp-ppc/arch/powerpc/cpu/mpc83xx/.depend.ddr-gen2', needed by 
> `/work/wd/tmp-ppc/arch/powerpc/cpu/mpc83xx/.depend'.  Stop.
> make[1]: *** No rule to make target 
> `/work/wd/tmp-ppc/arch/powerpc/cpu/mpc83xx/.depend.ddr-gen2', needed by 
> `/work/wd/tmp-ppc/arch/powerpc/cpu/mpc83xx/.depend'.  Stop.
> make: *** [depend] Error 2
> 
> [Local building works.]
> 
> 
> Note also that "make distclean" still leaves a file
> arch/powerpc/cpu/mpc83xx/ddr-gen2.c
> 
> Please fix this, too.
> 
> Best regards,
> 
> Wolfgang Denk

I'll yell at the guys to fix this :)

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

Reply via email to