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: ti: k3-am62-verdin: Add Mezzanine with LG LP156WF1 LVDS panel Date: Mon, 25 May 2026 19:56:41 +1000 Message-ID: In-Reply-To: <20260521150038.103538-32-ivitro@gmail.com> References: <20260521150038.103538-17-ivitro@gmail.com> <20260521150038.103538-32-ivitro@gmail.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 Dual-channel LVDS overlay using `oldi0` + `oldi1` with `ti,companion-oldi` = and `ti,secondary-oldi` properties. This is the AM62 dual OLDI pattern. The panel node uses `ports` with `dual-lvds-odd-pixels` / `dual-lvds-even-p= ixels`, which correctly references the new schema from Patch 13. **`data-mapping =3D "jeida-24"`** =E2=80=94 different from the other panels= which use `"vesa-24"`. This matches the LG LP156WF1 datasheet which uses J= EIDA mapping. **No `power-supply`** property on the panel node. The backlight has `enable= -gpios` but the panel itself has no power supply reference. If the panel re= quires a separate power rail, this could be an issue. If it's always powere= d, it's fine. The `dss_ports` node uses multiple endpoints (`endpoint@0`, `endpoint@1`) w= ithin a single `port@0` to drive both OLDI channels from the same DSS video= port =E2=80=94 this matches the AM62 hardware architecture where VP1 feeds= both OLDI0 and OLDI1. --- Generated by Claude Code Patch Reviewer