diff options
| author | Chris Wilson | 2017-01-14 01:28:26 +0100 |
|---|---|---|
| committer | Chris Wilson | 2017-01-14 17:18:35 +0100 |
| commit | 7b92c047bae2210874d64ae8bbb56fbd18ab6731 (patch) | |
| tree | 14863a4ca98367524272419563ab8195582a54aa /drivers/gpu/drm/i915/intel_display.c | |
| parent | drm/i915: Convert i915_ggtt_view to use an anonymous union (diff) | |
| download | kernel-qcow2-linux-7b92c047bae2210874d64ae8bbb56fbd18ab6731.tar.gz kernel-qcow2-linux-7b92c047bae2210874d64ae8bbb56fbd18ab6731.tar.xz kernel-qcow2-linux-7b92c047bae2210874d64ae8bbb56fbd18ab6731.zip | |
drm/i915: Eliminate superfluous i915_ggtt_view_rotated
It is only being used to clear a struct and set the type, after which it
is overwritten. Since we no longer check the unset bits of the union,
skipping the clear is permissible.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20170114002827.31315-6-chris@chris-wilson.co.uk
Diffstat (limited to 'drivers/gpu/drm/i915/intel_display.c')
| -rw-r--r-- | drivers/gpu/drm/i915/intel_display.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index f4be20f0198a..f523256ef77c 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c @@ -2137,11 +2137,10 @@ intel_fill_fb_ggtt_view(struct i915_ggtt_view *view, const struct drm_framebuffer *fb, unsigned int rotation) { + view->type = I915_GGTT_VIEW_NORMAL; if (drm_rotation_90_or_270(rotation)) { - *view = i915_ggtt_view_rotated; + view->type = I915_GGTT_VIEW_ROTATED; view->rotated = to_intel_framebuffer(fb)->rot_info; - } else { - *view = i915_ggtt_view_normal; } } |