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: panel: Align style of "true" properties Date: Sat, 14 Mar 2026 07:18:06 +1000 Message-ID: In-Reply-To: <20260313-dt-bindings-display-panel-clean-v2-1-d49615218f92@oss.qualcomm.com> References: <20260313-dt-bindings-display-panel-clean-v2-1-d49615218f92@oss.qualcomm.com> <20260313-dt-bindings-display-panel-clean-v2-1-d49615218f92@oss.qualcomm.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 **Changes across 11 YAML binding files:** The patch consistently applies two style rules: 1. **Remove blank lines between consecutive `": true"` properties** =E2=80= =94 e.g., in `himax,hx8394.yaml`: ```yaml - reset-gpios: true - - backlight: true - - rotation: true - - port: true + reset-gpios: true + backlight: true + rotation: true + port: true ``` 2. **Add a blank line between the `": true"` block and properties with sub-= fields** =E2=80=94 e.g., in `leadtek,ltk050h3146w.yaml`: ```yaml reset-gpios: true + iovcc-supply: description: regulator that supplies the iovcc voltage ``` 3. **Reorder some `": true"` properties to group them together at the end o= f the properties block** =E2=80=94 e.g., in `mantix,mlaf057we51-x.yaml`, `p= ort: true` and `reset-gpios: true` are moved from their scattered locations= to a grouped block alongside `backlight: true`. Similarly in `rocktech,jh0= 57n00900.yaml`, `novatek,nt35510.yaml`, and `startek,kd070fhfid015.yaml`. **Assessment:** All changes are consistent and correctly applied. The reord= ering of properties within the `properties:` block in YAML schemas has no f= unctional impact since property order does not affect DT binding validation= . The patch improves readability by making the style uniform across these f= iles. No issues found. This is a clean cosmetic patch ready to be applied. --- Generated by Claude Code Patch Reviewer