From e061af7bc37bfc77893dae8dab93d712d39d18df Mon Sep 17 00:00:00 2001 From: Juha-Pekka Heikkila Date: Tue, 22 Jan 2019 16:04:56 +0200 Subject: tests/kms_rotation_crc: Fix multiplane-rotation-cropping* tests MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Make certain viewports are divisible by four due to intel hw workarounds for NV12. Signed-off-by: Juha-Pekka Heikkila Reviewed-by: Ville Syrjälä --- tests/kms_rotation_crc.c | 29 +++++++++++++++++++++++++---- 1 file changed, 25 insertions(+), 4 deletions(-) (limited to 'tests/kms_rotation_crc.c') diff --git a/tests/kms_rotation_crc.c b/tests/kms_rotation_crc.c index 8f343e01..fc995d07 100644 --- a/tests/kms_rotation_crc.c +++ b/tests/kms_rotation_crc.c @@ -486,10 +486,31 @@ static bool get_multiplane_crc(data_t *data, igt_output_t *output, static void pointlocation(data_t *data, planeinfos *p, drmModeModeInfo *mode, int c) { - p[c].x1 = (int32_t)(data->planepos[c].x * mode->hdisplay) - + ((data->planepos[c].origo & p_right) ? mode->hdisplay : 0); - p[c].y1 = (int32_t)(data->planepos[c].y * mode->vdisplay) - + ((data->planepos[c].origo & p_bottom) ? mode->vdisplay : 0); + if (data->planepos[c].origo & p_right) { + p[c].x1 = (int32_t)(data->planepos[c].x * mode->hdisplay + + mode->hdisplay); + p[c].x1 &= ~3; + /* + * At this point is handled surface on right side. If display + * mode is not divisible by 4 but with 2 point location is + * fixed to match requirements. Because of YUV planes here is + * intentionally ignored bit 1. + */ + p[c].x1 -= mode->hdisplay & 2; + } else { + p[c].x1 = (int32_t)(data->planepos[c].x * mode->hdisplay); + p[c].x1 &= ~3; + } + + if (data->planepos[c].origo & p_bottom) { + p[c].y1 = (int32_t)(data->planepos[c].y * mode->vdisplay + + mode->vdisplay); + p[c].y1 &= ~3; + p[c].y1 -= mode->vdisplay & 2; + } else { + p[c].y1 = (int32_t)(data->planepos[c].y * mode->vdisplay); + p[c].y1 &= ~3; + } } /* -- cgit v1.2.3