zephyr: sys: sync: semaphore: Remove a few 'mut' declarations

The semaphore API shouldn't have `&mut self` declarations, as the
purpose of the API is to be that thread synchronization.  Even the less
save ones, such as `count_get` and `reset` have well defined
multi-threaded behavior.

Signed-off-by: David Brown <[email protected]>

Author: d3zd3z

zephyr/src/sys/sync/semaphore.rs

@@ -106,14 +106,14 @@ impl Semaphore {
     /// `Error(EAGAIN)`.
     ///
     /// [`take`]: Self::take
-    pub fn reset(&mut self) {
+    pub fn reset(&self) {
         unsafe { k_sem_reset(self.item.get()) }
     }
 
     /// Get a semaphore's count.
     ///
     /// Returns the current count.
-    pub fn count_get(&mut self) -> usize {
+    pub fn count_get(&self) -> usize {
         unsafe { k_sem_count_get(self.item.get()) as usize }
     }
 }