Re: [PATCH v2 4/7] drm/msm/dp: replug event is converted into an unplug followed by an plug events

Quoting Kuogee Hsieh (2021-07-13 08:54:04)

Remove special handling of replug interrupt and instead treat replug event as a sequential unplug followed by a plugin event. This is needed to meet the requirements of DP Link Layer CTS test case 4.2.1.3.

Changes in V2: -- add fixes statement

Fixes: f21c8a276c2d ("drm/msm/dp: handle irq_hpd with sink_count = 0 correctly")

Signed-off-by: Kuogee Hsieh khsieh@codeaurora.org

drivers/gpu/drm/msm/dp/dp_display.c | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/drivers/gpu/drm/msm/dp/dp_display.c b/drivers/gpu/drm/msm/dp/dp_display.c index 78c5301..d089ada 100644 --- a/drivers/gpu/drm/msm/dp/dp_display.c +++ b/drivers/gpu/drm/msm/dp/dp_display.c @@ -1146,9 +1146,6 @@ static int hpd_event_thread(void *data) case EV_IRQ_HPD_INT: dp_irq_hpd_handle(dp_priv, todo->data); break;

•           case EV_HPD_REPLUG_INT:
  

Please remove the enum as well.

•                   /* do nothing */
  

•                   break;
            case EV_USER_NOTIFICATION:
                    dp_display_send_hpd_notification(dp_priv,
                                            todo->data);