Commit dc659a4e authored by Noralf Trønnes's avatar Noralf Trønnes

drm/probe-helper: Check epoch counter in output_poll_execute()

drm_helper_hpd_irq_event() checks the epoch counter to determine
connector status change. This was introduced in
commit 5186421c ("drm: Introduce epoch counter to drm_connector").
Do the same for output_poll_execute() so it can detect other changes
beside connection status value changes.

v2:
- Add Fixes tag (Daniel)

Fixes: 5186421c ("drm: Introduce epoch counter to drm_connector")
Reviewed-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: default avatarNoralf Trønnes <noralf@tronnes.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20210313112545.37527-3-noralf@tronnes.org
parent 757e2671
...@@ -624,6 +624,7 @@ static void output_poll_execute(struct work_struct *work) ...@@ -624,6 +624,7 @@ static void output_poll_execute(struct work_struct *work)
struct drm_connector_list_iter conn_iter; struct drm_connector_list_iter conn_iter;
enum drm_connector_status old_status; enum drm_connector_status old_status;
bool repoll = false, changed; bool repoll = false, changed;
u64 old_epoch_counter;
if (!dev->mode_config.poll_enabled) if (!dev->mode_config.poll_enabled)
return; return;
...@@ -660,8 +661,9 @@ static void output_poll_execute(struct work_struct *work) ...@@ -660,8 +661,9 @@ static void output_poll_execute(struct work_struct *work)
repoll = true; repoll = true;
old_epoch_counter = connector->epoch_counter;
connector->status = drm_helper_probe_detect(connector, NULL, false); connector->status = drm_helper_probe_detect(connector, NULL, false);
if (old_status != connector->status) { if (old_epoch_counter != connector->epoch_counter) {
const char *old, *new; const char *old, *new;
/* /*
...@@ -690,6 +692,9 @@ static void output_poll_execute(struct work_struct *work) ...@@ -690,6 +692,9 @@ static void output_poll_execute(struct work_struct *work)
connector->base.id, connector->base.id,
connector->name, connector->name,
old, new); old, new);
DRM_DEBUG_KMS("[CONNECTOR:%d:%s] epoch counter %llu -> %llu\n",
connector->base.id, connector->name,
old_epoch_counter, connector->epoch_counter);
changed = true; changed = true;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment