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

Sharma, Swati2 swati2.sharma at intel.com
Mon Oct 21 20:22:00 UTC 2024


Hi Pranay,

Please fix the subject. Imperative mood should be used in subject line. 
For guidance on writing effectivecommitmessages, please refer to 
this10-minute read <https://cbea.ms/git-commit/>.

Also,

s/kms_flip.c/kms_flip

On 15-10-2024 22:48, 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");
>   
>   	if (duration) {
>   		duration = duration * 1000 / modes;


More information about the igt-dev mailing list