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: Remove unused header in drm_dumb_buffers.c Date: Sun, 12 Apr 2026 10:55:27 +1000 Message-ID: In-Reply-To: <20260409154826.8955-1-yiconghui@gmail.com> References: <20260409154826.8955-1-yiconghui@gmail.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Overall Series Review Subject: drm: Remove unused header in drm_dumb_buffers.c Author: Yicong Hui Patches: 1 Reviewed: 2026-04-12T10:55:27.497281 --- This is a single-patch series that removes an unused `#include "drm_internal.h"` from `drm_dumb_buffers.c`. The change is trivially correct. The commit message is well-written: it explains the provenance of the include (added by commit `47f10854ca89` when functions were moved to this file) and why it became unnecessary (commit `96a7b60f6ddb` removed `drm_gem_dumb_destroy`, which was the last user of declarations from that header). The testing described (build with DRM enabled, kunit, IGT in a VM) is appropriate for the scope. **Verdict: Looks good.** No issues found. --- Generated by Claude Code Patch Reviewer