summaryrefslogtreecommitdiff
path: root/tests/kms_ccs.c
diff options
context:
space:
mode:
authorVille Syrjälä <ville.syrjala@linux.intel.com>2018-07-17 18:32:38 +0300
committerVille Syrjälä <ville.syrjala@linux.intel.com>2018-09-27 23:15:58 +0300
commit2238361afbfee71827e2ee6e71fc43f1a39dcefe (patch)
tree87a679513a25bf93ac774dcea3d8e8ffc7910d63 /tests/kms_ccs.c
parentef43fce48d55eb0d37f9fb2f97785228dc844584 (diff)
lib/igt_fb: Pass around igt_fb internally
Instead of passing around a boatload of integers everywhere let's just pass around the igt_fb struct. That obviously means we have to populate it first sufficiently, to which end we'll add a small helper. Later on the stride/size calculations will consult the already pre-populated igt_fb and fill in the rest as needed. This makes the whole thing a lot less error prone as it's impossible to accidentally pass the arguments in the wrong order when there's just the one of them, and it's a pointer. v2: Rebase due to uint64_t size Cc: Paulo Zanoni <paulo.r.zanoni@intel.com> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Diffstat (limited to 'tests/kms_ccs.c')
-rw-r--r--tests/kms_ccs.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/kms_ccs.c b/tests/kms_ccs.c
index e1ee5880..e03f947e 100644
--- a/tests/kms_ccs.c
+++ b/tests/kms_ccs.c
@@ -378,7 +378,7 @@ static void generate_fb(data_t *data, struct igt_fb *fb,
fb->drm_format = f.pixel_format;
fb->width = f.width;
fb->height = f.height;
- fb->stride = f.pitches[0];
+ fb->strides[0] = f.pitches[0];
fb->tiling = f.modifier[0];
fb->size = size[0];
fb->cairo_surface = NULL;