Hi Tomi, I did some more testing, and I discovered a bug in this code, but I am not sure how to solve it. On 04/14/2017 12:25 PM, Hans Verkuil wrote: > From: Hans Verkuil <hans.verkuil@xxxxxxxxx> > > Hook up the HDMI CEC support in the hdmi4 driver. > > It add the CEC irq handler, the CEC (un)init calls and tells the CEC > implementation when the physical address changes. > > Signed-off-by: Hans Verkuil <hans.verkuil@xxxxxxxxx> > --- > drivers/gpu/drm/omapdrm/dss/Kconfig | 9 +++++++++ > drivers/gpu/drm/omapdrm/dss/Makefile | 1 + > drivers/gpu/drm/omapdrm/dss/hdmi4.c | 23 ++++++++++++++++++++++- > 3 files changed, 32 insertions(+), 1 deletion(-) > <snip> > diff --git a/drivers/gpu/drm/omapdrm/dss/hdmi4.c b/drivers/gpu/drm/omapdrm/dss/hdmi4.c > index e371b47ff6ff..ebe5b27cee6f 100644 > --- a/drivers/gpu/drm/omapdrm/dss/hdmi4.c > +++ b/drivers/gpu/drm/omapdrm/dss/hdmi4.c <snip> > @@ -392,6 +401,8 @@ static void hdmi_display_disable(struct omap_dss_device *dssdev) > > DSSDBG("Enter hdmi_display_disable\n"); > > + hdmi4_cec_set_phys_addr(&hdmi.core, CEC_PHYS_ADDR_INVALID); > + > mutex_lock(&hdmi.lock); > > spin_lock_irqsave(&hdmi.audio_playing_lock, flags); My assumption was that hdmi_display_disable() was called when the hotplug would go away. But I discovered that that isn't the case, or at least not when X is running. It seems that the actual HPD check is done in hdmic_detect() in omapdrm/displays/connector-hdmi.c. But there I have no access to hdmi.core (needed for the hdmi4_cec_set_phys_addr() call). Any idea how to solve this? I am not all that familiar with drm, let alone omapdrm, so if you can point me in the right direction, then that would be very helpful. Regards, Hans _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel