[PATCH i-g-t] tests/intel/kms_draw_crc: Skip formats not supported by the plane

[Kamil Konieczny]

Hi Juha-Pekka,
On 2025-03-05 at 16:40:07 +0200, Juha-Pekka Heikkilä wrote:
> Reviewed-by: Juha-Pekka Heikkila <juhapekka.heikkila at gmail.com>
> 

Thank you, applied.

Regards,
Kamil

> On Mon, Mar 3, 2025 at 1:02 PM Ville Syrjala
> <ville.syrjala at linux.intel.com> wrote:
> >
> > From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> >
> > The test only checks igt_display_has_format_mod() before
> > deciding to test a specific format. But that doesn't
> > mean the actual plane supports that format. Also check
> > for that once we know which plane to use.
> >
> > Should turn the fp16 failures into skips on IVB because it
> > doesn't have fp16 support on the primary plane.
> >
> > Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
> > ---
> >  tests/intel/kms_draw_crc.c | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/tests/intel/kms_draw_crc.c b/tests/intel/kms_draw_crc.c
> > index 9c350853cfd9..62c6c64be710 100644
> > --- a/tests/intel/kms_draw_crc.c
> > +++ b/tests/intel/kms_draw_crc.c
> > @@ -158,6 +158,8 @@ static void get_method_crc(enum igt_draw_method method, uint32_t drm_format,
> >
> >         primary = igt_output_get_plane_type(output, DRM_PLANE_TYPE_PRIMARY);
> >
> > +       igt_require(igt_plane_has_format_mod(primary, drm_format, modifier));
> > +
> >         igt_create_fb(drm_fd, mode->hdisplay, mode->vdisplay,
> >                       drm_format, modifier, &fb);
> >         igt_plane_set_fb(primary, &fb);
> > --
> > 2.45.3
> >