<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Thu, Mar 27, 2025 at 4:54 PM Doug Anderson <<a href="mailto:dianders@chromium.org">dianders@chromium.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
On Wed, Mar 26, 2025 at 8:29 PM Anusha Srivatsa <<a href="mailto:asrivats@redhat.com" target="_blank">asrivats@redhat.com</a>> wrote:<br>
><br>
> Move away from using deprecated API and use _multi variants<br>
> if available. Use mipi_dsi_msleep() and mipi_dsi_usleep_range()<br>
> instead of msleep() and usleep_range() respectively.<br>
><br>
> Used Coccinelle to find the _multi variant APIs,replacing<br>
> mpi_dsi_msleep() where necessary and for returning<br>
> dsi_ctx.accum_err in these functions. mipi_dsi_dcs_write()<br>
> does not have a corresponding _multi() variant. Replacing it with<br>
> mipi_dsi_dcs_write_seq_multi() instead. This change is manual.<br>
><br>
> The Coccinelle script is the same as the one in commit c8ba07caaecc<br>
> ("drm/panel/synaptics-r63353: Use _multi variants")<br>
><br>
> v2: Use mipi_dsi_write_buffer_multi() in place of<br>
> mipi_dsi_dcs_write(). (Dmitry)<br>
><br>
> v3: add commit details where the same coccinelle script is<br>
> used and remove the actual script from commit log.<br>
> Use mipi_dsi_dcs_write_seq_multi() for mipi_dsi_dcs_write() (Doug)<br>
><br>
> Cc: Maxime Ripard <<a href="mailto:mripard@kernel.org" target="_blank">mripard@kernel.org</a>><br>
> Cc: Dmitry Baryshkov <<a href="mailto:dmitry.baryshkov@linaro.org" target="_blank">dmitry.baryshkov@linaro.org</a>><br>
> Cc: Tejas Vipin <<a href="mailto:tejasvipin76@gmail.com" target="_blank">tejasvipin76@gmail.com</a>><br>
> Cc: Doug Anderson <<a href="mailto:dianders@chromium.org" target="_blank">dianders@chromium.org</a>><br>
> Signed-off-by: Anusha Srivatsa <<a href="mailto:asrivats@redhat.com" target="_blank">asrivats@redhat.com</a>><br>
> ---<br>
> Changes in v3:<br>
> - Simplify commit log by adding a reference to a patch that uses the<br>
>   same script.<br>
> - Simplify code by using a helper that doesnt need additional code<br>
>   changes other then using the helper itself.<br>
><br>
> - Link to v2: <a href="https://lore.kernel.org/r/20250324-b4-panel-ls043t1le01-v2-1-e43aedc115be@redhat.com" rel="noreferrer" target="_blank">https://lore.kernel.org/r/20250324-b4-panel-ls043t1le01-v2-1-e43aedc115be@redhat.com</a><br>
><br>
> Changes in v2:<br>
> - While mipi_dsi_dcs_write() does not have a corresponding _multi()<br>
>   variant replace it with mipi_dsi_dcs_write_buffer_multi() to have all<br>
>   APIs following _multi() usage for easier error handling<br>
><br>
> - Link to v1: <a href="https://lore.kernel.org/r/20250316-b4-panel-ls043t1le01-v1-1-ee38371b0ba0@redhat.com" rel="noreferrer" target="_blank">https://lore.kernel.org/r/20250316-b4-panel-ls043t1le01-v1-1-ee38371b0ba0@redhat.com</a><br>
> ---<br>
>  drivers/gpu/drm/panel/panel-sharp-ls043t1le01.c | 41 +++++++++----------------<br>
>  1 file changed, 15 insertions(+), 26 deletions(-)<br>
<br>
Reviewed-by: Douglas Anderson <<a href="mailto:dianders@chromium.org" target="_blank">dianders@chromium.org</a>><br></blockquote><div><br></div><div>Thanks Doug and Neil!</div><div><br></div><div>Anusha <br></div></div></div>