On Mon, Feb 23, 2015 at 03:55:59PM +0000, Tvrtko Ursulin wrote: > From: Damien Lespiau <damien.lespiau@xxxxxxxxx> > > v2: Rebased for addfb2 interface and consolidated a bit. (Tvrtko Ursulin) > v3: Rebased for fb modifier changes. (Tvrtko Ursulin) > > Signed-off-by: Damien Lespiau <damien.lespiau@xxxxxxxxx> > Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > --- This patch looks like it could reuse the newly introduced intel_fb_stride_alignment(). Otherwise: Reviewed-by: Damien Lespiau <damien.lespiau@xxxxxxxxx> > drivers/gpu/drm/i915/intel_display.c | 45 ++++++++++++++++++++++-------------- > 1 file changed, 28 insertions(+), 17 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c > index 358a97e..c622b11 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -7718,7 +7718,7 @@ skylake_get_initial_plane_config(struct intel_crtc *crtc, > { > struct drm_device *dev = crtc->base.dev; > struct drm_i915_private *dev_priv = dev->dev_private; > - u32 val, base, offset, stride_mult; > + u32 val, base, offset, stride_mult, tiling; > int pipe = crtc->pipe; > int fourcc, pixel_format; > int aligned_height; > @@ -7737,11 +7737,6 @@ skylake_get_initial_plane_config(struct intel_crtc *crtc, > if (!(val & PLANE_CTL_ENABLE)) > goto error; > > - if (val & PLANE_CTL_TILED_MASK) { > - plane_config->tiling = I915_TILING_X; > - fb->modifier[0] = I915_FORMAT_MOD_X_TILED; > - } > - > pixel_format = val & PLANE_CTL_FORMAT_MASK; > fourcc = skl_format_to_fourcc(pixel_format, > val & PLANE_CTL_ORDER_RGBX, > @@ -7749,6 +7744,33 @@ skylake_get_initial_plane_config(struct intel_crtc *crtc, > fb->pixel_format = fourcc; > fb->bits_per_pixel = drm_format_plane_cpp(fourcc, 0) * 8; > > + tiling = val & PLANE_CTL_TILED_MASK; > + switch (tiling) { > + case PLANE_CTL_TILED_LINEAR: > + fb->modifier[0] = DRM_FORMAT_MOD_NONE; > + stride_mult = 64; > + break; > + case PLANE_CTL_TILED_X: > + plane_config->tiling = I915_TILING_X; > + fb->modifier[0] = I915_FORMAT_MOD_X_TILED; > + stride_mult = 512; > + break; > + case PLANE_CTL_TILED_Y: > + fb->modifier[0] = I915_FORMAT_MOD_Y_TILED; > + stride_mult = 128; > + break; > + case PLANE_CTL_TILED_YF: > + fb->modifier[0] = I915_FORMAT_MOD_Yf_TILED; > + if (fb->bits_per_pixel == 8) > + stride_mult = 64; > + else > + stride_mult = 128; > + break; > + default: > + MISSING_CASE(tiling); > + goto error; > + } > + > base = I915_READ(PLANE_SURF(pipe, 0)) & 0xfffff000; > plane_config->base = base; > > @@ -7759,17 +7781,6 @@ skylake_get_initial_plane_config(struct intel_crtc *crtc, > fb->width = ((val >> 0) & 0x1fff) + 1; > > val = I915_READ(PLANE_STRIDE(pipe, 0)); > - switch (plane_config->tiling) { > - case I915_TILING_NONE: > - stride_mult = 64; > - break; > - case I915_TILING_X: > - stride_mult = 512; > - break; > - default: > - MISSING_CASE(plane_config->tiling); > - goto error; > - } > fb->pitches[0] = (val & 0x3ff) * stride_mult; > > aligned_height = intel_fb_align_height(dev, fb->height, > -- > 2.3.0 > _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx