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_fbdev.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_fbdev.c')
-rw-r--r-- | drivers/gpu/drm/i915/intel_fbdev.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/gpu/drm/i915/intel_fbdev.c b/drivers/gpu/drm/i915/intel_fbdev.c index 850cf7d6578c..04d582b50364 100644 --- a/drivers/gpu/drm/i915/intel_fbdev.c +++ b/drivers/gpu/drm/i915/intel_fbdev.c | |||
@@ -581,7 +581,7 @@ static bool intel_fbdev_init_bios(struct drm_device *dev, | |||
581 | * pipe. Note we need to use the selected fb's pitch and bpp | 581 | * pipe. Note we need to use the selected fb's pitch and bpp |
582 | * rather than the current pipe's, since they differ. | 582 | * rather than the current pipe's, since they differ. |
583 | */ | 583 | */ |
584 | cur_size = intel_crtc->config.adjusted_mode.crtc_hdisplay; | 584 | cur_size = intel_crtc->config.base.adjusted_mode.crtc_hdisplay; |
585 | cur_size = cur_size * fb->base.bits_per_pixel / 8; | 585 | cur_size = cur_size * fb->base.bits_per_pixel / 8; |
586 | if (fb->base.pitches[0] < cur_size) { | 586 | if (fb->base.pitches[0] < cur_size) { |
587 | DRM_DEBUG_KMS("fb not wide enough for plane %c (%d vs %d)\n", | 587 | DRM_DEBUG_KMS("fb not wide enough for plane %c (%d vs %d)\n", |
@@ -592,13 +592,13 @@ static bool intel_fbdev_init_bios(struct drm_device *dev, | |||
592 | break; | 592 | break; |
593 | } | 593 | } |
594 | 594 | ||
595 | cur_size = intel_crtc->config.adjusted_mode.crtc_vdisplay; | 595 | cur_size = intel_crtc->config.base.adjusted_mode.crtc_vdisplay; |
596 | cur_size = ALIGN(cur_size, plane_config->tiled ? (IS_GEN2(dev) ? 16 : 8) : 1); | 596 | cur_size = ALIGN(cur_size, plane_config->tiled ? (IS_GEN2(dev) ? 16 : 8) : 1); |
597 | cur_size *= fb->base.pitches[0]; | 597 | cur_size *= fb->base.pitches[0]; |
598 | DRM_DEBUG_KMS("pipe %c area: %dx%d, bpp: %d, size: %d\n", | 598 | DRM_DEBUG_KMS("pipe %c area: %dx%d, bpp: %d, size: %d\n", |
599 | pipe_name(intel_crtc->pipe), | 599 | pipe_name(intel_crtc->pipe), |
600 | intel_crtc->config.adjusted_mode.crtc_hdisplay, | 600 | intel_crtc->config.base.adjusted_mode.crtc_hdisplay, |
601 | intel_crtc->config.adjusted_mode.crtc_vdisplay, | 601 | intel_crtc->config.base.adjusted_mode.crtc_vdisplay, |
602 | fb->base.bits_per_pixel, | 602 | fb->base.bits_per_pixel, |
603 | cur_size); | 603 | cur_size); |
604 | 604 | ||