16 May
2013
16 May
'13
8:55 p.m.
"Tom" == Tom Rini trini@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