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/amd/display: clarify average bandwidth comment in dcn4_calcs Date: Tue, 28 Apr 2026 15:11:58 +1000 Message-ID: In-Reply-To: <20260426191825.9727-1-adri.vero.dev@gmail.com> References: <20260426191825.9727-1-adri.vero.dev@gmail.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Overall Series Review Subject: drm/amd/display: clarify average bandwidth comment in dcn4_calcs Author: Adriano Vero Patches: 1 Reviewed: 2026-04-28T15:11:58.284280 --- This is a single comment-only patch that replaces a stale `FIXME_DCN4` with a proper block comment explaining the design rationale. There are no functional code changes. The patch is **correct and well-motivated**. The old comment expressed uncertainty about whether `cursor_bw` and `tdlut` bandwidth should be included. Inspecting the four accumulation sites at lines 2867-2871 confirms that `cursor_bw[k]` is already included in all four (sys_active SDP, sys_active DRAM, svp_prefetch SDP, svp_prefetch DRAM), and `tdlut` bandwidth is absent from all of them. The new comment accurately documents this state and provides the rationale for tdlut's absence (fetched only during prefetch/blanking). No concerns with this patch. --- Generated by Claude Code Patch Reviewer