summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEdward O'Callaghan <eocallaghan@alterapraxis.com>2014-03-30 22:28:26 +1100
committerPatrick Georgi <patrick@georgi-clan.de>2014-04-03 10:00:52 +0200
commitc814be415839920b11277db1f2c2754d8ed27887 (patch)
tree78164d7a0e324d45031e96154d23e30a543a9d38
parentab180d85f76e8e8cc68ead2f0c89f29b9fe5fd53 (diff)
downloadcoreboot-staging-c814be415839920b11277db1f2c2754d8ed27887.zip
coreboot-staging-c814be415839920b11277db1f2c2754d8ed27887.tar.gz
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 <eocallaghan@alterapraxis.com> Reviewed-on: http://review.coreboot.org/5429 Reviewed-by: Patrick Georgi <patrick@georgi-clan.de> Tested-by: build bot (Jenkins)
-rw-r--r--src/cpu/amd/agesa/s3_resume.c2
-rw-r--r--src/cpu/amd/agesa/s3_resume.h2
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 e056a18..7b1157d 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 5ee4f38..6e719c9 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);
OpenPOWER on IntegriCloud