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

[Naladala, Ramanaidu]

On 11/7/2024 7:10 PM, Samala, Pranay wrote:
>
> -----Original Message-----
> From: Samala, Pranay<pranay.samala at intel.com> 
> Sent: Wednesday, November 6, 2024 3:30 PM
> To:igt-dev at lists.freedesktop.org
> Cc: B S, Karthik<karthik.b.s at intel.com>; Naladala, Ramanaidu<ramanaidu.naladala at intel.com>; Lattannavar, Sameer<sameer.lattannavar at intel.com>; Samala, Pranay<pranay.samala at intel.com>
> Subject: [PATCH i-g-t v2 1/2] tests/kms_flip: Updating the skip message
>
> 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;
> --
> 2.34.1
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/igt-dev/attachments/20241107/2a5d5f04/attachment-0001.htm>