diff options
author | Jani Nikula <jani.nikula@intel.com> | 2014-06-03 07:56:21 -0400 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2014-06-03 23:18:40 -0400 |
commit | 8e329a039b510b9f701ad2d536ce6aa0a1544f10 (patch) | |
tree | 8d32f3687848f50e859de1690bb27ad818e81908 | |
parent | 259338205c428454515a09654be69a9f6c983513 (diff) |
drm/i915: replace drm_get_encoder_name() with direct name field use
Generated using semantic patches:
@@
expression E;
@@
- drm_get_encoder_name(&E)
+ E.name
@@
expression E;
@@
- drm_get_encoder_name(E)
+ E->name
v2: Turn drm_get_encoder_name(&E) into E.name instead of &(E)->name.
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/i915/i915_debugfs.c | 2 | ||||
-rw-r--r-- | drivers/gpu/drm/i915/intel_display.c | 14 | ||||
-rw-r--r-- | drivers/gpu/drm/i915/intel_dp.c | 2 |
3 files changed, 9 insertions, 9 deletions
diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c index 8f49a2964ed5..36f4c15f538b 100644 --- a/drivers/gpu/drm/i915/i915_debugfs.c +++ b/drivers/gpu/drm/i915/i915_debugfs.c | |||
@@ -2197,7 +2197,7 @@ static void intel_encoder_info(struct seq_file *m, | |||
2197 | 2197 | ||
2198 | encoder = &intel_encoder->base; | 2198 | encoder = &intel_encoder->base; |
2199 | seq_printf(m, "\tencoder %d: type: %s, connectors:\n", | 2199 | seq_printf(m, "\tencoder %d: type: %s, connectors:\n", |
2200 | encoder->base.id, drm_get_encoder_name(encoder)); | 2200 | encoder->base.id, encoder->name); |
2201 | for_each_connector_on_encoder(dev, encoder, intel_connector) { | 2201 | for_each_connector_on_encoder(dev, encoder, intel_connector) { |
2202 | struct drm_connector *connector = &intel_connector->base; | 2202 | struct drm_connector *connector = &intel_connector->base; |
2203 | seq_printf(m, "\t\tconnector %d: type: %s, status: %s", | 2203 | seq_printf(m, "\t\tconnector %d: type: %s, status: %s", |
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index e3232ce2b5fa..e80b2917e3cb 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c | |||
@@ -7855,7 +7855,7 @@ void intel_write_eld(struct drm_encoder *encoder, | |||
7855 | connector->base.id, | 7855 | connector->base.id, |
7856 | connector->name, | 7856 | connector->name, |
7857 | connector->encoder->base.id, | 7857 | connector->encoder->base.id, |
7858 | drm_get_encoder_name(connector->encoder)); | 7858 | connector->encoder->name); |
7859 | 7859 | ||
7860 | connector->eld[6] = drm_av_sync_delay(connector, mode) / 2; | 7860 | connector->eld[6] = drm_av_sync_delay(connector, mode) / 2; |
7861 | 7861 | ||
@@ -8321,7 +8321,7 @@ bool intel_get_load_detect_pipe(struct drm_connector *connector, | |||
8321 | 8321 | ||
8322 | DRM_DEBUG_KMS("[CONNECTOR:%d:%s], [ENCODER:%d:%s]\n", | 8322 | DRM_DEBUG_KMS("[CONNECTOR:%d:%s], [ENCODER:%d:%s]\n", |
8323 | connector->base.id, connector->name, | 8323 | connector->base.id, connector->name, |
8324 | encoder->base.id, drm_get_encoder_name(encoder)); | 8324 | encoder->base.id, encoder->name); |
8325 | 8325 | ||
8326 | /* | 8326 | /* |
8327 | * Algorithm gets a little messy: | 8327 | * Algorithm gets a little messy: |
@@ -8433,7 +8433,7 @@ void intel_release_load_detect_pipe(struct drm_connector *connector, | |||
8433 | 8433 | ||
8434 | DRM_DEBUG_KMS("[CONNECTOR:%d:%s], [ENCODER:%d:%s]\n", | 8434 | DRM_DEBUG_KMS("[CONNECTOR:%d:%s], [ENCODER:%d:%s]\n", |
8435 | connector->base.id, connector->name, | 8435 | connector->base.id, connector->name, |
8436 | encoder->base.id, drm_get_encoder_name(encoder)); | 8436 | encoder->base.id, encoder->name); |
8437 | 8437 | ||
8438 | if (old->load_detect_temp) { | 8438 | if (old->load_detect_temp) { |
8439 | to_intel_connector(connector)->new_encoder = NULL; | 8439 | to_intel_connector(connector)->new_encoder = NULL; |
@@ -10050,7 +10050,7 @@ check_encoder_state(struct drm_device *dev) | |||
10050 | 10050 | ||
10051 | DRM_DEBUG_KMS("[ENCODER:%d:%s]\n", | 10051 | DRM_DEBUG_KMS("[ENCODER:%d:%s]\n", |
10052 | encoder->base.base.id, | 10052 | encoder->base.base.id, |
10053 | drm_get_encoder_name(&encoder->base)); | 10053 | encoder->base.name); |
10054 | 10054 | ||
10055 | WARN(&encoder->new_crtc->base != encoder->base.crtc, | 10055 | WARN(&encoder->new_crtc->base != encoder->base.crtc, |
10056 | "encoder's stage crtc doesn't match current crtc\n"); | 10056 | "encoder's stage crtc doesn't match current crtc\n"); |
@@ -11960,7 +11960,7 @@ static void intel_sanitize_encoder(struct intel_encoder *encoder) | |||
11960 | if (encoder->connectors_active && !has_active_crtc) { | 11960 | if (encoder->connectors_active && !has_active_crtc) { |
11961 | DRM_DEBUG_KMS("[ENCODER:%d:%s] has active connectors but no active pipe!\n", | 11961 | DRM_DEBUG_KMS("[ENCODER:%d:%s] has active connectors but no active pipe!\n", |
11962 | encoder->base.base.id, | 11962 | encoder->base.base.id, |
11963 | drm_get_encoder_name(&encoder->base)); | 11963 | encoder->base.name); |
11964 | 11964 | ||
11965 | /* Connector is active, but has no active pipe. This is | 11965 | /* Connector is active, but has no active pipe. This is |
11966 | * fallout from our resume register restoring. Disable | 11966 | * fallout from our resume register restoring. Disable |
@@ -11968,7 +11968,7 @@ static void intel_sanitize_encoder(struct intel_encoder *encoder) | |||
11968 | if (encoder->base.crtc) { | 11968 | if (encoder->base.crtc) { |
11969 | DRM_DEBUG_KMS("[ENCODER:%d:%s] manually disabled\n", | 11969 | DRM_DEBUG_KMS("[ENCODER:%d:%s] manually disabled\n", |
11970 | encoder->base.base.id, | 11970 | encoder->base.base.id, |
11971 | drm_get_encoder_name(&encoder->base)); | 11971 | encoder->base.name); |
11972 | encoder->disable(encoder); | 11972 | encoder->disable(encoder); |
11973 | } | 11973 | } |
11974 | 11974 | ||
@@ -12086,7 +12086,7 @@ static void intel_modeset_readout_hw_state(struct drm_device *dev) | |||
12086 | encoder->connectors_active = false; | 12086 | encoder->connectors_active = false; |
12087 | DRM_DEBUG_KMS("[ENCODER:%d:%s] hw state readout: %s, pipe %c\n", | 12087 | DRM_DEBUG_KMS("[ENCODER:%d:%s] hw state readout: %s, pipe %c\n", |
12088 | encoder->base.base.id, | 12088 | encoder->base.base.id, |
12089 | drm_get_encoder_name(&encoder->base), | 12089 | encoder->base.name, |
12090 | encoder->base.crtc ? "enabled" : "disabled", | 12090 | encoder->base.crtc ? "enabled" : "disabled", |
12091 | pipe_name(pipe)); | 12091 | pipe_name(pipe)); |
12092 | } | 12092 | } |
diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c index 8ef9034b68f3..d94f0271e422 100644 --- a/drivers/gpu/drm/i915/intel_dp.c +++ b/drivers/gpu/drm/i915/intel_dp.c | |||
@@ -3267,7 +3267,7 @@ intel_dp_check_link_status(struct intel_dp *intel_dp) | |||
3267 | 3267 | ||
3268 | if (!drm_dp_channel_eq_ok(link_status, intel_dp->lane_count)) { | 3268 | if (!drm_dp_channel_eq_ok(link_status, intel_dp->lane_count)) { |
3269 | DRM_DEBUG_KMS("%s: channel EQ not ok, retraining\n", | 3269 | DRM_DEBUG_KMS("%s: channel EQ not ok, retraining\n", |
3270 | drm_get_encoder_name(&intel_encoder->base)); | 3270 | intel_encoder->base.name); |
3271 | intel_dp_start_link_train(intel_dp); | 3271 | intel_dp_start_link_train(intel_dp); |
3272 | intel_dp_complete_link_train(intel_dp); | 3272 | intel_dp_complete_link_train(intel_dp); |
3273 | intel_dp_stop_link_train(intel_dp); | 3273 | intel_dp_stop_link_train(intel_dp); |