* [PATCH v2] drm/omap: dss: fix handling deferrals
@ 2026-11-11 21:14 Andreas Kemnade
2026-11-12 7:43 ` Luca Ceresoli
2026-11-12 10:25 ` Tomi Valkeinen
0 siblings, 2 replies; 4+ messages in thread
From: Andreas Kemnade @ 2026-11-11 21:14 UTC (permalink / raw)
To: Tomi Valkeinen, Maarten Lankhorst, Maxime Ripard,
Thomas Zimmermann, David Airlie, Simona Vetter, Luca Ceresoli
Cc: dri-devel, linux-omap, hns, linux-kernel, Andreas Kemnade
Forward the error given by of_drm_get_bridge_by_endpoint() if it is
EPROBE_DEFER to ensure output gets initialized later when available.
On GTA04/OMAP3 no display was found, but it appeared after doing
rmmod omapdrm ; modprobe omapdrm
Fixes: b7e9b93fc8ecf ("drm/omap: dss: don't create a panel_bridge")
Signed-off-by: Andreas Kemnade <andreas@kemnade.info>
---
Changes in v2:
- do not use the error goto
- Link to v1: https://patch.msgid.link/20261111-gta04-drm-regression-fix-v1-1-1de27d5ba46a@kemnade.info
---
drivers/gpu/drm/omapdrm/dss/output.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/gpu/drm/omapdrm/dss/output.c b/drivers/gpu/drm/omapdrm/dss/output.c
index 36d37deac8026..b5cef5093a883 100644
--- a/drivers/gpu/drm/omapdrm/dss/output.c
+++ b/drivers/gpu/drm/omapdrm/dss/output.c
@@ -24,6 +24,9 @@ int omapdss_device_init_output(struct omap_dss_device *out,
bridge = of_drm_get_bridge_by_endpoint(out->dev->of_node, out->of_port, 0);
if (IS_ERR(bridge)) {
+ if (PTR_ERR(bridge) == -EPROBE_DEFER)
+ return PTR_ERR(bridge);
+
dev_dbg(out->dev, "failed to find video sink\n");
return 0;
}
---
base-commit: 7475ea8b58616b33c04afb976cf0fbaad9dd63fb
change-id: 20261111-gta04-drm-regression-fix-f7ef51c66ed3
Best regards,
--
Andreas Kemnade <andreas@kemnade.info>
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH v2] drm/omap: dss: fix handling deferrals
2026-11-11 21:14 [PATCH v2] drm/omap: dss: fix handling deferrals Andreas Kemnade
@ 2026-11-12 7:43 ` Luca Ceresoli
2026-11-12 10:25 ` Tomi Valkeinen
1 sibling, 0 replies; 4+ messages in thread
From: Luca Ceresoli @ 2026-11-12 7:43 UTC (permalink / raw)
To: Andreas Kemnade, Tomi Valkeinen, Maarten Lankhorst,
Maxime Ripard, Thomas Zimmermann, David Airlie, Simona Vetter,
Luca Ceresoli
Cc: dri-devel, linux-omap, hns, linux-kernel
On Wed Nov 11, 2026 at 10:14 PM CET, Andreas Kemnade wrote:
> Forward the error given by of_drm_get_bridge_by_endpoint() if it is
> EPROBE_DEFER to ensure output gets initialized later when available.
>
> On GTA04/OMAP3 no display was found, but it appeared after doing
> rmmod omapdrm ; modprobe omapdrm
>
> Fixes: b7e9b93fc8ecf ("drm/omap: dss: don't create a panel_bridge")
> Signed-off-by: Andreas Kemnade <andreas@kemnade.info>
Reviewed-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
--
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH v2] drm/omap: dss: fix handling deferrals
2026-11-11 21:14 [PATCH v2] drm/omap: dss: fix handling deferrals Andreas Kemnade
2026-11-12 7:43 ` Luca Ceresoli
@ 2026-11-12 10:25 ` Tomi Valkeinen
2026-11-12 11:35 ` Andreas Kemnade
1 sibling, 1 reply; 4+ messages in thread
From: Tomi Valkeinen @ 2026-11-12 10:25 UTC (permalink / raw)
To: Andreas Kemnade
Cc: dri-devel, linux-omap, hns, linux-kernel, Maarten Lankhorst,
Maxime Ripard, Thomas Zimmermann, David Airlie, Simona Vetter,
Luca Ceresoli
Hi,
On 11/11/2026 23:14, Andreas Kemnade wrote:
> Forward the error given by of_drm_get_bridge_by_endpoint() if it is
> EPROBE_DEFER to ensure output gets initialized later when available.
>
> On GTA04/OMAP3 no display was found, but it appeared after doing
> rmmod omapdrm ; modprobe omapdrm
>
> Fixes: b7e9b93fc8ecf ("drm/omap: dss: don't create a panel_bridge")
> Signed-off-by: Andreas Kemnade <andreas@kemnade.info>
> ---
> Changes in v2:
> - do not use the error goto
> - Link to v1: https://patch.msgid.link/20261111-gta04-drm-regression-fix-v1-1-1de27d5ba46a@kemnade.info
> ---
> drivers/gpu/drm/omapdrm/dss/output.c | 3 +++
> 1 file changed, 3 insertions(+)
Thanks, I'll push to drm-misc-next.
Tomi
> diff --git a/drivers/gpu/drm/omapdrm/dss/output.c b/drivers/gpu/drm/omapdrm/dss/output.c
> index 36d37deac8026..b5cef5093a883 100644
> --- a/drivers/gpu/drm/omapdrm/dss/output.c
> +++ b/drivers/gpu/drm/omapdrm/dss/output.c
> @@ -24,6 +24,9 @@ int omapdss_device_init_output(struct omap_dss_device *out,
>
> bridge = of_drm_get_bridge_by_endpoint(out->dev->of_node, out->of_port, 0);
> if (IS_ERR(bridge)) {
> + if (PTR_ERR(bridge) == -EPROBE_DEFER)
> + return PTR_ERR(bridge);
> +
> dev_dbg(out->dev, "failed to find video sink\n");
> return 0;
> }
>
> ---
> base-commit: 7475ea8b58616b33c04afb976cf0fbaad9dd63fb
> change-id: 20261111-gta04-drm-regression-fix-f7ef51c66ed3
>
> Best regards,
> --
> Andreas Kemnade <andreas@kemnade.info>
>
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH v2] drm/omap: dss: fix handling deferrals
2026-11-12 10:25 ` Tomi Valkeinen
@ 2026-11-12 11:35 ` Andreas Kemnade
0 siblings, 0 replies; 4+ messages in thread
From: Andreas Kemnade @ 2026-11-12 11:35 UTC (permalink / raw)
To: Tomi Valkeinen
Cc: dri-devel, linux-omap, hns, linux-kernel, Maarten Lankhorst,
Maxime Ripard, Thomas Zimmermann, David Airlie, Simona Vetter,
Luca Ceresoli
On Thu, 12 Nov 2026 12:25:20 +0200
Tomi Valkeinen <tomi.valkeinen@ideasonboard.com> wrote:
> Hi,
>
> On 11/11/2026 23:14, Andreas Kemnade wrote:
> > Forward the error given by of_drm_get_bridge_by_endpoint() if it is
> > EPROBE_DEFER to ensure output gets initialized later when available.
> >
> > On GTA04/OMAP3 no display was found, but it appeared after doing
> > rmmod omapdrm ; modprobe omapdrm
> >
> > Fixes: b7e9b93fc8ecf ("drm/omap: dss: don't create a panel_bridge")
> > Signed-off-by: Andreas Kemnade <andreas@kemnade.info>
> > ---
> > Changes in v2:
> > - do not use the error goto
> > - Link to v1: https://patch.msgid.link/20261111-gta04-drm-regression-fix-v1-1-1de27d5ba46a@kemnade.info
> > ---
> > drivers/gpu/drm/omapdrm/dss/output.c | 3 +++
> > 1 file changed, 3 insertions(+)
>
> Thanks, I'll push to drm-misc-next.
>
Isn't it drm-misc-fixes? This issue is new in v7.4-rc1.
Regards,
Andreas
> Tomi
>
> > diff --git a/drivers/gpu/drm/omapdrm/dss/output.c b/drivers/gpu/drm/omapdrm/dss/output.c
> > index 36d37deac8026..b5cef5093a883 100644
> > --- a/drivers/gpu/drm/omapdrm/dss/output.c
> > +++ b/drivers/gpu/drm/omapdrm/dss/output.c
> > @@ -24,6 +24,9 @@ int omapdss_device_init_output(struct omap_dss_device *out,
> >
> > bridge = of_drm_get_bridge_by_endpoint(out->dev->of_node, out->of_port, 0);
> > if (IS_ERR(bridge)) {
> > + if (PTR_ERR(bridge) == -EPROBE_DEFER)
> > + return PTR_ERR(bridge);
> > +
> > dev_dbg(out->dev, "failed to find video sink\n");
> > return 0;
> > }
> >
> > ---
> > base-commit: 7475ea8b58616b33c04afb976cf0fbaad9dd63fb
> > change-id: 20261111-gta04-drm-regression-fix-f7ef51c66ed3
> >
> > Best regards,
> > --
> > Andreas Kemnade <andreas@kemnade.info>
> >
>
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2026-11-12 11:35 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-11-11 21:14 [PATCH v2] drm/omap: dss: fix handling deferrals Andreas Kemnade
2026-11-12 7:43 ` Luca Ceresoli
2026-11-12 10:25 ` Tomi Valkeinen
2026-11-12 11:35 ` Andreas Kemnade
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®