Re: [PATCH v3 4/4] rust: serdev: Simplify callbacks

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] rust: missing #[inline] on OpenGuard::drop
--

commit bed85685e823335d80519c5fa72de2689c661486
Author: Markus Probst <markus.probst@posteo.de>

rust: serdev: Simplify callbacks

This commit simplifies the serdev callbacks by initializing the driver's
private data directly on PrivateData. It also introduces the OpenGuard
struct to handle device resource cleanup.

> diff --git a/rust/kernel/serdev.rs b/rust/kernel/serdev.rs
> index c16d6593a8d2d..3f9165e167764 100644
> --- a/rust/kernel/serdev.rs
> +++ b/rust/kernel/serdev.rs

[ ... ]

> @@ -92,24 +88,36 @@ unsafe fn unregister(sdrv: &Opaque<Self::DriverType>) {
>      }
>  }
>  
> +struct OpenGuard<'bound> {
> +    sdev: &'bound Device<device::Bound>,
> +}
> +
> +impl Drop for OpenGuard<'_> {
> +    fn drop(&mut self) {

[Severity: Low]
Does this small abstraction method forwarding to a C binding need the
#[inline] annotation? The Rust subsystem guidelines recommend #[inline]
for small functions or functions forwarding to a C binding call in
abstractions to ensure compiler optimizations across crate boundaries.

> +        // SAFETY:
> +        // - `self.sdev.as_raw()` is guaranteed to be a pointer to a valid
> +        //   `struct serdev_device`.
> +        // - The existence of self proves that the device is open.
> +        unsafe { bindings::serdev_device_close(self.sdev.as_raw()) };
> +    }
> +}

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260930-rust_serdev_probe_refactor-v3-0-d1b6e77135e0@posteo.de?part=4