kapi/device: remove init()

6 files changed, 2 insertions, 67 deletions

diff --git a/kernel/kapi/boot_module/bus.cpp b/kernel/kapi/boot_module/bus.cpp
index b917866c..3e95a3ac 100644
--- a/kernel/kapi/boot_module/bus.cpp
+++ b/kernel/kapi/boot_module/bus.cpp
@@ -20,7 +20,8 @@ namespace kapi::boot_modules
     {
       auto const text = std::string_view{token};
       auto const equals = text.find('=');
-      if (equals != std::string_view::npos && text.substr(equals) == key && text.substr(equals + 1) == value)
+      if (equals != std::string_view::npos && text.substr(0, equals) == key &&
+          text.substr(equals + 1, value.size()) == value)
       {
         return true;
       }
@@ -33,11 +34,6 @@ namespace kapi::boot_modules
       , m_module(module)
   {}
 
-  auto boot_module_device::init() -> bool
-  {
-    return true;
-  }
-
   auto boot_module_device::command_line() const -> std::string_view
   {
     return m_module.name;
diff --git a/kernel/kapi/devices.cpp b/kernel/kapi/devices.cpp
index 5eed1b87..267bfb4d 100644
--- a/kernel/kapi/devices.cpp
+++ b/kernel/kapi/devices.cpp
@@ -35,7 +35,6 @@ namespace kapi::devices
 
     root_bus = kstd::make_shared<kernel::devices::root_bus>();
     register_device(root_bus);
-    root_bus->init();
   }
 
   auto get_root_bus() -> kstd::shared_ptr<bus>
diff --git a/kernel/kapi/devices.tests.cpp b/kernel/kapi/devices.tests.cpp
index 236a812d..627c5ce0 100644
--- a/kernel/kapi/devices.tests.cpp
+++ b/kernel/kapi/devices.tests.cpp
@@ -15,11 +15,6 @@ namespace
   struct test_device final : kapi::devices::device
   {
     using kapi::devices::device::device;
-
-    auto init() -> bool override
-    {
-      return true;
-    }
   };
 
 }  // namespace
diff --git a/kernel/kapi/devices/bus.cpp b/kernel/kapi/devices/bus.cpp
index 25e35733..db693eac 100644
--- a/kernel/kapi/devices/bus.cpp
+++ b/kernel/kapi/devices/bus.cpp
@@ -5,11 +5,9 @@
 #include <kapi/system.hpp>
 
 #include <kstd/memory.hpp>
-#include <kstd/print.hpp>
 #include <kstd/string.hpp>
 #include <kstd/vector.hpp>
 
-#include <algorithm>
 #include <span>
 #include <utility>
 
@@ -19,30 +17,6 @@ namespace kapi::devices
       : device{name}
   {}
 
-  auto bus::init() -> bool
-  {
-    if (m_init_was_called.test_and_set())
-    {
-      kstd::println(kstd::print_sink::stderr, "[OS:DEV] Bus {} already initialized", name());
-      return true;
-    }
-
-    if (!enumerate_old())
-    {
-      kstd::println(kstd::print_sink::stderr, "[OS:DEV] Bus {} enumeration failed", name());
-      return false;
-    }
-
-    auto child_status = std::ranges::fold_left(m_devices, true, [&](bool acc, auto & child) -> bool {
-      kstd::println("[OS:DEV] Initializing child device {}@{}", child->name(), name());
-      return child->init() && acc;
-    });
-
-    m_initialized.test_and_set();
-
-    return child_status;
-  }
-
   auto bus::add_child(kstd::shared_ptr<device> child) -> void
   {
     child->set_parent(kstd::static_pointer_cast<bus>(shared_from_this()));
@@ -56,15 +30,6 @@ namespace kapi::devices
     attached->set_state(state::present);
 
     driver_registry::get().device_attached(attached);
-
-    if (m_initialized.test())
-    {
-      kstd::println("[OS:DEV] Initializing child device {}@{}", attached->name(), name());
-      if (!attached->init())
-      {
-        kapi::system::panic("[OS:DEV] Failed to initialize child device");
-      }
-    }
   }
 
   [[nodiscard]] auto bus::children() const -> std::span<kstd::shared_ptr<device> const>
@@ -77,9 +42,4 @@ namespace kapi::devices
     return nullptr;
   }
 
-  auto bus::enumerate_old() -> bool
-  {
-    return true;
-  }
-
 }  // namespace kapi::devices
\ No newline at end of file
diff --git a/kernel/kapi/devices/bus.tests.cpp b/kernel/kapi/devices/bus.tests.cpp
index d5a2d1cf..9f95513b 100644
--- a/kernel/kapi/devices/bus.tests.cpp
+++ b/kernel/kapi/devices/bus.tests.cpp
@@ -16,11 +16,6 @@ namespace
   struct test_device final : kapi::devices::device
   {
     using kapi::devices::device::device;
-
-    auto init() -> bool override
-    {
-      return true;
-    }
   };
 
   struct flagging_device final : kapi::devices::device
@@ -35,11 +30,6 @@ namespace
       *m_destroyed_flag = true;
     }
 
-    auto init() -> bool override
-    {
-      return true;
-    }
-
   private:
     bool * m_destroyed_flag;
   };
diff --git a/kernel/kapi/devices/interface_registry.tests.cpp b/kernel/kapi/devices/interface_registry.tests.cpp
index aea7776c..78cd7ecf 100644
--- a/kernel/kapi/devices/interface_registry.tests.cpp
+++ b/kernel/kapi/devices/interface_registry.tests.cpp
@@ -35,11 +35,6 @@ namespace
         , m_const_device{&const_device}
     {}
 
-    auto init() -> bool override
-    {
-      return true;
-    }
-
     [[nodiscard]] constexpr auto get_value() const noexcept -> int override
     {
       return m_value;