Re: [PATCH v2 6/8] rust: drm: file: Add File abstraction

[Alyssa Rosenzweig <alyssa@rosenzweig.io>]

> +        // SAFETY: A callback from `struct drm_driver::open` guarantees that
> +        // - `raw_dev` is valid pointer to a `sturct drm_device`

struct

> +        // - the corresponding `sturct drm_device` has been registered.

struct


> +        // SAFETY: `raw_file` valid pointer to a `struct drm_file`.

`raw_file` is a valid pointer..

With those comments fixed, this is

   Reviewed-by: Alyssa Rosenzweig <alyssa@rosenzweig.io>

---

Tangential food-for-thought: might be possible to introduce a helper to
kernel::error::Result to make the error handling here more
ergonomic, but I don't know what that would look like exactly.