drm/crc: Only open CRC on atomic drivers when the CRTC is active.
authorMaarten Lankhorst <maarten.lankhorst@linux.intel.com>
Thu, 6 Jul 2017 13:03:15 +0000 (15:03 +0200)
committerMaarten Lankhorst <maarten.lankhorst@linux.intel.com>
Mon, 17 Jul 2017 14:34:51 +0000 (16:34 +0200)
commit8038e09be5a3ac061118bd80c7a505829920b50f
tree8855386457a0b84cfd111e7a37f5798401981724
parenteb42ea6d0b8ed9ca8e73cc24fa801f0d8ab28905
drm/crc: Only open CRC on atomic drivers when the CRTC is active.

Commit e8fa5671183c ("drm: crc: Wait for a frame before returning
from open()") adds a wait for CRC frame, but with the CRTC off
this will never be generated. For atomic drivers we know if a CRTC
is active through crtc_state->active, so when inactive reject the
open with -EIO.

Just like with the previous patch changing debugfs opening semantics,
this patch has been tested against igt.

Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Fixes: e8fa5671183c ("drm: crc: Wait for a frame before returning from open()")
Testcase: debugfs_test.read_all_entries
Link: http://patchwork.freedesktop.org/patch/msgid/15f9d300-65d3-63aa-00e3-e83f5e4d5a7a@linux.intel.com
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/drm_debugfs_crc.c