diff options
author | Edward O'Callaghan <eocallaghan@alterapraxis.com> | 2014-03-30 22:28:26 +1100 |
---|---|---|
committer | Patrick Georgi <patrick@georgi-clan.de> | 2014-04-03 10:00:52 +0200 |
commit | c814be415839920b11277db1f2c2754d8ed27887 (patch) | |
tree | 78164d7a0e324d45031e96154d23e30a543a9d38 | |
parent | ab180d85f76e8e8cc68ead2f0c89f29b9fe5fd53 (diff) | |
download | coreboot-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.c | 2 | ||||
-rw-r--r-- | 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 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); |