<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 6, 2025 at 4:31 AM Maxime Ripard <<a href="mailto:mripard@kernel.org">mripard@kernel.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 Anusha,<br>
<br>
On Wed, Mar 05, 2025 at 07:01:41PM -0500, Anusha Srivatsa wrote:<br>
> Move away from using deprecated API and use _multi<br>
> variants if available. Use mipi_dsi_msleep()<br>
> and mipi_dsi_usleep_range() instead of msleep()<br>
> and usleep_range() respectively.<br>
> <br>
> Used Coccinelle to find the multiple occurences.<br>
> SmPl patch:<br>
> @rule@<br>
> identifier dsi_var;<br>
> identifier r;<br>
> identifier func;<br>
> type t;<br>
> position p;<br>
> expression dsi_device;<br>
> expression list es;<br>
> @@<br>
> t func(...) {<br>
> ...<br>
> struct mipi_dsi_device *dsi_var = dsi_device;<br>
> +struct mipi_dsi_multi_context dsi_ctx = { .dsi = dsi_var };<br>
> <+...<br>
> (<br>
> -mipi_dsi_dcs_write_seq(dsi_var,es)@p;<br>
> +mipi_dsi_dcs_write_seq_multi(&dsi_ctx,es);<br>
> |<br>
> -mipi_dsi_generic_write_seq(dsi_var,es)@p;<br>
> +mipi_dsi_generic_write_seq_multi(&dsi_ctx,es);<br>
> |<br>
> -mipi_dsi_generic_write(dsi_var,es)@p;<br>
> +mipi_dsi_generic_write_multi(&dsi_ctx,es);<br>
> |<br>
> -r = mipi_dsi_dcs_nop(dsi_var)@p;<br>
> +mipi_dsi_dcs_nop_multi(&dsi_ctx);<br>
> |<br>
> ....rest of API<br>
> ..<br>
> )<br>
> -if(r < 0) {<br>
> -...<br>
> -}<br>
> ...+><br>
<br>
The point of sending a single patch was to review the coccinelle script,<br>
so you must put the entire script you used here.<br>
<br></blockquote><div><br></div><div>I was actually thinking of sending patches per driver this time around since Tejas also seems to be looking into similar parts....Thoughts?</div><div> </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> Cc: Maxime Ripard <<a href="mailto:maxime.ripard@bootlin.com" target="_blank">maxime.ripard@bootlin.com</a>><br>
<br>
That hasn't been my email address for 6 years :)<br>
<br></blockquote><div>My bad. Will change this.</div><div><br></div><div>Anusha </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Maxime<br>
</blockquote></div></div>