<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Wed, Mar 5, 2025 at 4:29 PM 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 Wed, Mar 05, 2025 at 12:29:06PM -0500, Anusha Srivatsa wrote:<br>
> On Sat, Feb 8, 2025 at 3:43 PM Doug Anderson <<a href="mailto:dianders@chromium.org" target="_blank">dianders@chromium.org</a>> wrote:<br>
> <br>
> > 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>><br>
> > 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>
> ><br>
> I thought I had replied to this  :( (face-palm) sincere apologies. Luckily<br>
> I bumped into this today.  Thanks for the feedback. I agree with what you<br>
> said. While the script catches the patterns , a little of manual conversion<br>
> and proof checking is needed. I am taking this into account while I spin<br>
> the next version. WIll CC everyone in individual patches who have taken<br>
> their time to take a look at my clearly incomplete approach to this<br>
> redesign.<br>
<br>
- Please post them one by one rather than the whole series<br>
- As you are looking into the same set of items as Tejas has been<br>
  working on, please make sure to discuss the order, so that you don't<br>
  do the same job twice.<br>
<br></blockquote><div><br></div><div>Sounds good!</div><div><a href="https://lore.kernel.org/dri-devel/20250305-mipi-synaptic-1-v1-1-92017cd19ef6@redhat.com/T/#u">https://lore.kernel.org/dri-devel/20250305-mipi-synaptic-1-v1-1-92017cd19ef6@redhat.com/T/#u</a></div><div></div><div><br></div><div>Anusha <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> <br>
> Again, thanks Dough<br>
> <br>
> Anusha<br>
> <br>
> > -Doug<br>
> ><br>
> > [1]<br>
> > <a href="https://lore.kernel.org/r/mz6usiheb2lx6wlk47z3btvf6t6kbo4ja4n6mli5hosrpcvdwp@wmvfvhaqhpkm" rel="noreferrer" target="_blank">https://lore.kernel.org/r/mz6usiheb2lx6wlk47z3btvf6t6kbo4ja4n6mli5hosrpcvdwp@wmvfvhaqhpkm</a><br>
> > [2]<br>
> > <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>
> ><br>
<br>
-- <br>
With best wishes<br>
Dmitry<br>
<br>
</blockquote></div></div>