From ee8db28dcd51f4080784c0048c1da7a27be91065 Mon Sep 17 00:00:00 2001 From: Alex Martens Date: Sun, 29 Oct 2023 15:48:29 -0700 Subject: [PATCH] semihosting: add Safety docstring to syscall --- cortex-m-semihosting/src/lib.rs | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/cortex-m-semihosting/src/lib.rs b/cortex-m-semihosting/src/lib.rs index 83063072..e72dbe67 100644 --- a/cortex-m-semihosting/src/lib.rs +++ b/cortex-m-semihosting/src/lib.rs @@ -184,12 +184,23 @@ pub mod hio; pub mod nr; /// Performs a semihosting operation, takes a pointer to an argument block +/// +/// # Safety +/// +/// The syscall number must be a valid [semihosting operation], +/// and the arguments must be valid for the associated operation. +/// +/// [semihosting operation]: https://developer.arm.com/documentation/dui0471/i/semihosting/semihosting-operations?lang=en #[inline(always)] pub unsafe fn syscall(nr: usize, arg: &T) -> usize { syscall1(nr, arg as *const T as usize) } /// Performs a semihosting operation, takes one integer as an argument +/// +/// # Safety +/// +/// Same as [`syscall`]. #[inline(always)] pub unsafe fn syscall1(_nr: usize, _arg: usize) -> usize { match () {