diff options
author | Simon Ser <simon.ser@intel.com> | 2019-07-02 10:57:37 +0300 |
---|---|---|
committer | Simon Ser <simon.ser@intel.com> | 2019-07-05 16:06:40 +0300 |
commit | 219430933ac316313b71832f6a65e2e08c08cbd7 (patch) | |
tree | e047fb451be5f93c8daf072581266f62536ad019 /tests/kms_3d.c | |
parent | f0e39642f6f8da5406627bfa79c6600df949e203 (diff) |
lib/igt_kms: remove length parameter from kmstest_force_edid
Given an EDID, computing the size is trivial. Instead of having one size
constant per EDID and hope the callers use the right one (ie. *not* EDID_LENGTH
when there's an extension), we can make functions that take EDIDs compute the
size if they need it.
We have tests in lib/tests/igt_edid.c which assert the number of extensions
present in the EDID anyway.
Signed-off-by: Simon Ser <simon.ser@intel.com>
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Diffstat (limited to 'tests/kms_3d.c')
-rw-r--r-- | tests/kms_3d.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/kms_3d.c b/tests/kms_3d.c index df8185ab..a170814f 100644 --- a/tests/kms_3d.c +++ b/tests/kms_3d.c @@ -60,7 +60,7 @@ igt_simple_main kmstest_edid_add_3d(igt_kms_get_base_edid(), EDID_LENGTH, &edid, &length); - kmstest_force_edid(drm_fd, connector, edid, length); + kmstest_force_edid(drm_fd, connector, edid); if (!kmstest_force_connector(drm_fd, connector, FORCE_CONNECTOR_ON)) igt_skip("Could not force connector on\n"); @@ -113,7 +113,7 @@ igt_simple_main } kmstest_force_connector(drm_fd, connector, FORCE_CONNECTOR_UNSPECIFIED); - kmstest_force_edid(drm_fd, connector, NULL, 0); + kmstest_force_edid(drm_fd, connector, NULL); drmModeFreeConnector(connector); free(edid); |