Claude review: (no cover subject)

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

Overall Series Review

Subject: (no cover subject)
Author: Abel Vesa <abel.vesa@oss.qualcomm.com>
Patches: 2
Reviewed: 2026-03-03T13:40:01.758671

---

This is a 2-patch series fixing Qualcomm Glymur DP controller device tree bindings. The changes are logically sound: Glymur's DP controllers have 4 ports in their register space (unlike similar platforms that only have 2), requiring 9 `reg` entries instead of 5. The series correctly separates Glymur into its own conditional block and updates the example node to match. Both patches are tagged for stable (v6.19), which is appropriate for binding fixes.

**One notable issue**: The patch perpetuates a pre-existing typo (`clocks-names` instead of `clock-names`) present in all existing conditional blocks in `dp-controller.yaml`. Since the property is defined as `clock-names` (line 101), all the `clocks-names` constraints in the `allOf` conditionals are effectively dead — they constrain a non-existent property and thus never enforce anything. This is worth flagging even though it's not introduced by this series.

The v2 was sent without a cover subject and the author noted it should be ignored; v3 is the corrected submission with identical patches.

---
Generated by Claude Code Patch Reviewer