fix: fix build with clang-17
Target triple `aarch64-none-eabi` was incorrect, but old versions of
clang accepted it anyway. clang-17 is stricter and does not accept it
(https://releases.llvm.org/17.0.1/tools/clang/docs/ReleaseNotes.html#arm-and-aarch64-support).
The correct replacement is `aarch64-none-elf`.
The local variable `struct ffa_partition_rxtx_header header` was
assigned a different address that was not aligned to 8 bytes by
clang-17, and so `memcpy_trapped` would fail. This is fixed by
annotating it with `alignas(8)`.
Fix new warnings from `clang-tidy`.
Ignore `misc-include-cleaner` warning from `clang-tidy`, because it
produces a lot of incorrect warnings.
Apply `make format`.
Change-Id: I605e23ce258de9bfd49d717ef5864292a30235e1
Signed-off-by: Karl Meakin <karl.meakin@arm.com>
diff --git a/src/arch/aarch64/hypervisor/handler.c b/src/arch/aarch64/hypervisor/handler.c
index c30d52a..5f3812d 100644
--- a/src/arch/aarch64/hypervisor/handler.c
+++ b/src/arch/aarch64/hypervisor/handler.c
@@ -805,9 +805,12 @@
case FFA_ERROR_32:
*args = plat_ffa_error_32(current, next, args->arg2);
return true;
- }
- return false;
+ default:
+ dlog_verbose("Unsupported function %#x (%s)\n", func,
+ ffa_func_name(func));
+ return false;
+ }
}
/**