<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Mon, Mar 17, 2025 at 6:14 AM Neil Armstrong <<a href="mailto:neil.armstrong@linaro.org">neil.armstrong@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 16/03/2025 18:40, Anusha Srivatsa wrote:<br>
> <br>
> <br>
> On Fri, Mar 14, 2025 at 10:20 AM Doug Anderson <<a href="mailto:dianders@chromium.org" target="_blank">dianders@chromium.org</a> <mailto:<a href="mailto:dianders@chromium.org" target="_blank">dianders@chromium.org</a>>> wrote:<br>
> <br>
>     Hi,<br>
> <br>
>     On Thu, Mar 13, 2025 at 9:47 PM Anusha Srivatsa <<a href="mailto:asrivats@redhat.com" target="_blank">asrivats@redhat.com</a> <mailto:<a href="mailto:asrivats@redhat.com" target="_blank">asrivats@redhat.com</a>>> wrote:<br>
>      ><br>
>      > @@ -181,24 +162,15 @@ static int r63353_panel_prepare(struct drm_panel *panel)<br>
>      >  static int r63353_panel_deactivate(struct r63353_panel *rpanel)<br>
>      >  {<br>
>      >         struct mipi_dsi_device *dsi = rpanel->dsi;<br>
>      > -       struct device *dev = &dsi->dev;<br>
>      > -       int ret;<br>
>      > +       struct mipi_dsi_multi_context dsi_ctx = { .dsi = dsi };<br>
>      ><br>
>      > -       ret = mipi_dsi_dcs_set_display_off(dsi);<br>
>      > -       if (ret < 0) {<br>
>      > -               dev_err(dev, "Failed to set display OFF (%d)\n", ret);<br>
>      > -               return ret;<br>
>      > -       }<br>
>      > +       mipi_dsi_dcs_set_display_off_multi(&dsi_ctx);<br>
>      ><br>
>      > -       usleep_range(5000, 10000);<br>
>      > +       mipi_dsi_usleep_range(&dsi_ctx, 5000, 10000);<br>
>      ><br>
>      > -       ret = mipi_dsi_dcs_enter_sleep_mode(dsi);<br>
>      > -       if (ret < 0) {<br>
>      > -               dev_err(dev, "Failed to enter sleep mode (%d)\n", ret);<br>
>      > -               return ret;<br>
>      > -       }<br>
>      > +       mipi_dsi_dcs_enter_sleep_mode_multi(&dsi_ctx);<br>
>      ><br>
>      > -       return 0;<br>
>      > +       return dsi_ctx.accum_err;<br>
> <br>
>     nit: the one caller of r63353_panel_deactivate() doesn't actually look<br>
>     at the error code, so this could be a function that returns "void".<br>
>     That was true even before your patch, though. I wouldn't mind a<br>
>     followup patch that fixed this. ;-)<br>
> <br>
> <br>
> This is anyway not merged, Maybe better to fix right now instead of a follow up patch?<br>
<br>
If you can, yeah fix it now !<br></blockquote><div>On it.</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>
>     In any case, the patch looks reasonable to me now.<br>
> <br>
>     Reviewed-by: Douglas Anderson <<a href="mailto:dianders@chromium.org" target="_blank">dianders@chromium.org</a> <mailto:<a href="mailto:dianders@chromium.org" target="_blank">dianders@chromium.org</a>>><br>
> <br>
> <br>
> Thanks :)<br>
> <br>
> Anusha<br>
<br>
<br>
Thanks,<br>
Neil<br>
> <br>
> <br>
>     Happy for someone else to apply it if they want. If not, I'll snooze<br>
>     this for ~a week to give others a chance to comment and then plan to<br>
>     push to drm-misc-next. <br>
> <br>
> <br>
> <br>
>     -Doug<br>
> <br>
<br>
</blockquote></div></div>