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-c814be415839920b11277db1f2c2754d8ed27887.tar.xz | |
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 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); |