<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Fri, Jun 6, 2025 at 7:03 AM Maxime Ripard <<a href="mailto:mripard@kernel.org">mripard@kernel.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, Jun 04, 2025 at 10:45:11PM -0500, Anusha Srivatsa wrote:<br>
> Put the panel reference back when driver is no<br>
> longer using it.<br>
> <br>
> Signed-off-by: Anusha Srivatsa <<a href="mailto:asrivats@redhat.com" target="_blank">asrivats@redhat.com</a>><br>
> ---<br>
>  drivers/gpu/drm/bridge/samsung-dsim.c | 1 +<br>
>  1 file changed, 1 insertion(+)<br>
> <br>
> diff --git a/drivers/gpu/drm/bridge/samsung-dsim.c b/drivers/gpu/drm/bridge/samsung-dsim.c<br>
> index 0014c497e3fe7d8349a119dbdda30d65d816cccf..3667855ff0d6d1b608c579573de657af7fd14388 100644<br>
> --- a/drivers/gpu/drm/bridge/samsung-dsim.c<br>
> +++ b/drivers/gpu/drm/bridge/samsung-dsim.c<br>
> @@ -1748,6 +1748,7 @@ static int samsung_dsim_host_attach(struct mipi_dsi_host *host,<br>
>                       dsi->out_bridge = ERR_PTR(-EINVAL);<br>
>       }<br>
>  <br>
> +     drm_panel_put(panel);<br>
>       of_node_put(remote);<br>
>  <br>
>       if (IS_ERR(dsi->out_bridge)) {<br>
<br>
Explaining in your commit log why you think it's a good idea to put it<br>
there would be really nice. In particular, it looks super odd to me that<br>
you would put the panel reference in attach. <br></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Also, your patch doesn't work, and you have a reference inbalance. You<br>
have one taken by the panel driver, put in remove. You have one in<br>
drm_panel_add, put in drm_panel_remove. Which reference do you put here?<br>
<br>
This applies to your other patches too.<br>
<br></blockquote><div>Yes, I should get the ref in of_drm_find_panel() to put it here. With that said, all callers </div><div>of of_drm_find_panel() should be converted instead of the small batch that this series addresses.</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">
Maxime<br>
</blockquote></div></div>