drm/bridge: lvds-codec: Constify the drm_bridge_funcs structure
authorLaurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Mon, 24 Feb 2020 23:00:56 +0000 (01:00 +0200)
committerBoris Brezillon <boris.brezillon@collabora.com>
Tue, 25 Feb 2020 10:14:43 +0000 (11:14 +0100)
The drm_bridge_funcs structure is never modified, make it const. Making
it read-only can improve security as the structure contains function
pointers.

Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Reviewed-by: Boris Brezillon <boris.brezillon@collabora.com>
Signed-off-by: Boris Brezillon <boris.brezillon@collabora.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200224230056.2157-1-laurent.pinchart+renesas@ideasonboard.com
drivers/gpu/drm/bridge/lvds-codec.c

index ae173d8..f7ae28e 100644 (file)
@@ -50,7 +50,7 @@ static void lvds_codec_disable(struct drm_bridge *bridge)
                gpiod_set_value_cansleep(lvds_codec->powerdown_gpio, 1);
 }
 
-static struct drm_bridge_funcs funcs = {
+static const struct drm_bridge_funcs funcs = {
        .attach = lvds_codec_attach,
        .enable = lvds_codec_enable,
        .disable = lvds_codec_disable,