<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 11:29 AM Dmitry Baryshkov <<a href="mailto:dmitry.baryshkov@linaro.org">dmitry.baryshkov@linaro.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">On Thu, 6 Mar 2025 at 17:10, Anusha Srivatsa <<a href="mailto:asrivats@redhat.com" target="_blank">asrivats@redhat.com</a>> wrote:<br>
><br>
><br>
><br>
> On Thu, Mar 6, 2025 at 4:31 AM Maxime Ripard <<a href="mailto:mripard@kernel.org" target="_blank">mripard@kernel.org</a>> wrote:<br>
>><br>
>> 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>
><br>
> I was actually thinking of sending patches per driver this time around since Tejas also seems to be looking into similar parts....Thoughts?<br>
<br>
Have you discussed it with Tejas? What is his next target?<br>
<br></blockquote><div>I was hoping he will have some feedback on this patch and we could take it from there..... </div><div>It *should* be okay for me to send all changes in a single series...</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">
>> > 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>
> My bad. Will change this.<br>
<br>
<br>
<br>
-- <br>
With best wishes<br>
Dmitry<br>
<br>
</blockquote></div></div>