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-binding: display: Add ITE IT61620 MIPI DSI to HDMI bridge Date: Sun, 22 Mar 2026 03:53:20 +1000 Message-ID: In-Reply-To: <20260320-it61620-0714-v8-1-0e70271cf5a9@ite.com.tw> References: <20260320-it61620-0714-v8-0-0e70271cf5a9@ite.com.tw> <20260320-it61620-0714-v8-1-0e70271cf5a9@ite.com.tw> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Patch Review The binding looks clean overall. **Minor: pinctrl in example but not in schema** The example includes `pinctrl-names` and `pinctrl-0` properties, but these = are not listed in the schema `properties` section. While `unevaluatedProper= ties: false` won't reject them (since pinctrl is handled by a common schema= ), the v4 changelog says "Drop pinctrl-names and pinctrl-0; driver does not= require them" =E2=80=94 yet they remain in the example. Consider removing = them from the example for consistency. **Nit: missing period in description** ```yaml + The main differences are listed below ``` Missing a colon or period at the end of this line. Otherwise this patch looks fine. --- --- Generated by Claude Code Patch Reviewer