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: chrontel-ch7033: 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-5-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-5-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** This patch takes the approach of moving to `bridge.next_bridge` (the auto-put field in `struct drm_bridge`), removing the separate `next_bridge` pointer from the driver's private struct. All usages of `priv->next_bridge` are converted to `priv->bridge.next_bridge`. This is the cleanest approach for bridge drivers that store the downstream bridge pointer for their lifetime. The `bridge.next_bridge` field is documented as "automatically put when this bridge is freed", so no explicit cleanup is needed. --- Generated by Claude Code Patch Reviewer