agp: kill agp_flush_chipset and corresponding ioctl
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 5 Nov 2010 17:40:56 +0000 (18:40 +0100)
committerChris Wilson <chris@chris-wilson.co.uk>
Tue, 23 Nov 2010 20:14:45 +0000 (20:14 +0000)
The intel drm calls the chipset functions now directly. Userspace
never called the corresponding ioctl, hence it can be killed, too.

Cc: Dave Airlie <airlied@gmail.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
drivers/char/agp/agp.h
drivers/char/agp/compat_ioctl.c
drivers/char/agp/compat_ioctl.h
drivers/char/agp/frontend.c
drivers/char/agp/generic.c
drivers/char/agp/intel-gtt.c
include/linux/agp_backend.h

index 5259065..3e67ddd 100644 (file)
@@ -120,7 +120,6 @@ struct agp_bridge_driver {
        void (*agp_destroy_page)(struct page *, int flags);
        void (*agp_destroy_pages)(struct agp_memory *);
        int (*agp_type_to_mask_type) (struct agp_bridge_data *, int);
-       void (*chipset_flush)(struct agp_bridge_data *);
 };
 
 struct agp_bridge_data {
index 9d2c97a..a48e05b 100644 (file)
@@ -276,7 +276,6 @@ long compat_agp_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
                break;
 
        case AGPIOC_CHIPSET_FLUSH32:
-               ret_val = agpioc_chipset_flush_wrap(curr_priv);
                break;
        }
 
index 0c9678a..f30e0fd 100644 (file)
@@ -102,6 +102,5 @@ void agp_free_memory_wrap(struct agp_memory *memory);
 struct agp_memory *agp_allocate_memory_wrap(size_t pg_count, u32 type);
 struct agp_memory *agp_find_mem_by_key(int key);
 struct agp_client *agp_find_client_by_pid(pid_t id);
-int agpioc_chipset_flush_wrap(struct agp_file_private *priv);
 
 #endif /* _AGP_COMPAT_H */
index 3cb4539..2e04433 100644 (file)
@@ -957,13 +957,6 @@ static int agpioc_unbind_wrap(struct agp_file_private *priv, void __user *arg)
        return agp_unbind_memory(memory);
 }
 
-int agpioc_chipset_flush_wrap(struct agp_file_private *priv)
-{
-       DBG("");
-       agp_flush_chipset(agp_bridge);
-       return 0;
-}
-
 static long agp_ioctl(struct file *file,
                     unsigned int cmd, unsigned long arg)
 {
@@ -1039,7 +1032,6 @@ static long agp_ioctl(struct file *file,
                break;
               
        case AGPIOC_CHIPSET_FLUSH:
-               ret_val = agpioc_chipset_flush_wrap(curr_priv);
                break;
        }
 
index 4956f1c..78bc8de 100644 (file)
@@ -81,13 +81,6 @@ static int agp_get_key(void)
        return -1;
 }
 
-void agp_flush_chipset(struct agp_bridge_data *bridge)
-{
-       if (bridge->driver->chipset_flush)
-               bridge->driver->chipset_flush(bridge);
-}
-EXPORT_SYMBOL(agp_flush_chipset);
-
 /*
  * Use kmalloc if possible for the page list. Otherwise fall back to
  * vmalloc. This speeds things up and also saves memory for small AGP
index 8e2e208..1603e4f 100644 (file)
@@ -984,11 +984,6 @@ static int intel_fake_agp_remove_entries(struct agp_memory *mem,
        return 0;
 }
 
-static void intel_fake_agp_chipset_flush(struct agp_bridge_data *bridge)
-{
-       intel_private.driver->chipset_flush();
-}
-
 static struct agp_memory *intel_fake_agp_alloc_by_type(size_t pg_count,
                                                       int type)
 {
@@ -1222,7 +1217,6 @@ static const struct agp_bridge_driver intel_fake_agp_driver = {
        .agp_alloc_pages        = agp_generic_alloc_pages,
        .agp_destroy_page       = agp_generic_destroy_page,
        .agp_destroy_pages      = agp_generic_destroy_pages,
-       .chipset_flush          = intel_fake_agp_chipset_flush,
 };
 
 static const struct intel_gtt_driver i81x_gtt_driver = {
index 09ea4a1..a479b48 100644 (file)
@@ -106,6 +106,5 @@ extern int agp_rebind_memory(void);
 extern void agp_enable(struct agp_bridge_data *, u32);
 extern struct agp_bridge_data *agp_backend_acquire(struct pci_dev *);
 extern void agp_backend_release(struct agp_bridge_data *);
-extern void agp_flush_chipset(struct agp_bridge_data *);
 
 #endif                         /* _AGP_BACKEND_H */