From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: drm/ttm: Support 52-bit PAs in ttm_place Date: Sat, 16 May 2026 11:53:55 +1000 Message-ID: In-Reply-To: <20260513141253.20410-1-felix.kuehling@amd.com> References: <20260513141253.20410-1-felix.kuehling@amd.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Overall Series Review Subject: drm/ttm: Support 52-bit PAs in ttm_place Author: Felix Kuehling Patches: 6 Reviewed: 2026-05-16T11:53:55.057936 --- This is a single patch (v2) that widens `fpfn` and `lpfn` in `struct ttm_pl= ace` from `unsigned` (32-bit) to `uint64_t` (64-bit) to support physical ad= dresses beyond 44 bits (32-bit page number =C3=97 4KB page =3D 44-bit PA). The change is straightforward and minimal =E2=80=94 only the struct definit= ion is modified. However, there is a notable discrepancy between the commit= message and the actual code, and the patch lacks accompanying driver-side = audit work. **Note:** The drm-next tree already contains this exact change (`uint64_t` = for both fields), so this patch has either already been merged or an equiva= lent fix landed independently. --- Generated by Claude Code Patch Reviewer