diff options
| author | Trent Piepho <tpiepho@kymetacorp.com> | 2015-12-04 19:24:19 +0000 |
|---|---|---|
| committer | Sascha Hauer <s.hauer@pengutronix.de> | 2015-12-07 10:44:45 +0100 |
| commit | a2b3a25417593e97500cd746f0632d36edfac1a5 (patch) | |
| tree | d7fdf1cb8319d51f8b3b0f259d56621a211e2282 /include/common.h | |
| parent | de7a09bb4326fd9f014303a45e3ad5a729ac3a3c (diff) | |
| download | barebox-a2b3a25417593e97500cd746f0632d36edfac1a5.tar.gz barebox-a2b3a25417593e97500cd746f0632d36edfac1a5.tar.xz | |
ARM: Remove do_execute and thumb2_execute
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@kymetacorp.com>
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Diffstat (limited to 'include/common.h')
| -rw-r--r-- | include/common.h | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/include/common.h b/include/common.h index 03ceec257d..38a6dbf933 100644 --- a/include/common.h +++ b/include/common.h @@ -113,12 +113,6 @@ extern int (*barebox_main)(void); void __noreturn start_barebox(void); void shutdown_barebox(void); -/* - * architectures which have special calling conventions for - * executing programs should set this. Used by the 'go' command - */ -extern void (*do_execute)(void *func, int argc, char *argv[]); - int run_shell(void); #ifdef CONFIG_SHELL_HUSH |