first try to simplify the architecture (remove redundant inode_file and open() methods), add ext2_file placeholder struct

1 files changed, 4 insertions, 28 deletions

diff --git a/kernel/src/filesystem/device_inode.cpp b/kernel/src/filesystem/device_inode.cpp
index 18e3b24..592637d 100644
--- a/kernel/src/filesystem/device_inode.cpp
+++ b/kernel/src/filesystem/device_inode.cpp
@@ -3,12 +3,12 @@
 #include "kapi/system.hpp"
 
 #include "kernel/devices/device.hpp"
+#include "kernel/filesystem/device_file.hpp"
+#include "kernel/filesystem/file.hpp"
 #include "kernel/filesystem/inode.hpp"
 
 #include <kstd/memory>
 
-#include <cstddef>
-
 namespace filesystem
 {
   device_inode::device_inode(kstd::shared_ptr<devices::device> const & device)
@@ -21,32 +21,8 @@ namespace filesystem
     }
   }
 
-  auto device_inode::backing_device() const -> kstd::shared_ptr<devices::device> const &
-  {
-    return m_device;
-  }
-
-  auto device_inode::read(void * /*buffer*/, size_t /*offset*/, size_t /*size*/) const -> size_t
+  auto device_inode::open_file() const -> kstd::shared_ptr<file>
   {
-    if (!m_device)
-    {
-      kapi::system::panic("[FILESYSTEM] device_inode has null device.");
-    }
-
-    // TODO BA-FS26 use device file?
-    // return m_device->read(buffer, offset, size);
-    return 0;
-  }
-
-  auto device_inode::write(void const * /*buffer*/, size_t /*offset*/, size_t /*size*/) -> size_t
-  {
-    if (!m_device)
-    {
-      kapi::system::panic("[FILESYSTEM] device_inode has null device.");
-    }
-
-    // TODO BA-FS26 use device file?
-    // return m_device->write(buffer, offset, size);
-    return 0;
+    return kstd::make_shared<device_file>(m_device);
   }
 }  // namespace filesystem
\ No newline at end of file