On Mon, 2010-11-22 at 21:58 -0700, Wolfgang Denk wrote:
Dear Haiying.Wang@freescale.com,
In message 1290462446-1459-3-git-send-email-Haiying.Wang@freescale.com you wrote:
From: Haiying Wang Haiying.Wang@freescale.com
This patch introduces the third program loader(TPL) to load the
final uboot
image after the spl code. Once the CONFIG_SYS_TPL_BOOT is defined, the CONFIG_TPL_BOOT is enabled to generate the u-boot-tpl.bin. There
are two
This makes no sense to me. Why do you need an additional variable CONFIG_TPL_BOOT when you already have CONFIG_SYS_TPL_BOOT ?
Please use only one such name (CONFIG_SYS_TPL_BOOT).
CONFIG_SYS_TPL_BOOT is used as CONFIG_NAND_U_BOOT in Makefile and header file. CONFIG_TPL_BOOT is used as CONFIG_NAND_SPL in header file.
Does CONFIG_TPL_UBOOT make sense to you if I replace CONFIG_SYS_TPL_BOOT with it?
Thanks.
Haiying