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/bridge: dw-hdmi-qp: use drm_hdmi_acr_get_n_cts() helper for audio N/CTS Date: Mon, 25 May 2026 20:47:24 +1000 Message-ID: In-Reply-To: <86fcf349-0a7a-4618-9001-612371b0f71b@symple.nz> References: <86fcf349-0a7a-4618-9001-612371b0f71b@symple.nz> 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/bridge: dw-hdmi-qp: use drm_hdmi_acr_get_n_cts() helper for audio N/CTS Author: Simon Wright Patches: 1 Reviewed: 2026-05-25T20:47:24.201809 --- This is a clean, well-motivated single patch that replaces ~200 lines of open-coded HDMI Audio Clock Regeneration (ACR) N/CTS lookup tables and computation functions with a single call to the existing shared helper `drm_hdmi_acr_get_n_cts()`. The patch fixes a real audio bug on strict HDMI sinks (LG G3 OLED) at TMDS rates not present in the removed CTS table, and the commit message is thorough with excellent testing documentation. **Verdict: Looks good.** No correctness issues found. --- Generated by Claude Code Patch Reviewer