summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJulius Werner <jwerner@chromium.org>2018-06-25 17:59:13 -0700
committerJulius Werner <jwerner@chromium.org>2018-06-26 23:59:08 +0000
commita98b5bf89b39fede95c34bf81e9decc1e6b6d38f (patch)
tree7b32d53ee95131f5fd9e30f54b6780e0ee7dbc00
parent905e1e763e0a8922cade359c01dfdee92834b473 (diff)
downloadcoreboot-a98b5bf89b39fede95c34bf81e9decc1e6b6d38f.tar.gz
coreboot-a98b5bf89b39fede95c34bf81e9decc1e6b6d38f.tar.bz2
coreboot-a98b5bf89b39fede95c34bf81e9decc1e6b6d38f.zip
arm64: Switch remaining uses of __ASSEMBLY__ to __ASSEMBLER__
Some arm64 files that were imported from other projects use the __ASSEMBLY__ macro to test whether a header is included from a C or an assembly file. This patch switches them to the coreboot standard __ASSEMBLER__, which has the advantage of being a GCC builtin so that the including file doesn't have to supply it explicitly. Change-Id: I1023f72dd13857b14ce060388e97c658e748928f Signed-off-by: Julius Werner <jwerner@chromium.org> Reviewed-on: https://review.coreboot.org/27237 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Aaron Durbin <adurbin@chromium.org>
-rw-r--r--src/arch/arm64/include/arch/transition.h4
-rw-r--r--src/arch/arm64/include/armv8/arch/barrier.h4
-rw-r--r--src/arch/arm64/include/armv8/arch/lib_helpers.h4
-rw-r--r--src/arch/arm64/transition_asm.S1
4 files changed, 6 insertions, 7 deletions
diff --git a/src/arch/arm64/include/arch/transition.h b/src/arch/arm64/include/arch/transition.h
index 4793b840c28e..5cd73c15d878 100644
--- a/src/arch/arm64/include/arch/transition.h
+++ b/src/arch/arm64/include/arch/transition.h
@@ -56,7 +56,7 @@
#define EXC_VID_LOW32_SERR 15
#define NUM_EXC_VIDS 16
-#ifndef __ASSEMBLY__
+#ifndef __ASSEMBLER__
#include <stdint.h>
#include <arch/lib_helpers.h>
@@ -197,6 +197,6 @@ void exc_dispatch(struct exc_state *exc_state, uint64_t id);
*/
void exc_entry(struct exc_state *exc_state, uint64_t id);
-#endif /* __ASSEMBLY__ */
+#endif /* __ASSEMBLER__ */
#endif /* __ARCH_ARM64_TRANSITION_H__ */
diff --git a/src/arch/arm64/include/armv8/arch/barrier.h b/src/arch/arm64/include/armv8/arch/barrier.h
index b653d24bee1c..397ac2aa67d8 100644
--- a/src/arch/arm64/include/armv8/arch/barrier.h
+++ b/src/arch/arm64/include/armv8/arch/barrier.h
@@ -15,7 +15,7 @@
#ifndef __ASM_ARM_BARRIER_H
#define __ASM_ARM_BARRIER_H
-#ifndef __ASSEMBLY__
+#ifndef __ASSEMBLER__
#define sevl() asm volatile("sevl" : : : "memory")
#define sev() asm volatile("sev" : : : "memory")
@@ -103,6 +103,6 @@ do { \
!ret; \
})
-#endif /* __ASSEMBLY__ */
+#endif /* __ASSEMBLER__ */
#endif /* __ASM_ARM_BARRIER_H */
diff --git a/src/arch/arm64/include/armv8/arch/lib_helpers.h b/src/arch/arm64/include/armv8/arch/lib_helpers.h
index f4b6ba6e6afb..d2d30e12aa95 100644
--- a/src/arch/arm64/include/armv8/arch/lib_helpers.h
+++ b/src/arch/arm64/include/armv8/arch/lib_helpers.h
@@ -136,7 +136,7 @@
#define CPACR_TRAP_FP_EL0 (1 << CPACR_FPEN_SHIFT)
#define CPACR_TRAP_FP_DISABLE (3 << CPACR_FPEN_SHIFT)
-#ifdef __ASSEMBLY__
+#ifdef __ASSEMBLER__
/* Macro to switch to label based on current el */
.macro switch_el xreg label1 label2 label3
@@ -585,6 +585,6 @@ void tlbiallis_current(void);
void tlbiallis(uint32_t el);
void tlbivaa_el1(uint64_t va);
-#endif // __ASSEMBLY__
+#endif /* __ASSEMBLER__ */
#endif /* __ARCH_LIB_HELPERS_H__ */
diff --git a/src/arch/arm64/transition_asm.S b/src/arch/arm64/transition_asm.S
index 8e61246db17e..924a4bc388b0 100644
--- a/src/arch/arm64/transition_asm.S
+++ b/src/arch/arm64/transition_asm.S
@@ -53,7 +53,6 @@
* to be executed.
*/
-#define __ASSEMBLY__
#include <arch/asm.h>
#include <arch/lib_helpers.h>
#include <arch/transition.h>