From mboxrd@z Thu Jan  1 00:00:00 1970
From: Claude Code Review Bot <claude-review@example.com>
To: dri-devel-reviews@example.com
Subject: Claude review: dt-bindings: display: msm: Document DSI controller and DSI PHY on SC8280XP
Date: Fri, 27 Feb 2026 14:04:45 +1000
Message-ID: <review-patch3-20260225054525.6803-4-mitltlatltl@gmail.com>
In-Reply-To: <20260225054525.6803-4-mitltlatltl@gmail.com>
References: <20260225054525.6803-1-mitltlatltl@gmail.com> <20260225054525.6803-4-mitltlatltl@gmail.com>
X-Mailer: Claude Code Patch Reviewer
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0

Patch Review

Adds DSI controller and PHY pattern properties to the SC8280XP MDSS binding:

```yaml
+  "^dsi@[0-9a-f]+$":
+    type: object
+    additionalProperties: true
+    properties:
+      compatible:
+        contains:
+          const: qcom,sc8280xp-dsi-ctrl
+
+  "^phy@[0-9a-f]+$":
+    type: object
+    additionalProperties: true
+    properties:
+      compatible:
+        contains:
+          const: qcom,sc8280xp-dsi-phy-5nm
```

Also adds port@1 and port@2 example endpoints for intf1/intf2 (DSI0/DSI1). Looks correct and follows the pattern of other MDSS bindings.

---

---
Generated by Claude Code Patch Reviewer