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/display: bridge_connector: init scdc debugfs for HDMI Date: Mon, 25 May 2026 21:49:33 +1000 Message-ID: In-Reply-To: <20260520-scdc-link-health-v2-2-511af18cd64b@collabora.com> References: <20260520-scdc-link-health-v2-0-511af18cd64b@collabora.com> <20260520-scdc-link-health-v2-2-511af18cd64b@collabora.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review Clean and correct. Just 4 lines adding the SCDC debugfs init call when a bridge has HDMI support. ```c if (bridge_connector->bridge_hdmi) drm_scdc_debugfs_init(connector, root); ``` This is placed after the per-bridge `debugfs_init` loop, which is the right location. The `drm_scdc_debugfs_init` function already handles NULL checks and allocation failure gracefully. No issues with this patch. --- Generated by Claude Code Patch Reviewer