[U-Boot] [Patch v2 6/7] powerpc/mpc8349emds: Migrate from spd_sdram to unified DDR driver
Kumar Gala
kumar.gala at freescale.com
Mon Oct 24 19:57:17 CEST 2011
On Oct 23, 2011, at 4:53 AM, Wolfgang Denk wrote:
> Dear Kumar Gala,
>
> In message <2A6300E8-A874-49AC-84CB-BE681AF1BF90 at 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 <yorksun at freescale.com>
>>> Signed-off-by: Kim Phillips <kim.phillips at 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
More information about the U-Boot
mailing list