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>
This commit is contained in:
Paulo Zanoni 2013-11-08 17:05:00 -02:00
parent 514efb507d
commit 2fd0eb53aa

View File

@ -600,12 +600,12 @@ static int count_i2c_valid_edids(void)
return ret; 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 i2c_edids = count_i2c_valid_edids();
int drm_edids = count_drm_valid_edids(data); int drm_edids = count_drm_valid_edids(data);
return i2c_edids == drm_edids; igt_assert(i2c_edids == drm_edids);
} }
static void setup_environment(void) static void setup_environment(void)
@ -733,7 +733,7 @@ static void i2c_subtest(void)
disable_all_screens(&ms_data); disable_all_screens(&ms_data);
igt_assert(pc8_plus_enabled()); igt_assert(pc8_plus_enabled());
igt_assert(test_i2c(&ms_data)); test_i2c(&ms_data);
igt_assert(pc8_plus_enabled()); igt_assert(pc8_plus_enabled());
enable_one_screen(&ms_data); enable_one_screen(&ms_data);