Re: [PATCH v2] drm/panel: samsung-s6d16d0: Use mipi_dsi_*_multi(); fix minor bugs
Linus Walleij <[email protected]>
| Newsgroups | gmane.linux.kernel,gmane.comp.video.dri.devel |
|---|---|
| Message-ID | <CAD++jLkms1eTK5_73M_p0BTCjrxhKKjrTfhy-a3=_hLAehVYTw@mail.gmail.com> |
On Tue, Aug 11, 2026 at 1:45 AM Akash Sukhavasi <[email protected]> wrote: > The mipi_dsi_dcs_*() functions used by this driver are deprecated > in favour of their _multi() counterparts, as noted in > Documentation/gpu/todo.rst. The _multi() variants record the > first error in a context structure and skip subsequent calls once > an error is set, removing the need to check the return value > after each command. They also log failures internally, making the > per-call dev_err() calls redundant. > > Convert prepare(), enable(), disable(), and unprepare() to use > mipi_dsi_dcs_*_multi(). > > unprepare() previously returned an error if > mipi_dsi_dcs_enter_sleep_mode() failed, skipping RESET assertion > and regulator_disable(). Because drm_panel_unprepare() does not > clear panel->prepared when the callback returns an error, > drm_panel_prepare() would then return early on the next call, > leaving the panel powered and unable to be re-initialised. The > converted code always asserts RESET, disables the regulator, and > returns 0. > > Also fix a typo in a comment ("Enabe" -> "Enable"). > > Signed-off-by: Akash Sukhavasi <[email protected]> Patch applied to drm-misc-next. I can't test it for the moment but that is no reason to hold back the patch, I will deal with the fallout of there is some problem! Yours, Linus Walleij