From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: dt-bindings: display: ti,am65x-dss: Add AM62P DSS Date: Thu, 23 Apr 2026 09:45:58 +1000 Message-ID: In-Reply-To: <20260420-beagley-ai-display-v1-5-f628543dfd14@ideasonboard.com> References: <20260420-beagley-ai-display-v1-0-f628543dfd14@ideasonboard.com> <20260420-beagley-ai-display-v1-5-f628543dfd14@ideasonboard.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit MIME-Version: 1.0 Patch Review The lengthy description in the binding about OLDI control path muxing is valuable documentation. Adding `ti,am62p-dss` to the compatible enum is correct. Minor: The binding doesn't add conditional rules for `ti,am62p-dss` (e.g., `oldi-transmitters: false` or `true` depending on instance). This seems intentional since both DSS0 and DSS1 on AM62P can use OLDI, and the binding is meant to describe the IP, not per-instance SoC integration. Acceptable. --- Generated by Claude Code Patch Reviewer