drm/i915: Fix intel_tile_width() parameters
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Wed, 20 Jan 2016 19:05:28 +0000 (21:05 +0200)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Thu, 28 Jan 2016 18:57:08 +0000 (20:57 +0200)
The fb_modifiers and cpp arguments passed to intel_tile_width() in
intel_fill_fb_ggtt_view() got accidentally swapped around. I'm pretty
sure I fixed this already, but could be I lost the fix accidentally
during some rebases or something. Anyway, fix it up for real.

Fixes: d9b3288ecf2f ("drm/i915: change intel_fill_fb_ggtt_view() to use the real tile size")
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1453316739-13296-8-git-send-email-ville.syrjala@linux.intel.com
Testcase: igt/kms_rotation_crc/primary-rotation-90
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: drm-intel-fixes@lists.freedesktop.org
drivers/gpu/drm/i915/intel_display.c

index 23cce7df74b0a3f34e6141b2e130e29a7c83275a..304fc9637026172899ae17e953a4ee34a3fbbf0b 100644 (file)
@@ -2306,7 +2306,7 @@ intel_fill_fb_ggtt_view(struct i915_ggtt_view *view, struct drm_framebuffer *fb,
        tile_size = intel_tile_size(dev_priv);
 
        cpp = drm_format_plane_cpp(fb->pixel_format, 0);
-       tile_width = intel_tile_width(dev_priv, cpp, fb->modifier[0]);
+       tile_width = intel_tile_width(dev_priv, fb->modifier[0], cpp);
        tile_height = tile_size / tile_width;
 
        info->width_pages = DIV_ROUND_UP(fb->pitches[0], tile_width);