Merge pull request #580 from phip1611/bump-uart

deps: bump uart_16550 to 0.8.0 + fix UEFI weirdness

Author: Freax13

Cargo.lock

@@ -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

Cargo.toml

@@ -1,5 +1,10 @@
 # Unreleased
 
+- When booting on UEFI, whatever is written to the UEFI console
+  (e.g., via the SIMPLE_TEXT_OUTPUT_PROTOCOL) is no longer replicated to the
+  serial device. The bootloader's own logger now takes full ownership of the
+  serial device, ensuring safe and reliable logging.
+
 # 0.11.17 - 2026-07-27
 
 * [Revert `uart_16550` version bump to 0.6.0](https://github.com/rust-osdev/bootloader/pull/575)

Changelog.md

@@ -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.8.0"
 log = "0.4.17"
 
 [dependencies.noto-sans-mono-bitmap]

common/Cargo.toml

@@ -34,8 +34,6 @@ pub mod level_4_entries;
 pub mod load_kernel;
 /// Provides a logger that logs output as text in various formats.
 pub mod logger;
-/// Provides a type that logs output as text to a Serial Being port.
-pub mod serial;
 
 const PAGE_SIZE: u64 = 4096;
 

common/src/lib.rs

@@ -1,16 +1,18 @@
-use crate::{framebuffer::FrameBufferWriter, serial::SerialPort};
+use crate::framebuffer::FrameBufferWriter;
 use bootloader_api::info::FrameBufferInfo;
 use conquer_once::spin::OnceCell;
 use core::fmt::Write;
 use spinning_top::Spinlock;
+use uart_16550::backend::PioBackend;
+use uart_16550::{Config, Uart16550Tty};
 
 /// The global logger instance used for the `log` crate.
 pub static LOGGER: OnceCell<LockedLogger> = OnceCell::uninit();
 
 /// A logger instance protected by a spinlock.
 pub struct LockedLogger {
     framebuffer: Option<Spinlock<FrameBufferWriter>>,
-    serial: Option<Spinlock<SerialPort>>,
+    serial: Option<Spinlock<Uart16550Tty<PioBackend>>>,
 }
 
 impl LockedLogger {
@@ -27,7 +29,17 @@ impl LockedLogger {
         };
 
         let serial = match serial_logger_status {
-            true => Some(Spinlock::new(unsafe { SerialPort::init() })),
+            true => {
+                // SAFETY: We have exclusive access to the device.
+                //
+                // This returns `None` if the config is invalid or the self-test fails.
+                // We do not panic here because we want to continue booting.
+                unsafe {
+                    Uart16550Tty::new_port(0x3f8, Config::default())
+                        .ok()
+                        .map(Spinlock::new)
+                }
+            }
             false => None,
         };