diff options
author | Jani Nikula <jani.nikula@intel.com> | 2014-06-03 07:56:19 -0400 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2014-06-03 23:17:30 -0400 |
commit | 72082093aefa5294801de91acdc25f5ea6722000 (patch) | |
tree | e47480e81ab20b3b3470aaeed57e508f5acbb46a | |
parent | 8c6c361ac6490eb3b74593a85ef5ea7d86a8669f (diff) |
drm/radeon: replace drm_get_connector_name() with direct name field use
Generated using semantic patch:
@@
expression E;
@@
- drm_get_connector_name(E)
+ E->name
[airlied: regenerated]
Acked-by: Alex Deucher <alexander.deucher@amd.com>
Acked-by: David Herrmann <dh.herrmann@gmail.com>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
-rw-r--r-- | drivers/gpu/drm/radeon/evergreen_hdmi.c | 6 | ||||
-rw-r--r-- | drivers/gpu/drm/radeon/radeon_connectors.c | 25 | ||||
-rw-r--r-- | drivers/gpu/drm/radeon/radeon_display.c | 2 |
3 files changed, 19 insertions, 14 deletions
diff --git a/drivers/gpu/drm/radeon/evergreen_hdmi.c b/drivers/gpu/drm/radeon/evergreen_hdmi.c index b760ef52fdcd..1ec0e6e83f9f 100644 --- a/drivers/gpu/drm/radeon/evergreen_hdmi.c +++ b/drivers/gpu/drm/radeon/evergreen_hdmi.c | |||
@@ -342,19 +342,19 @@ void evergreen_hdmi_setmode(struct drm_encoder *encoder, struct drm_display_mode | |||
342 | case 16: | 342 | case 16: |
343 | default: | 343 | default: |
344 | DRM_DEBUG("%s: Disabling hdmi deep color for %d bpc.\n", | 344 | DRM_DEBUG("%s: Disabling hdmi deep color for %d bpc.\n", |
345 | drm_get_connector_name(connector), bpc); | 345 | connector->name, bpc); |
346 | break; | 346 | break; |
347 | case 10: | 347 | case 10: |
348 | val |= HDMI_DEEP_COLOR_ENABLE; | 348 | val |= HDMI_DEEP_COLOR_ENABLE; |
349 | val |= HDMI_DEEP_COLOR_DEPTH(HDMI_30BIT_DEEP_COLOR); | 349 | val |= HDMI_DEEP_COLOR_DEPTH(HDMI_30BIT_DEEP_COLOR); |
350 | DRM_DEBUG("%s: Enabling hdmi deep color 30 for 10 bpc.\n", | 350 | DRM_DEBUG("%s: Enabling hdmi deep color 30 for 10 bpc.\n", |
351 | drm_get_connector_name(connector)); | 351 | connector->name); |
352 | break; | 352 | break; |
353 | case 12: | 353 | case 12: |
354 | val |= HDMI_DEEP_COLOR_ENABLE; | 354 | val |= HDMI_DEEP_COLOR_ENABLE; |
355 | val |= HDMI_DEEP_COLOR_DEPTH(HDMI_36BIT_DEEP_COLOR); | 355 | val |= HDMI_DEEP_COLOR_DEPTH(HDMI_36BIT_DEEP_COLOR); |
356 | DRM_DEBUG("%s: Enabling hdmi deep color 36 for 12 bpc.\n", | 356 | DRM_DEBUG("%s: Enabling hdmi deep color 36 for 12 bpc.\n", |
357 | drm_get_connector_name(connector)); | 357 | connector->name); |
358 | break; | 358 | break; |
359 | } | 359 | } |
360 | 360 | ||
diff --git a/drivers/gpu/drm/radeon/radeon_connectors.c b/drivers/gpu/drm/radeon/radeon_connectors.c index 0dc61dde4c3e..31d4cf60bae3 100644 --- a/drivers/gpu/drm/radeon/radeon_connectors.c +++ b/drivers/gpu/drm/radeon/radeon_connectors.c | |||
@@ -150,7 +150,7 @@ int radeon_get_monitor_bpc(struct drm_connector *connector) | |||
150 | /* hdmi deep color only implemented on DCE4+ */ | 150 | /* hdmi deep color only implemented on DCE4+ */ |
151 | if ((bpc > 8) && !ASIC_IS_DCE4(rdev)) { | 151 | if ((bpc > 8) && !ASIC_IS_DCE4(rdev)) { |
152 | DRM_DEBUG("%s: HDMI deep color %d bpc unsupported. Using 8 bpc.\n", | 152 | DRM_DEBUG("%s: HDMI deep color %d bpc unsupported. Using 8 bpc.\n", |
153 | drm_get_connector_name(connector), bpc); | 153 | connector->name, bpc); |
154 | bpc = 8; | 154 | bpc = 8; |
155 | } | 155 | } |
156 | 156 | ||
@@ -162,13 +162,13 @@ int radeon_get_monitor_bpc(struct drm_connector *connector) | |||
162 | */ | 162 | */ |
163 | if (bpc > 12) { | 163 | if (bpc > 12) { |
164 | DRM_DEBUG("%s: HDMI deep color %d bpc unsupported. Using 12 bpc.\n", | 164 | DRM_DEBUG("%s: HDMI deep color %d bpc unsupported. Using 12 bpc.\n", |
165 | drm_get_connector_name(connector), bpc); | 165 | connector->name, bpc); |
166 | bpc = 12; | 166 | bpc = 12; |
167 | } | 167 | } |
168 | } | 168 | } |
169 | 169 | ||
170 | DRM_DEBUG("%s: Display bpc=%d, returned bpc=%d\n", | 170 | DRM_DEBUG("%s: Display bpc=%d, returned bpc=%d\n", |
171 | drm_get_connector_name(connector), connector->display_info.bpc, bpc); | 171 | connector->name, connector->display_info.bpc, bpc); |
172 | 172 | ||
173 | return bpc; | 173 | return bpc; |
174 | } | 174 | } |
@@ -285,13 +285,17 @@ radeon_connector_analog_encoder_conflict_solve(struct drm_connector *connector, | |||
285 | continue; | 285 | continue; |
286 | 286 | ||
287 | if (priority == true) { | 287 | if (priority == true) { |
288 | DRM_DEBUG_KMS("1: conflicting encoders switching off %s\n", drm_get_connector_name(conflict)); | 288 | DRM_DEBUG_KMS("1: conflicting encoders switching off %s\n", |
289 | DRM_DEBUG_KMS("in favor of %s\n", drm_get_connector_name(connector)); | 289 | conflict->name); |
290 | DRM_DEBUG_KMS("in favor of %s\n", | ||
291 | connector->name); | ||
290 | conflict->status = connector_status_disconnected; | 292 | conflict->status = connector_status_disconnected; |
291 | radeon_connector_update_scratch_regs(conflict, connector_status_disconnected); | 293 | radeon_connector_update_scratch_regs(conflict, connector_status_disconnected); |
292 | } else { | 294 | } else { |
293 | DRM_DEBUG_KMS("2: conflicting encoders switching off %s\n", drm_get_connector_name(connector)); | 295 | DRM_DEBUG_KMS("2: conflicting encoders switching off %s\n", |
294 | DRM_DEBUG_KMS("in favor of %s\n", drm_get_connector_name(conflict)); | 296 | connector->name); |
297 | DRM_DEBUG_KMS("in favor of %s\n", | ||
298 | conflict->name); | ||
295 | current_status = connector_status_disconnected; | 299 | current_status = connector_status_disconnected; |
296 | } | 300 | } |
297 | break; | 301 | break; |
@@ -812,7 +816,7 @@ radeon_vga_detect(struct drm_connector *connector, bool force) | |||
812 | 816 | ||
813 | if (!radeon_connector->edid) { | 817 | if (!radeon_connector->edid) { |
814 | DRM_ERROR("%s: probed a monitor but no|invalid EDID\n", | 818 | DRM_ERROR("%s: probed a monitor but no|invalid EDID\n", |
815 | drm_get_connector_name(connector)); | 819 | connector->name); |
816 | ret = connector_status_connected; | 820 | ret = connector_status_connected; |
817 | } else { | 821 | } else { |
818 | radeon_connector->use_digital = !!(radeon_connector->edid->input & DRM_EDID_INPUT_DIGITAL); | 822 | radeon_connector->use_digital = !!(radeon_connector->edid->input & DRM_EDID_INPUT_DIGITAL); |
@@ -1035,12 +1039,13 @@ radeon_dvi_detect(struct drm_connector *connector, bool force) | |||
1035 | 1039 | ||
1036 | if (!radeon_connector->edid) { | 1040 | if (!radeon_connector->edid) { |
1037 | DRM_ERROR("%s: probed a monitor but no|invalid EDID\n", | 1041 | DRM_ERROR("%s: probed a monitor but no|invalid EDID\n", |
1038 | drm_get_connector_name(connector)); | 1042 | connector->name); |
1039 | /* rs690 seems to have a problem with connectors not existing and always | 1043 | /* rs690 seems to have a problem with connectors not existing and always |
1040 | * return a block of 0's. If we see this just stop polling on this output */ | 1044 | * return a block of 0's. If we see this just stop polling on this output */ |
1041 | if ((rdev->family == CHIP_RS690 || rdev->family == CHIP_RS740) && radeon_connector->base.null_edid_counter) { | 1045 | if ((rdev->family == CHIP_RS690 || rdev->family == CHIP_RS740) && radeon_connector->base.null_edid_counter) { |
1042 | ret = connector_status_disconnected; | 1046 | ret = connector_status_disconnected; |
1043 | DRM_ERROR("%s: detected RS690 floating bus bug, stopping ddc detect\n", drm_get_connector_name(connector)); | 1047 | DRM_ERROR("%s: detected RS690 floating bus bug, stopping ddc detect\n", |
1048 | connector->name); | ||
1044 | radeon_connector->ddc_bus = NULL; | 1049 | radeon_connector->ddc_bus = NULL; |
1045 | } else { | 1050 | } else { |
1046 | ret = connector_status_connected; | 1051 | ret = connector_status_connected; |
diff --git a/drivers/gpu/drm/radeon/radeon_display.c b/drivers/gpu/drm/radeon/radeon_display.c index d96d1ca74e70..e330e762c360 100644 --- a/drivers/gpu/drm/radeon/radeon_display.c +++ b/drivers/gpu/drm/radeon/radeon_display.c | |||
@@ -703,7 +703,7 @@ static void radeon_print_display_setup(struct drm_device *dev) | |||
703 | list_for_each_entry(connector, &dev->mode_config.connector_list, head) { | 703 | list_for_each_entry(connector, &dev->mode_config.connector_list, head) { |
704 | radeon_connector = to_radeon_connector(connector); | 704 | radeon_connector = to_radeon_connector(connector); |
705 | DRM_INFO("Connector %d:\n", i); | 705 | DRM_INFO("Connector %d:\n", i); |
706 | DRM_INFO(" %s\n", drm_get_connector_name(connector)); | 706 | DRM_INFO(" %s\n", connector->name); |
707 | if (radeon_connector->hpd.hpd != RADEON_HPD_NONE) | 707 | if (radeon_connector->hpd.hpd != RADEON_HPD_NONE) |
708 | DRM_INFO(" %s\n", hpd_names[radeon_connector->hpd.hpd]); | 708 | DRM_INFO(" %s\n", hpd_names[radeon_connector->hpd.hpd]); |
709 | if (radeon_connector->ddc_bus) { | 709 | if (radeon_connector->ddc_bus) { |