diff options
| author | Liav A <liavalb@gmail.com> | 2021-09-23 10:20:54 +0300 |
|---|---|---|
| committer | Andreas Kling <kling@serenityos.org> | 2021-09-29 11:24:33 +0200 |
| commit | 057f5a12c2c69b1bad1d8b778d3cb934b3a8f346 (patch) | |
| tree | e982c24df38e71e0d1c7dc0745c8ceec99c926cd /Kernel/Net/RTL8168NetworkAdapter.cpp | |
| parent | da327746a2702bd40ae6fa79017850ed357f5c1f (diff) | |
| download | serenity-057f5a12c2c69b1bad1d8b778d3cb934b3a8f346.zip | |
Kernel/PCI: Propagate usage of DeviceIdentifier everywhere
This allows us to remove a bunch of PCI API functions, and instead to
leverage the cached data from DeviceIdentifier object in many places.
Diffstat (limited to 'Kernel/Net/RTL8168NetworkAdapter.cpp')
| -rw-r--r-- | Kernel/Net/RTL8168NetworkAdapter.cpp | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/Kernel/Net/RTL8168NetworkAdapter.cpp b/Kernel/Net/RTL8168NetworkAdapter.cpp index 787ce9472f..3b2459b86d 100644 --- a/Kernel/Net/RTL8168NetworkAdapter.cpp +++ b/Kernel/Net/RTL8168NetworkAdapter.cpp @@ -181,15 +181,14 @@ namespace Kernel { #define TX_BUFFER_SIZE 0x1FF8 #define RX_BUFFER_SIZE 0x1FF8 // FIXME: this should be increased (0x3FFF) -UNMAP_AFTER_INIT RefPtr<RTL8168NetworkAdapter> RTL8168NetworkAdapter::try_to_initialize(PCI::Address address) +UNMAP_AFTER_INIT RefPtr<RTL8168NetworkAdapter> RTL8168NetworkAdapter::try_to_initialize(PCI::DeviceIdentifier const& pci_device_identifier) { - auto id = PCI::get_hardware_id(address); - if (id.vendor_id != PCI::VendorID::Realtek) + if (pci_device_identifier.hardware_id().vendor_id != PCI::VendorID::Realtek) return {}; - if (id.device_id != 0x8168) + if (pci_device_identifier.hardware_id().device_id != 0x8168) return {}; - u8 irq = PCI::get_interrupt_line(address); - return adopt_ref_if_nonnull(new (nothrow) RTL8168NetworkAdapter(address, irq)); + u8 irq = PCI::get_interrupt_line(pci_device_identifier.address()); + return adopt_ref_if_nonnull(new (nothrow) RTL8168NetworkAdapter(pci_device_identifier.address(), irq)); } bool RTL8168NetworkAdapter::determine_supported_version() const |