diff options
author | Daniel Stone <daniels@collabora.com> | 2015-03-19 00:33:26 -0400 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-03-23 11:22:31 -0400 |
commit | 27798365a60eaa61d8d9010466ffd57368b98ba9 (patch) | |
tree | f03978f2add0fe4ffd4e7e15a10ce590e97d6658 | |
parent | 8f164ce41d1beaebfde99bcbf8bdaafb868eb7ae (diff) |
drm: atomic: Allow setting CRTC active property
Before, we would set the property, but also return -EINVAL because of a
broken fallthrough.
Signed-off-by: Daniel Stone <daniels@collabora.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
-rw-r--r-- | drivers/gpu/drm/drm_atomic.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomic.c index e7af6b89b392..5d3abe3904f0 100644 --- a/drivers/gpu/drm/drm_atomic.c +++ b/drivers/gpu/drm/drm_atomic.c | |||
@@ -248,11 +248,14 @@ int drm_atomic_crtc_set_property(struct drm_crtc *crtc, | |||
248 | struct drm_mode_config *config = &dev->mode_config; | 248 | struct drm_mode_config *config = &dev->mode_config; |
249 | 249 | ||
250 | /* FIXME: Mode prop is missing, which also controls ->enable. */ | 250 | /* FIXME: Mode prop is missing, which also controls ->enable. */ |
251 | if (property == config->prop_active) { | 251 | if (property == config->prop_active) |
252 | state->active = val; | 252 | state->active = val; |
253 | } else if (crtc->funcs->atomic_set_property) | 253 | else if (crtc->funcs->atomic_set_property) |
254 | return crtc->funcs->atomic_set_property(crtc, state, property, val); | 254 | return crtc->funcs->atomic_set_property(crtc, state, property, val); |
255 | return -EINVAL; | 255 | else |
256 | return -EINVAL; | ||
257 | |||
258 | return 0; | ||
256 | } | 259 | } |
257 | EXPORT_SYMBOL(drm_atomic_crtc_set_property); | 260 | EXPORT_SYMBOL(drm_atomic_crtc_set_property); |
258 | 261 | ||