From: "Onur Özkan" <work@onurozkan.dev>
To: Alvin Sun <alvin.sun@linux.dev>
Cc: "Arnd Bergmann" <arnd@arndb.de>,
"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
"Miguel Ojeda" <ojeda@kernel.org>,
"Boqun Feng" <boqun@kernel.org>, "Gary Guo" <gary@garyguo.net>,
"Björn Roy Baron" <bjorn3_gh@protonmail.com>,
"Benno Lossin" <lossin@kernel.org>,
"Andreas Hindborg" <a.hindborg@kernel.org>,
"Alice Ryhl" <aliceryhl@google.com>,
"Trevor Gross" <tmgross@umich.edu>,
"Danilo Krummrich" <dakr@kernel.org>,
rust-for-linux@vger.kernel.org, linux-block@vger.kernel.org,
"Onur Özkan" <work@onurozkan.dev>
Subject: Re: [PATCH v2 2/5] samples: rust_misc_device: use vertical import style
Date: Wed, 20 May 2026 05:58:39 +0300 [thread overview]
Message-ID: <20260520025842.34118-1-work@onurozkan.dev> (raw)
In-Reply-To: <20260520-miscdev-use-format-v2-2-64dc48fc1345@linux.dev>
On Wed, 20 May 2026 10:40:09 +0800
Alvin Sun <alvin.sun@linux.dev> wrote:
> Convert `use` imports to vertical layout for better readability and
> maintainability.
>
> Signed-off-by: Alvin Sun <alvin.sun@linux.dev>
Reviewed-by: Onur Özkan <work@onurozkan.dev>
> ---
> samples/rust/rust_misc_device.rs | 34 ++++++++++++++++++++++++++++------
> 1 file changed, 28 insertions(+), 6 deletions(-)
>
> diff --git a/samples/rust/rust_misc_device.rs b/samples/rust/rust_misc_device.rs
> index 87a1fe63533ae..41e26c825060b 100644
> --- a/samples/rust/rust_misc_device.rs
> +++ b/samples/rust/rust_misc_device.rs
> @@ -97,14 +97,36 @@
>
> use kernel::{
> device::Device,
> - fs::{File, Kiocb},
> - ioctl::{_IO, _IOC_SIZE, _IOR, _IOW},
> - iov::{IovIterDest, IovIterSource},
> - miscdevice::{MiscDevice, MiscDeviceOptions, MiscDeviceRegistration},
> + fs::{
> + File,
> + Kiocb, //
> + },
> + ioctl::{
> + _IO,
> + _IOC_SIZE,
> + _IOR,
> + _IOW, //
> + },
> + iov::{
> + IovIterDest,
> + IovIterSource, //
> + },
> + miscdevice::{
> + MiscDevice,
> + MiscDeviceOptions,
> + MiscDeviceRegistration, //
> + },
> new_mutex,
> prelude::*,
> - sync::{aref::ARef, Mutex},
> - uaccess::{UserSlice, UserSliceReader, UserSliceWriter},
> + sync::{
> + aref::ARef,
> + Mutex, //
> + },
> + uaccess::{
> + UserSlice,
> + UserSliceReader,
> + UserSliceWriter, //
> + },
> };
>
> const RUST_MISC_DEV_HELLO: u32 = _IO('|' as u32, 0x80);
>
> --
> 2.43.0
>
>
next prev parent reply other threads:[~2026-05-20 2:58 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-20 2:40 [PATCH v2 0/5] rust: use vertical import style and remove redundant imports Alvin Sun
2026-05-20 2:40 ` [PATCH v2 1/5] rust: miscdevice: use vertical import style Alvin Sun
2026-05-20 2:57 ` Onur Özkan
2026-05-20 2:40 ` [PATCH v2 2/5] samples: rust_misc_device: " Alvin Sun
2026-05-20 2:58 ` Onur Özkan [this message]
2026-05-20 2:40 ` [PATCH v2 3/5] rust: miscdevice: remove redundant imports Alvin Sun
2026-05-20 2:59 ` Onur Özkan
2026-05-20 2:40 ` [PATCH v2 4/5] rust: block: mq: use vertical import style Alvin Sun
2026-05-20 2:59 ` Onur Özkan
2026-05-20 2:40 ` [PATCH v2 5/5] rust: block: mq: remove redundant imports and format Alvin Sun
2026-05-20 2:59 ` Onur Özkan
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260520025842.34118-1-work@onurozkan.dev \
--to=work@onurozkan.dev \
--cc=a.hindborg@kernel.org \
--cc=aliceryhl@google.com \
--cc=alvin.sun@linux.dev \
--cc=arnd@arndb.de \
--cc=bjorn3_gh@protonmail.com \
--cc=boqun@kernel.org \
--cc=dakr@kernel.org \
--cc=gary@garyguo.net \
--cc=gregkh@linuxfoundation.org \
--cc=linux-block@vger.kernel.org \
--cc=lossin@kernel.org \
--cc=ojeda@kernel.org \
--cc=rust-for-linux@vger.kernel.org \
--cc=tmgross@umich.edu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox