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: add of_drm_get_bridge_by_endpoint() Date: Sat, 16 May 2026 15:02:22 +1000 Message-ID: In-Reply-To: <20260511-drm-bridge-alloc-getput-panel_or_bridge-v6-2-f61c9e498b3f@bootlin.com> References: <20260511-drm-bridge-alloc-getput-panel_or_bridge-v6-0-f61c9e498b3f@bootlin.com> <20260511-drm-bridge-alloc-getput-panel_or_bridge-v6-2-f61c9e498b3f@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 The new function is a clean wrapper around `of_drm_find_and_get_bridge()` with OF graph resolution. Key details look correct: ```c bridge = of_drm_find_and_get_bridge(remote); if (!bridge) return ERR_PTR(-EPROBE_DEFER); ``` This correctly converts the NULL return from `of_drm_find_and_get_bridge()` (bridge registered but not yet available) into `-EPROBE_DEFER`, matching the semantics of the old `drm_of_find_panel_or_bridge()`. The non-OF inline stub correctly returns `ERR_PTR(-ENODEV)` rather than NULL, consistent with the new API contract. **Very minor nit:** The kdoc return description is missing a trailing period: ``` * Returns a pointer to the connected drm_bridge, or a negative error on failure ``` Already has R-by from Dmitry and Laurent. No issues. --- Generated by Claude Code Patch Reviewer