[U-Boot] [PATCH v2 7/7] am335x_evm: Add support to boot from NOR.

[Peter Korsgaard]

>>>>> "Tom" == Tom Rini <trini at ti.com> writes:

Hi,

 Tom> +#ifdef CONFIG_TI81XX
 >> 
 >> Why are you adding the TI81XX dependency here? That doesn't have
 >> anything to do with nor boot, does it?

 Tom> dmm is TI81XX-only (for this file, it's also omap4+) and since main
 Tom> U-Boot doesn't use --ffunction-sections/--fdata-sections/--gc-sections
 Tom> we end up with a link error about not having enable_dmm_clocks().  This
 Tom> is OK on SPL since we do use --gc-sections and the link error is avoided
 Tom> as we discard config_dmm on am33xx builds.

Ok, that's interesting info which imho belongs in the commit message.

-- 
Bye, Peter Korsgaard