diff --git a/Kernel/Bus/USB/UHCI/UHCIController.cpp b/Kernel/Bus/USB/UHCI/UHCIController.cpp index e50f421d3b..403ac08888 100644 --- a/Kernel/Bus/USB/UHCI/UHCIController.cpp +++ b/Kernel/Bus/USB/UHCI/UHCIController.cpp @@ -76,7 +76,7 @@ ErrorOr UHCIController::initialize() dmesgln("UHCI: I/O base {}", m_io_base); dmesgln("UHCI: Interrupt line: {}", interrupt_number()); - spawn_port_process(); + TRY(spawn_port_process()); TRY(reset()); return start(); @@ -464,15 +464,10 @@ size_t UHCIController::poll_transfer_queue(QueueHead& transfer_queue) return transfer_size; } -void UHCIController::spawn_port_process() +ErrorOr UHCIController::spawn_port_process() { RefPtr usb_hotplug_thread; - - auto process_name = KString::try_create("UHCI hotplug"); - if (process_name.is_error()) - TODO(); - - (void)Process::create_kernel_process(usb_hotplug_thread, process_name.release_value(), [&] { + (void)Process::create_kernel_process(usb_hotplug_thread, TRY(KString::try_create("UHCI hotplug")), [&] { for (;;) { if (m_root_hub) m_root_hub->check_for_port_updates(); @@ -480,6 +475,7 @@ void UHCIController::spawn_port_process() (void)Thread::current()->sleep(Time::from_seconds(1)); } }); + return {}; } bool UHCIController::handle_irq(const RegisterState&) diff --git a/Kernel/Bus/USB/UHCI/UHCIController.h b/Kernel/Bus/USB/UHCI/UHCIController.h index 6b800dceb0..0cd243cc92 100644 --- a/Kernel/Bus/USB/UHCI/UHCIController.h +++ b/Kernel/Bus/USB/UHCI/UHCIController.h @@ -42,7 +42,7 @@ public: virtual ErrorOr reset() override; virtual ErrorOr stop() override; virtual ErrorOr start() override; - void spawn_port_process(); + ErrorOr spawn_port_process(); virtual ErrorOr submit_control_transfer(Transfer& transfer) override;