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 142184c..4d7ea65 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 1b111e2..d80b0d1 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 a9d4698..2788105 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,
 };