Hi Laurentiu, Thank you for the patch. On Thu, Jul 21, 2022 at 03:09:12PM +0300, Laurentiu Palcu wrote: > When DCSS + MIPI_DSI is used, and the last bridge in the chain supports > HPD, we can see a "Hot plug detection already enabled" warning stack > trace dump that's thrown when DCSS is initialized. > > The problem appeared when HPD was enabled by default in the > bridge_connector initialization, which made the > drm_bridge_connector_enable_hpd() call, in DCSS init path, redundant. > So, let's remove that call. > > Fixes: 09077bc311658 ("drm/bridge_connector: enable HPD by default if supported") > Signed-off-by: Laurentiu Palcu <laurentiu.palcu@xxxxxxxxxxx> Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > --- > drivers/gpu/drm/imx/dcss/dcss-kms.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/gpu/drm/imx/dcss/dcss-kms.c b/drivers/gpu/drm/imx/dcss/dcss-kms.c > index 9b84df34a6a12..8cf3352d88582 100644 > --- a/drivers/gpu/drm/imx/dcss/dcss-kms.c > +++ b/drivers/gpu/drm/imx/dcss/dcss-kms.c > @@ -142,8 +142,6 @@ struct dcss_kms_dev *dcss_kms_attach(struct dcss_dev *dcss) > > drm_kms_helper_poll_init(drm); > > - drm_bridge_connector_enable_hpd(kms->connector); > - > ret = drm_dev_register(drm, 0); > if (ret) > goto cleanup_crtc; -- Regards, Laurent Pinchart