On Fri, Dec 04, 2015 at 07:24:19PM +0000, Trent Piepho wrote: > In commit 104a6a7ccfb7928ca5dc28c8cbe0ea231ffc45ee support was added > for Thumb2. It added do_execute() as a way to provide arch dependent > calling veneers for use in "go" and thumb2_execute() as the thumb2 to > arm veneer. > > But thumb2_execute() isn't necessary as gcc generates a proper calling > sequence from a standard function pointer call. Thumb2 barebox is > compiled with the AAPCS ABI which requires this. > > It also had a bug and didn't pass the arguments properly, but code > execute via "go" rarely uses arguments so this wasn't very noticeable. > > Since thumb2 was always the only user of do_execute(), go ahead and > delete that too. > > Signed-off-by: Trent Piepho <tpiepho@xxxxxxxxxxxxxx> Applied, thanks Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox
