Hi Andreas,
On 2014-12-15 15:24, Andreas Färber wrote:
Hi Stefan,
Am 03.12.2014 um 18:04 schrieb Stefan Agner:
Can this be fixed by the merger or should I create a new revision?
It looks as if this was neither applied nor respun? I have some more patches to make CONFIG_USE_PRIVATE_LIBGCC build for Thumb that I would like to rebase on it.
Not sure what the expectation is... Probably I should just do a respin since there are now three minor nits...
In particular, I have pieced together a tiny asm/unified.h file with THUMB() and ARM() macros and doing the .syntax unified in that central place, and resyncing the various arch/arm/lib/_*.S files from Linux. https://github.com/afaerber/u-boot/commits/stm32
Is there a particular reason you are doing the .syntax inline here?
No particular reason other than laziness :-)
And looking beyond this patch, assuming this is for your Vybrid's M4, do you already have a concept for how to go about vectors.S, start.S and the like for ARMv7-M? :) I've stubbed them out for now but haven't managed to successfully link yet...
Actually no, I don't use U-Boot on the M4 (so far). For Linux on the Vybrid M4, I use a special boot loader which runs on the A5 Linux. On Vybrid (the variant we use at least) boots always from the A5 first, hence I have a running Linux. Or alternatively, I could run the M4 from U-Boot running on the A5. So far, I had no need to run U-Boot on the M4.
The reason I was interested in Thumb2 was the size: Due to lack of 5V on the Colibri module we need to load the boot loader using Vybrid's serial loader over UART which is somewhat slow. So the 25% reduction of size means 25% less download time...
-- Stefan