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/damage-helper: Remove old state from drm_atomic_helper_damage_iter_init() Date: Thu, 04 Jun 2026 15:21:22 +1000 Message-ID: In-Reply-To: <20260530185716.65688-8-tzimmermann@suse.de> References: <20260530185716.65688-1-tzimmermann@suse.de> <20260530185716.65688-8-tzimmermann@suse.de> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review Mass-conversion patch removing the `old_state` parameter from `drm_atomic_helper_damage_iter_init()`. Touches 17 files across many drivers (ast, gud, hyperv, mgag200, sitronix, solomon, sysfb, appletbdrm, bochs, cirrus, udl, vmwgfx, and tests). Every call site is mechanically converted. Many callers also get their now-unused `old_plane_state` variable removed from `atomic_update` functions, which is the nice payoff of this cleanup. The test file removes `old_state` from the mock struct and all `set_plane_src(&mock->old_state, ...)` calls. **No issues.** Clean mechanical conversion. --- Generated by Claude Code Patch Reviewer