From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: arm64: dts: imx8mp: remove LDB reg and reg-names property Date: Tue, 05 May 2026 07:44:16 +1000 Message-ID: In-Reply-To: <20260504-v7-1-topic-ldb-fixes-v1-4-0125d937ff7f@pengutronix.de> References: <20260504-v7-1-topic-ldb-fixes-v1-0-0125d937ff7f@pengutronix.de> <20260504-v7-1-topic-ldb-fixes-v1-4-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 Removes `reg`, `reg-names`, and the unit address from the bridge node in `i= mx8mp.dtsi`. **Changes reviewed:** ```dts - lvds_bridge: bridge@5c { + lvds_bridge: bridge { compatible =3D "fsl,imx8mp-ldb"; - reg =3D <0x5c 0x4>, <0x128 0x4>; - reg-names =3D "ldb", "lvds"; ``` **Assessment:** Correct. The offsets `0x5c` and `0x128` match the hardcoded= `.ldb_ctrl =3D 0x5c` and `.lvds_ctrl =3D 0x128` in `fsl_ldb_devdata[IMX8MP= _LDB]`, confirming redundancy. The parent `media_blk_ctrl` retains its `#ad= dress-cells`/`#size-cells` for other children. The commit message has the same "porperty" typo (should be "property"). Not= blocking. **No issues.** --- **Summary:** Clean, well-motivated series with correct ordering and no func= tional risk. The only cosmetic nits are minor typos ("controlls", "porperty= ") in commit messages. No code changes are needed =E2=80=94 this is purely = a DT cleanup that removes properties the driver never consumed. --- Generated by Claude Code Patch Reviewer