refactor(console): extract Console::read

src/arch/aarch64/kernel/serial.rs

@@ -34,11 +34,6 @@ impl SerialPort {
 		}
 	}
 
-	#[allow(dead_code)]
-	pub fn read(&mut self) -> Option<u8> {
-		None
-	}
-
 	pub fn init(&self, _baudrate: u32) {
 		// We don't do anything here (yet).
 	}

src/arch/x86_64/kernel/serial.rs

@@ -52,18 +52,11 @@ impl SerialPort {
 		}
 	}
 
-	#[allow(dead_code)]
 	#[cfg(feature = "shell")]
 	pub fn read(&mut self) -> Option<u8> {
 		self.buffer.pop_front()
 	}
 
-	#[allow(dead_code)]
-	#[cfg(not(feature = "shell"))]
-	pub fn read(&mut self) -> Option<u8> {
-		None
-	}
-
 	pub fn send(&mut self, buf: &[u8]) {
 		match &mut self.inner {
 			SerialInner::Uhyve(s) => {

src/console.rs

@@ -10,6 +10,14 @@ impl Console {
 	pub fn write(&mut self, buf: &[u8]) {
 		arch::output_message_buf(buf);
 	}
+
+	#[cfg(feature = "shell")]
+	pub fn read(&mut self) -> Option<u8> {
+		crate::arch::kernel::COM1
+			.lock()
+			.as_mut()
+			.map(|s| s.read())?
+	}
 }
 
 /// A collection of methods that are required to format

src/shell.rs

@@ -1,10 +1,9 @@
 use simple_shell::*;
 
-use crate::arch::kernel::COM1;
 use crate::interrupts::print_statistics;
 
 fn read() -> Option<u8> {
-	COM1.lock().as_mut().map(|s| s.read())?
+	crate::console::CONSOLE.lock().read()
 }
 
 pub(crate) fn init() {