drm/i915: fix oops on single crtc devices.
[safe/jmp/linux-2.6] / drivers / gpu / drm / i915 / intel_fb.c
index b0de9bb..dfbb0c6 100644 (file)
@@ -30,7 +30,6 @@
 #include <linux/string.h>
 #include <linux/mm.h>
 #include <linux/tty.h>
-#include <linux/slab.h>
 #include <linux/sysrq.h>
 #include <linux/delay.h>
 #include <linux/fb.h>
@@ -45,7 +44,7 @@
 #include "i915_drm.h"
 #include "i915_drv.h"
 
-struct intel_kernel_fbdev {
+struct intel_fbdev {
        struct drm_fb_helper helper;
        struct intel_framebuffer ifb;
        struct list_head fbdev_list;
@@ -56,7 +55,6 @@ static struct fb_ops intelfb_ops = {
        .owner = THIS_MODULE,
        .fb_check_var = drm_fb_helper_check_var,
        .fb_set_par = drm_fb_helper_set_par,
-       .fb_setcolreg = drm_fb_helper_setcolreg,
        .fb_fillrect = cfb_fillrect,
        .fb_copyarea = cfb_copyarea,
        .fb_imageblit = cfb_imageblit,
@@ -65,20 +63,12 @@ static struct fb_ops intelfb_ops = {
        .fb_setcmap = drm_fb_helper_setcmap,
 };
 
-static struct drm_fb_helper_funcs intel_fb_helper_funcs = {
-       .gamma_set = intel_crtc_fb_gamma_set,
-       .gamma_get = intel_crtc_fb_gamma_get,
-};
-
-
-static int intelfb_create(struct drm_device *dev,
-                         struct drm_fb_helper_surface_size *sizes,
-                         struct intel_kernel_fbdev **ifbdev_p)
+static int intelfb_create(struct intel_fbdev *ifbdev,
+                         struct drm_fb_helper_surface_size *sizes)
 {
+       struct drm_device *dev = ifbdev->helper.dev;
        struct fb_info *info;
-       struct intel_kernel_fbdev *ifbdev;
        struct drm_framebuffer *fb;
-       struct intel_framebuffer *intel_fb;
        struct drm_mode_fb_cmd mode_cmd;
        struct drm_gem_object *fbo = NULL;
        struct drm_i915_gem_object *obj_priv;
@@ -98,13 +88,13 @@ static int intelfb_create(struct drm_device *dev,
 
        size = mode_cmd.pitch * mode_cmd.height;
        size = ALIGN(size, PAGE_SIZE);
-       fbo = drm_gem_object_alloc(dev, size);
+       fbo = i915_gem_alloc_object(dev, size);
        if (!fbo) {
                DRM_ERROR("failed to allocate framebuffer\n");
                ret = -ENOMEM;
                goto out;
        }
-       obj_priv = fbo->driver_private;
+       obj_priv = to_intel_bo(fbo);
 
        mutex_lock(&dev->struct_mutex);
 
@@ -115,44 +105,43 @@ static int intelfb_create(struct drm_device *dev,
        }
 
        /* Flush everything out, we'll be doing GTT only from now on */
-       i915_gem_object_set_to_gtt_domain(fbo, 1);
+       ret = i915_gem_object_set_to_gtt_domain(fbo, 1);
+       if (ret) {
+               DRM_ERROR("failed to bind fb: %d.\n", ret);
+               goto out_unpin;
+       }
 
-       info = framebuffer_alloc(sizeof(struct intel_kernel_fbdev), device);
+       info = framebuffer_alloc(0, device);
        if (!info) {
                ret = -ENOMEM;
                goto out_unpin;
        }
 
-       ifbdev = info->par;
+       info->par = ifbdev;
+
        intel_framebuffer_init(dev, &ifbdev->ifb, &mode_cmd, fbo);
 
        fb = &ifbdev->ifb.base;
 
        ifbdev->helper.fb = fb;
        ifbdev->helper.fbdev = info;
-       ifbdev->helper.funcs = &intel_fb_helper_funcs;
-       ifbdev->helper.dev = dev;
-
-       *ifbdev_p = ifbdev;
-
-       ret = drm_fb_helper_init_crtc_count(&ifbdev->helper, 2,
-                                           INTELFB_CONN_LIMIT);
-       if (ret)
-               goto out_unref;
 
        strcpy(info->fix.id, "inteldrmfb");
 
        info->flags = FBINFO_DEFAULT;
-
        info->fbops = &intelfb_ops;
 
-
        /* setup aperture base/size for vesafb takeover */
-       info->aperture_base = dev->mode_config.fb_base;
+       info->apertures = alloc_apertures(1);
+       if (!info->apertures) {
+               ret = -ENOMEM;
+               goto out_unpin;
+       }
+       info->apertures->ranges[0].base = dev->mode_config.fb_base;
        if (IS_I9XX(dev))
-               info->aperture_size = pci_resource_len(dev->pdev, 2);
+               info->apertures->ranges[0].size = pci_resource_len(dev->pdev, 2);
        else
-               info->aperture_size = pci_resource_len(dev->pdev, 0);
+               info->apertures->ranges[0].size = pci_resource_len(dev->pdev, 0);
 
        info->fix.smem_start = dev->mode_config.fb_base + obj_priv->gtt_offset;
        info->fix.smem_len = size;
@@ -165,6 +154,12 @@ static int intelfb_create(struct drm_device *dev,
                ret = -ENOSPC;
                goto out_unpin;
        }
+
+       ret = fb_alloc_cmap(&info->cmap, 256, 0);
+       if (ret) {
+               ret = -ENOMEM;
+               goto out_unpin;
+       }
        info->screen_size = size;
 
 //     memset(info->screen_base, 0, size);
@@ -183,8 +178,8 @@ static int intelfb_create(struct drm_device *dev,
        info->pixmap.scan_align = 1;
 
        DRM_DEBUG_KMS("allocated %dx%d fb: 0x%08x, bo %p\n",
-                       intel_fb->base.width, intel_fb->base.height,
-                       obj_priv->gtt_offset, fbo);
+                     fb->width, fb->height,
+                     obj_priv->gtt_offset, fbo);
 
 
        mutex_unlock(&dev->struct_mutex);
@@ -200,76 +195,86 @@ out:
        return ret;
 }
 
-static int intel_fb_find_or_create_single(struct drm_device *dev,
-                                         struct drm_fb_helper_surface_size *sizes,
-                                         struct drm_fb_helper **fb_ptr)
+static int intel_fb_find_or_create_single(struct drm_fb_helper *helper,
+                                         struct drm_fb_helper_surface_size *sizes)
 {
-       drm_i915_private_t *dev_priv = dev->dev_private;
-       struct intel_kernel_fbdev *ifbdev = NULL;
+       struct intel_fbdev *ifbdev = (struct intel_fbdev *)helper;
        int new_fb = 0;
        int ret;
 
-       if (!dev_priv->fbdev) {
-               ret = intelfb_create(dev, sizes,
-                                    &ifbdev);
+       if (!helper->fb) {
+               ret = intelfb_create(ifbdev, sizes);
                if (ret)
                        return ret;
-
-               dev_priv->fbdev = ifbdev;
                new_fb = 1;
-       } else {
-               ifbdev = dev_priv->fbdev;
-               if (ifbdev->ifb.base.width < sizes->surface_width ||
-                   ifbdev->ifb.base.height < sizes->surface_height) {
-                       DRM_ERROR("Framebuffer not large enough to scale console onto.\n");
-                       return -EINVAL;
-               }
        }
-
-       *fb_ptr = &ifbdev->helper;
        return new_fb;
 }
 
-static int intelfb_probe(struct drm_device *dev)
-{
-       int ret;
-
-       DRM_DEBUG_KMS("\n");
-       ret = drm_fb_helper_single_fb_probe(dev, 32, intel_fb_find_or_create_single);
-       return ret;
-}
+static struct drm_fb_helper_funcs intel_fb_helper_funcs = {
+       .gamma_set = intel_crtc_fb_gamma_set,
+       .gamma_get = intel_crtc_fb_gamma_get,
+       .fb_probe = intel_fb_find_or_create_single,
+};
 
 int intel_fbdev_destroy(struct drm_device *dev,
-                       struct intel_kernel_fbdev *ifbdev)
+                       struct intel_fbdev *ifbdev)
 {
        struct fb_info *info;
        struct intel_framebuffer *ifb = &ifbdev->ifb;
 
-       info = ifbdev->helper.fbdev;
+       if (ifbdev->helper.fbdev) {
+               info = ifbdev->helper.fbdev;
+               unregister_framebuffer(info);
+               iounmap(info->screen_base);
+               if (info->cmap.len)
+                       fb_dealloc_cmap(&info->cmap);
+               framebuffer_release(info);
+       }
 
-       unregister_framebuffer(info);
-       iounmap(info->screen_base);
-       drm_fb_helper_free(&ifbdev->helper);
+       drm_fb_helper_fini(&ifbdev->helper);
 
        drm_framebuffer_cleanup(&ifb->base);
-       drm_gem_object_unreference_unlocked(ifb->obj);
-
-       framebuffer_release(info);
+       if (ifb->obj)
+               drm_gem_object_unreference_unlocked(ifb->obj);
 
        return 0;
 }
 
 int intel_fbdev_init(struct drm_device *dev)
 {
-       drm_helper_initial_config(dev);
-       intelfb_probe(dev);
+       struct intel_fbdev *ifbdev;
+       drm_i915_private_t *dev_priv = dev->dev_private;
+
+       ifbdev = kzalloc(sizeof(struct intel_fbdev), GFP_KERNEL);
+       if (!ifbdev)
+               return -ENOMEM;
+
+       dev_priv->fbdev = ifbdev;
+       ifbdev->helper.funcs = &intel_fb_helper_funcs;
+
+       drm_fb_helper_init(dev, &ifbdev->helper, dev_priv->num_pipe,
+                          INTELFB_CONN_LIMIT);
+
+       drm_fb_helper_single_add_all_connectors(&ifbdev->helper);
+       drm_fb_helper_initial_config(&ifbdev->helper, 32);
        return 0;
 }
 
 void intel_fbdev_fini(struct drm_device *dev)
 {
        drm_i915_private_t *dev_priv = dev->dev_private;
+       if (!dev_priv->fbdev)
+               return;
+
        intel_fbdev_destroy(dev, dev_priv->fbdev);
+       kfree(dev_priv->fbdev);
        dev_priv->fbdev = NULL;
 }
 MODULE_LICENSE("GPL and additional rights");
+
+void intel_fb_output_poll_changed(struct drm_device *dev)
+{
+       drm_i915_private_t *dev_priv = dev->dev_private;
+       drm_fb_helper_hotplug_event(&dev_priv->fbdev->helper);
+}