[PATCH i-g-t] lib/igt_kms.c: igt_require -> igt_require_f("two displays required\n")

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The bare "Test requirement: modes" message is too cryptic, I had to go and
read the source code to understand the missing requirement.

Signed-off-by: Marc Herbert <marc.herbert@xxxxxxxxx>
---

If there is a different message that you would like to push yourself
then please don't mind me - I think ANY message would do.

 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 b843478..cae02e9 100644
--- a/tests/kms_flip.c
+++ b/tests/kms_flip.c
@@ -1544,7 +1544,7 @@ static int 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(modes);
+	igt_require_f(modes, "At least two displays required\n");
 	duration = duration * 1000 / modes;
 	duration = max(duration, 500);
 
-- 
1.9.3

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx





[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux