sugar/linux
1
0
Fork 0
Code Activity

samples: rust: auxiliary: misc cleanup of ParentDriver::connect()

Browse Source 
In ParentDriver::connect() rename parent to dev, use it for the
dev_info!() call, call pdev.vendor_() directly in the print statement
and remove the unnecessary generic type of Result.

Reviewed-by: Alice Ryhl <aliceryhl@google.com>
Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Danilo Krummrich <dakr@kernel.org>
This commit is contained in:
Danilo Krummrich
2025-10-21 00:34:29 +02:00
parent 675e514edd
commit 710ac54688
1 changed files with 5 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
samples/rust/rust_driver_auxiliary.rs
View File

@@ -70,16 +70,15 @@ impl pci::Driver for ParentDriver {
} }
impl ParentDriver { impl ParentDriver {
fn connect(adev: &auxiliary::Device) -> Result<()> { fn connect(adev: &auxiliary::Device) -> Result {
let parent = adev.parent(); let dev = adev.parent();
let pdev: &pci::Device = parent.try_into()?; let pdev: &pci::Device = dev.try_into()?;
let vendor = pdev.vendor_id();
dev_info!( dev_info!(
adev.as_ref(), dev,
"Connect auxiliary {} with parent: VendorID={}, DeviceID={:#x}\n", "Connect auxiliary {} with parent: VendorID={}, DeviceID={:#x}\n",
adev.id(), adev.id(),
vendor, pdev.vendor_id(),
pdev.device_id() pdev.device_id()
); );