summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChris Wilson <chris@chris-wilson.co.uk>2012-12-21 21:36:30 +0000
committerChris Wilson <chris@chris-wilson.co.uk>2012-12-21 21:36:30 +0000
commit52fd223fc970118cbdcb31f9574414debc905e9c (patch)
tree968332e000bd68cd50ba95e6751059ad41181f30
parent3793ccf7804cfc870b46c623dfeefbe0c381c1d4 (diff)
downloadxf86-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.c5
-rw-r--r--src/sna/sna_video_textured.c2
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;