>>>>> "Tom" == Tom Rini <tr...@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
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to