[U-Boot] [RFC PATCH 2/4] OMAP3+: fix issues with Thumb build

Albert ARIBAUD albert.u.boot at aribaud.net
Mon Feb 6 22:06:24 CET 2012


Le 06/02/2012 12:37, Aneesh V a écrit :
> When U-Boot/SPL is built using the Thumb instruction set the
> toolchain has a  potential issue with weakly linked symbols.
> If a function has a weakly linked default implementation in C
> and a real implementation in assembly GCC is confused about the
> instruction set of the assembly implementation. As a result
> the assembly function that is built in ARM is executed as
> if it is Thumb. This results in a crash.
>
> We need to investigate further to see if this is a toolchain
> issue or an issue with our usage of it. In the meanwhile, we
> can workaround the issue by having both the weakly linked alias
> and the real implementation in C.

I would tend to NAK a patch submission where an issue is known and 
investigation is considered but is actually bypassed by a workaround. 
These tend to turn from 'temporary fix' to 'old crust' with time, 
because there is no incentive for a better solution when the quick fix 
"works, after all".

Amicalement,
-- 
Albert.


More information about the U-Boot mailing list