aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDaniel Vetter <daniel.vetter@ffwll.ch>2016-12-13 18:08:04 -0500
committerDaniel Vetter <daniel.vetter@ffwll.ch>2016-12-18 08:18:12 -0500
commit2babdc850cc4f58753b7fb5332a57b513e3313cb (patch)
treead3764a4ba1b43f2cc60d01b23f3b4dc76391eb5
parent5bc9cb4dfbe8fcae02f8421e3102c78681f53b8c (diff)
drm/radeon|amdgpu: Remove redundant num_connectors check
The list walk will shortcircuit anyway. Cc: Alex Deucher <alexdeucher@gmail.com> Reviewed-by: Alex Deucher <alexander.deucher@amd.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com> Link: http://patchwork.freedesktop.org/patch/msgid/20161213230814.19598-4-daniel.vetter@ffwll.ch
-rw-r--r--drivers/gpu/drm/amd/amdgpu/amdgpu_irq.c6
-rw-r--r--drivers/gpu/drm/radeon/radeon_irq_kms.c12
2 files changed, 6 insertions, 12 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_irq.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_irq.c
index fb902932f571..e63ece049b05 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_irq.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_irq.c
@@ -61,10 +61,8 @@ static void amdgpu_hotplug_work_func(struct work_struct *work)
61 struct drm_connector *connector; 61 struct drm_connector *connector;
62 62
63 mutex_lock(&mode_config->mutex); 63 mutex_lock(&mode_config->mutex);
64 if (mode_config->num_connector) { 64 list_for_each_entry(connector, &mode_config->connector_list, head)
65 list_for_each_entry(connector, &mode_config->connector_list, head) 65 amdgpu_connector_hotplug(connector);
66 amdgpu_connector_hotplug(connector);
67 }
68 mutex_unlock(&mode_config->mutex); 66 mutex_unlock(&mode_config->mutex);
69 /* Just fire off a uevent and let userspace tell us what to do */ 67 /* Just fire off a uevent and let userspace tell us what to do */
70 drm_helper_hpd_irq_event(dev); 68 drm_helper_hpd_irq_event(dev);
diff --git a/drivers/gpu/drm/radeon/radeon_irq_kms.c b/drivers/gpu/drm/radeon/radeon_irq_kms.c
index c084cadcbf21..1b7528df7f7f 100644
--- a/drivers/gpu/drm/radeon/radeon_irq_kms.c
+++ b/drivers/gpu/drm/radeon/radeon_irq_kms.c
@@ -85,10 +85,8 @@ static void radeon_hotplug_work_func(struct work_struct *work)
85 return; 85 return;
86 86
87 mutex_lock(&mode_config->mutex); 87 mutex_lock(&mode_config->mutex);
88 if (mode_config->num_connector) { 88 list_for_each_entry(connector, &mode_config->connector_list, head)
89 list_for_each_entry(connector, &mode_config->connector_list, head) 89 radeon_connector_hotplug(connector);
90 radeon_connector_hotplug(connector);
91 }
92 mutex_unlock(&mode_config->mutex); 90 mutex_unlock(&mode_config->mutex);
93 /* Just fire off a uevent and let userspace tell us what to do */ 91 /* Just fire off a uevent and let userspace tell us what to do */
94 drm_helper_hpd_irq_event(dev); 92 drm_helper_hpd_irq_event(dev);
@@ -103,10 +101,8 @@ static void radeon_dp_work_func(struct work_struct *work)
103 struct drm_connector *connector; 101 struct drm_connector *connector;
104 102
105 /* this should take a mutex */ 103 /* this should take a mutex */
106 if (mode_config->num_connector) { 104 list_for_each_entry(connector, &mode_config->connector_list, head)
107 list_for_each_entry(connector, &mode_config->connector_list, head) 105 radeon_connector_hotplug(connector);
108 radeon_connector_hotplug(connector);
109 }
110} 106}
111/** 107/**
112 * radeon_driver_irq_preinstall_kms - drm irq preinstall callback 108 * radeon_driver_irq_preinstall_kms - drm irq preinstall callback