diff options
| author | Damien Lespiau <damien.lespiau@intel.com> | 2013-10-15 19:55:32 +0200 |
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2013-10-16 13:32:11 +0200 |
| commit | ae676fcd2a93bf1809cddbd79e37b37609791dc2 (patch) | |
| tree | a96c69d390e41cc080a591d650aa42d3356635fa | |
| parent | drm/i915: Make switching to the same CRC source a no-op (diff) | |
| download | linux-ae676fcd2a93bf1809cddbd79e37b37609791dc2.tar.xz linux-ae676fcd2a93bf1809cddbd79e37b37609791dc2.zip | |
drm/i915: Enforce going back to none before changing CRC source
This way we can have some init/fini code on those transitions.
Signed-off-by: Damien Lespiau <damien.lespiau@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
| -rw-r--r-- | drivers/gpu/drm/i915/i915_debugfs.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c index 8c750d5110a1..787c50d194d4 100644 --- a/drivers/gpu/drm/i915/i915_debugfs.c +++ b/drivers/gpu/drm/i915/i915_debugfs.c @@ -1816,6 +1816,10 @@ static int pipe_crc_set_source(struct drm_device *dev, enum pipe pipe, if (pipe_crc->source == source) return 0; + /* forbid changing the source without going back to 'none' */ + if (pipe_crc->source && source) + return -EINVAL; + pipe_crc->source = source; switch (source) { |