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: bridge: Document Renesas R-Car V4H DSC bindings Date: Sat, 16 May 2026 09:29:57 +1000 Message-ID: In-Reply-To: <20260515-rcar-du-dsc-v3-2-164157820498@ideasonboard.com> References: <20260515-rcar-du-dsc-v3-0-164157820498@ideasonboard.com> <20260515-rcar-du-dsc-v3-2-164157820498@ideasonboard.com> 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 correct. Two minor observations: 1. The binding uses `unevaluatedProperties: false` at the top level. Since = there is no `allOf` inheriting from another schema, `additionalProperties: = false` would be the more conventional choice. Both work, but `additionalPro= perties` is the norm for standalone bindings (see most other files in the s= ame directory). 2. The binding documents `interrupts` and `resets` as required properties, = which is correct for completeness of the hardware description, even though = the current driver (patch 3) doesn't use either. This is fine =E2=80=94 bin= dings should describe the hardware, not the current driver capability. --- Generated by Claude Code Patch Reviewer