drm: Constify drm_mode_config_funcs pointer
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Thu, 17 May 2012 11:27:23 +0000 (13:27 +0200)
committerDave Airlie <airlied@redhat.com>
Tue, 22 May 2012 09:35:07 +0000 (10:35 +0100)
The DRM mode config functions structure declared by drivers and pointed
to by the drm_mode_config funcs field is never modified. Make it a const
pointer.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Inki Dae <inki.dae@samsung.com>
Cc: Alan Cox <alan@linux.intel.com>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Cc: Ben Skeggs <bskeggs@redhat.com>
Cc: Thomas Hellstrom <thellstrom@vmware.com>
Cc: Rob Clark <rob.clark@linaro.org>
Reviwed-by: Alex Deucher <alexdeucher@gmail.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
drivers/gpu/drm/exynos/exynos_drm_fb.c
drivers/gpu/drm/gma500/framebuffer.c
drivers/gpu/drm/i915/intel_display.c
drivers/gpu/drm/nouveau/nouveau_display.c
drivers/gpu/drm/radeon/radeon_display.c
drivers/gpu/drm/udl/udl_modeset.c
drivers/gpu/drm/vmwgfx/vmwgfx_kms.c
drivers/staging/omapdrm/omap_drv.c
include/drm/drm_crtc.h

index c38c8f468fa35351d89a1f3309cee0fa358a960c..f82a299553fb9278d8ebde3ede5382b521f9e525 100644 (file)
@@ -191,7 +191,7 @@ static void exynos_drm_output_poll_changed(struct drm_device *dev)
                drm_fb_helper_hotplug_event(fb_helper);
 }
 
-static struct drm_mode_config_funcs exynos_drm_mode_config_funcs = {
+static const struct drm_mode_config_funcs exynos_drm_mode_config_funcs = {
        .fb_create = exynos_user_fb_create,
        .output_poll_changed = exynos_drm_output_poll_changed,
 };
index 09cd23cc05a84d4cd6f1ef8b30137e581bdde06d..5732b5702e1cef5f002ac707fcb34302c33f53de 100644 (file)
@@ -782,7 +782,7 @@ void psb_modeset_init(struct drm_device *dev)
        dev->mode_config.min_width = 0;
        dev->mode_config.min_height = 0;
 
-       dev->mode_config.funcs = (void *) &psb_mode_funcs;
+       dev->mode_config.funcs = &psb_mode_funcs;
 
        /* set memory base */
        /* Oaktrail and Poulsbo should use BAR 2*/
index 3c71850ddf2077bf06f3e1ffb59caf39f93f3240..ee61ad1e642b06848f537fb6000b3d83c3bf1be1 100644 (file)
@@ -6941,7 +6941,7 @@ void intel_modeset_init(struct drm_device *dev)
        dev->mode_config.preferred_depth = 24;
        dev->mode_config.prefer_shadow = 1;
 
-       dev->mode_config.funcs = (void *)&intel_mode_funcs;
+       dev->mode_config.funcs = &intel_mode_funcs;
 
        intel_init_quirks(dev);
 
index a85e112863d1ae2062bdd7e4ea52fe9c4967bf27..4b1cf74579833102a6ddb323b527abc5d6340444 100644 (file)
@@ -300,7 +300,7 @@ nouveau_display_create(struct drm_device *dev)
                disp->color_vibrance_property->values[1] = 200; /* -100..+100 */
        }
 
-       dev->mode_config.funcs = (void *)&nouveau_mode_config_funcs;
+       dev->mode_config.funcs = &nouveau_mode_config_funcs;
        dev->mode_config.fb_base = pci_resource_start(dev->pdev, 1);
 
        dev->mode_config.min_width = 0;
index da3fe8a68f8d63ba958f1b226c3d763821740808..64a008d14493f16410342084bf0b2c7e9441e0ae 100644 (file)
@@ -1320,7 +1320,7 @@ int radeon_modeset_init(struct radeon_device *rdev)
        drm_mode_config_init(rdev->ddev);
        rdev->mode_info.mode_config_initialized = true;
 
-       rdev->ddev->mode_config.funcs = (void *)&radeon_mode_funcs;
+       rdev->ddev->mode_config.funcs = &radeon_mode_funcs;
 
        if (ASIC_IS_DCE5(rdev)) {
                rdev->ddev->mode_config.max_width = 16384;
index b3ecb3d12a1d592f1a3368a73f261f15cccb71a1..0d7816789da1460625491c9104d82e1d80fb07d7 100644 (file)
@@ -395,7 +395,7 @@ int udl_modeset_init(struct drm_device *dev)
        dev->mode_config.prefer_shadow = 0;
        dev->mode_config.preferred_depth = 24;
 
-       dev->mode_config.funcs = (void *)&udl_mode_funcs;
+       dev->mode_config.funcs = &udl_mode_funcs;
 
        drm_mode_create_dirty_info_property(dev);
 
index 2286d47e5022029af20bf4e03de6b81879bd911c..6b0078ffa7638b163abb2a286e6b8d579daae4a5 100644 (file)
@@ -1178,7 +1178,7 @@ err_out:
        return &vfb->base;
 }
 
-static struct drm_mode_config_funcs vmw_kms_funcs = {
+static const struct drm_mode_config_funcs vmw_kms_funcs = {
        .fb_create = vmw_kms_fb_create,
 };
 
index 071c7702d05df40726f2856981177760234fded8..1edbd777bf404f3ddeb86b66f4a5c8c28f7823b8 100644 (file)
@@ -58,7 +58,7 @@ static void omap_fb_output_poll_changed(struct drm_device *dev)
        }
 }
 
-static struct drm_mode_config_funcs omap_mode_config_funcs = {
+static const struct drm_mode_config_funcs omap_mode_config_funcs = {
        .fb_create = omap_framebuffer_create,
        .output_poll_changed = omap_fb_output_poll_changed,
 };
index 3ecee192db06babe0df604c916429bc8d510ea86..b88b28f45f9e76434c4a66c9d6eb2b200cd07082 100644 (file)
@@ -767,7 +767,7 @@ struct drm_mode_config {
 
        int min_width, min_height;
        int max_width, max_height;
-       struct drm_mode_config_funcs *funcs;
+       const struct drm_mode_config_funcs *funcs;
        resource_size_t fb_base;
 
        /* output poll support */