diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2018-10-03 20:43:49 +0100 |
---|---|---|
committer | Arkadiusz Hiler <arkadiusz.hiler@intel.com> | 2018-10-04 15:44:09 +0300 |
commit | 519003cace0c32aa226005dc7409544e26c243c2 (patch) | |
tree | ebbc2d82b3c547d33477c05417602ecb95d431cc /tests/testdisplay.c | |
parent | 596f48dcd59fd2f8c16671514f3e69d4a2891374 (diff) |
igt: Check drmModeGetResources()
If KMS is not supported on the device, drmModeGetResources() will return
NULL, often this is an indication that we should not attempt to run the
test. Although it would be preferred to use something like
igt_require_display() as the canonical check and assert that
drmModeGetResources() did not hit an error, it is not always practical
as the tests do not utilize the common igt_display abstraction.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
Diffstat (limited to 'tests/testdisplay.c')
-rw-r--r-- | tests/testdisplay.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/tests/testdisplay.c b/tests/testdisplay.c index 0ff98a2b..b13c3d70 100644 --- a/tests/testdisplay.c +++ b/tests/testdisplay.c @@ -156,12 +156,15 @@ static void dump_connectors_fd(int drmfd) static void dump_crtcs_fd(int drmfd) { - int i; - drmModeRes *mode_resources = drmModeGetResources(drmfd); + drmModeRes *mode_resources; + + mode_resources = drmModeGetResources(drmfd); + if (!mode_resources) + return; igt_info("CRTCs:\n"); igt_info("id\tfb\tpos\tsize\n"); - for (i = 0; i < mode_resources->count_crtcs; i++) { + for (int i = 0; i < mode_resources->count_crtcs; i++) { drmModeCrtc *crtc; crtc = drmModeGetCrtc(drmfd, mode_resources->crtcs[i]); |