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/vblank: timer: Reorganize get_vblank_timeout Date: Thu, 04 Jun 2026 14:03:59 +1000 Message-ID: In-Reply-To: <20260601141922.91498-5-tzimmermann@suse.de> References: <20260601141922.91498-1-tzimmermann@suse.de> <20260601141922.91498-5-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 **Concept:** Pure code motion to prepare for Patch 5. No functional changes claimed or observed. **Minor style note:** The `else { return false; }` at the end of the if/else chain is fine, though using early return for the disabled-with-no-clock case (as done in Patch 5's `else if (mode->crtc_clock)` fallthrough) would be more idiomatic kernel style. But this is a prep patch, so it's fine. No issues. --- --- Generated by Claude Code Patch Reviewer