From c814be415839920b11277db1f2c2754d8ed27887 Mon Sep 17 00:00:00 2001 From: Edward O'Callaghan Date: Sun, 30 Mar 2014 22:28:26 +1100 Subject: [PATCH] amd/agesa/s3_resume: Make compiler agnostic. Clang does not like inline functions defined in C files with prototypes in headers. Rather Clang expects inline function bodies to be in headers if they are to be used out of scope. Since inline is purely advisory to the compiler, drop its usage here. Change-Id: I08a7a3d2cdf841ffbab10c017c75917768aac209 Signed-off-by: Edward O'Callaghan Reviewed-on: http://review.coreboot.org/5429 Reviewed-by: Patrick Georgi Tested-by: build bot (Jenkins) --- src/cpu/amd/agesa/s3_resume.c | 2 +- src/cpu/amd/agesa/s3_resume.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/cpu/amd/agesa/s3_resume.c b/src/cpu/amd/agesa/s3_resume.c index e056a18b03..7b1157d358 100644 --- a/src/cpu/amd/agesa/s3_resume.c +++ b/src/cpu/amd/agesa/s3_resume.c @@ -103,7 +103,7 @@ void restore_mtrr(void) wrmsr(SYS_CFG, msr_data); } -inline void *backup_resume(void) +void *backup_resume(void) { void *resume_backup_memory; diff --git a/src/cpu/amd/agesa/s3_resume.h b/src/cpu/amd/agesa/s3_resume.h index 5ee4f38ab0..6e719c98a7 100644 --- a/src/cpu/amd/agesa/s3_resume.h +++ b/src/cpu/amd/agesa/s3_resume.h @@ -41,7 +41,7 @@ typedef enum { void restore_mtrr(void); void s3_resume(void); -inline void *backup_resume(void); +void *backup_resume(void); void set_resume_cache(void); void move_stack_high_mem(void);