From mboxrd@z Thu Jan 1 00:00:00 1970 From: Claude Code Review Bot To: dri-devel-reviews@example.com Subject: Claude review: drm/bridge: tc358762: Improve DPI enable handling Date: Sat, 16 May 2026 12:04:22 +1000 Message-ID: In-Reply-To: <20260513-tc358762-fixes-v3-6-6698b55008b9@ideasonboard.com> References: <20260513-tc358762-fixes-v3-0-6698b55008b9@ideasonboard.com> <20260513-tc358762-fixes-v3-6-6698b55008b9@ideasonboard.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 Adds `tc358762_write(ctx, LCDCTRL, 0)` at the start of init to clear DPIENA= BLE (which defaults to 1 on reset), preventing temporary DPI output with wr= ong timings during configuration. Also adds a matching `tc358762_write(ctx,= LCDCTRL, 0)` in `post_disable` to explicitly disable DPI output. Minor nit: the comment says "coniguration" =E2=80=94 typo for "configuratio= n": ```c * DPI until we have finished the coniguration. ``` This is cosmetic but worth fixing. Otherwise, the approach is correct. **Reviewed-by worthy** with the typo fi= x. --- Generated by Claude Code Patch Reviewer