Hi Laurent, On Mon, Feb 19, 2018 at 4:50 PM, Laurent Pinchart <[email protected]> wrote: > Hi Fabio, > > Thank you for the patch. > > On Friday, 16 February 2018 22:16:10 EET Fabio Estevam wrote: >> From: Fabio Estevam <[email protected]> >> >> The cable_plugin member never receives an assignment, so it is always >> false, which causes hdmi_enable_overflow_interrupts() to never >> be called as per the logic below: >> >> if (hdmi->cable_plugin && hdmi->sink_is_hdmi) >> hdmi_enable_overflow_interrupts(hdmi); >> >> This has been the case since the driver was originally introduced >> in commit 9aaf880ed4ee ("imx-drm: Add mx6 hdmi transmitter support"). >> >> Remove the cable_plugin element and the hdmi_enable_overflow_interrupts() >> function that is never called. >> >> Signed-off-by: Fabio Estevam <[email protected]> > > Tested-by: Laurent Pinchart <[email protected]> # On R-Car H3 > Reviewed-by: Laurent Pinchart <[email protected]>
Are you able to apply this patch or should Archit Taneja handle it? Thanks _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
