drm/omap: remove no-op cleanup code
authorTomi Valkeinen <tomi.valkeinen@ti.com>
Tue, 15 Aug 2017 13:04:20 +0000 (16:04 +0300)
committerTomi Valkeinen <tomi.valkeinen@ti.com>
Wed, 16 Aug 2017 13:21:07 +0000 (16:21 +0300)
The driver sets crtc and plane rotation properties back to 0 degrees in
dev_lastclose() using drm_object_property_set_value().
drm_object_property_set_value() doesn't do anything with atomic drivers,
and a recent change added WARN_ON() when atomic driver calls the
function.

So remove the code.

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
drivers/gpu/drm/omapdrm/omap_drv.c

index 9a14553265d8141e397144942756bf6e7b603ee4..9b3c36b48356145f20e2f874149179f1a95fdd97 100644 (file)
@@ -464,44 +464,11 @@ static int dev_open(struct drm_device *dev, struct drm_file *file)
  */
 static void dev_lastclose(struct drm_device *dev)
 {
-       int i;
-
-       /* we don't support vga_switcheroo.. so just make sure the fbdev
-        * mode is active
-        */
        struct omap_drm_private *priv = dev->dev_private;
        int ret;
 
        DBG("lastclose: dev=%p", dev);
 
-       /* need to restore default rotation state.. not sure
-        * if there is a cleaner way to restore properties to
-        * default state?  Maybe a flag that properties should
-        * automatically be restored to default state on
-        * lastclose?
-        */
-       for (i = 0; i < priv->num_crtcs; i++) {
-               struct drm_crtc *crtc = priv->crtcs[i];
-
-               if (!crtc->primary->rotation_property)
-                       continue;
-
-               drm_object_property_set_value(&crtc->base,
-                                             crtc->primary->rotation_property,
-                                             DRM_MODE_ROTATE_0);
-       }
-
-       for (i = 0; i < priv->num_planes; i++) {
-               struct drm_plane *plane = priv->planes[i];
-
-               if (!plane->rotation_property)
-                       continue;
-
-               drm_object_property_set_value(&plane->base,
-                                             plane->rotation_property,
-                                             DRM_MODE_ROTATE_0);
-       }
-
        if (priv->fbdev) {
                ret = drm_fb_helper_restore_fbdev_mode_unlocked(priv->fbdev);
                if (ret)