<div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Tue, 18 Mar 2025 at 12:00 PM, Dmitry Baryshkov <<a href="mailto:dmitry.baryshkov@oss.qualcomm.com">dmitry.baryshkov@oss.qualcomm.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, 18 Mar 2025 at 18:55, Anusha Srivatsa <<a href="mailto:asrivats@redhat.com" target="_blank">asrivats@redhat.com</a>> wrote:<br>
><br>
> On Sun, Mar 16, 2025 at 7:32 PM Dmitry Baryshkov <<a href="mailto:dmitry.baryshkov@oss.qualcomm.com" target="_blank">dmitry.baryshkov@oss.qualcomm.com</a>> wrote:<br>
>><br>
>> On Sun, Mar 16, 2025 at 02:58:22PM -0400, Anusha Srivatsa wrote:<br>
<br>
>> > ---<br>
>> >  drivers/gpu/drm/panel/panel-sharp-ls043t1le01.c | 36 ++++++++++---------------<br>
>> >  1 file changed, 14 insertions(+), 22 deletions(-)<br>
>> ><br>
>> > diff --git a/drivers/gpu/drm/panel/panel-sharp-ls043t1le01.c b/drivers/gpu/drm/panel/panel-sharp-ls043t1le01.c<br>
>> > index 729cbb0d8403ff7c0c4b9d21774909cc298904a2..9803eaf18e95d70742ece88e82932751476fdda8 100644<br>
>> > --- a/drivers/gpu/drm/panel/panel-sharp-ls043t1le01.c<br>
>> > +++ b/drivers/gpu/drm/panel/panel-sharp-ls043t1le01.c<br>
>> > @@ -36,15 +36,14 @@ static inline struct sharp_nt_panel *to_sharp_nt_panel(struct drm_panel *panel)<br>
>> >  static int sharp_nt_panel_init(struct sharp_nt_panel *sharp_nt)<br>
>> >  {<br>
>> >       struct mipi_dsi_device *dsi = sharp_nt->dsi;<br>
>> > +     struct mipi_dsi_multi_context dsi_ctx = { .dsi = dsi };<br>
>> >       int ret;<br>
>> ><br>
>> >       dsi->mode_flags |= MIPI_DSI_MODE_LPM;<br>
>> ><br>
>> > -     ret = mipi_dsi_dcs_exit_sleep_mode(dsi);<br>
>> > -     if (ret < 0)<br>
>> > -             return ret;<br>
>> > +     mipi_dsi_dcs_exit_sleep_mode_multi(&dsi_ctx);<br>
>> ><br>
>> > -     msleep(120);<br>
>> > +     mipi_dsi_msleep(&dsi_ctx, 120);<br>
>> ><br>
>> >       /* Novatek two-lane operation */<br>
>> >       ret = mipi_dsi_dcs_write(dsi, 0xae, (u8[]){ 0x03 }, 1);<br>
>><br>
>> Anusha. Please. Start reviewing your patches before you send them. This<br>
>> is a MIPI DSI call. And it wasn't switched to _multi / context. You<br>
>> should have cought this. I don't know why you didn't. Please review and<br>
>> check your code before sending it upstream.<br>
><br>
><br>
> Actually , mipi_dsi_dcs_write() doesnt have a _multi() variant. There are _multi() variants for other writes: mipi_dsi_dcs_write_buffer() has mipi_dsi_dcs_write_buffer_multi(), mipi_dsi_generic_write() has mipi_dsi_generic_write_multi() but not the one used above.<br>
<br>
That doesn't mean that it can be left unconverted. You can use<br>
mipi_dsi_dcs_write_buffer_multi().<br>
</blockquote><div dir="auto"><br></div><div dir="auto">Ah got it . Makes sense.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
P.S. Please wrap your replies ont some useful boundary (72-75 chars is<br>
the recommended value).<br>
</blockquote><div dir="auto"><br></div><div dir="auto">Sure.</div><div dir="auto"><br></div><div dir="auto">Anusha </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
-- <br>
With best wishes<br>
Dmitry<br>
<br>
</blockquote></div></div>