From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by gabe.freedesktop.org (Postfix) with ESMTPS id 28C3E6E23B for ; Mon, 20 Jul 2020 09:11:14 +0000 (UTC) Date: Mon, 20 Jul 2020 12:11:11 +0300 From: Petri Latvala Message-ID: <20200720091111.GN20883@platvala-desk.ger.corp.intel.com> References: <20200618152707.6855-1-matthew.s.atwood@intel.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20200618152707.6855-1-matthew.s.atwood@intel.com> Subject: Re: [igt-dev] [PATCH i-g-t] tests/kms_plane_scaling: set minimum height and width to 20 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" To: Matt Atwood Cc: igt-dev@lists.freedesktop.org List-ID: On Thu, Jun 18, 2020 at 08:27:07AM -0700, Matt Atwood wrote: > Depending on pixel format for gen11+ platforms minimum width can be as > high as 20. Set width and height to 20. Remove is_planar_yuv_format, no > longer used. > > Signed-off-by: Matt Atwood Reviewed-by: Petri Latvala > --- > tests/kms_plane_scaling.c | 17 +---------------- > 1 file changed, 1 insertion(+), 16 deletions(-) > > diff --git a/tests/kms_plane_scaling.c b/tests/kms_plane_scaling.c > index 19087286a..ecd2c2e30 100644 > --- a/tests/kms_plane_scaling.c > +++ b/tests/kms_plane_scaling.c > @@ -62,19 +62,6 @@ static int get_num_scalers(data_t* d, enum pipe pipe) > return 1; > } > > -static bool is_planar_yuv_format(uint32_t pixelformat) > -{ > - switch (pixelformat) { > - case DRM_FORMAT_NV12: > - case DRM_FORMAT_P010: > - case DRM_FORMAT_P012: > - case DRM_FORMAT_P016: > - return true; > - default: > - return false; > - } > -} > - > static void cleanup_fbs(data_t *data) > { > int i; > @@ -147,9 +134,7 @@ static void check_scaling_pipe_plane_rot(data_t *d, igt_plane_t *plane, > mode = igt_output_get_mode(output); > > /* create buffer in the range of min and max source side limit.*/ > - width = height = 8; > - if (is_i915_device(d->drm_fd) && is_planar_yuv_format(pixel_format)) > - width = height = 16; > + width = height = 20; > igt_create_color_fb(display->drm_fd, width, height, > pixel_format, tiling, 0.0, 1.0, 0.0, &d->fb[0]); > igt_plane_set_fb(plane, &d->fb[0]); > -- > 2.21.3 > _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev