From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
Even unscaled NV12 needs the plane scaler on SKL+, so when
unscaled NV12 setplane fails we should still take the GPU
scaling fallback path.
Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
---
src/sna/sna_video_sprite.c | 12 ++++++++++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/src/sna/sna_video_sprite.c b/src/sna/sna_video_sprite.c
index f6d6f0b45b6c..0f52f0328fc0 100644
--- a/src/sna/sna_video_sprite.c
+++ b/src/sna/sna_video_sprite.c
@@ -415,6 +415,15 @@ sna_video_sprite_show(struct sna *sna,
return true;
}
+static bool need_scaling(const struct sna_video_frame *frame,
+ const BoxRec *dst)
+{
+ /* SKL+ need the plane scaler even for unscaled NV12 */
+ return frame->id == FOURCC_NV12 ||
+ frame->src.x2 - frame->src.x1 != dst->x2 - dst->x1 ||
+ frame->src.y2 - frame->src.y1 != dst->y2 - dst->y1;
+}
+
static int sna_video_sprite_put_image(ddPutImage_ARGS)
{
struct sna_video *video = port->devPriv.ptr;
@@ -562,8 +571,7 @@ off:
}
if (!hw_scaling && sna->render.video &&
- !((frame.src.x2 - frame.src.x1) == (dst.x2 - dst.x1) &&
- (frame.src.y2 - frame.src.y1) == (dst.y2 - dst.y1))) {
+ need_scaling(&frame, &dst)) {
ScreenPtr screen = to_screen_from_sna(sna);
PixmapPtr scaled;
RegionRec r;
--
2.16.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
