Re: [PATCH i-g-t] tests/kms_bw: Skip tests when outputs don't support the mode

["Jason-JH Lin (林睿祥)" <Jason-JH.Lin@mediatek.com>]

[snip]

> > +static bool output_mode_supported(igt_output_t *output, const
> > drmModeModeInfo *mode)
> > +{
> > +	drmModeConnector *connector = output->config.connector;
> > +	int i;
> > +
> > +	/* Virtual/forced sinks support all modes */
> > +	if (!igt_output_is_connected(output))
> > +		return true;> +	for (i = 0; i < connector-
> > >count_modes; i++) {
> > +		drmModeModeInfo *conn_mode = &connector->modes[i];
> > +
> > +		if (conn_mode->hdisplay == mode->hdisplay &&
> > +		    conn_mode->vdisplay == mode->vdisplay &&
> > +		    conn_mode->vrefresh == mode->vrefresh) {
> > +			igt_info("Found matching mode for
> > %dx%d@%dHz on %s\n",
> > +				 mode->hdisplay, mode->vdisplay,
> > mode->vrefresh,
> > +				 igt_output_name(output));
> 
> Hello,
> 
> I think this should be a igt_debug, I don't think this is useful to
> see 
> it every time in normal operations.
> 
> With or without this:
> Reviewed-by: Louis Chauvet <louis.chauvet@bootlin.com>
> 

Hi Louis,

Thanks for the review.
OK! I'll change this to igt_debug.

Regards,
Jason-JH Lin

> Thanks,
> Louis Chauvet