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/ingenic: Remove calls to drm_atomic_helper_check_plane_damage() Date: Thu, 04 Jun 2026 15:21:21 +1000 Message-ID: In-Reply-To: <20260530185716.65688-4-tzimmermann@suse.de> References: <20260530185716.65688-1-tzimmermann@suse.de> <20260530185716.65688-4-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 Removes the driver's own damage evaluation since the atomic helper framework now handles it. Two call sites removed: 1. `ingenic-drm-drv.c`: Conditional on `priv->soc_info->map_noncoherent` 2. `ingenic-ipu.c`: Two `goto out_check_damage` paths replaced with `return 0` The `goto` to `return` conversion is safe since the label only called the damage check followed by `return 0`. **No issues.** Clean removal of driver-specific workaround that patch 2 makes unnecessary. --- Generated by Claude Code Patch Reviewer