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/exynos: exynos_dp: Remove &exynos_dp_device.ptn_bridge Date: Wed, 11 Feb 2026 16:37:16 +1000 Message-ID: In-Reply-To: <20260210071225.2566099-4-damon.ding@rock-chips.com> References: <20260210071225.2566099-1-damon.ding@rock-chips.com> <20260210071225.2566099-4-damon.ding@rock-chips.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Mailer: Claude Code Patch Reviewer Patch Review **Summary:** Uses platform data field instead of device-local field for bridge pointer. ```c - if (dp->ptn_bridge) { - ret = drm_bridge_attach(&dp->encoder, dp->ptn_bridge, bridge, + if (plat_data->next_bridge) { + ret = drm_bridge_attach(&dp->encoder, plat_data->next_bridge, bridge, ``` **Technical review:** - Reduces redundant state storage - Makes ownership clearer (platform data owns the bridge reference) - The assignment happens in probe: `dp->plat_data.next_bridge = bridge;` **Verdict:** Good cleanup, improves data flow. --- --- Generated by Claude Code Patch Reviewer