From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: ARM: dts: imx6sx: remove LDB reg property Date: Tue, 05 May 2026 07:44:16 +1000 Message-ID: In-Reply-To: <20260504-v7-1-topic-ldb-fixes-v1-3-0125d937ff7f@pengutronix.de> References: <20260504-v7-1-topic-ldb-fixes-v1-0-0125d937ff7f@pengutronix.de> <20260504-v7-1-topic-ldb-fixes-v1-3-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 the `reg` property and unit address from the bridge node in `imx6sx= .dtsi`. **Changes reviewed:** ```dts - lvds_bridge: bridge@18 { + lvds_bridge: bridge { compatible =3D "fsl,imx6sx-ldb"; - reg =3D <0x18 0x4>; ``` **Assessment:** Correct. The offset `0x18` matches the hardcoded `.ldb_ctrl= =3D 0x18` in the driver's `fsl_ldb_devdata[IMX6SX_LDB]`, confirming the `r= eg` was redundant. The parent `gpr: syscon@20e4000` still has its own `reg`= and the `#address-cells`/`#size-cells` properties =E2=80=94 these remain b= ecause other children of the syscon node may still have unit addresses (I v= erified the parent node retains these). The node label `lvds_bridge` is preserved, so all overlay files referencing= `&lvds_bridge` continue to work. **No issues.** --- --- Generated by Claude Code Patch Reviewer