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/msm: Fix typo in clock-names property Date: Tue, 28 Apr 2026 14:29:39 +1000 Message-ID: In-Reply-To: <20260427154658.276737-1-prabhakar.mahadev-lad.rj@bp.renesas.com> References: <20260427154658.276737-1-prabhakar.mahadev-lad.rj@bp.renesas.com> X-Mailer: Claude Code Patch Reviewer Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Overall Series Review Subject: dt-bindings: display/msm: Fix typo in clock-names property Author: Prabhakar Patches: 1 Reviewed: 2026-04-28T14:29:39.561063 --- This is a single-patch fix for a straightforward typo in the MSM DisplayPor= t controller device-tree binding schema. The property name `clocks-names` (= with a spurious "s") was used in five `allOf/if/then` conditional blocks in= stead of the correct `clock-names`. Because YAML schema validators match pr= operty names exactly, these misspelled entries were silently **not constrai= ning anything** =E2=80=94 the `minItems`/`maxItems` limits on `clock-names`= within these conditional blocks were dead code. The fix is correct and com= plete. **Verdict: Patch looks good.** Straightforward, low-risk, obviously correct. --- Generated by Claude Code Patch Reviewer