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/hisilicon/kirin: switch to of_drm_get_bridge_by_endpoint() Date: Tue, 05 May 2026 08:26:09 +1000 Message-ID: In-Reply-To: <20260504-drm-bridge-alloc-getput-panel_or_bridge-v4-4-b578c3daaf10@bootlin.com> References: <20260504-drm-bridge-alloc-getput-panel_or_bridge-v4-0-b578c3daaf10@bootlin.com> <20260504-drm-bridge-alloc-getput-panel_or_bridge-v4-4-b578c3daaf10@bootlin.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review **Status: Good** Simple conversion. The bridge is a local variable used only for `drm_bridge_attach()`, so the `__free(drm_bridge_put)` cleanup handles the refcount drop on scope exit. Since `drm_bridge_attach()` takes its own reference internally (as part of the refcounting work already merged), dropping the local reference after attach is correct. No `-ENODEV` handling needed here because the original code treated all errors equally (no optional bridge case). --- Generated by Claude Code Patch Reviewer