* [PATCH] drm/bridge: sn65dsi86: defer if there is no dsi host
@ 2021-12-07 20:09 Rob Clark
2021-12-07 20:26 ` Doug Anderson
0 siblings, 1 reply; 3+ messages in thread
From: Rob Clark @ 2021-12-07 20:09 UTC (permalink / raw)
To: dri-devel
Cc: linux-arm-msm, Maxime Ripard, Douglas Anderson, Stephen Boyd,
Rob Clark, Andrzej Hajda, Neil Armstrong, Robert Foss,
Laurent Pinchart, Jonas Karlman, Jernej Skrabec, David Airlie,
Daniel Vetter, Sam Ravnborg, open list
From: Rob Clark <robdclark@chromium.org>
Otherwise we don't get another shot at it if the bridge probes before
the dsi host is registered. It seems like this is what *most* (but not
all) of the other bridges do.
It looks like this was missed in the conversion to attach dsi host at
probe time.
Fixes: c3b75d4734cb ("drm/bridge: sn65dsi86: Register and attach our DSI device at probe")
Signed-off-by: Rob Clark <robdclark@chromium.org>
---
drivers/gpu/drm/bridge/ti-sn65dsi86.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi86.c b/drivers/gpu/drm/bridge/ti-sn65dsi86.c
index 02b490671f8f..3684e12fbac3 100644
--- a/drivers/gpu/drm/bridge/ti-sn65dsi86.c
+++ b/drivers/gpu/drm/bridge/ti-sn65dsi86.c
@@ -716,7 +716,7 @@ static int ti_sn_attach_host(struct ti_sn65dsi86 *pdata)
host = of_find_mipi_dsi_host_by_node(pdata->host_node);
if (!host) {
DRM_ERROR("failed to find dsi host\n");
- return -ENODEV;
+ return -EPROBE_DEFER;
}
dsi = devm_mipi_dsi_device_register_full(dev, host, &info);
--
2.33.1
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] drm/bridge: sn65dsi86: defer if there is no dsi host
2021-12-07 20:09 [PATCH] drm/bridge: sn65dsi86: defer if there is no dsi host Rob Clark
@ 2021-12-07 20:26 ` Doug Anderson
2021-12-07 20:35 ` Jernej Škrabec
0 siblings, 1 reply; 3+ messages in thread
From: Doug Anderson @ 2021-12-07 20:26 UTC (permalink / raw)
To: Rob Clark
Cc: dri-devel, linux-arm-msm, Maxime Ripard, Stephen Boyd, Rob Clark,
Andrzej Hajda, Neil Armstrong, Robert Foss, Laurent Pinchart,
Jonas Karlman, Jernej Skrabec, David Airlie, Daniel Vetter,
Sam Ravnborg, open list
Hi,
On Tue, Dec 7, 2021 at 12:03 PM Rob Clark <robdclark@gmail.com> wrote:
>
> From: Rob Clark <robdclark@chromium.org>
>
> Otherwise we don't get another shot at it if the bridge probes before
> the dsi host is registered. It seems like this is what *most* (but not
> all) of the other bridges do.
>
> It looks like this was missed in the conversion to attach dsi host at
> probe time.
>
> Fixes: c3b75d4734cb ("drm/bridge: sn65dsi86: Register and attach our DSI device at probe")
> Signed-off-by: Rob Clark <robdclark@chromium.org>
> ---
> drivers/gpu/drm/bridge/ti-sn65dsi86.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi86.c b/drivers/gpu/drm/bridge/ti-sn65dsi86.c
> index 02b490671f8f..3684e12fbac3 100644
> --- a/drivers/gpu/drm/bridge/ti-sn65dsi86.c
> +++ b/drivers/gpu/drm/bridge/ti-sn65dsi86.c
> @@ -716,7 +716,7 @@ static int ti_sn_attach_host(struct ti_sn65dsi86 *pdata)
> host = of_find_mipi_dsi_host_by_node(pdata->host_node);
> if (!host) {
> DRM_ERROR("failed to find dsi host\n");
> - return -ENODEV;
> + return -EPROBE_DEFER;
Can you delete the DRM_ERROR line?
Thanks!
-Doug
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] drm/bridge: sn65dsi86: defer if there is no dsi host
2021-12-07 20:26 ` Doug Anderson
@ 2021-12-07 20:35 ` Jernej Škrabec
0 siblings, 0 replies; 3+ messages in thread
From: Jernej Škrabec @ 2021-12-07 20:35 UTC (permalink / raw)
To: Rob Clark, Doug Anderson
Cc: dri-devel, linux-arm-msm, Maxime Ripard, Stephen Boyd, Rob Clark,
Andrzej Hajda, Neil Armstrong, Robert Foss, Laurent Pinchart,
Jonas Karlman, David Airlie, Daniel Vetter, Sam Ravnborg,
open list
Dne torek, 07. december 2021 ob 21:26:50 CET je Doug Anderson napisal(a):
> Hi,
>
> On Tue, Dec 7, 2021 at 12:03 PM Rob Clark <robdclark@gmail.com> wrote:
> > From: Rob Clark <robdclark@chromium.org>
> >
> > Otherwise we don't get another shot at it if the bridge probes before
> > the dsi host is registered. It seems like this is what *most* (but not
> > all) of the other bridges do.
> >
> > It looks like this was missed in the conversion to attach dsi host at
> > probe time.
> >
> > Fixes: c3b75d4734cb ("drm/bridge: sn65dsi86: Register and attach our DSI
> > device at probe") Signed-off-by: Rob Clark <robdclark@chromium.org>
> > ---
> >
> > drivers/gpu/drm/bridge/ti-sn65dsi86.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi86.c
> > b/drivers/gpu/drm/bridge/ti-sn65dsi86.c index 02b490671f8f..3684e12fbac3
> > 100644
> > --- a/drivers/gpu/drm/bridge/ti-sn65dsi86.c
> > +++ b/drivers/gpu/drm/bridge/ti-sn65dsi86.c
> > @@ -716,7 +716,7 @@ static int ti_sn_attach_host(struct ti_sn65dsi86
> > *pdata)>
> > host = of_find_mipi_dsi_host_by_node(pdata->host_node);
> > if (!host) {
> >
> > DRM_ERROR("failed to find dsi host\n");
> >
> > - return -ENODEV;
> > + return -EPROBE_DEFER;
>
> Can you delete the DRM_ERROR line?
or use dev_err_probe()
Best regards,
Jernej
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-12-07 20:35 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-12-07 20:09 [PATCH] drm/bridge: sn65dsi86: defer if there is no dsi host Rob Clark
2021-12-07 20:26 ` Doug Anderson
2021-12-07 20:35 ` Jernej Škrabec
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox