diff options
author | Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com> | 2015-01-15 07:55:22 -0500 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-01-27 03:50:48 -0500 |
commit | 2d112de7db9d2cb0bd43f67120acd6c028bb60e8 (patch) | |
tree | d14633773369d33598fda44a647a84e72a119aa8 /drivers/gpu/drm/i915/intel_hdmi.c | |
parent | 5cec258b4f185a5011165099a49757516f90de2b (diff) |
drm/i915: Embedded struct drm_crtc_state in intel_crtc_state
And get rid of the duplicate mode structures. This patch was generated
with the following semantic patch:
@@ @@
struct intel_crtc_state {
+struct drm_crtc_state base;
+
...
-struct drm_display_mode requested_mode;
-struct drm_display_mode adjusted_mode;
...
}
@@ struct intel_crtc_state *state; @@
-state->adjusted_mode
+state->base.adjusted_mode
@@ struct intel_crtc_state *state; @@
-state->requested_mode
+state->base.mode
@@ struct intel_crtc_state state; @@
-state.adjusted_mode
+state.base.adjusted_mode
@@ struct intel_crtc_state state; @@
-state.requested_mode
+state.base.mode
@@ struct drm_crtc *crtc; @@
-to_intel_crtc(crtc)->config.adjusted_mode
+to_intel_crtc(crtc)->config.base.adjusted_mode
@@ identifier member; expression E; @@
-PIPE_CONF_CHECK_FLAGS(adjusted_mode.member, E);
+PIPE_CONF_CHECK_FLAGS(base.adjusted_mode.member, E);
@@ identifier member; @@
-PIPE_CONF_CHECK_I(adjusted_mode.member);
+PIPE_CONF_CHECK_I(base.adjusted_mode.member);
@@ identifier member; @@
-PIPE_CONF_CHECK_CLOCK_FUZZY(adjusted_mode.member);
+PIPE_CONF_CHECK_CLOCK_FUZZY(base.adjusted_mode.member);
v2: Completely generate the patch with cocci. (Ander)
Signed-off-by: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
Reviewed-by: Matt Roper <matthew.d.roper@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'drivers/gpu/drm/i915/intel_hdmi.c')
-rw-r--r-- | drivers/gpu/drm/i915/intel_hdmi.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/drivers/gpu/drm/i915/intel_hdmi.c b/drivers/gpu/drm/i915/intel_hdmi.c index 657452b471ec..02ff3e2de88b 100644 --- a/drivers/gpu/drm/i915/intel_hdmi.c +++ b/drivers/gpu/drm/i915/intel_hdmi.c | |||
@@ -700,7 +700,7 @@ static void intel_hdmi_prepare(struct intel_encoder *encoder) | |||
700 | struct drm_i915_private *dev_priv = dev->dev_private; | 700 | struct drm_i915_private *dev_priv = dev->dev_private; |
701 | struct intel_crtc *crtc = to_intel_crtc(encoder->base.crtc); | 701 | struct intel_crtc *crtc = to_intel_crtc(encoder->base.crtc); |
702 | struct intel_hdmi *intel_hdmi = enc_to_intel_hdmi(&encoder->base); | 702 | struct intel_hdmi *intel_hdmi = enc_to_intel_hdmi(&encoder->base); |
703 | struct drm_display_mode *adjusted_mode = &crtc->config.adjusted_mode; | 703 | struct drm_display_mode *adjusted_mode = &crtc->config.base.adjusted_mode; |
704 | u32 hdmi_val; | 704 | u32 hdmi_val; |
705 | 705 | ||
706 | hdmi_val = SDVO_ENCODING_HDMI; | 706 | hdmi_val = SDVO_ENCODING_HDMI; |
@@ -792,7 +792,7 @@ static void intel_hdmi_get_config(struct intel_encoder *encoder, | |||
792 | tmp & HDMI_COLOR_RANGE_16_235) | 792 | tmp & HDMI_COLOR_RANGE_16_235) |
793 | pipe_config->limited_color_range = true; | 793 | pipe_config->limited_color_range = true; |
794 | 794 | ||
795 | pipe_config->adjusted_mode.flags |= flags; | 795 | pipe_config->base.adjusted_mode.flags |= flags; |
796 | 796 | ||
797 | if ((tmp & SDVO_COLOR_FORMAT_MASK) == HDMI_COLOR_FORMAT_12bpc) | 797 | if ((tmp & SDVO_COLOR_FORMAT_MASK) == HDMI_COLOR_FORMAT_12bpc) |
798 | dotclock = pipe_config->port_clock * 2 / 3; | 798 | dotclock = pipe_config->port_clock * 2 / 3; |
@@ -802,7 +802,7 @@ static void intel_hdmi_get_config(struct intel_encoder *encoder, | |||
802 | if (HAS_PCH_SPLIT(dev_priv->dev)) | 802 | if (HAS_PCH_SPLIT(dev_priv->dev)) |
803 | ironlake_check_encoder_dotclock(pipe_config, dotclock); | 803 | ironlake_check_encoder_dotclock(pipe_config, dotclock); |
804 | 804 | ||
805 | pipe_config->adjusted_mode.crtc_clock = dotclock; | 805 | pipe_config->base.adjusted_mode.crtc_clock = dotclock; |
806 | } | 806 | } |
807 | 807 | ||
808 | static void intel_enable_hdmi(struct intel_encoder *encoder) | 808 | static void intel_enable_hdmi(struct intel_encoder *encoder) |
@@ -979,8 +979,8 @@ bool intel_hdmi_compute_config(struct intel_encoder *encoder, | |||
979 | { | 979 | { |
980 | struct intel_hdmi *intel_hdmi = enc_to_intel_hdmi(&encoder->base); | 980 | struct intel_hdmi *intel_hdmi = enc_to_intel_hdmi(&encoder->base); |
981 | struct drm_device *dev = encoder->base.dev; | 981 | struct drm_device *dev = encoder->base.dev; |
982 | struct drm_display_mode *adjusted_mode = &pipe_config->adjusted_mode; | 982 | struct drm_display_mode *adjusted_mode = &pipe_config->base.adjusted_mode; |
983 | int clock_12bpc = pipe_config->adjusted_mode.crtc_clock * 3 / 2; | 983 | int clock_12bpc = pipe_config->base.adjusted_mode.crtc_clock * 3 / 2; |
984 | int portclock_limit = hdmi_portclock_limit(intel_hdmi, false); | 984 | int portclock_limit = hdmi_portclock_limit(intel_hdmi, false); |
985 | int desired_bpp; | 985 | int desired_bpp; |
986 | 986 | ||
@@ -1252,7 +1252,7 @@ static void intel_hdmi_pre_enable(struct intel_encoder *encoder) | |||
1252 | struct intel_hdmi *intel_hdmi = enc_to_intel_hdmi(&encoder->base); | 1252 | struct intel_hdmi *intel_hdmi = enc_to_intel_hdmi(&encoder->base); |
1253 | struct intel_crtc *intel_crtc = to_intel_crtc(encoder->base.crtc); | 1253 | struct intel_crtc *intel_crtc = to_intel_crtc(encoder->base.crtc); |
1254 | struct drm_display_mode *adjusted_mode = | 1254 | struct drm_display_mode *adjusted_mode = |
1255 | &intel_crtc->config.adjusted_mode; | 1255 | &intel_crtc->config.base.adjusted_mode; |
1256 | 1256 | ||
1257 | intel_hdmi_prepare(encoder); | 1257 | intel_hdmi_prepare(encoder); |
1258 | 1258 | ||
@@ -1270,7 +1270,7 @@ static void vlv_hdmi_pre_enable(struct intel_encoder *encoder) | |||
1270 | struct intel_crtc *intel_crtc = | 1270 | struct intel_crtc *intel_crtc = |
1271 | to_intel_crtc(encoder->base.crtc); | 1271 | to_intel_crtc(encoder->base.crtc); |
1272 | struct drm_display_mode *adjusted_mode = | 1272 | struct drm_display_mode *adjusted_mode = |
1273 | &intel_crtc->config.adjusted_mode; | 1273 | &intel_crtc->config.base.adjusted_mode; |
1274 | enum dpio_channel port = vlv_dport_to_channel(dport); | 1274 | enum dpio_channel port = vlv_dport_to_channel(dport); |
1275 | int pipe = intel_crtc->pipe; | 1275 | int pipe = intel_crtc->pipe; |
1276 | u32 val; | 1276 | u32 val; |
@@ -1467,7 +1467,7 @@ static void chv_hdmi_pre_enable(struct intel_encoder *encoder) | |||
1467 | struct intel_crtc *intel_crtc = | 1467 | struct intel_crtc *intel_crtc = |
1468 | to_intel_crtc(encoder->base.crtc); | 1468 | to_intel_crtc(encoder->base.crtc); |
1469 | struct drm_display_mode *adjusted_mode = | 1469 | struct drm_display_mode *adjusted_mode = |
1470 | &intel_crtc->config.adjusted_mode; | 1470 | &intel_crtc->config.base.adjusted_mode; |
1471 | enum dpio_channel ch = vlv_dport_to_channel(dport); | 1471 | enum dpio_channel ch = vlv_dport_to_channel(dport); |
1472 | int pipe = intel_crtc->pipe; | 1472 | int pipe = intel_crtc->pipe; |
1473 | int data, i; | 1473 | int data, i; |