]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
drm/exynos: Constify struct exynos_drm_ipp_funcs
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Sun, 14 Jul 2024 07:37:20 +0000 (09:37 +0200)
committerInki Dae <inki.dae@samsung.com>
Fri, 6 Sep 2024 06:59:13 +0000 (15:59 +0900)
'struct exynos_drm_ipp_funcs' are not modified in these drivers.

Constifying this structure moves some data to a read-only section, so
increase overall security.

On a x86_64, with allmodconfig, as an example:
Before:
======
   text    data     bss     dec     hex filename
  20446    1746      16   22208    56c0 drivers/gpu/drm/exynos/exynos_drm_fimc.o

After:
=====
   text    data     bss     dec     hex filename
  20446    1714      16   22176    56a0 drivers/gpu/drm/exynos/exynos_drm_fimc.o

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Signed-off-by: Inki Dae <inki.dae@samsung.com>
drivers/gpu/drm/exynos/exynos_drm_fimc.c
drivers/gpu/drm/exynos/exynos_drm_gsc.c
drivers/gpu/drm/exynos/exynos_drm_scaler.c

index 142184c8c3bc55665c224b38eb37c8a191e177a9..4d7ea65b7dd8366a2ca1c24f1c01d94105501b87 100644 (file)
@@ -1125,7 +1125,7 @@ static void fimc_abort(struct exynos_drm_ipp *ipp,
        }
 }
 
-static struct exynos_drm_ipp_funcs ipp_funcs = {
+static const struct exynos_drm_ipp_funcs ipp_funcs = {
        .commit = fimc_commit,
        .abort = fimc_abort,
 };
index 1b111e2c33472bf65a1460bcb5d97f597cd9efa8..d80b0d1eb734e4be2d7399c60e4500450946ef06 100644 (file)
@@ -1162,7 +1162,7 @@ static void gsc_abort(struct exynos_drm_ipp *ipp,
        }
 }
 
-static struct exynos_drm_ipp_funcs ipp_funcs = {
+static const struct exynos_drm_ipp_funcs ipp_funcs = {
        .commit = gsc_commit,
        .abort = gsc_abort,
 };
index a9d46989682420c0611cb6255733fb5099c81cb3..2788105ac780bc90d395978d7c91d4b8db340021 100644 (file)
@@ -403,7 +403,7 @@ static int scaler_commit(struct exynos_drm_ipp *ipp,
        return 0;
 }
 
-static struct exynos_drm_ipp_funcs ipp_funcs = {
+static const struct exynos_drm_ipp_funcs ipp_funcs = {
        .commit = scaler_commit,
 };