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/panel: r61307/r69328: remove redundant checks Date: Sun, 15 Feb 2026 20:24:19 +1000 Message-ID: In-Reply-To: <20260215085140.20499-4-clamor95@gmail.com> References: <20260215085140.20499-1-clamor95@gmail.com> <20260215085140.20499-4-clamor95@gmail.com> 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 `bool prepared` field and `if (priv->prepared)` / `if (!priv->pre= pared)` guards from prepare/unprepare. The commit message states "This is now= done by the DRM framework itself." Verified: `drm_panel_prepare()` (drm_panel.c:122) checks `panel->prepared` an= d skips if already true, sets it to true after the callback succeeds. `drm_pa= nel_unprepare()` (drm_panel.c:177) checks `!panel->prepared` and skips if alr= eady false. The framework tracking is equivalent to the removed driver-level = tracking. No regressions. --- Generated by Claude Code Patch Reviewer