diff options
| author | Khaled Almahallawy <khaled.almahallawy@intel.com> | 2021-04-01 11:21:41 -0700 |
|---|---|---|
| committer | Petri Latvala <petri.latvala@intel.com> | 2021-04-06 10:28:15 +0300 |
| commit | 67c297bb3ed3aacac406eca454de41c4d7848d48 (patch) | |
| tree | a4d667efb6945aa20a3e662e78fc7d8d53f98d4b /tools/intel_dp_compliance.c | |
| parent | 84e6a7e19ccc7fafc46f372e756cad9d4aa093f7 (diff) | |
Revert "Add eDP support in Link Layer Compliance Tool"
The eDP change didn’t take into account choosing the right CRTC when compliance is requested on external DP.
This causes Hotplug Uevent sent from the kernel (intel_dp_short_pulse) to always trigger LT on eDP.
Without reverting this change the only way to run compliance on external display is to disconnect the eDP cable which is pain for ODMs.
So reverting this change until a full solution that handles multiple connected displays in place.
This reverts commit 391929b7402fd0417dcb9026c202241413653942.
Signed-off-by: Khaled Almahallawy <khaled.almahallawy@intel.com>
Acked-by: Petri Latvala <petri.latvala@intel.com>
Diffstat (limited to 'tools/intel_dp_compliance.c')
| -rw-r--r-- | tools/intel_dp_compliance.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/tools/intel_dp_compliance.c b/tools/intel_dp_compliance.c index 217a1247..fb946056 100644 --- a/tools/intel_dp_compliance.c +++ b/tools/intel_dp_compliance.c @@ -709,8 +709,7 @@ int update_display(int mode, bool is_compliance_test) conn = &connectors[cnt]; conn->id = resources->connectors[cnt]; c = drmModeGetConnector(drm_fd, conn->id); - if ((c->connector_type == DRM_MODE_CONNECTOR_DisplayPort || - c->connector_type == DRM_MODE_CONNECTOR_eDP) && + if (c->connector_type == DRM_MODE_CONNECTOR_DisplayPort && c->connection == DRM_MODE_CONNECTED) { test_connector_id = c->connector_id; conn->connector = c; |