drm/exynos: consistent use of cpp
authorTobias Jakobi <tjakobi@math.uni-bielefeld.de>
Tue, 22 Aug 2017 14:19:43 +0000 (16:19 +0200)
committerInki Dae <inki.dae@samsung.com>
Fri, 25 Aug 2017 05:06:48 +0000 (14:06 +0900)
A recent commit (272725c7db4da1fd3229d944fc76d2e98e3a144e) has removed
the use of 'bits_per_pixel' in DRM. However the corresponding Exynos
driver code still uses the ambiguous 'bpp', even though it is now
initialized from fb->cpp[0].

Consistenly use 'cpp' in FIMD, DECON7 and DECON5433 drivers.

Signed-off-by: Tobias Jakobi <tjakobi@math.uni-bielefeld.de>
drivers/gpu/drm/exynos/exynos5433_drm_decon.c
drivers/gpu/drm/exynos/exynos7_drm_decon.c
drivers/gpu/drm/exynos/exynos_drm_fimd.c

index da183e05f2671a2195cbcf7d2c784511f1b6ce40..f7c5e340bc71d8de6646e2ad14d911075d36244c 100644 (file)
@@ -288,7 +288,7 @@ static void decon_win_set_pixfmt(struct decon_context *ctx, unsigned int win,
                return;
        }
 
-       DRM_DEBUG_KMS("bpp = %u\n", fb->format->cpp[0] * 8);
+       DRM_DEBUG_KMS("cpp = %u\n", fb->format->cpp[0]);
 
        /*
         * In case of exynos, setting dma-burst to 16Word causes permanent
@@ -331,7 +331,7 @@ static void decon_update_plane(struct exynos_drm_crtc *crtc,
        struct decon_context *ctx = crtc->ctx;
        struct drm_framebuffer *fb = state->base.fb;
        unsigned int win = plane->index;
-       unsigned int bpp = fb->format->cpp[0];
+       unsigned int cpp = fb->format->cpp[0];
        unsigned int pitch = fb->pitches[0];
        dma_addr_t dma_addr = exynos_drm_fb_dma_addr(fb, 0);
        u32 val;
@@ -367,11 +367,11 @@ static void decon_update_plane(struct exynos_drm_crtc *crtc,
        writel(val, ctx->addr + DECON_VIDW0xADD1B0(win));
 
        if (!(ctx->out_type & IFTYPE_HDMI))
-               val = BIT_VAL(pitch - state->crtc.w * bpp, 27, 14)
-                       | BIT_VAL(state->crtc.w * bpp, 13, 0);
+               val = BIT_VAL(pitch - state->crtc.w * cpp, 27, 14)
+                       | BIT_VAL(state->crtc.w * cpp, 13, 0);
        else
-               val = BIT_VAL(pitch - state->crtc.w * bpp, 29, 15)
-                       | BIT_VAL(state->crtc.w * bpp, 14, 0);
+               val = BIT_VAL(pitch - state->crtc.w * cpp, 29, 15)
+                       | BIT_VAL(state->crtc.w * cpp, 14, 0);
        writel(val, ctx->addr + DECON_VIDW0xADD2(win));
 
        decon_win_set_pixfmt(ctx, win, fb);
index 3e88269fdc2e11a5a6ee867bcb4415e48e0cbb66..4662d55ed988de9670b9ca75b1c3ecaf49b902d9 100644 (file)
@@ -321,7 +321,7 @@ static void decon_win_set_pixfmt(struct decon_context *ctx, unsigned int win,
                break;
        }
 
-       DRM_DEBUG_KMS("bpp = %d\n", fb->format->cpp[0] * 8);
+       DRM_DEBUG_KMS("cpp = %d\n", fb->format->cpp[0]);
 
        /*
         * In case of exynos, setting dma-burst to 16Word causes permanent
@@ -398,7 +398,7 @@ static void decon_update_plane(struct exynos_drm_crtc *crtc,
        unsigned int last_x;
        unsigned int last_y;
        unsigned int win = plane->index;
-       unsigned int bpp = fb->format->cpp[0];
+       unsigned int cpp = fb->format->cpp[0];
        unsigned int pitch = fb->pitches[0];
 
        if (ctx->suspended)
@@ -418,7 +418,7 @@ static void decon_update_plane(struct exynos_drm_crtc *crtc,
        val = (unsigned long)exynos_drm_fb_dma_addr(fb, 0);
        writel(val, ctx->regs + VIDW_BUF_START(win));
 
-       padding = (pitch / bpp) - fb->width;
+       padding = (pitch / cpp) - fb->width;
 
        /* buffer size */
        writel(fb->width + padding, ctx->regs + VIDW_WHOLE_X(win));
index 60f93cad6643d9881b25b5dbc25fa1ebaabf88b8..ebc2077b1253a112dd83f625ea7b6a04a9eff492 100644 (file)
@@ -718,13 +718,13 @@ static void fimd_update_plane(struct exynos_drm_crtc *crtc,
        unsigned long val, size, offset;
        unsigned int last_x, last_y, buf_offsize, line_size;
        unsigned int win = plane->index;
-       unsigned int bpp = fb->format->cpp[0];
+       unsigned int cpp = fb->format->cpp[0];
        unsigned int pitch = fb->pitches[0];
 
        if (ctx->suspended)
                return;
 
-       offset = state->src.x * bpp;
+       offset = state->src.x * cpp;
        offset += state->src.y * pitch;
 
        /* buffer start address */
@@ -743,8 +743,8 @@ static void fimd_update_plane(struct exynos_drm_crtc *crtc,
                        state->crtc.w, state->crtc.h);
 
        /* buffer size */
-       buf_offsize = pitch - (state->crtc.w * bpp);
-       line_size = state->crtc.w * bpp;
+       buf_offsize = pitch - (state->crtc.w * cpp);
+       line_size = state->crtc.w * cpp;
        val = VIDW_BUF_SIZE_OFFSET(buf_offsize) |
                VIDW_BUF_SIZE_PAGEWIDTH(line_size) |
                VIDW_BUF_SIZE_OFFSET_E(buf_offsize) |