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: rockchip: analogix-dp: Expose inherited properties Date: Sat, 16 May 2026 11:13:13 +1000 Message-ID: In-Reply-To: <20260514070133.2275069-2-damon.ding@rock-chips.com> References: <20260514070133.2275069-1-damon.ding@rock-chips.com> <20260514070133.2275069-2-damon.ding@rock-chips.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 **Status: Looks correct.** This patch adds `true` declarations for properties inherited from the base = `analogix,dp.yaml` schema to satisfy `unevaluatedProperties: false` in the = Rockchip binding file. The properties exposed (`reg`, `interrupts`, `phys`,= `phy-names`, `force-hpd`, `ports`) all exist in the parent schema and are = already used in the example DT node at the bottom of the file (e.g., `reg`,= `interrupts`, `phys`, `phy-names`). ```yaml + reg: true + interrupts: true + phys: true + phy-names: true + force-hpd: true + ports: true ``` This is a straightforward schema fix. It's technically a prerequisite for t= he series (since patch 2 modifies the parent schema), but it's also a valid= standalone fix =E2=80=94 these properties should have been listed already = given the `unevaluatedProperties: false` constraint. No issues. --- --- Generated by Claude Code Patch Reviewer