drm/vc4: Make sure that planes aren't scaled.
authorEric Anholt <eric@anholt.net>
Fri, 23 Oct 2015 09:36:27 +0000 (10:36 +0100)
committerEric Anholt <eric@anholt.net>
Tue, 17 Nov 2015 20:27:03 +0000 (12:27 -0800)
We would scan out the memory around them if an upscale was attempted,
and would just scan out incorrectly for downscaling.

Signed-off-by: Eric Anholt <eric@anholt.net>
drivers/gpu/drm/vc4/vc4_plane.c

index 2db509210d62a342d5ebe935dec22b3c5062ba1c..887f3caad0be6b6fd115b4e9430abfd3aeadd53f 100644 (file)
@@ -157,6 +157,16 @@ static int vc4_plane_mode_set(struct drm_plane *plane,
        int crtc_w = state->crtc_w;
        int crtc_h = state->crtc_h;
 
+       if (state->crtc_w << 16 != state->src_w ||
+           state->crtc_h << 16 != state->src_h) {
+               /* We don't support scaling yet, which involves
+                * allocating the LBM memory for scaling temporary
+                * storage, and putting filter kernels in the HVS
+                * context.
+                */
+               return -EINVAL;
+       }
+
        if (crtc_x < 0) {
                offset += drm_format_plane_cpp(fb->pixel_format, 0) * -crtc_x;
                crtc_w += crtc_x;