Re: [PATCH v5 0/4] rust: miscdevice: Provide sample driver using the new MiscDevice bindings

From: Danilo Krummrich
Date: Fri Dec 06 2024 - 07:53:56 EST


On Fri, Dec 06, 2024 at 12:42:11PM +0000, Lee Jones wrote:
> It has been suggested that the driver should use dev_info() instead of
> pr_info() however there is currently no scaffolding to successfully pull
> a 'struct device' out from driver data post register(). This is being
> worked on and we will convert this over in due course.

I think you're going too fast with this series.

Please address the comments you receive before sending out new versions.

Also, please document the changes you have made from one version to the next,
otherwise it's gonna be very hard to review this.

Thanks,
Danilo

>
> Lee Jones (4):
> Documentation: ioctl-number: Carve out some identifiers for use by
> sample drivers
> samples: rust: Provide example using the new Rust MiscDevice
> abstraction
> sample: rust_misc_device: Demonstrate additional get/set value
> functionality
> MAINTAINERS: Add Rust Misc Sample to MISC entry
>
> .../userspace-api/ioctl/ioctl-number.rst | 1 +
> MAINTAINERS | 1 +
> samples/rust/Kconfig | 10 ++
> samples/rust/Makefile | 1 +
> samples/rust/rust_misc_device.rs | 132 ++++++++++++++++++
> 5 files changed, 145 insertions(+)
> create mode 100644 samples/rust/rust_misc_device.rs
>
> --
> 2.47.0.338.g60cca15819-goog
>
>