fix: fix build with clang-18
Some uses of inline assembly for accessing floating point registers did
not enable the FP extension and were failing with clang-18.
The check in the linker script for the `hftest_enable` variable fails
under clang-18. However, hftest still compiles and passes without the
check, so it was removed.
Fix new warnings from `clang-tidy`.
Apply `make format`.
Change-Id: I43996abb4c42de54be807dcdb76107b9752c62fb
Signed-off-by: Karl Meakin <karl.meakin@arm.com>
diff --git a/inc/hf/arch/std.h b/inc/hf/arch/std.h
index e0f2e0a..91f808e 100644
--- a/inc/hf/arch/std.h
+++ b/inc/hf/arch/std.h
@@ -31,7 +31,7 @@
#if __has_builtin(__builtin_is_aligned)
#define is_aligned(v, a) __builtin_is_aligned((v), (a))
#else
-#define is_aligned(v, a) (((uintptr_t)(v) & ((a)-1)) == 0)
+#define is_aligned(v, a) (((uintptr_t)(v) & ((a) - 1)) == 0)
#endif
/**
@@ -40,7 +40,7 @@
#if __has_builtin(__builtin_align_up)
#define align_up(v, a) __builtin_align_up((v), (a))
#else
-#define align_up(v, a) (((uintptr_t)(v) + ((a)-1)) & ~((a)-1))
+#define align_up(v, a) (((uintptr_t)(v) + ((a) - 1)) & ~((a) - 1))
#endif
/**
@@ -49,7 +49,7 @@
#if __has_builtin(__builtin_align_down)
#define align_down(v, a) __builtin_align_down((v), (a))
#else
-#define align_down(v, a) ((uintptr_t)(v) & ~((a)-1))
+#define align_down(v, a) ((uintptr_t)(v) & ~((a) - 1))
#endif
#ifndef be16toh
diff --git a/inc/hf/list.h b/inc/hf/list.h
index 0f200ce..1f13697 100644
--- a/inc/hf/list.h
+++ b/inc/hf/list.h
@@ -16,10 +16,7 @@
struct list_entry *prev;
};
-#define LIST_INIT(l) \
- { \
- .next = &l, .prev = &l \
- }
+#define LIST_INIT(l) {.next = &l, .prev = &l}
#define CONTAINER_OF(ptr, type, field) \
((type *)((char *)ptr - offsetof(type, field)))
diff --git a/inc/vmapi/hf/ffa.h b/inc/vmapi/hf/ffa.h
index 5a0a468..a13e505 100644
--- a/inc/vmapi/hf/ffa.h
+++ b/inc/vmapi/hf/ffa.h
@@ -676,6 +676,7 @@
static inline enum ffa_error ffa_error_code(struct ffa_value val)
{
+ /* NOLINTNEXTLINE(EnumCastOutOfRange) */
return (enum ffa_error)val.arg2;
}