From mboxrd@z Thu Jan  1 00:00:00 1970
From: Claude Code Review Bot <claude-review@example.com>
To: dri-devel-reviews@example.com
Subject: Claude review: Revert "dt-bindings: display: bridge: ldb: Add check for reg and reg-names"
Date: Tue, 05 May 2026 07:44:15 +1000
Message-ID: <review-patch1-20260504-v7-1-topic-ldb-fixes-v1-1-0125d937ff7f@pengutronix.de>
In-Reply-To: <20260504-v7-1-topic-ldb-fixes-v1-1-0125d937ff7f@pengutronix.de>
References: <20260504-v7-1-topic-ldb-fixes-v1-0-0125d937ff7f@pengutronix.de> <20260504-v7-1-topic-ldb-fixes-v1-1-0125d937ff7f@pengutronix.de>
X-Mailer: Claude Code Patch Reviewer
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0

Patch Review

This reverts commit `8aa2f0ac08d3`, removing the `reg` requirement from the=
 `required:` list and the conditional `reg-names` requirement for non-imx6s=
x variants.

**Changes reviewed:**

```yaml
-  - reg
```
removed from the `required:` block, and the entire conditional block:
```yaml
-  - if:
-      not:
-        properties:
-          compatible:
-            contains:
-              const: fsl,imx6sx-ldb
-    then:
-      required:
-        - reg-names
```
is also removed.

**Assessment:** Correct. The commit message clearly explains why =E2=80=94 =
the syscon register ranges have multi-purpose registers and `reg` per sub-d=
evice is wrong. The revert is clean and the remaining schema (`fsl,ldb.yaml=
`) remains valid without the `reg` requirement.

**Minor nit:** The commit message has a typo "controlls" (should be "contro=
ls") =E2=80=94 same typo appears in patches 2 and 3. Not blocking.

**No issues.**

---

---
Generated by Claude Code Patch Reviewer