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: arm, komeda: add Arm China Linlon D6 compatible Date: Sat, 14 Mar 2026 07:31:00 +1000 Message-ID: In-Reply-To: <20260313033119.33686-3-cunyuan.liu@cixtech.com> References: <20260313033119.33686-1-cunyuan.liu@cixtech.com> <20260313033119.33686-3-cunyuan.liu@cixtech.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 Adds `armchina,linlon-d6` as a new compatible string with `arm,mali-d71` as= the required fallback. The v2 rework correctly uses `enum` instead of the = previous approach (as suggested by Krzysztof): ```yaml - items: - - const: arm,mali-d32 + - enum: + - arm,mali-d32 + - armchina,linlon-d6 - const: arm,mali-d71 ``` This is the correct DT binding pattern =E2=80=94 the Linlon-D6 must be desc= ribed with `compatible =3D "armchina,linlon-d6", "arm,mali-d71"` in the dev= ice tree, which the schema now validates. The `arm,mali-d32` case is also p= roperly folded into the same `enum` to keep the schema clean. No issues. --- Generated by Claude Code Patch Reviewer