diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2016-07-27 13:17:38 +0200 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2016-07-27 15:43:25 +0200 |
commit | 582ce4cd19c627606047b1a8fdd987c4dc07353c (patch) | |
tree | 67b198d809719ae0f1ede6c96a605b8bd49a9986 /lib/igt_debugfs.c | |
parent | 734ae3ebcfc0eacd988325e9c399cc7b8ad0220c (diff) |
lib/debugs: nuke igt_crc_equal again
This was intentionally left out of the abi since CRC have fairly low
entropy and can't be reliably compared for inequality in testcases.
Spotted while fixing up warnigns in docs, but given that igt_assert_crc_equal
has a big comment explaining why only the positive assert exists I'm not
sure all those docs are all that useful :(
Cc: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Cc: Matt Roper <matthew.d.roper@intel.com>
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'lib/igt_debugfs.c')
-rw-r--r-- | lib/igt_debugfs.c | 17 |
1 files changed, 0 insertions, 17 deletions
diff --git a/lib/igt_debugfs.c b/lib/igt_debugfs.c index 103fe4d4..afb83617 100644 --- a/lib/igt_debugfs.c +++ b/lib/igt_debugfs.c @@ -252,23 +252,6 @@ bool igt_debugfs_search(const char *filename, const char *substring) * @a: first pipe CRC value * @b: second pipe CRC value * - * Compares two CRC values. - */ -bool igt_crc_equal(igt_crc_t *a, igt_crc_t *b) -{ - int i; - - for (i = 0; i < a->n_words; i++) - if (a->crc[i] != b->crc[i]) - return false; - return true; -} - -/** - * igt_assert_crc_equal: - * @a: first pipe CRC value - * @b: second pipe CRC value - * * Compares two CRC values and fails the testcase if they don't match with * igt_fail(). Note that due to CRC collisions CRC based testcase can only * assert that CRCs match, never that they are different. Otherwise there might |