Re: [PATCH] ARM: Remove do_execute and thumb2_execute

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux