diff options
| author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-12-01 11:24:18 +0100 |
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-12-04 10:48:05 +0100 |
| commit | cdb398b5b9a6a1895bbc8213f88bc9e45795db23 (patch) | |
| tree | f51502aa353b48c4b7baa73231b138efbe8fb6ff /tests | |
| parent | 02cdd9899dbf56ae8b9ae8ee2b83e2afcd479193 (diff) | |
tests/kms_force_connector: Fixes
The edid we inject stayed the same, but the kernel started to list
more modes for it. No idea whether that's the right thing here since
I'm not really an EDID expert. But then again the testcase wants to
check that the injection works, not validate the kernel's parser.
v2: Only check the preferred mode for more future-proofing (Thomas).
v3: Clarify commit message (Jani).
Cc: Thomas Wood <thomas.wood@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/kms_force_connector.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/tests/kms_force_connector.c b/tests/kms_force_connector.c index 838a3b0a..99a5649e 100644 --- a/tests/kms_force_connector.c +++ b/tests/kms_force_connector.c @@ -150,10 +150,8 @@ int main(int argc, char **argv) vga_connector->connector_id); CHECK_MODE(temp->modes[0], 1920, 1080, 60); - CHECK_MODE(temp->modes[1], 1280, 720, 60); - CHECK_MODE(temp->modes[2], 1024, 768, 60); - CHECK_MODE(temp->modes[3], 800, 600, 60); - CHECK_MODE(temp->modes[4], 640, 480, 60); + /* Don't check non-preferred modes to avoid to tight coupling + * with the in-kernel EDID parser. */ drmModeFreeConnector(temp); |