diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2012-12-21 21:36:30 +0000 |
---|---|---|
committer | Chris Wilson <chris@chris-wilson.co.uk> | 2012-12-21 21:36:30 +0000 |
commit | 52fd223fc970118cbdcb31f9574414debc905e9c (patch) | |
tree | 968332e000bd68cd50ba95e6751059ad41181f30 | |
parent | 3793ccf7804cfc870b46c623dfeefbe0c381c1d4 (diff) | |
download | xf86-video-intel-52fd223fc970118cbdcb31f9574414debc905e9c.tar.gz xf86-video-intel-52fd223fc970118cbdcb31f9574414debc905e9c.tar.bz2 xf86-video-intel-52fd223fc970118cbdcb31f9574414debc905e9c.zip |
sna/video: Initialise alignment for video ports > 0
We repeatedly set the alignment value on the first port, rather than
once for each.
Reported-by: Jiri Slaby <jirislaby@gmail.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=47597
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
-rw-r--r-- | src/sna/sna_video.c | 5 | ||||
-rw-r--r-- | src/sna/sna_video_textured.c | 2 |
2 files changed, 5 insertions, 2 deletions
diff --git a/src/sna/sna_video.c b/src/sna/sna_video.c index ebcf48da6..d5b9c0f02 100644 --- a/src/sna/sna_video.c +++ b/src/sna/sna_video.c @@ -200,6 +200,10 @@ sna_video_frame_init(struct sna *sna, { int align; + DBG(("%s: id=%d [planar? %d], width=%d, height=%d, align=%d\n", + __FUNCTION__, id, is_planar_fourcc(id), width, height, video->alignment)); + assert(width && height); + frame->bo = NULL; frame->id = id; frame->width = width; @@ -212,7 +216,6 @@ sna_video_frame_init(struct sna *sna, align = 1024; #endif - /* Determine the desired destination pitch (representing the chroma's pitch, * in the planar case. */ diff --git a/src/sna/sna_video_textured.c b/src/sna/sna_video_textured.c index c5947ed58..e5cae859e 100644 --- a/src/sna/sna_video_textured.c +++ b/src/sna/sna_video_textured.c @@ -453,7 +453,7 @@ XF86VideoAdaptorPtr sna_video_textured_setup(struct sna *sna, struct sna_video *v = &video[i]; v->textured = true; - video->alignment = 4; + v->alignment = 4; v->rotation = RR_Rotate_0; v->SyncToVblank = 1; |