Claude review: drm/radeon/radeon_connectors: use struct drm_edid

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

Overall Series Review

Subject: drm/radeon/radeon_connectors: use struct drm_edid
Author: Joshua Peisach <jpeisach@ubuntu.com>
Patches: 4
Reviewed: 2026-05-25T17:37:44.687299

---

This is a 2-patch series converting the radeon connector code from the deprecated `struct edid` / `drm_get_edid()` / `drm_connector_update_edid_property()` API to the newer `struct drm_edid` / `drm_edid_read_ddc()` / `drm_edid_connector_update()` API. This mirrors an equivalent conversion already done for amdgpu.

The overall approach is correct and the API migration is sound. The new functions exist in the kernel tree with the expected signatures. The series is logically split: patch 1 does the type/API conversion, patch 2 inlines and removes the now-trivial `radeon_connector_free_edid` helper. However, there are issues with the ordering of patches in the mbox (patch 2/2 appears before patch 1/2), and there are a few correctness concerns worth raising.

**Verdict: Needs minor revision** — the series is on the right track but has a few issues to address.

---

---
Generated by Claude Code Patch Reviewer