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: adv7511: switch to of_drm_get_bridge_by_endpoint() Date: Tue, 05 May 2026 08:26:10 +1000 Message-ID: In-Reply-To: <20260504-drm-bridge-alloc-getput-panel_or_bridge-v4-8-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-8-b578c3daaf10@bootlin.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Patch Review **Status: Good** Like patch 3, this handles the optional bridge case (`-ENODEV` =E2=86=92 `N= ULL`). It uses the `bridge.next_bridge` approach (not `__free`) since the b= ridge pointer is needed for the lifetime of the driver. The `next_bridge` f= ield is removed from the `adv7511` header struct. The conditional `if (adv-= >bridge.next_bridge)` in `adv7511_bridge_attach()` correctly handles the ca= se where no downstream bridge is present. --- Generated by Claude Code Patch Reviewer