diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-02-23 06:03:31 -0500 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-02-27 12:10:53 -0500 |
commit | 3f52c6edf7d72ef71d82c1f5ad4038ec83a31549 (patch) | |
tree | b8d8e2a67f0815fa508a6ce2d27614340fcce2bd | |
parent | 8634bd4aea1b59777e5fd63be15300107ab454cc (diff) |
drm/i915: Remove DRIVER_MODESET checks from modeset code
Mostly just checks in i915-private modeset ioctls.
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
-rw-r--r-- | drivers/gpu/drm/i915/intel_display.c | 3 | ||||
-rw-r--r-- | drivers/gpu/drm/i915/intel_opregion.c | 6 | ||||
-rw-r--r-- | drivers/gpu/drm/i915/intel_overlay.c | 2 | ||||
-rw-r--r-- | drivers/gpu/drm/i915/intel_sprite.c | 6 |
4 files changed, 2 insertions, 15 deletions
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index 93f1ff05a6f4..ca6a24064cab 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c | |||
@@ -12394,9 +12394,6 @@ int intel_get_pipe_from_crtc_id(struct drm_device *dev, void *data, | |||
12394 | struct drm_crtc *drmmode_crtc; | 12394 | struct drm_crtc *drmmode_crtc; |
12395 | struct intel_crtc *crtc; | 12395 | struct intel_crtc *crtc; |
12396 | 12396 | ||
12397 | if (!drm_core_check_feature(dev, DRIVER_MODESET)) | ||
12398 | return -ENODEV; | ||
12399 | |||
12400 | drmmode_crtc = drm_crtc_find(dev, pipe_from_crtc_id->crtc_id); | 12397 | drmmode_crtc = drm_crtc_find(dev, pipe_from_crtc_id->crtc_id); |
12401 | 12398 | ||
12402 | if (!drmmode_crtc) { | 12399 | if (!drmmode_crtc) { |
diff --git a/drivers/gpu/drm/i915/intel_opregion.c b/drivers/gpu/drm/i915/intel_opregion.c index d8de1d5140a7..71e87abdcae7 100644 --- a/drivers/gpu/drm/i915/intel_opregion.c +++ b/drivers/gpu/drm/i915/intel_opregion.c | |||
@@ -744,10 +744,8 @@ void intel_opregion_init(struct drm_device *dev) | |||
744 | return; | 744 | return; |
745 | 745 | ||
746 | if (opregion->acpi) { | 746 | if (opregion->acpi) { |
747 | if (drm_core_check_feature(dev, DRIVER_MODESET)) { | 747 | intel_didl_outputs(dev); |
748 | intel_didl_outputs(dev); | 748 | intel_setup_cadls(dev); |
749 | intel_setup_cadls(dev); | ||
750 | } | ||
751 | 749 | ||
752 | /* Notify BIOS we are ready to handle ACPI video ext notifs. | 750 | /* Notify BIOS we are ready to handle ACPI video ext notifs. |
753 | * Right now, all the events are handled by the ACPI video module. | 751 | * Right now, all the events are handled by the ACPI video module. |
diff --git a/drivers/gpu/drm/i915/intel_overlay.c b/drivers/gpu/drm/i915/intel_overlay.c index f93dfc174495..823d1d97a000 100644 --- a/drivers/gpu/drm/i915/intel_overlay.c +++ b/drivers/gpu/drm/i915/intel_overlay.c | |||
@@ -1065,7 +1065,6 @@ int intel_overlay_put_image(struct drm_device *dev, void *data, | |||
1065 | struct put_image_params *params; | 1065 | struct put_image_params *params; |
1066 | int ret; | 1066 | int ret; |
1067 | 1067 | ||
1068 | /* No need to check for DRIVER_MODESET - we don't set it up then. */ | ||
1069 | overlay = dev_priv->overlay; | 1068 | overlay = dev_priv->overlay; |
1070 | if (!overlay) { | 1069 | if (!overlay) { |
1071 | DRM_DEBUG("userspace bug: no overlay\n"); | 1070 | DRM_DEBUG("userspace bug: no overlay\n"); |
@@ -1261,7 +1260,6 @@ int intel_overlay_attrs(struct drm_device *dev, void *data, | |||
1261 | struct overlay_registers __iomem *regs; | 1260 | struct overlay_registers __iomem *regs; |
1262 | int ret; | 1261 | int ret; |
1263 | 1262 | ||
1264 | /* No need to check for DRIVER_MODESET - we don't set it up then. */ | ||
1265 | overlay = dev_priv->overlay; | 1263 | overlay = dev_priv->overlay; |
1266 | if (!overlay) { | 1264 | if (!overlay) { |
1267 | DRM_DEBUG("userspace bug: no overlay\n"); | 1265 | DRM_DEBUG("userspace bug: no overlay\n"); |
diff --git a/drivers/gpu/drm/i915/intel_sprite.c b/drivers/gpu/drm/i915/intel_sprite.c index 5ae56ecbca72..f0b06c09da40 100644 --- a/drivers/gpu/drm/i915/intel_sprite.c +++ b/drivers/gpu/drm/i915/intel_sprite.c | |||
@@ -1301,9 +1301,6 @@ int intel_sprite_set_colorkey(struct drm_device *dev, void *data, | |||
1301 | struct intel_plane *intel_plane; | 1301 | struct intel_plane *intel_plane; |
1302 | int ret = 0; | 1302 | int ret = 0; |
1303 | 1303 | ||
1304 | if (!drm_core_check_feature(dev, DRIVER_MODESET)) | ||
1305 | return -ENODEV; | ||
1306 | |||
1307 | /* Make sure we don't try to enable both src & dest simultaneously */ | 1304 | /* Make sure we don't try to enable both src & dest simultaneously */ |
1308 | if ((set->flags & (I915_SET_COLORKEY_DESTINATION | I915_SET_COLORKEY_SOURCE)) == (I915_SET_COLORKEY_DESTINATION | I915_SET_COLORKEY_SOURCE)) | 1305 | if ((set->flags & (I915_SET_COLORKEY_DESTINATION | I915_SET_COLORKEY_SOURCE)) == (I915_SET_COLORKEY_DESTINATION | I915_SET_COLORKEY_SOURCE)) |
1309 | return -EINVAL; | 1306 | return -EINVAL; |
@@ -1332,9 +1329,6 @@ int intel_sprite_get_colorkey(struct drm_device *dev, void *data, | |||
1332 | struct intel_plane *intel_plane; | 1329 | struct intel_plane *intel_plane; |
1333 | int ret = 0; | 1330 | int ret = 0; |
1334 | 1331 | ||
1335 | if (!drm_core_check_feature(dev, DRIVER_MODESET)) | ||
1336 | return -ENODEV; | ||
1337 | |||
1338 | drm_modeset_lock_all(dev); | 1332 | drm_modeset_lock_all(dev); |
1339 | 1333 | ||
1340 | plane = drm_plane_find(dev, get->plane_id); | 1334 | plane = drm_plane_find(dev, get->plane_id); |