Re: [PATCH] drm/panel: tdo-tl070wsh30: Use mipi_dsi_*_multi() functions
Akash Sukhavasi <[email protected]>
| Newsgroups | dev.linux.lists.sashiko-reviews,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <[email protected]> |
On Sat, Jul 25, 2026 at 05:01:33PM +0000, [email protected] wrote: > [Severity: High] > Does returning early here on a DSI transmission error leave the panel in an > unrecoverable state? > > Because panel->prepared is never set to false on error, a subsequent call > to drm_panel_prepare() will immediately return 0 without re-running the > hardware initialization sequence, leaving the display permanently blank > until a system reboot. > > Could we unconditionally disable the regulator and return 0 here to ensure > the state machine remains synchronized, even if there are DSI errors? Agreed, and this is the alternative I raised below the cut in this patch. Unless anyone objects, I will send a v2 that disables the regulator unconditionally and returns 0. Sashiko AI review: https://sashiko.dev/#/patchset/[email protected]?part=1 -- Thanks, Akash