drm, kdb, kms: Change mode_set_base_atomic() enter argument to be an enum
authorJason Wessel <jason.wessel@windriver.com>
Wed, 13 Oct 2010 19:09:44 +0000 (14:09 -0500)
committerDave Airlie <airlied@redhat.com>
Tue, 19 Oct 2010 04:13:33 +0000 (14:13 +1000)
The enter argument as implemented by commit 413d45d3627 (drm, kdb, kms:
Add an enter argument to mode_set_base_atomic() API) should be more
descriptive as to what it does vs just passing 1 and 0 around.

There is no runtime behavior change as a result of this patch.

Reported-by: Jesse Barnes <jbarnes@virtuousgeek.org>
Signed-off-by: Jason Wessel <jason.wessel@windriver.com>
CC: David Airlie <airlied@linux.ie>
CC: dri-devel@lists.freedesktop.org
Signed-off-by: Dave Airlie <airlied@redhat.com>
drivers/gpu/drm/drm_fb_helper.c
drivers/gpu/drm/i915/intel_display.c
drivers/gpu/drm/nouveau/nv04_crtc.c
drivers/gpu/drm/nouveau/nv50_crtc.c
drivers/gpu/drm/radeon/atombios_crtc.c
drivers/gpu/drm/radeon/radeon_legacy_crtc.c
drivers/gpu/drm/radeon/radeon_mode.h
include/drm/drm_crtc_helper.h

index 8208e190faaad22df581bca1bea1917d95f365a1..d2849e4ea4d00a45b6f9dfdfe3da1f0da7824533 100644 (file)
@@ -289,8 +289,7 @@ int drm_fb_helper_debug_enter(struct fb_info *info)
                                                    mode_set->fb,
                                                    mode_set->x,
                                                    mode_set->y,
-                                                   1);
-
+                                                   ENTER_ATOMIC_MODE_SET);
                }
        }
 
@@ -336,7 +335,7 @@ int drm_fb_helper_debug_leave(struct fb_info *info)
 
                drm_fb_helper_restore_lut_atomic(mode_set->crtc);
                funcs->mode_set_base_atomic(mode_set->crtc, fb, crtc->x,
-                                           crtc->y, 0);
+                                           crtc->y, LEAVE_ATOMIC_MODE_SET);
        }
 
        return 0;
index 9109c00f3ead42f9f04601a6922fa1f28a2478cc..96d08a9f3aaaa8e9b8351ccad80f882fabc6e614 100644 (file)
@@ -1492,7 +1492,7 @@ err_unpin:
 /* Assume fb object is pinned & idle & fenced and just update base pointers */
 static int
 intel_pipe_set_base_atomic(struct drm_crtc *crtc, struct drm_framebuffer *fb,
-                          int x, int y, int enter)
+                          int x, int y, enum mode_set_atomic state)
 {
        struct drm_device *dev = crtc->dev;
        struct drm_i915_private *dev_priv = dev->dev_private;
@@ -1614,7 +1614,8 @@ intel_pipe_set_base(struct drm_crtc *crtc, int x, int y,
                           atomic_read(&obj_priv->pending_flip) == 0);
        }
 
-       ret = intel_pipe_set_base_atomic(crtc, crtc->fb, x, y, 0);
+       ret = intel_pipe_set_base_atomic(crtc, crtc->fb, x, y,
+                                        LEAVE_ATOMIC_MODE_SET);
        if (ret) {
                i915_gem_object_unpin(to_intel_framebuffer(crtc->fb)->obj);
                mutex_unlock(&dev->struct_mutex);
index 17f7cf0c11a8f45483c834eb7d69b08d11f3a7a1..c71abc2a34d5d9cf22a01d3c3bf455a29c5d1dbf 100644 (file)
@@ -860,12 +860,12 @@ nv04_crtc_mode_set_base(struct drm_crtc *crtc, int x, int y,
 static int
 nv04_crtc_mode_set_base_atomic(struct drm_crtc *crtc,
                               struct drm_framebuffer *fb,
-                              int x, int y, int enter)
+                              int x, int y, enum mode_set_atomic state)
 {
        struct drm_nouveau_private *dev_priv = crtc->dev->dev_private;
        struct drm_device *dev = dev_priv->dev;
 
-       if (enter)
+       if (state == ENTER_ATOMIC_MODE_SET)
                nouveau_fbcon_save_disable_accel(dev);
        else
                nouveau_fbcon_restore_accel(dev);
index ba91befd373445becacfd9e0a73e8dcc448fe96f..16380d52cd885599f349219c903f12cf7b9c6159 100644 (file)
@@ -708,7 +708,7 @@ nv50_crtc_mode_set_base(struct drm_crtc *crtc, int x, int y,
 static int
 nv50_crtc_mode_set_base_atomic(struct drm_crtc *crtc,
                               struct drm_framebuffer *fb,
-                              int x, int y, int enter)
+                              int x, int y, enum mode_set_atomic state)
 {
        return nv50_crtc_do_mode_set_base(crtc, fb, x, y, true, true);
 }
index 176f424975ac291f296b871a970e803e82171fc1..df2b6f2b35f893d00b475147f8d995a7b3c15bc2 100644 (file)
@@ -1270,7 +1270,7 @@ int atombios_crtc_set_base(struct drm_crtc *crtc, int x, int y,
 
 int atombios_crtc_set_base_atomic(struct drm_crtc *crtc,
                                   struct drm_framebuffer *fb,
-                                  int x, int y, int enter)
+                                 int x, int y, enum mode_set_atomic state)
 {
        struct drm_device *dev = crtc->dev;
        struct radeon_device *rdev = dev->dev_private;
index 326843ec51f628f74ff97c47b5381e09aa49fd0b..ace2e6384d4093a688b856c7a5369a9b4124fe94 100644 (file)
@@ -353,7 +353,7 @@ int radeon_crtc_set_base(struct drm_crtc *crtc, int x, int y,
 
 int radeon_crtc_set_base_atomic(struct drm_crtc *crtc,
                                struct drm_framebuffer *fb,
-                               int x, int y, int enter)
+                               int x, int y, enum mode_set_atomic state)
 {
        return radeon_crtc_do_set_base(crtc, fb, x, y, 1);
 }
index f99e12daa81d7849f70c02e575cbe539015c56dd..61b9243db217b93bcfbc5f53a89196a49d8fb1e8 100644 (file)
@@ -35,6 +35,7 @@
 #include <drm_edid.h>
 #include <drm_dp_helper.h>
 #include <drm_fixed.h>
+#include <drm_crtc_helper.h>
 #include <linux/i2c.h>
 #include <linux/i2c-id.h>
 #include <linux/i2c-algo-bit.h>
@@ -526,7 +527,8 @@ extern int atombios_crtc_set_base(struct drm_crtc *crtc, int x, int y,
                                   struct drm_framebuffer *old_fb);
 extern int atombios_crtc_set_base_atomic(struct drm_crtc *crtc,
                                         struct drm_framebuffer *fb,
-                                        int x, int y, int enter);
+                                        int x, int y,
+                                        enum mode_set_atomic state);
 extern int atombios_crtc_mode_set(struct drm_crtc *crtc,
                                   struct drm_display_mode *mode,
                                   struct drm_display_mode *adjusted_mode,
@@ -538,7 +540,8 @@ extern int radeon_crtc_set_base(struct drm_crtc *crtc, int x, int y,
                                 struct drm_framebuffer *old_fb);
 extern int radeon_crtc_set_base_atomic(struct drm_crtc *crtc,
                                       struct drm_framebuffer *fb,
-                                      int x, int y, int enter);
+                                      int x, int y,
+                                      enum mode_set_atomic state);
 extern int radeon_crtc_do_set_base(struct drm_crtc *crtc,
                                   struct drm_framebuffer *fb,
                                   int x, int y, int atomic);
index 6a9f3935ea0b83df8e2bec54c35f0fe3c92c8189..73b071203dcc8bf61d65a75bac74524ba579fbcb 100644 (file)
 
 #include <linux/fb.h>
 
+enum mode_set_atomic {
+       LEAVE_ATOMIC_MODE_SET,
+       ENTER_ATOMIC_MODE_SET,
+};
+
 struct drm_crtc_helper_funcs {
        /*
         * Control power levels on the CRTC.  If the mode passed in is
@@ -62,7 +67,7 @@ struct drm_crtc_helper_funcs {
                             struct drm_framebuffer *old_fb);
        int (*mode_set_base_atomic)(struct drm_crtc *crtc,
                                    struct drm_framebuffer *fb, int x, int y,
-                                   int is_enter);
+                                   enum mode_set_atomic);
 
        /* reload the current crtc LUT */
        void (*load_lut)(struct drm_crtc *crtc);