deps: bump uart_16550 to 0.6.0

Cargo.toml

@@ -19,6 +19,7 @@ members = [
     "tests/runner",
 ]
 exclude = ["examples/basic", "examples/test_framework", "tests/test_kernels/*"]
+resolver = "3"
 
 [workspace.package]
 # don't forget to update `workspace.dependencies` below

common/Cargo.toml

@@ -19,7 +19,7 @@ xmas-elf = "0.8.0"
 raw-cpuid = "10.2.0"
 rand = { version = "0.8.6", default-features = false }
 rand_hc = "0.3.1"
-uart_16550 = "0.3.2"
+uart_16550 = "0.6.0"
 log = "0.4.17"
 
 [dependencies.noto-sans-mono-bitmap]

common/src/serial.rs

@@ -1,16 +1,21 @@
 use core::fmt;
+use uart_16550::backend::PioBackend;
 
+// TODO this type can be replaced with Uart16550Tty but using it currently panics
+// in the new constructor.
 pub struct SerialPort {
-    port: uart_16550::SerialPort,
+    port: uart_16550::Uart16550<PioBackend>,
 }
 
 impl SerialPort {
     /// # Safety
     ///
     /// unsafe because this function must only be called once
     pub unsafe fn init() -> Self {
-        let mut port = unsafe { uart_16550::SerialPort::new(0x3F8) };
-        port.init();
+        let mut port =
+            unsafe { uart_16550::Uart16550::new_port(0x3F8) }.expect("should be valid port");
+        port.init(uart_16550::Config::default())
+            .expect("should init successfully");
         Self { port }
     }
 }
@@ -19,8 +24,8 @@ impl fmt::Write for SerialPort {
     fn write_str(&mut self, s: &str) -> fmt::Result {
         for char in s.bytes() {
             match char {
-                b'\n' => self.port.write_str("\r\n").unwrap(),
-                byte => self.port.send(byte),
+                b'\n' => self.port.send_bytes_exact(b"\r\n"),
+                byte => self.port.send_bytes_exact(&[byte]),
             }
         }
         Ok(())

examples/basic/kernel/Cargo.toml

@@ -5,5 +5,5 @@ edition = "2024"
 
 [dependencies]
 bootloader_api = "0.11.12"
-uart_16550 = "0.4.0"
+uart_16550 = "0.6.0"
 x86_64 = "0.15.2"

examples/basic/kernel/src/main.rs

@@ -3,6 +3,8 @@
 
 use bootloader_api::{BootInfo, entry_point};
 use core::fmt::Write;
+use uart_16550::backend::PioBackend;
+use uart_16550::{Config, Uart16550Tty};
 
 #[derive(Debug, Clone, Copy, PartialEq, Eq)]
 #[repr(u32)]
@@ -24,10 +26,9 @@ pub fn exit_qemu(exit_code: QemuExitCode) -> ! {
     }
 }
 
-pub fn serial() -> uart_16550::SerialPort {
-    let mut port = unsafe { uart_16550::SerialPort::new(0x3F8) };
-    port.init();
-    port
+pub fn serial() -> Uart16550Tty<PioBackend> {
+    unsafe { Uart16550Tty::new_port(0x3F8, Config::default()) }
+        .expect("should initialize serial device from valid config and valid port")
 }
 
 entry_point!(kernel_main);

examples/basic/src/main.rs

@@ -62,8 +62,8 @@ fn main() {
     let mut child = cmd.spawn().expect("failed to start qemu-system-x86_64");
     let status = child.wait().expect("failed to wait on qemu");
     match status.code().unwrap_or(1) {
-        0x10 => 0,  // success
-        0x11 => 1,  // failure
-        _    => 2,  // unknown fault
+        0x10 => 0, // success
+        0x11 => 1, // failure
+        _ => 2,    // unknown fault
     };
 }