Claude review: drm/bridge: analogix_dp: Extract error pointer from correct variable

[Claude Code Review Bot <claude-review@example.com>]

Overall Series Review

Subject: drm/bridge: analogix_dp: Extract error pointer from correct variable
Author: Ethan Tidmore <ethantidmore06@gmail.com>
Patches: 4
Reviewed: 2026-05-05T11:59:49.661899

---

This is a single-patch series that fixes a real, Smatch-detected bug in `analogix_dp_bind()` where `PTR_ERR()` is called on the wrong variable. The fix is straightforward and correct: the `IS_ERR()` check tests `dp->plat_data->next_bridge`, but the subsequent `PTR_ERR()` extracts the error code from `bridge` — a completely different, valid pointer. This means the error path would always return a bogus error code (likely 0 or a valid pointer cast to int), masking the real error.

**However**, this patch may already be obsolete. On drm-next, the `devm_drm_panel_bridge_add()` call and associated panel_bridge logic have been entirely removed from `analogix_dp_bind()`, meaning this code no longer exists in the current development tree. The patch would not apply cleanly to drm-next. The maintainers should evaluate whether this fix is still needed for stable backports to older kernels.

**Verdict**: The fix itself is correct but may be superseded by recent refactoring. Worth applying to stable branches if the affected code still exists there.

---
Generated by Claude Code Patch Reviewer