drm: const'ify ioctls table (v2)
authorzrj <rimvydas.jasinskas@gmail.com>
Mon, 3 Aug 2015 08:01:17 +0000 (11:01 +0300)
committerFran├žois Tigeot <ftigeot@wolfpond.org>
Sat, 22 Aug 2015 06:49:15 +0000 (08:49 +0200)
Because, there is no reason for it not to be const.
We never modify the contents of drm_ioctls, so make it const.

sys/dev/drm/drm_drv.c
sys/dev/drm/i915/i915_dma.c
sys/dev/drm/i915/i915_drv.h
sys/dev/drm/include/drm/drmP.h
sys/dev/drm/radeon/radeon_drv.c
sys/dev/drm/radeon/radeon_kms.c

index 349cb20..bbeb115 100644 (file)
@@ -99,7 +99,7 @@ MODULE_DEPEND(drm, iicbus, 1, 1, 1);
        [DRM_IOCTL_NR(ioctl)] = {.cmd = ioctl, .func = _func, .flags = _flags, .cmd_drv = 0, .name = #ioctl}
 
 /** Ioctl table */
-static struct drm_ioctl_desc drm_ioctls[] = {
+static const struct drm_ioctl_desc drm_ioctls[] = {
        DRM_IOCTL_DEF(DRM_IOCTL_VERSION, drm_version, DRM_UNLOCKED),
        DRM_IOCTL_DEF(DRM_IOCTL_GET_UNIQUE, drm_getunique, 0),
        DRM_IOCTL_DEF(DRM_IOCTL_GET_MAGIC, drm_getmagic, 0),
@@ -720,7 +720,7 @@ int drm_ioctl(struct dev_ioctl_args *ap)
 {
        struct cdev *kdev = ap->a_head.a_dev;
        struct drm_device *dev;
-       struct drm_ioctl_desc *ioctl = NULL;
+       const struct drm_ioctl_desc *ioctl = NULL;
        u_long cmd = ap->a_cmd;
        unsigned int nr = DRM_IOCTL_NR(cmd);
        int retcode = 0;
index 57d51ca..f0cc5b7 100644 (file)
@@ -1995,7 +1995,7 @@ void i915_driver_postclose(struct drm_device *dev, struct drm_file *file)
        kfree(file_priv);
 }
 
-struct drm_ioctl_desc i915_ioctls[] = {
+const struct drm_ioctl_desc i915_ioctls[] = {
        DRM_IOCTL_DEF_DRV(I915_INIT, i915_dma_init, DRM_AUTH|DRM_MASTER|DRM_ROOT_ONLY),
        DRM_IOCTL_DEF_DRV(I915_FLUSH, i915_flush_ioctl, DRM_AUTH),
        DRM_IOCTL_DEF_DRV(I915_FLIP, i915_flip_bufs, DRM_AUTH),
index e7fb5f8..64eeb53 100644 (file)
@@ -2023,7 +2023,7 @@ struct drm_i915_cmd_table {
 
 #include "i915_trace.h"
 
-extern struct drm_ioctl_desc i915_ioctls[];
+extern const struct drm_ioctl_desc i915_ioctls[];
 extern int i915_max_ioctl;
 
 extern int i915_suspend(device_t kdev);
index 3673b0d..cc3d6a3 100644 (file)
@@ -968,7 +968,7 @@ struct drm_driver {
 
        u32 driver_features;
        int dev_priv_size;
-       struct drm_ioctl_desc *ioctls;
+       const struct drm_ioctl_desc *ioctls;
        int num_ioctls;
 };
 
index 9cb9433..7fcdbad 100644 (file)
@@ -88,7 +88,7 @@ extern int radeon_get_crtc_scanoutpos(struct drm_device *dev, int crtc,
                                      unsigned int flags,
                                      int *vpos, int *hpos, ktime_t *stime,
                                      ktime_t *etime);
-extern struct drm_ioctl_desc radeon_ioctls_kms[];
+extern const struct drm_ioctl_desc radeon_ioctls_kms[];
 extern int radeon_max_kms_ioctl;
 #ifdef DUMBBELL_WIP
 int radeon_mmap(struct file *filp, struct vm_area_struct *vma);
index b321487..e7d4c67 100644 (file)
@@ -716,7 +716,7 @@ KMS_INVALID_IOCTL(radeon_surface_alloc_kms)
 KMS_INVALID_IOCTL(radeon_surface_free_kms)
 
 
-struct drm_ioctl_desc radeon_ioctls_kms[] = {
+const struct drm_ioctl_desc radeon_ioctls_kms[] = {
        DRM_IOCTL_DEF_DRV(RADEON_CP_INIT, radeon_cp_init_kms, DRM_AUTH|DRM_MASTER|DRM_ROOT_ONLY),
        DRM_IOCTL_DEF_DRV(RADEON_CP_START, radeon_cp_start_kms, DRM_AUTH|DRM_MASTER|DRM_ROOT_ONLY),
        DRM_IOCTL_DEF_DRV(RADEON_CP_STOP, radeon_cp_stop_kms, DRM_AUTH|DRM_MASTER|DRM_ROOT_ONLY),