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/mipi-dbi: Only modify planes on enabled CRTCs Date: Sun, 22 Mar 2026 04:15:32 +1000 Message-ID: In-Reply-To: <20260319160110.109610-2-tzimmermann@suse.de> References: <20260319160110.109610-1-tzimmermann@suse.de> <20260319160110.109610-2-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 Sound change. Switching to `drm_atomic_helper_commit_tail_rpm()` ensures CRTC enable happens before plane update, eliminating the redundant plane flush from each driver's enable callback. The removal of `mipi_dbi_enable_flush()` and inlining of `backlight_enable()` into each driver is correct. Each driver properly replaces the combined flush+backlight call with just `backlight_enable()`. No issues found. --- Generated by Claude Code Patch Reviewer