diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2016-12-13 18:08:04 -0500 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2016-12-18 08:18:12 -0500 |
commit | 2babdc850cc4f58753b7fb5332a57b513e3313cb (patch) | |
tree | ad3764a4ba1b43f2cc60d01b23f3b4dc76391eb5 | |
parent | 5bc9cb4dfbe8fcae02f8421e3102c78681f53b8c (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.c | 6 | ||||
-rw-r--r-- | drivers/gpu/drm/radeon/radeon_irq_kms.c | 12 |
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 |