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 Toradex Capacitive Touch Display 10.1" LVDS Date: Mon, 25 May 2026 18:25:01 +1000 Message-ID: In-Reply-To: <20260522161105.277519-15-ivitro@gmail.com> References: <20260522161105.277519-13-ivitro@gmail.com> <20260522161105.277519-15-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 This overlay drives the same panel (LT170410-2WHC) but via the native AM62 = OLDI output rather than a DSI bridge, for carrier boards like Mallow. - The overlay correctly defines `pinctrl_i2s_2_d_in_gpio` and `pinctrl_i2s_= 2_sync_gpio` in `&main_pmx0` within the overlay itself. These pin controls = are NOT in the base dtsi, so they must be defined here. This is correct =E2= =80=94 they're only needed when this specific overlay is applied. - The `reg_3v3_lvds` fixed regulator has `regulator-name =3D "+V3.3_LVDS"` = =E2=80=94 good, this was a v3 fix. - The touchscreen is on `&main_i2c2` (Verdin I2C_2_DSI), which is the corre= ct bus for display touch on this platform. - The `backlight-pwm2` uses `epwm0 1` (PWM_2 / SODIMM 16), while patch 1 us= es `epwm1 0` (PWM_3_DSI / SODIMM 19). The different PWM channels match the = different Verdin signal routing for LVDS vs DSI. Correct. **No blocking issues.** --- Generated by Claude Code Patch Reviewer