From: Ville Syrjälä <ville.syrj...@linux.intel.com>

In preparation for changing the primary plane position pass the clipped
position to .update_primary_plane().

Cc: Sonika Jindal <sonika.jin...@intel.com>
Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>
---
 drivers/gpu/drm/i915/i915_drv.h      |  1 +
 drivers/gpu/drm/i915/intel_display.c | 20 +++++++++++++++-----
 2 files changed, 16 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index e99eef0..8d8b9a1 100644
--- a/drivers/gpu/drm/i915/i915_drv.h
+++ b/drivers/gpu/drm/i915/i915_drv.h
@@ -579,6 +579,7 @@ struct drm_i915_display_funcs {
        void (*update_primary_plane)(struct drm_crtc *crtc,
                                     struct drm_framebuffer *fb,
                                     int x, int y,
+                                    int crtc_x, int crtc_y,
                                     int crtc_w, int crtc_h);
        void (*hpd_irq_setup)(struct drm_device *dev);
        /* clock updates for mode set */
diff --git a/drivers/gpu/drm/i915/intel_display.c 
b/drivers/gpu/drm/i915/intel_display.c
index 1a789f0..33680a8 100644
--- a/drivers/gpu/drm/i915/intel_display.c
+++ b/drivers/gpu/drm/i915/intel_display.c
@@ -2483,6 +2483,7 @@ intel_find_plane_obj(struct intel_crtc *intel_crtc,
 static void i9xx_update_primary_plane(struct drm_crtc *crtc,
                                      struct drm_framebuffer *fb,
                                      int x, int y,
+                                     int crtc_x, int crtc_y,
                                      int crtc_w, int crtc_h)
 {
        struct drm_device *dev = crtc->dev;
@@ -2519,13 +2520,14 @@ static void i9xx_update_primary_plane(struct drm_crtc 
*crtc,
                if (intel_crtc->pipe == PIPE_B)
                        dspcntr |= DISPPLANE_SEL_PIPE_B;
                I915_WRITE(DSPSIZE(plane), ((crtc_h - 1) << 16) | (crtc_w - 1));
-               I915_WRITE(DSPPOS(plane), 0);
+               I915_WRITE(DSPPOS(plane), (crtc_y << 16) | crtc_x);
        } else if (IS_CHERRYVIEW(dev) && plane == PLANE_B) {
                I915_WRITE(PRIMSIZE(plane), ((crtc_h - 1) << 16) | (crtc_w - 
1));
-               I915_WRITE(PRIMPOS(plane), 0);
+               I915_WRITE(PRIMPOS(plane), (crtc_y << 16) | crtc_x);
                I915_WRITE(PRIMCNSTALPHA(plane), 0);
        } else {
-               WARN_ONCE(crtc_w != intel_crtc->config->pipe_src_w ||
+               WARN_ONCE(crtc_x != 0 || crtc_y != 0 ||
+                         crtc_w != intel_crtc->config->pipe_src_w ||
                          crtc_h != intel_crtc->config->pipe_src_h,
                          "primary plane size doesn't match pipe size\n");
        }
@@ -2609,6 +2611,7 @@ static void i9xx_update_primary_plane(struct drm_crtc 
*crtc,
 static void ironlake_update_primary_plane(struct drm_crtc *crtc,
                                          struct drm_framebuffer *fb,
                                          int x, int y,
+                                         int crtc_x, int crtc_y,
                                          int crtc_w, int crtc_h)
 {
        struct drm_device *dev = crtc->dev;
@@ -2641,7 +2644,8 @@ static void ironlake_update_primary_plane(struct drm_crtc 
*crtc,
        if (IS_HASWELL(dev) || IS_BROADWELL(dev))
                dspcntr |= DISPPLANE_PIPE_CSC_ENABLE;
 
-       WARN_ONCE(crtc_w != intel_crtc->config->pipe_src_w ||
+       WARN_ONCE(crtc_x != 0 || crtc_y != 0 ||
+                 crtc_w != intel_crtc->config->pipe_src_w ||
                  crtc_h != intel_crtc->config->pipe_src_h,
                  "primary plane size doesn't match pipe size\n");
 
@@ -2750,6 +2754,7 @@ u32 intel_fb_stride_alignment(struct drm_device *dev, 
uint64_t fb_modifier,
 static void skylake_update_primary_plane(struct drm_crtc *crtc,
                                         struct drm_framebuffer *fb,
                                         int x, int y,
+                                        int crtc_x, int crtc_y,
                                         int crtc_w, int crtc_h)
 {
        struct drm_device *dev = crtc->dev;
@@ -2827,7 +2832,7 @@ static void skylake_update_primary_plane(struct drm_crtc 
*crtc,
 
        I915_WRITE(PLANE_CTL(pipe, 0), plane_ctl);
 
-       I915_WRITE(PLANE_POS(pipe, 0), 0);
+       I915_WRITE(PLANE_POS(pipe, 0), (crtc_y << 16) | crtc_x);
        I915_WRITE(PLANE_OFFSET(pipe, 0), (y << 16) | x);
        I915_WRITE(PLANE_SIZE(pipe, 0), ((crtc_h - 1) << 16) | (crtc_w - 1));
        I915_WRITE(PLANE_STRIDE(pipe, 0), fb->pitches[0] / stride_div);
@@ -2854,6 +2859,7 @@ intel_pipe_set_base_atomic(struct drm_crtc *crtc, struct 
drm_framebuffer *fb,
        /* FIXME: this will go badly if the fb isn't big enough */
        to_intel_crtc(crtc)->primary_enabled = true;
        dev_priv->display.update_primary_plane(crtc, fb, x, y,
+                                              0, 0,
                                               intel_crtc->config->pipe_src_w,
                                               intel_crtc->config->pipe_src_h);
 
@@ -2891,6 +2897,8 @@ static void intel_update_primary_planes(struct drm_device 
*dev)
                                                               state->base.fb,
                                                               state->src.x1 >> 
16,
                                                               state->src.y1 >> 
16,
+                                                              state->dst.x1,
+                                                              state->dst.y1,
                                                               
drm_rect_width(&state->dst),
                                                               
drm_rect_height(&state->dst));
                }
@@ -12027,6 +12035,8 @@ intel_commit_primary_plane(struct drm_plane *plane,
                                                       state->base.fb,
                                                       state->src.x1 >> 16,
                                                       state->src.y1 >> 16,
+                                                      state->dst.x1,
+                                                      state->dst.y1,
                                                       
drm_rect_width(&state->dst),
                                                       
drm_rect_height(&state->dst));
        }
-- 
2.0.5

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to