drm-tests: draw.c: rename pixel skip to byte skip
Now we've changed how draw YUV surfaces, pixel_skip needs to mean
the number of bytes between the same draw component on the same
plane. For example for NV12, the number bytes between U components
on the second plane is 2. This is analogous to chroma step in
struct android_ycbcr.
BUG=b:37526302
TEST=plane_test (defaults to NV12) on minnie draws striped pattern
Change-Id: Iaff77bbbd5538d9c821e2cc52dc36d46249cbff8
Reviewed-on: https://chromium-review.googlesource.com/489811
Commit-Ready: Gurchetan Singh <gurchetansingh@chromium.org>
Tested-by: Gurchetan Singh <gurchetansingh@chromium.org>
Reviewed-by: Dongseong Hwang <dongseong.hwang@intel.com>
Reviewed-by: Zach Reizner <zachr@chromium.org>
diff --git a/bsdrm/src/draw.c b/bsdrm/src/draw.c
index c383b8e..553da28 100644
--- a/bsdrm/src/draw.c
+++ b/bsdrm/src/draw.c
@@ -12,7 +12,7 @@
float value_offset;
uint32_t horizontal_subsample_rate;
uint32_t vertical_subsample_rate;
- uint32_t pixel_skip;
+ uint32_t byte_skip;
uint32_t plane_index;
uint32_t plane_offset;
};
@@ -43,8 +43,8 @@
3,
{
{ { 0.2567890625f, 0.50412890625f, 0.09790625f }, 16.0f, 1, 1, 1, 0, 0 },
- { { -0.14822265625f, -0.2909921875f, 0.43921484375f }, 128.0f, 2, 2, 1, 1, 0 },
- { { 0.43921484375f, -0.3677890625f, -0.07142578125f }, 128.0f, 2, 2, 1, 1, 1 },
+ { { -0.14822265625f, -0.2909921875f, 0.43921484375f }, 128.0f, 2, 2, 2, 1, 0 },
+ { { 0.43921484375f, -0.3677890625f, -0.07142578125f }, 128.0f, 2, 2, 2, 1, 1 },
},
},
{
@@ -198,7 +198,7 @@
}
}
- *(rows[comp_index] + x * comp->pixel_skip) = color / samples;
+ *(rows[comp_index] + x * comp->byte_skip) = color / samples;
}
}
}