From 36e3a8a9e43e2cb9efb15c04c43ddae16db73aef Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Rom=C3=A1n=20C=C3=A1rdenas=20Rodr=C3=ADguez?=
 <rcardenas.rod@gmail.com>
Date: Fri, 11 Oct 2024 14:37:37 +0200
Subject: [PATCH] replace static muts with pointers

---
 hifive1/CHANGELOG.md  |  1 +
 hifive1/Cargo.toml    |  1 +
 hifive1/src/stdout.rs | 23 +++++++++++------------
 3 files changed, 13 insertions(+), 12 deletions(-)

diff --git a/hifive1/CHANGELOG.md b/hifive1/CHANGELOG.md
index 4924efa..40286ad 100644
--- a/hifive1/CHANGELOG.md
+++ b/hifive1/CHANGELOG.md
@@ -6,6 +6,7 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/)
 and this project adheres to [Semantic Versioning](http://semver.org/).
 
 ## [Unreleased]
+- Replace static muts with Mutexes
 - Apply clippy changes
 - Bump MSRV to 1.72
 - Adapt to new Cargo workspace
diff --git a/hifive1/Cargo.toml b/hifive1/Cargo.toml
index efa0546..ac01280 100644
--- a/hifive1/Cargo.toml
+++ b/hifive1/Cargo.toml
@@ -11,6 +11,7 @@ edition = "2021"
 rust-version = "1.72"
 
 [dependencies]
+critical-section = { version = "1.1.3" }
 e310x-hal = { path = "../e310x-hal", version = "0.11.0" }
 embedded-hal = "0.2.7"
 riscv = "0.10.1"
diff --git a/hifive1/src/stdout.rs b/hifive1/src/stdout.rs
index 59be531..c325747 100644
--- a/hifive1/src/stdout.rs
+++ b/hifive1/src/stdout.rs
@@ -1,6 +1,6 @@
 //! Stdout based on the UART hooked up to FTDI or J-Link
 
-use core::fmt;
+use core::{fmt, ptr};
 use e310x_hal::{
     clock::Clocks,
     e310x::Uart0,
@@ -10,12 +10,11 @@ use e310x_hal::{
     time::Bps,
 };
 use nb::block;
-use riscv::interrupt;
-
-static mut STDOUT: Option<SerialWrapper> = None;
 
 struct SerialWrapper(Tx<Uart0>);
 
+static mut STDOUT: Option<SerialWrapper> = None;
+
 impl core::fmt::Write for SerialWrapper {
     fn write_str(&mut self, s: &str) -> fmt::Result {
         for byte in s.as_bytes() {
@@ -50,28 +49,28 @@ pub fn configure<X, Y>(
     let serial = Serial::new(uart, (tx, rx), baud_rate, clocks);
     let (tx, rx) = serial.split();
 
-    interrupt::free(|| unsafe {
-        STDOUT.replace(SerialWrapper(tx));
+    critical_section::with(|_| {
+        unsafe { &mut *ptr::addr_of_mut!(STDOUT) }.replace(SerialWrapper(tx));
     });
     rx
 }
 
 /// Writes string to stdout
 pub fn write_str(s: &str) {
-    interrupt::free(|| unsafe {
-        if let Some(stdout) = STDOUT.as_mut() {
+    critical_section::with(|_| {
+        if let Some(stdout) = unsafe { &mut *ptr::addr_of_mut!(STDOUT) } {
             let _ = stdout.write_str(s);
         }
-    })
+    });
 }
 
 /// Writes formatted string to stdout
 pub fn write_fmt(args: fmt::Arguments) {
-    interrupt::free(|| unsafe {
-        if let Some(stdout) = STDOUT.as_mut() {
+    critical_section::with(|_| {
+        if let Some(stdout) = unsafe { &mut *ptr::addr_of_mut!(STDOUT) } {
             let _ = stdout.write_fmt(args);
         }
-    })
+    });
 }
 
 /// Macro for printing to the serial standard output