diff options
author | Liav A <liavalb@gmail.com> | 2021-10-22 19:50:28 +0300 |
---|---|---|
committer | Andreas Kling <kling@serenityos.org> | 2021-10-23 19:17:44 +0200 |
commit | ca32ee571447a0f780ce92a364a63a5b9537032f (patch) | |
tree | 2eb63258425252fbaedbb861bac328fa1b41448a | |
parent | 79522db5d02986d7185e08b4e46dde9c5bdfd309 (diff) | |
download | serenity-ca32ee571447a0f780ce92a364a63a5b9537032f.zip |
Kernel/PCI: Simplify detect_optimal_access_type function
Instead of getting the kernel commandline argument as function parameter
we just take internally in the function.
-rw-r--r-- | Kernel/Bus/PCI/Initializer.cpp | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/Kernel/Bus/PCI/Initializer.cpp b/Kernel/Bus/PCI/Initializer.cpp index 203b9c264e..24bc607a0a 100644 --- a/Kernel/Bus/PCI/Initializer.cpp +++ b/Kernel/Bus/PCI/Initializer.cpp @@ -19,8 +19,9 @@ namespace PCI { static bool test_pci_io(); -UNMAP_AFTER_INIT static PCIAccessLevel detect_optimal_access_type(PCIAccessLevel boot_determined) +UNMAP_AFTER_INIT static PCIAccessLevel detect_optimal_access_type() { + auto boot_determined = kernel_command_line().pci_access_level(); if (!ACPI::is_enabled() || !ACPI::Parser::the()->find_table("MCFG").has_value()) return PCIAccessLevel::IOAddressing; @@ -35,9 +36,7 @@ UNMAP_AFTER_INIT static PCIAccessLevel detect_optimal_access_type(PCIAccessLevel UNMAP_AFTER_INIT void initialize() { - auto boot_determined = kernel_command_line().pci_access_level(); - - switch (detect_optimal_access_type(boot_determined)) { + switch (detect_optimal_access_type()) { case PCIAccessLevel::MemoryAddressing: { auto mcfg = ACPI::Parser::the()->find_table("MCFG"); VERIFY(mcfg.has_value()); |