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_merged() Date: Thu, 04 Jun 2026 15:21:22 +1000 Message-ID: In-Reply-To: <20260530185716.65688-9-tzimmermann@suse.de> References: <20260530185716.65688-1-tzimmermann@suse.de> <20260530185716.65688-9-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 Similar mass-conversion for `drm_atomic_helper_damage_merged()`. Touches 13 files including ast cursor, mipi_dbi, i915 (intel_plane.c and intel_psr.c), sitronix st7586, various tiny drivers, virtio, and vmwgfx stdu. The i915 changes are notable -- `intel_plane_copy_uapi_plane_damage()` loses its `old_uapi_plane_state` parameter, and the `plane_atomic_check` function no longer needs `old_primary_crtc_plane_state`. The small bonus typo fix `"Incase"` -> `"In case the"` in `intel_plane.c` is fine but should ideally be a separate trivial patch per maintainer conventions. It's minor enough not to block. **No issues.** --- Generated by Claude Code Patch Reviewer