Commit d88d2463 authored by Gustavo Padovan's avatar Gustavo Padovan Committed by Inki Dae

drm/exynos: unify exynos_drm_plane names with drm core

Rename crtc_{widht,height} to crtc_{w,h} and src_{width,height} to
src_{w,h} to make it similar to the atomic state names.
Signed-off-by: default avatarGustavo Padovan <gustavo.padovan@collabora.co.uk>
Reviewed-by: default avatarJoonyoung Shim <jy0922.shim@samsung.com>
Signed-off-by: default avatarInki Dae <inki.dae@samsung.com>
parent a1bcc9da
...@@ -237,8 +237,8 @@ static void decon_update_plane(struct exynos_drm_crtc *crtc, ...@@ -237,8 +237,8 @@ static void decon_update_plane(struct exynos_drm_crtc *crtc,
val = COORDINATE_X(plane->crtc_x) | COORDINATE_Y(plane->crtc_y); val = COORDINATE_X(plane->crtc_x) | COORDINATE_Y(plane->crtc_y);
writel(val, ctx->addr + DECON_VIDOSDxA(win)); writel(val, ctx->addr + DECON_VIDOSDxA(win));
val = COORDINATE_X(plane->crtc_x + plane->crtc_width - 1) | val = COORDINATE_X(plane->crtc_x + plane->crtc_w - 1) |
COORDINATE_Y(plane->crtc_y + plane->crtc_height - 1); COORDINATE_Y(plane->crtc_y + plane->crtc_h - 1);
writel(val, ctx->addr + DECON_VIDOSDxB(win)); writel(val, ctx->addr + DECON_VIDOSDxB(win));
val = VIDOSD_Wx_ALPHA_R_F(0x0) | VIDOSD_Wx_ALPHA_G_F(0x0) | val = VIDOSD_Wx_ALPHA_R_F(0x0) | VIDOSD_Wx_ALPHA_G_F(0x0) |
...@@ -251,11 +251,11 @@ static void decon_update_plane(struct exynos_drm_crtc *crtc, ...@@ -251,11 +251,11 @@ static void decon_update_plane(struct exynos_drm_crtc *crtc,
writel(plane->dma_addr[0], ctx->addr + DECON_VIDW0xADD0B0(win)); writel(plane->dma_addr[0], ctx->addr + DECON_VIDW0xADD0B0(win));
val = plane->dma_addr[0] + pitch * plane->crtc_height; val = plane->dma_addr[0] + pitch * plane->crtc_h;
writel(val, ctx->addr + DECON_VIDW0xADD1B0(win)); writel(val, ctx->addr + DECON_VIDW0xADD1B0(win));
val = OFFSIZE(pitch - plane->crtc_width * bpp) val = OFFSIZE(pitch - plane->crtc_w * bpp)
| PAGEWIDTH(plane->crtc_width * bpp); | PAGEWIDTH(plane->crtc_w * bpp);
writel(val, ctx->addr + DECON_VIDW0xADD2(win)); writel(val, ctx->addr + DECON_VIDW0xADD2(win));
decon_win_set_pixfmt(ctx, win, state->fb); decon_win_set_pixfmt(ctx, win, state->fb);
......
...@@ -430,25 +430,25 @@ static void decon_update_plane(struct exynos_drm_crtc *crtc, ...@@ -430,25 +430,25 @@ static void decon_update_plane(struct exynos_drm_crtc *crtc,
DRM_DEBUG_KMS("start addr = 0x%lx\n", DRM_DEBUG_KMS("start addr = 0x%lx\n",
(unsigned long)val); (unsigned long)val);
DRM_DEBUG_KMS("ovl_width = %d, ovl_height = %d\n", DRM_DEBUG_KMS("ovl_width = %d, ovl_height = %d\n",
plane->crtc_width, plane->crtc_height); plane->crtc_w, plane->crtc_h);
/* /*
* OSD position. * OSD position.
* In case the window layout goes of LCD layout, DECON fails. * In case the window layout goes of LCD layout, DECON fails.
*/ */
if ((plane->crtc_x + plane->crtc_width) > mode->hdisplay) if ((plane->crtc_x + plane->crtc_w) > mode->hdisplay)
plane->crtc_x = mode->hdisplay - plane->crtc_width; plane->crtc_x = mode->hdisplay - plane->crtc_w;
if ((plane->crtc_y + plane->crtc_height) > mode->vdisplay) if ((plane->crtc_y + plane->crtc_h) > mode->vdisplay)
plane->crtc_y = mode->vdisplay - plane->crtc_height; plane->crtc_y = mode->vdisplay - plane->crtc_h;
val = VIDOSDxA_TOPLEFT_X(plane->crtc_x) | val = VIDOSDxA_TOPLEFT_X(plane->crtc_x) |
VIDOSDxA_TOPLEFT_Y(plane->crtc_y); VIDOSDxA_TOPLEFT_Y(plane->crtc_y);
writel(val, ctx->regs + VIDOSD_A(win)); writel(val, ctx->regs + VIDOSD_A(win));
last_x = plane->crtc_x + plane->crtc_width; last_x = plane->crtc_x + plane->crtc_w;
if (last_x) if (last_x)
last_x--; last_x--;
last_y = plane->crtc_y + plane->crtc_height; last_y = plane->crtc_y + plane->crtc_h;
if (last_y) if (last_y)
last_y--; last_y--;
......
...@@ -44,12 +44,12 @@ enum exynos_drm_output_type { ...@@ -44,12 +44,12 @@ enum exynos_drm_output_type {
* - the unit is screen coordinates. * - the unit is screen coordinates.
* @src_y: offset y on a framebuffer to be displayed. * @src_y: offset y on a framebuffer to be displayed.
* - the unit is screen coordinates. * - the unit is screen coordinates.
* @src_width: width of a partial image to be displayed from framebuffer. * @src_w: width of a partial image to be displayed from framebuffer.
* @src_height: height of a partial image to be displayed from framebuffer. * @src_h: height of a partial image to be displayed from framebuffer.
* @crtc_x: offset x on hardware screen. * @crtc_x: offset x on hardware screen.
* @crtc_y: offset y on hardware screen. * @crtc_y: offset y on hardware screen.
* @crtc_width: window width to be displayed (hardware screen). * @crtc_w: window width to be displayed (hardware screen).
* @crtc_height: window height to be displayed (hardware screen). * @crtc_h: window height to be displayed (hardware screen).
* @h_ratio: horizontal scaling ratio, 16.16 fixed point * @h_ratio: horizontal scaling ratio, 16.16 fixed point
* @v_ratio: vertical scaling ratio, 16.16 fixed point * @v_ratio: vertical scaling ratio, 16.16 fixed point
* @dma_addr: array of bus(accessed by dma) address to the memory region * @dma_addr: array of bus(accessed by dma) address to the memory region
...@@ -64,12 +64,12 @@ struct exynos_drm_plane { ...@@ -64,12 +64,12 @@ struct exynos_drm_plane {
struct drm_plane base; struct drm_plane base;
unsigned int src_x; unsigned int src_x;
unsigned int src_y; unsigned int src_y;
unsigned int src_width; unsigned int src_w;
unsigned int src_height; unsigned int src_h;
unsigned int crtc_x; unsigned int crtc_x;
unsigned int crtc_y; unsigned int crtc_y;
unsigned int crtc_width; unsigned int crtc_w;
unsigned int crtc_height; unsigned int crtc_h;
unsigned int h_ratio; unsigned int h_ratio;
unsigned int v_ratio; unsigned int v_ratio;
dma_addr_t dma_addr[MAX_FB_BUFFER]; dma_addr_t dma_addr[MAX_FB_BUFFER];
......
...@@ -645,18 +645,18 @@ static void fimd_update_plane(struct exynos_drm_crtc *crtc, ...@@ -645,18 +645,18 @@ static void fimd_update_plane(struct exynos_drm_crtc *crtc,
writel(val, ctx->regs + VIDWx_BUF_START(win, 0)); writel(val, ctx->regs + VIDWx_BUF_START(win, 0));
/* buffer end address */ /* buffer end address */
size = pitch * plane->crtc_height; size = pitch * plane->crtc_h;
val = (unsigned long)(dma_addr + size); val = (unsigned long)(dma_addr + size);
writel(val, ctx->regs + VIDWx_BUF_END(win, 0)); writel(val, ctx->regs + VIDWx_BUF_END(win, 0));
DRM_DEBUG_KMS("start addr = 0x%lx, end addr = 0x%lx, size = 0x%lx\n", DRM_DEBUG_KMS("start addr = 0x%lx, end addr = 0x%lx, size = 0x%lx\n",
(unsigned long)dma_addr, val, size); (unsigned long)dma_addr, val, size);
DRM_DEBUG_KMS("ovl_width = %d, ovl_height = %d\n", DRM_DEBUG_KMS("ovl_width = %d, ovl_height = %d\n",
plane->crtc_width, plane->crtc_height); plane->crtc_w, plane->crtc_h);
/* buffer size */ /* buffer size */
buf_offsize = pitch - (plane->crtc_width * bpp); buf_offsize = pitch - (plane->crtc_w * bpp);
line_size = plane->crtc_width * bpp; line_size = plane->crtc_w * bpp;
val = VIDW_BUF_SIZE_OFFSET(buf_offsize) | val = VIDW_BUF_SIZE_OFFSET(buf_offsize) |
VIDW_BUF_SIZE_PAGEWIDTH(line_size) | VIDW_BUF_SIZE_PAGEWIDTH(line_size) |
VIDW_BUF_SIZE_OFFSET_E(buf_offsize) | VIDW_BUF_SIZE_OFFSET_E(buf_offsize) |
...@@ -670,10 +670,10 @@ static void fimd_update_plane(struct exynos_drm_crtc *crtc, ...@@ -670,10 +670,10 @@ static void fimd_update_plane(struct exynos_drm_crtc *crtc,
VIDOSDxA_TOPLEFT_Y_E(plane->crtc_y); VIDOSDxA_TOPLEFT_Y_E(plane->crtc_y);
writel(val, ctx->regs + VIDOSD_A(win)); writel(val, ctx->regs + VIDOSD_A(win));
last_x = plane->crtc_x + plane->crtc_width; last_x = plane->crtc_x + plane->crtc_w;
if (last_x) if (last_x)
last_x--; last_x--;
last_y = plane->crtc_y + plane->crtc_height; last_y = plane->crtc_y + plane->crtc_h;
if (last_y) if (last_y)
last_y--; last_y--;
...@@ -690,7 +690,7 @@ static void fimd_update_plane(struct exynos_drm_crtc *crtc, ...@@ -690,7 +690,7 @@ static void fimd_update_plane(struct exynos_drm_crtc *crtc,
u32 offset = VIDOSD_D(win); u32 offset = VIDOSD_D(win);
if (win == 0) if (win == 0)
offset = VIDOSD_C(win); offset = VIDOSD_C(win);
val = plane->crtc_width * plane->crtc_height; val = plane->crtc_w * plane->crtc_h;
writel(val, ctx->regs + offset); writel(val, ctx->regs + offset);
DRM_DEBUG_KMS("osd size = 0x%x\n", (unsigned int)val); DRM_DEBUG_KMS("osd size = 0x%x\n", (unsigned int)val);
......
...@@ -97,17 +97,18 @@ static void exynos_plane_mode_set(struct drm_plane *plane, ...@@ -97,17 +97,18 @@ static void exynos_plane_mode_set(struct drm_plane *plane,
/* set drm framebuffer data. */ /* set drm framebuffer data. */
exynos_plane->src_x = src_x; exynos_plane->src_x = src_x;
exynos_plane->src_y = src_y; exynos_plane->src_y = src_y;
exynos_plane->src_width = (actual_w * exynos_plane->h_ratio) >> 16; exynos_plane->src_w = (actual_w * exynos_plane->h_ratio) >> 16;
exynos_plane->src_height = (actual_h * exynos_plane->v_ratio) >> 16; exynos_plane->src_h = (actual_h * exynos_plane->v_ratio) >> 16;
/* set plane range to be displayed. */ /* set plane range to be displayed. */
exynos_plane->crtc_x = crtc_x; exynos_plane->crtc_x = crtc_x;
exynos_plane->crtc_y = crtc_y; exynos_plane->crtc_y = crtc_y;
exynos_plane->crtc_width = actual_w; exynos_plane->crtc_w = actual_w;
exynos_plane->crtc_height = actual_h; exynos_plane->crtc_h = actual_h;
DRM_DEBUG_KMS("plane : offset_x/y(%d,%d), width/height(%d,%d)", DRM_DEBUG_KMS("plane : offset_x/y(%d,%d), width/height(%d,%d)",
exynos_plane->crtc_x, exynos_plane->crtc_y, exynos_plane->crtc_x, exynos_plane->crtc_y,
exynos_plane->crtc_width, exynos_plane->crtc_height); exynos_plane->crtc_w, exynos_plane->crtc_h);
plane->crtc = crtc; plane->crtc = crtc;
} }
......
...@@ -446,19 +446,19 @@ static void vp_video_buffer(struct mixer_context *ctx, ...@@ -446,19 +446,19 @@ static void vp_video_buffer(struct mixer_context *ctx,
vp_reg_write(res, VP_IMG_SIZE_C, VP_IMG_HSIZE(fb->pitches[0]) | vp_reg_write(res, VP_IMG_SIZE_C, VP_IMG_HSIZE(fb->pitches[0]) |
VP_IMG_VSIZE(fb->height / 2)); VP_IMG_VSIZE(fb->height / 2));
vp_reg_write(res, VP_SRC_WIDTH, plane->src_width); vp_reg_write(res, VP_SRC_WIDTH, plane->src_w);
vp_reg_write(res, VP_SRC_HEIGHT, plane->src_height); vp_reg_write(res, VP_SRC_HEIGHT, plane->src_h);
vp_reg_write(res, VP_SRC_H_POSITION, vp_reg_write(res, VP_SRC_H_POSITION,
VP_SRC_H_POSITION_VAL(plane->src_x)); VP_SRC_H_POSITION_VAL(plane->src_x));
vp_reg_write(res, VP_SRC_V_POSITION, plane->src_y); vp_reg_write(res, VP_SRC_V_POSITION, plane->src_y);
vp_reg_write(res, VP_DST_WIDTH, plane->crtc_width); vp_reg_write(res, VP_DST_WIDTH, plane->crtc_w);
vp_reg_write(res, VP_DST_H_POSITION, plane->crtc_x); vp_reg_write(res, VP_DST_H_POSITION, plane->crtc_x);
if (ctx->interlace) { if (ctx->interlace) {
vp_reg_write(res, VP_DST_HEIGHT, plane->crtc_height / 2); vp_reg_write(res, VP_DST_HEIGHT, plane->crtc_h / 2);
vp_reg_write(res, VP_DST_V_POSITION, plane->crtc_y / 2); vp_reg_write(res, VP_DST_V_POSITION, plane->crtc_y / 2);
} else { } else {
vp_reg_write(res, VP_DST_HEIGHT, plane->crtc_height); vp_reg_write(res, VP_DST_HEIGHT, plane->crtc_h);
vp_reg_write(res, VP_DST_V_POSITION, plane->crtc_y); vp_reg_write(res, VP_DST_V_POSITION, plane->crtc_y);
} }
...@@ -495,15 +495,15 @@ static void mixer_layer_update(struct mixer_context *ctx) ...@@ -495,15 +495,15 @@ static void mixer_layer_update(struct mixer_context *ctx)
static int mixer_setup_scale(const struct exynos_drm_plane *plane, static int mixer_setup_scale(const struct exynos_drm_plane *plane,
unsigned int *x_ratio, unsigned int *y_ratio) unsigned int *x_ratio, unsigned int *y_ratio)
{ {
if (plane->crtc_width != plane->src_width) { if (plane->crtc_w != plane->src_w) {
if (plane->crtc_width == 2 * plane->src_width) if (plane->crtc_w == 2 * plane->src_w)
*x_ratio = 1; *x_ratio = 1;
else else
goto fail; goto fail;
} }
if (plane->crtc_height != plane->src_height) { if (plane->crtc_h != plane->src_h) {
if (plane->crtc_height == 2 * plane->src_height) if (plane->crtc_h == 2 * plane->src_h)
*y_ratio = 1; *y_ratio = 1;
else else
goto fail; goto fail;
...@@ -592,8 +592,8 @@ static void mixer_graph_buffer(struct mixer_context *ctx, ...@@ -592,8 +592,8 @@ static void mixer_graph_buffer(struct mixer_context *ctx,
mixer_reg_write(res, MXR_RESOLUTION, val); mixer_reg_write(res, MXR_RESOLUTION, val);
} }
val = MXR_GRP_WH_WIDTH(plane->src_width); val = MXR_GRP_WH_WIDTH(plane->src_w);
val |= MXR_GRP_WH_HEIGHT(plane->src_height); val |= MXR_GRP_WH_HEIGHT(plane->src_h);
val |= MXR_GRP_WH_H_SCALE(x_ratio); val |= MXR_GRP_WH_H_SCALE(x_ratio);
val |= MXR_GRP_WH_V_SCALE(y_ratio); val |= MXR_GRP_WH_V_SCALE(y_ratio);
mixer_reg_write(res, MXR_GRAPHIC_WH(win), val); mixer_reg_write(res, MXR_GRAPHIC_WH(win), val);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment