From mboxrd@z Thu Jan  1 00:00:00 1970
From: Claude Code Review Bot <claude-review@example.com>
To: dri-devel-reviews@example.com
Subject: Claude review: rust: driver: move 'static bounds to constructor
Date: Mon, 18 May 2026 16:24:32 +1000
Message-ID: <review-patch2-20260517000149.3226762-3-dakr@kernel.org>
In-Reply-To: <20260517000149.3226762-3-dakr@kernel.org>
References: <20260517000149.3226762-1-dakr@kernel.org> <20260517000149.3226762-3-dakr@kernel.org>
X-Mailer: Claude Code Patch Reviewer
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0

Patch Review

Removes the `'static` bound from `set_drvdata` and `get_drvdata`, changing =
from:

```rust
-    pub fn set_drvdata<T: ForeignOwnable + 'static>(&self, data: T) {
+    pub fn set_drvdata<T: ForeignOwnable>(&self, data: T) {
```

This is a prerequisite for allowing non-`'static` driver data. The `'static=
` bound was overly conservative =E2=80=94 `ForeignOwnable` already ensures =
the data can be round-tripped through a raw pointer. Clean change.

---
Generated by Claude Code Patch Reviewer