diff options
| author | Paulo Zanoni <paulo.r.zanoni@intel.com> | 2013-11-08 17:05:00 -0200 |
|---|---|---|
| committer | Paulo Zanoni <paulo.r.zanoni@intel.com> | 2013-11-08 18:40:22 -0200 |
| commit | 2fd0eb53aa33662f44d6373c4562ba7cd4523400 (patch) | |
| tree | a4d49fa95e471dc378274ed4dcbeba09c2fc5ea6 | |
| parent | 514efb507d5bb1cd8774b25f4419c948b575a36b (diff) | |
tests/pm_pc8: return void on test_i2c
And do the assertion in the code line that actually verifies the
condition we need. Makes it easier to debug failed tests.
Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
| -rw-r--r-- | tests/pm_pc8.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tests/pm_pc8.c b/tests/pm_pc8.c index 4ecfdfdf..5a0b1fb0 100644 --- a/tests/pm_pc8.c +++ b/tests/pm_pc8.c @@ -600,12 +600,12 @@ static int count_i2c_valid_edids(void) return ret; } -static bool test_i2c(struct mode_set_data *data) +static void test_i2c(struct mode_set_data *data) { int i2c_edids = count_i2c_valid_edids(); int drm_edids = count_drm_valid_edids(data); - return i2c_edids == drm_edids; + igt_assert(i2c_edids == drm_edids); } static void setup_environment(void) @@ -733,7 +733,7 @@ static void i2c_subtest(void) disable_all_screens(&ms_data); igt_assert(pc8_plus_enabled()); - igt_assert(test_i2c(&ms_data)); + test_i2c(&ms_data); igt_assert(pc8_plus_enabled()); enable_one_screen(&ms_data); |