drm/bridge: make drm_connector_funcs structures const
authorBhumika Goyal <bhumirks@gmail.com>
Tue, 8 Aug 2017 11:28:30 +0000 (16:58 +0530)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Wed, 9 Aug 2017 16:37:17 +0000 (18:37 +0200)
Make these const as they are only passed to the function
drm_connector_init and the corresponding argument is of type const.
Done using Coccinelle

@match disable optional_qualifier@
identifier s;
@@
static struct drm_connector_funcs s = {...};

@ref@
position p;
identifier match.s;
@@
s@p

@good1@
identifier match.s;
expression e1,e2;
position ref.p;
@@
drm_connector_init(e1,e2,&s@p,...)

@bad depends on  !good1@
position ref.p;
identifier match.s;
@@
s@p

@depends on forall !bad disable optional_qualifier@
identifier match.s;
@@
static
+ const
struct drm_connector_funcs s;

Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: https://patchwork.freedesktop.org/patch/msgid/1502191712-11231-2-git-send-email-bhumirks@gmail.com
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
drivers/gpu/drm/bridge/adv7511/adv7511_drv.c

index a773a19c3440f05e036ee0c433293961b0a71b8e..b2431aee788795cfa078dbf4b5515c75dbe1961c 100644 (file)
@@ -785,7 +785,7 @@ adv7511_connector_detect(struct drm_connector *connector, bool force)
        return adv7511_detect(adv, connector);
 }
 
-static struct drm_connector_funcs adv7511_connector_funcs = {
+static const struct drm_connector_funcs adv7511_connector_funcs = {
        .fill_modes = drm_helper_probe_single_connector_modes,
        .detect = adv7511_connector_detect,
        .destroy = drm_connector_cleanup,