[PATCH i-g-t 1/2] tests/kms_flip.c: Updating the skip message

Naladala, Ramanaidu Ramanaidu.naladala at intel.com
Thu Nov 7 13:26:30 UTC 2024


On 10/15/2024 10:48 PM, Pranay Samala wrote:
> Debugging would be easier if we had logs available for
> test failures and skips. This patch rephrases the skip
> message for better understanding.
>
> Signed-off-by: Pranay Samala<pranay.samala at intel.com>
> ---
>   tests/kms_flip.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/kms_flip.c b/tests/kms_flip.c
> index cbabbe74f..88cd41083 100755
> --- a/tests/kms_flip.c
> +++ b/tests/kms_flip.c
> @@ -1896,7 +1896,7 @@ static void run_pair(int duration, int flags)
>   
>   	/* If we have fewer than 2 connected outputs then we won't have any
>   	 * configuration at all. So skip in that case. */
> -	igt_require_f(modes, "At least two displays required\n");
> +	igt_require_f(modes, "At least two displays with same modes are required\n");

LGTM,

Reviewed-by: Naladala Ramanaidu <ramanaidu.naladala at intel.com> 
<mailto:ramanaidu.naladala at intel.com>

>   
>   	if (duration) {
>   		duration = duration * 1000 / modes;
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/igt-dev/attachments/20241107/0dba128b/attachment-0001.htm>


More information about the igt-dev mailing list