[igt-dev] [i-g-t] tests/kms_plane_scaling: set minimum height and width to 20

[Kaparthi, SowmyaX]

Hi Petri,

  This patch got merged in internal igt but not in open source igt.  This causing the kms_plane_scaling test case fail. So I requested Mika to push this patch to open source igt. 

Thanks and Regards
Sowmya

-----Original Message-----
From: Latvala, Petri <petri.latvala at intel.com> 
Sent: 09 September 2020 14:38
To: Kahola, Mika <mika.kahola at intel.com>
Cc: Kaparthi, SowmyaX <sowmyax.kaparthi at intel.com>; igt-dev at lists.freedesktop.org; Ausmus, James <james.ausmus at intel.com>
Subject: Re: [igt-dev] [i-g-t] tests/kms_plane_scaling: set minimum height and width to 20

On Wed, Sep 09, 2020 at 06:51:13AM +0000, Kahola, Mika wrote:
> > -----Original Message-----
> > From: Kaparthi, SowmyaX <sowmyax.kaparthi at intel.com>
> > Sent: Tuesday, September 8, 2020 12:05 PM
> > To: igt-dev at lists.freedesktop.org; Ausmus, James 
> > <james.ausmus at intel.com>; Kahola, Mika <mika.kahola at intel.com>; 
> > Kaparthi, SowmyaX <sowmyax.kaparthi at intel.com>
> > Subject: [i-g-t] tests/kms_plane_scaling: set minimum height and 
> > width to 20
> > 
> > Depending on pixel format minimum width and height can be as high as 20.
> > Set width and height to 20. Remove is_planar_yuv_format, no longer used.
> > 
> > Cc: Ausmus, James <james.ausmus at intel.com>
> > Signed-off-by: Sowmya Kaparthi <sowmyax.kaparthi at intel.com>
> 
> The patch look ok to me. The width and height can be divided by 4 and there's no need to have separate width and height for yuv and planar formats.
> 
> Reviewed-by: Mika Kahola <mika.kahola at intel.com>

I'm confused by what's going on here. This patch was originated by Matt Atwood, resent, re-reviewed...

I merged the patch now, with all S-o-b:s and R-b:s combined.



--
Petri Latvala