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/dp: Store coasting vtotal in struct drm_dp_as_sdp Date: Sun, 12 Apr 2026 12:51:25 +1000 Message-ID: In-Reply-To: <20260408084239.1295325-6-ankit.k.nautiyal@intel.com> References: <20260408084239.1295325-1-ankit.k.nautiyal@intel.com> <20260408084239.1295325-6-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 Adds `coasting_vtotal` to `struct drm_dp_as_sdp` and logs it. The field is `int`, which is fine since vtotal values fit easily. The log string uses a space between "coasting" and "vtotal": ```c drm_printf(p, " coasting vtotal: %d\n", as_sdp->coasting_vtotal); ``` This is slightly inconsistent with other fields that use underscores (e.g., `target_rr`, `duration_incr_ms`). Minor style nit. Missing R-b from Ville (unlike neighboring patches). --- Generated by Claude Code Patch Reviewer