<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Fri, Mar 14, 2025 at 10:20 AM 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, Mar 13, 2025 at 9:47 PM Anusha Srivatsa <<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></blockquote><div><br></div><div>This is anyway not merged, Maybe better to fix right now instead of a follow up patch? </div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
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>><br></blockquote><div><br></div><div>Thanks :)</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">
<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. </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
<br>
-Doug<br>
<br>
</blockquote></div></div>