<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Sat, Feb 8, 2025 at 3:43 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 Thu, Feb 6, 2025 at 1:06 PM Anusha Srivatsa <<a href="mailto:asrivats@redhat.com" target="_blank">asrivats@redhat.com</a>> wrote:<br>
><br>
> Transition away from mipi_dsi_generic_write_seq() and<br>
> mipi_dsi_dcs_write_seq() to mipi_dsi_generic_write_seq_multi()<br>
> and mipi_dsi_dcs_write_seq_multi() respectively.<br>
><br>
> This is addressing one of the gpu todo items [1]<br>
><br>
> Used Coccinelle to make the code changes.<br>
<br>
As Dmitry mentioned [1], I think a real cleanup needs more thought<br>
than can be done in a Coccinelle script. Maybe you can make a script<br>
that's super fancy and does a perfect conversion, but I sorta doubt it<br>
in this case.<br>
<br>
A few other note:<br>
<br>
* Tejas Vipin has been slowly whittling down this TODO item. It would<br>
be good to CC him on any attempts to avoid duplicating work. He just<br>
submitted a patch [2] that duplicates one of the panels you ran your<br>
script on. It would also be good to look at the patches he has been<br>
posting to see some of the extra complexities.<br>
<br>
* I'm happy you CCed on the cover letter, but given that I've been<br>
helping with this TODO entry it would have been nice to have been CCed<br>
on all the patches.<br>
<br>
For now I'm not going to review the individual patches in this series.<br>
If there are any where you think your Coccinelle script really got<br>
everything adjusted perfectly then feel free to point them out and<br>
I'll review them. If you want to help with this effort, my own<br>
personal preference would be to at least start w/ one panel at a time<br>
since probably review feedback on all the conversions will be similar.<br>
<br>
Thanks!<br>
<br></blockquote><div><br></div><div>I thought I had replied to this  :( (face-palm) sincere apologies. Luckily I bumped into this today.  Thanks for the feedback. I agree with what you said. While the script catches the patterns , a little of manual conversion and proof checking is needed. I am taking this into account while I spin the next version. WIll CC everyone in individual patches who have taken their time to take a look at my clearly incomplete approach to this redesign. </div><div><br></div><div>Again, thanks Dough</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">
-Doug<br>
<br>
[1] <a href="https://lore.kernel.org/r/mz6usiheb2lx6wlk47z3btvf6t6kbo4ja4n6mli5hosrpcvdwp@wmvfvhaqhpkm" rel="noreferrer" target="_blank">https://lore.kernel.org/r/mz6usiheb2lx6wlk47z3btvf6t6kbo4ja4n6mli5hosrpcvdwp@wmvfvhaqhpkm</a><br>
[2] <a href="http://lore.kernel.org/r/20250208051541.176667-1-tejasvipin76@gmail.com" rel="noreferrer" target="_blank">http://lore.kernel.org/r/20250208051541.176667-1-tejasvipin76@gmail.com</a><br>
<br>
</blockquote></div></div>