> + // 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 <aly...@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.