Claude review: drm: Improve logic behind damage handling

[Claude Code Review Bot <claude-review@example.com>]

Overall Series Review

Subject: drm: Improve logic behind damage handling
Author: Thomas Zimmermann <tzimmermann@suse.de>
Patches: 22
Reviewed: 2026-06-04T15:21:20.564314

---

This is a well-structured 10-patch series from Thomas Zimmermann that cleans up and consolidates DRM damage clipping logic. The core insight is correct: damage evaluation was happening too early (before `atomic_check`), which meant driver `atomic_check` callbacks could trigger a modeset that invalidated the already-evaluated damage state. The series addresses this by:

1. Moving damage evaluation to **after** all `atomic_check` callbacks run
2. Using the `ignore_damage_clips` flag instead of destroying the damage blob
3. Moving src-coordinate comparison from the iterator into the check phase
4. Progressively removing the now-unnecessary `old_state` parameter from damage helpers

The series is well-ordered in v4 to avoid error-prone intermediate states, has appropriate acks/reviews from Zack Rusin and others, and includes kunit test updates. The approach is sound and the code is clean. No correctness bugs found.

---
Generated by Claude Code Patch Reviewer