[PATCH] drm/bridge: dw-hdmi: Remove unused hdmi_enable_overflow_interrupts()

Fabio Estevam festevam at gmail.com
Wed Mar 14 15:25:46 UTC 2018


Hi Laurent,

On Mon, Feb 19, 2018 at 4:50 PM, Laurent Pinchart
<laurent.pinchart at ideasonboard.com> wrote:
> Hi Fabio,
>
> Thank you for the patch.
>
> On Friday, 16 February 2018 22:16:10 EET Fabio Estevam wrote:
>> From: Fabio Estevam <fabio.estevam at nxp.com>
>>
>> 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 <fabio.estevam at nxp.com>
>
> Tested-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com> # On R-Car H3
> Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

Are you able to apply this patch or should Archit Taneja handle it?

Thanks


More information about the dri-devel mailing list