From ae3f3024d9ef60c6e12d9b49b2fd955939616504 Mon Sep 17 00:00:00 2001 From: Aaron Durbin Date: Wed, 10 Feb 2016 22:54:07 -0600 Subject: arch: remove stage_exit() It's no longer used. Remove it. Change-Id: Id6f4084ab9d671e94f0eee76bf36fad9a174ef14 Signed-off-by: Aaron Durbin Reviewed-on: https://review.coreboot.org/13678 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Georgi --- src/arch/arm/include/arch/stages.h | 1 - src/arch/arm/stages.c | 16 ---------------- 2 files changed, 17 deletions(-) (limited to 'src/arch/arm') diff --git a/src/arch/arm/include/arch/stages.h b/src/arch/arm/include/arch/stages.h index ef4658ef89..2d20604589 100644 --- a/src/arch/arm/include/arch/stages.h +++ b/src/arch/arm/include/arch/stages.h @@ -19,6 +19,5 @@ extern void main(void); void stage_entry(void); -void stage_exit(void *); #endif diff --git a/src/arch/arm/stages.c b/src/arch/arm/stages.c index 6e5b8cd4a5..2beaacd1f8 100644 --- a/src/arch/arm/stages.c +++ b/src/arch/arm/stages.c @@ -34,19 +34,3 @@ __attribute__((weak)) void stage_entry(void) { main(); } - -/* we had marked 'doit' as 'noreturn'. - * There is no apparent harm in leaving it as something we can return from, and in the one - * case where we call a payload, the payload is allowed to return. - * Hence, leave it as something we can return from. - */ -void stage_exit(void *addr) -{ - void (*doit)(void) = addr; - /* - * Most stages load code so we need to sync caches here. Should maybe - * go into cbfs_load_stage() instead... - */ - cache_sync_instructions(); - doit(); -} -- cgit v1.2.3