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_force_connector_basic.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_force_connector_basic.c')
-rw-r--r-- | tests/kms_force_connector_basic.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/tests/kms_force_connector_basic.c b/tests/kms_force_connector_basic.c index 20812d5e..f1533e54 100644 --- a/tests/kms_force_connector_basic.c +++ b/tests/kms_force_connector_basic.c @@ -48,7 +48,7 @@ static void reset_connectors(void) 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); } @@ -247,7 +247,7 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL) /* test edid forcing */ kmstest_force_edid(drm_fd, vga_connector, - igt_kms_get_base_edid(), EDID_LENGTH); + igt_kms_get_base_edid()); temp = drmModeGetConnectorCurrent(drm_fd, vga_connector->connector_id); @@ -260,7 +260,7 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL) drmModeFreeConnector(temp); /* remove edid */ - kmstest_force_edid(drm_fd, vga_connector, NULL, 0); + kmstest_force_edid(drm_fd, vga_connector, NULL); kmstest_force_connector(drm_fd, vga_connector, FORCE_CONNECTOR_UNSPECIFIED); temp = drmModeGetConnectorCurrent(drm_fd, @@ -280,7 +280,7 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL) /* test pruning of stale modes */ kmstest_force_edid(drm_fd, vga_connector, - igt_kms_get_alt_edid(), EDID_LENGTH); + igt_kms_get_alt_edid()); temp = drmModeGetConnectorCurrent(drm_fd, vga_connector->connector_id); @@ -294,7 +294,7 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL) drmModeFreeConnector(temp); kmstest_force_edid(drm_fd, vga_connector, - igt_kms_get_base_edid(), EDID_LENGTH); + igt_kms_get_base_edid()); temp = drmModeGetConnectorCurrent(drm_fd, vga_connector->connector_id); @@ -307,7 +307,7 @@ igt_main_args("", long_opts, help_str, opt_handler, NULL) drmModeFreeConnector(temp); - kmstest_force_edid(drm_fd, vga_connector, NULL, 0); + kmstest_force_edid(drm_fd, vga_connector, NULL); kmstest_force_connector(drm_fd, vga_connector, FORCE_CONNECTOR_UNSPECIFIED); } |