* [PATCH] samples: rust_dma: use vertical import style
@ 2026-07-06 1:25 Guru Das Srinagesh
2026-07-08 7:37 ` Link Mauve
0 siblings, 1 reply; 2+ messages in thread
From: Guru Das Srinagesh @ 2026-07-06 1:25 UTC (permalink / raw)
To: Danilo Krummrich, Abdiel Janulgue, Daniel Almeida, Robin Murphy,
Andreas Hindborg, Miguel Ojeda, Boqun Feng, Gary Guo,
Björn Roy Baron, Benno Lossin, Alice Ryhl, Trevor Gross,
Tamir Duberstein, Alexandre Courbot, Onur Özkan
Cc: driver-core, rust-for-linux, linux-kernel, Guru Das Srinagesh
Convert `use` imports to vertical layout for better readability and
maintainability.
Signed-off-by: Guru Das Srinagesh <linux@gurudas.dev>
---
Came across a recent commit bc58905eb07 ("samples: rust_misc_device: use
vertical import style") and found a few more locations that could
benefit from this cleanup. No functional changes.
Separating out patches per-subsystem as per the review feedback in [0].
Tested via:
$ make LLVM=1 rustfmtcheck || echo "fail"
$
[0]: https://lore.kernel.org/lkml/20260628-b4-rust-vertical-imports-v1-0-98bc71d4810b@gurudas.dev/
---
samples/rust/rust_dma.rs | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/samples/rust/rust_dma.rs b/samples/rust/rust_dma.rs
index 5046b4628d0e..c46af109f4aa 100644
--- a/samples/rust/rust_dma.rs
+++ b/samples/rust/rust_dma.rs
@@ -12,9 +12,13 @@
Device,
DmaMask, //
},
- page, pci,
+ page,
+ pci, //
prelude::*,
- scatterlist::{Owned, SGTable},
+ scatterlist::{
+ Owned,
+ SGTable, //
+ },
sync::aref::ARef,
};
---
base-commit: dc59e4fea9d83f03bad6bddf3fa2e52491777482
change-id: 20260705-rfl-vert-imp-d12bdbe397c7
Best regards,
--
Guru Das Srinagesh <linux@gurudas.dev>
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] samples: rust_dma: use vertical import style
2026-07-06 1:25 [PATCH] samples: rust_dma: use vertical import style Guru Das Srinagesh
@ 2026-07-08 7:37 ` Link Mauve
0 siblings, 0 replies; 2+ messages in thread
From: Link Mauve @ 2026-07-08 7:37 UTC (permalink / raw)
To: Guru Das Srinagesh
Cc: Danilo Krummrich, Abdiel Janulgue, Daniel Almeida, Robin Murphy,
Andreas Hindborg, Miguel Ojeda, Boqun Feng, Gary Guo,
Björn Roy Baron, Benno Lossin, Alice Ryhl, Trevor Gross,
Tamir Duberstein, Alexandre Courbot, Onur Özkan, driver-core,
rust-for-linux, linux-kernel
Hi,
On Sun, Jul 05, 2026 at 06:25:37PM -0700, Guru Das Srinagesh wrote:
> Convert `use` imports to vertical layout for better readability and
> maintainability.
>
> Signed-off-by: Guru Das Srinagesh <linux@gurudas.dev>
> ---
> Came across a recent commit bc58905eb07 ("samples: rust_misc_device: use
> vertical import style") and found a few more locations that could
> benefit from this cleanup. No functional changes.
>
> Separating out patches per-subsystem as per the review feedback in [0].
>
> Tested via:
>
> $ make LLVM=1 rustfmtcheck || echo "fail"
> $
>
> [0]: https://lore.kernel.org/lkml/20260628-b4-rust-vertical-imports-v1-0-98bc71d4810b@gurudas.dev/
> ---
> samples/rust/rust_dma.rs | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/samples/rust/rust_dma.rs b/samples/rust/rust_dma.rs
> index 5046b4628d0e..c46af109f4aa 100644
> --- a/samples/rust/rust_dma.rs
> +++ b/samples/rust/rust_dma.rs
> @@ -12,9 +12,13 @@
> Device,
> DmaMask, //
> },
> - page, pci,
> + page,
> + pci, //
Usually we put the empty comment at the very end of the imports, not in
the middle.
> prelude::*,
> - scatterlist::{Owned, SGTable},
> + scatterlist::{
> + Owned,
> + SGTable, //
> + },
> sync::aref::ARef,
So here: ↑
> };
>
>
> ---
> base-commit: dc59e4fea9d83f03bad6bddf3fa2e52491777482
> change-id: 20260705-rfl-vert-imp-d12bdbe397c7
>
> Best regards,
> --
> Guru Das Srinagesh <linux@gurudas.dev>
>
>
Thanks,
--
Link Mauve
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-07-08 7:37 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-07-06 1:25 [PATCH] samples: rust_dma: use vertical import style Guru Das Srinagesh
2026-07-08 7:37 ` Link Mauve
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox