Claude review: dma::Coherent & dma::CoherentInit API

[Claude Code Review Bot <claude-review@example.com>]

Overall Series Review

Subject: dma::Coherent & dma::CoherentInit API
Author: Danilo Krummrich <dakr@kernel.org>
Patches: 12
Reviewed: 2026-03-04T07:03:01.673413

---

This is a well-structured 8-patch series that refactors the Rust DMA coherent allocation API from `CoherentAllocation<T>` (always a slice container) to a generalized `Coherent<T>` that supports both single-object and slice types, plus a `CoherentInit<T>` wrapper for safe initialization before DMA address exposure. The design is sound and the nova-core conversions in patches 6-8 demonstrate real ergonomic improvements (e.g., eliminating `[0]?` indexing for single-object allocations and removing stack-side initialization of large firmware structures).

**Key concern**: The series depends on `crate::ptr::KnownSize`, which does not exist in the current drm-next tree. This is presumably from Gary Guo's I/O projection work referenced in the cover letter. The base-commit dependency should be explicitly stated, or the series should be rebased onto a tree that includes that prerequisite.

Overall the API evolution is clean and the safety reasoning is mostly correct. A few minor issues are noted below.

---
Generated by Claude Code Patch Reviewer