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/i915/dp: Compute and include coasting vtotal for AS SDP Date: Sun, 12 Apr 2026 12:51:30 +1000 Message-ID: In-Reply-To: <20260408084239.1295325-26-ankit.k.nautiyal@intel.com> References: <20260408084239.1295325-1-ankit.k.nautiyal@intel.com> <20260408084239.1295325-26-ankit.k.nautiyal@intel.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review **R-b: Ville Syrjala** Adds coasting vtotal to the AS SDP pack, unpack, and comparison. Sets it during late compute for PR with Aux-less ALPM when async video timing is supported: ```c if (intel_alpm_is_alpm_aux_less(intel_dp, crtc_state) && intel_psr_pr_async_video_timing_supported(intel_dp)) as_sdp->coasting_vtotal = crtc_state->vrr.vmax; ``` Using `vrr.vmax` as the coasting vtotal (minimum refresh rate) is the safe default. No issues. --- Generated by Claude Code Patch Reviewer