drm/msm/dpu: drop (mostly) unused DPU_NAME_SIZE define
authorDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Sun, 21 May 2023 19:22:29 +0000 (22:22 +0300)
committerDmitry Baryshkov <dmitry.baryshkov@linaro.org>
Sun, 4 Jun 2023 01:54:14 +0000 (04:54 +0300)
This define is used only in one place, in dpu_encoder debugfs code.
Inline the value and drop the define completely.

Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Reviewed-by: Marijn Suijten <marijn.suijten@somainline.org>
Reviewed-by: Abhinav Kumar <quic_abhinavk@quicinc.com>
Patchwork: https://patchwork.freedesktop.org/patch/538303/
Link: https://lore.kernel.org/r/20230521192230.9747-2-dmitry.baryshkov@linaro.org
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c
drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h
drivers/gpu/drm/msm/disp/dpu1/dpu_plane.c

index c771383..af34932 100644 (file)
@@ -2116,14 +2116,14 @@ static int _dpu_encoder_init_debugfs(struct drm_encoder *drm_enc)
 {
        struct dpu_encoder_virt *dpu_enc = to_dpu_encoder_virt(drm_enc);
 
-       char name[DPU_NAME_SIZE];
+       char name[12];
 
        if (!drm_enc->dev) {
                DPU_ERROR("invalid encoder or kms\n");
                return -EINVAL;
        }
 
-       snprintf(name, DPU_NAME_SIZE, "encoder%u", drm_enc->base.id);
+       snprintf(name, sizeof(name), "encoder%u", drm_enc->base.id);
 
        /* create overall sub-directory for the encoder */
        dpu_enc->debugfs_root = debugfs_create_dir(name,
index aca39a4..197f352 100644 (file)
@@ -63,8 +63,6 @@
 #define ktime_compare_safe(A, B) \
        ktime_compare(ktime_sub((A), (B)), ktime_set(0, 0))
 
-#define DPU_NAME_SIZE  12
-
 struct dpu_kms {
        struct msm_kms base;
        struct drm_device *dev;
index 14b5cfe..ac75ba1 100644 (file)
@@ -42,8 +42,6 @@
 #define SHARP_SMOOTH_THR_DEFAULT       8
 #define SHARP_NOISE_THR_DEFAULT        2
 
-#define DPU_NAME_SIZE  12
-
 #define DPU_PLANE_COLOR_FILL_FLAG      BIT(31)
 #define DPU_ZPOS_MAX 255